Skip to content

Commit

Permalink
Re-enables cert-related tests by disabling individual cases
Browse files Browse the repository at this point in the history
b/331472624

Change-Id: Ief67f72123846ff91681db163b6fab85c2043d8b
  • Loading branch information
johnxwork committed Mar 27, 2024
1 parent c21ab66 commit f878117
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
4 changes: 0 additions & 4 deletions net/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -3341,10 +3341,6 @@ test("net_unittests") {
# TODO: b/331469815
"socket/tcp_client_socket_unittest.cc",

# TODO: b/331472624
"cert/pki/path_builder_unittest.cc",
"cert/multi_threaded_cert_verifier_unittest.cc",

# Re-enable once we're on c++20
"base/backoff_entry_serializer_unittest.cc",
]
Expand Down
3 changes: 3 additions & 0 deletions net/cert/multi_threaded_cert_verifier_unittest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -249,6 +249,8 @@ TEST_F(MultiThreadedCertVerifierTest, CancelRequestThenQuit) {
// Destroy |verifier_| by going out of scope.
}

// Starboard disables cert_net_fetcher which results in different error cause.
#if !defined(STARBOARD)
// Tests propagation of configuration options into CertVerifyProc flags
TEST_F(MultiThreadedCertVerifierTest, ConvertsConfigToFlags) {
base::FilePath certs_dir = GetTestCertsDirectory();
Expand Down Expand Up @@ -297,6 +299,7 @@ TEST_F(MultiThreadedCertVerifierTest, ConvertsConfigToFlags) {
testing::Mock::VerifyAndClearExpectations(mock_verify_proc_.get());
}
}
#endif

// Tests propagation of CertVerifier flags into CertVerifyProc flags
TEST_F(MultiThreadedCertVerifierTest, ConvertsFlagsToFlags) {
Expand Down

0 comments on commit f878117

Please sign in to comment.