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

Unified Diff: net/cert/ct_objects_extractor_unittest.cc

Issue 67513008: Certificate Transparency: Add the high-level interface for verifying SCTs over multiple logs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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/cert/ct_objects_extractor_unittest.cc
diff --git a/net/cert/ct_objects_extractor_unittest.cc b/net/cert/ct_objects_extractor_unittest.cc
index 30ababd5b9e0a7c1d5c2144e919a1309baa267ba..d9cb8a7d9390d4590d7c1bffc61a63db06dc12e1 100644
--- a/net/cert/ct_objects_extractor_unittest.cc
+++ b/net/cert/ct_objects_extractor_unittest.cc
@@ -105,7 +105,7 @@ TEST_F(CTObjectsExtractorTest, ExtractedSCTVerifies) {
precert_chain_[1]->os_cert_handle(),
&entry));
- EXPECT_TRUE(log_->Verify(entry, sct));
+ ASSERT_EQ(CTLogVerifier::SCT_VERIFIED_OK, log_->Verify(entry, sct));
}
// Test that an externally-provided SCT verifies over the LogEntry
@@ -117,7 +117,7 @@ TEST_F(CTObjectsExtractorTest, ComplementarySCTVerifies) {
LogEntry entry;
ASSERT_TRUE(GetX509LogEntry(test_cert_->os_cert_handle(), &entry));
- EXPECT_TRUE(log_->Verify(entry, sct));
+ ASSERT_EQ(CTLogVerifier::SCT_VERIFIED_OK, log_->Verify(entry, sct));
}
} // namespace ct

Powered by Google App Engine
This is Rietveld 408576698