net/base/cert_database_nss_unittest.cc - Issue 8566056: This applies GUIDs to certificate and key nicknames when

Unified Diff: net/base/cert_database_nss_unittest.cc

Issue 8566056: This applies GUIDs to certificate and key nicknames when (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: minor fixes Created 9 years, 1 month ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: net/base/cert_database_nss_unittest.cc

diff --git a/net/base/cert_database_nss_unittest.cc b/net/base/cert_database_nss_unittest.cc

index 1038e76402c122cd8b6fe9eb3616ad665b03af4a..a5ea1668769e7d3392a584e2dc9e230d799daef5 100644

--- a/net/base/cert_database_nss_unittest.cc

+++ b/net/base/cert_database_nss_unittest.cc

@@ -153,7 +153,8 @@ TEST_F(CertDatabaseNSSTest, ImportFromPKCS12WrongPassword) {

cert_db_.ImportFromPKCS12(slot_,

pkcs12_data,

ASCIIToUTF16(""),

- true)); // is_extractable

+ true, // is_extractable

+ NULL));

// Test db should still be empty.

EXPECT_EQ(0U, ListCertsInSlot(slot_->os_module_handle()).size());

@@ -165,7 +166,8 @@ TEST_F(CertDatabaseNSSTest, ImportFromPKCS12AsExtractableAndExportAgain) {

EXPECT_EQ(OK, cert_db_.ImportFromPKCS12(slot_,

pkcs12_data,

ASCIIToUTF16("12345"),

- true)); // is_extractable

+ true, // is_extractable

+ NULL));

CertificateList cert_list = ListCertsInSlot(slot_->os_module_handle());

ASSERT_EQ(1U, cert_list.size());

@@ -188,7 +190,8 @@ TEST_F(CertDatabaseNSSTest, ImportFromPKCS12Twice) {

EXPECT_EQ(OK, cert_db_.ImportFromPKCS12(slot_,

pkcs12_data,

ASCIIToUTF16("12345"),

- true)); // is_extractable

+ true, // is_extractable

+ NULL));

EXPECT_EQ(1U, ListCertsInSlot(slot_->os_module_handle()).size());

// NSS has a SEC_ERROR_PKCS12_DUPLICATE_DATA error, but it doesn't look like

@@ -196,7 +199,8 @@ TEST_F(CertDatabaseNSSTest, ImportFromPKCS12Twice) {

EXPECT_EQ(OK, cert_db_.ImportFromPKCS12(slot_,

pkcs12_data,

ASCIIToUTF16("12345"),

- true)); // is_extractable

+ true, // is_extractable

+ NULL));

EXPECT_EQ(1U, ListCertsInSlot(slot_->os_module_handle()).size());

}

@@ -206,7 +210,8 @@ TEST_F(CertDatabaseNSSTest, ImportFromPKCS12AsUnextractableAndExportAgain) {

EXPECT_EQ(OK, cert_db_.ImportFromPKCS12(slot_,

pkcs12_data,

ASCIIToUTF16("12345"),

- false)); // is_extractable

+ false, // is_extractable

+ NULL));

CertificateList cert_list = ListCertsInSlot(slot_->os_module_handle());

ASSERT_EQ(1U, cert_list.size());

@@ -227,7 +232,8 @@ TEST_F(CertDatabaseNSSTest, ImportFromPKCS12OnlyMarkIncludedKey) {

EXPECT_EQ(OK, cert_db_.ImportFromPKCS12(slot_,

pkcs12_data,

ASCIIToUTF16("12345"),

- true)); // is_extractable

+ true, // is_extractable

+ NULL));

CertificateList cert_list = ListCertsInSlot(slot_->os_module_handle());

ASSERT_EQ(1U, cert_list.size());

@@ -237,7 +243,8 @@ TEST_F(CertDatabaseNSSTest, ImportFromPKCS12OnlyMarkIncludedKey) {

EXPECT_EQ(OK, cert_db_.ImportFromPKCS12(slot_,

pkcs12_data,

ASCIIToUTF16("12345"),

- false)); // is_extractable

+ false, // is_extractable

+ NULL));

cert_list = ListCertsInSlot(slot_->os_module_handle());

ASSERT_EQ(1U, cert_list.size());

@@ -256,7 +263,8 @@ TEST_F(CertDatabaseNSSTest, ImportFromPKCS12InvalidFile) {

cert_db_.ImportFromPKCS12(slot_,

pkcs12_data,

ASCIIToUTF16(""),

- true)); // is_extractable

+ true, // is_extractable

+ NULL));

// Test db should still be empty.

EXPECT_EQ(0U, ListCertsInSlot(slot_->os_module_handle()).size());

« no previous file with comments | « net/base/cert_database_nss.cc ('k') | net/base/cert_database_openssl.cc » ('j') | net/base/x509_certificate_nss.cc » ('J')