Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(165)

Unified Diff: chrome/common/safe_browsing/binary_feature_extractor_win_unittest.cc

Issue 2308403002: Use a new dual-signed binary in safe browsing tests. (Closed)
Patch Set: rebase to position 416899 Created 4 years, 3 months 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
« no previous file with comments | « no previous file | chrome/common/safe_browsing/pe_image_reader_win_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/safe_browsing/binary_feature_extractor_win_unittest.cc
diff --git a/chrome/common/safe_browsing/binary_feature_extractor_win_unittest.cc b/chrome/common/safe_browsing/binary_feature_extractor_win_unittest.cc
index cf878e8b32e5ea8d1ac8802d4fdd06ac61df0d8f..4b2f9df2a280a9fd6a88998e251d46a35af90379 100644
--- a/chrome/common/safe_browsing/binary_feature_extractor_win_unittest.cc
+++ b/chrome/common/safe_browsing/binary_feature_extractor_win_unittest.cc
@@ -65,10 +65,11 @@ TEST_F(BinaryFeatureExtractorWinTest, UntrustedSignedBinary) {
}
TEST_F(BinaryFeatureExtractorWinTest, TrustedBinary) {
- // wow_helper.exe is signed using Google's signing certifiacte.
+ // disable_outdated_build_detector.exe is dual signed using Google's signing
+ // certifiacte.
ClientDownloadRequest_SignatureInfo signature_info;
binary_feature_extractor_->CheckSignature(
- testdata_path_.Append(L"wow_helper.exe"),
+ testdata_path_.Append(L"disable_outdated_build_detector.exe"),
&signature_info);
ASSERT_EQ(1, signature_info.certificate_chain_size());
std::vector<scoped_refptr<net::X509Certificate> > certs;
@@ -76,9 +77,9 @@ TEST_F(BinaryFeatureExtractorWinTest, TrustedBinary) {
ASSERT_EQ(3u, certs.size());
EXPECT_EQ("Google Inc", certs[0]->subject().common_name);
- EXPECT_EQ("VeriSign Class 3 Code Signing 2009-2 CA",
+ EXPECT_EQ("VeriSign Class 3 Code Signing 2010 CA",
certs[1]->subject().common_name);
- EXPECT_EQ("Class 3 Public Primary Certification Authority",
+ EXPECT_EQ("VeriSign Trust Network",
certs[2]->subject().organization_unit_names[0]);
EXPECT_TRUE(signature_info.trusted());
@@ -198,7 +199,7 @@ TEST_F(BinaryFeatureExtractorWinTest, ExtractImageFeaturesTrustedSigned) {
ClientDownloadRequest_ImageHeaders image_headers;
google::protobuf::RepeatedPtrField<std::string> signed_data;
ASSERT_TRUE(binary_feature_extractor_->ExtractImageFeatures(
- testdata_path_.AppendASCII("wow_helper.exe"),
+ testdata_path_.AppendASCII("disable_outdated_build_detector.exe"),
BinaryFeatureExtractor::kDefaultOptions, &image_headers, &signed_data));
ASSERT_EQ(1, signed_data.size());
ASSERT_LT(0U, signed_data.Get(0).size());
« no previous file with comments | « no previous file | chrome/common/safe_browsing/pe_image_reader_win_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698