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

Side by Side Diff: net/url_request/url_request_unittest.cc

Issue 2164303002: Disable flaky HTTPSOCSPVerifyTest on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <memory> 5 #include <memory>
6 #include <utility> 6 #include <utility>
7 7
8 #include "base/memory/ptr_util.h" 8 #include "base/memory/ptr_util.h"
9 #include "base/run_loop.h" 9 #include "base/run_loop.h"
10 #include "build/build_config.h" 10 #include "build/build_config.h"
(...skipping 9445 matching lines...) Expand 10 before | Expand all | Expand 10 after
9456 9456
9457 {{{SpawnedTestServer::SSLOptions::OCSP_MISMATCHED_SERIAL, 9457 {{{SpawnedTestServer::SSLOptions::OCSP_MISMATCHED_SERIAL,
9458 SpawnedTestServer::SSLOptions::OCSP_DATE_EARLY}}, 9458 SpawnedTestServer::SSLOptions::OCSP_DATE_EARLY}},
9459 SpawnedTestServer::SSLOptions::OCSP_PRODUCED_VALID, 9459 SpawnedTestServer::SSLOptions::OCSP_PRODUCED_VALID,
9460 OCSPVerifyResult::NO_MATCHING_RESPONSE, 9460 OCSPVerifyResult::NO_MATCHING_RESPONSE,
9461 false, 9461 false,
9462 OCSPRevocationStatus::UNKNOWN}, 9462 OCSPRevocationStatus::UNKNOWN},
9463 9463
9464 }; 9464 };
9465 9465
9466 // Flaky on Android Tests (dbg) bot, crbug.com/630286.
9467 #if !defined(OS_ANDROID)
9466 class HTTPSOCSPVerifyTest 9468 class HTTPSOCSPVerifyTest
9467 : public HTTPSOCSPTest, 9469 : public HTTPSOCSPTest,
9468 public testing::WithParamInterface<OCSPVerifyTestData> {}; 9470 public testing::WithParamInterface<OCSPVerifyTestData> {};
9469 9471
9470 TEST_P(HTTPSOCSPVerifyTest, VerifyResult) { 9472 TEST_P(HTTPSOCSPVerifyTest, VerifyResult) {
9471 SpawnedTestServer::SSLOptions ssl_options( 9473 SpawnedTestServer::SSLOptions ssl_options(
9472 SpawnedTestServer::SSLOptions::CERT_AUTO); 9474 SpawnedTestServer::SSLOptions::CERT_AUTO);
9473 OCSPVerifyTestData test = GetParam(); 9475 OCSPVerifyTestData test = GetParam();
9474 9476
9475 ssl_options.ocsp_responses = test.ocsp_responses; 9477 ssl_options.ocsp_responses = test.ocsp_responses;
(...skipping 14 matching lines...) Expand all
9490 9492
9491 EXPECT_EQ(test.response_status, ssl_info.ocsp_result.response_status); 9493 EXPECT_EQ(test.response_status, ssl_info.ocsp_result.response_status);
9492 9494
9493 if (test.has_revocation_status) 9495 if (test.has_revocation_status)
9494 EXPECT_EQ(test.cert_status, ssl_info.ocsp_result.revocation_status); 9496 EXPECT_EQ(test.cert_status, ssl_info.ocsp_result.revocation_status);
9495 } 9497 }
9496 9498
9497 INSTANTIATE_TEST_CASE_P(OCSPVerify, 9499 INSTANTIATE_TEST_CASE_P(OCSPVerify,
9498 HTTPSOCSPVerifyTest, 9500 HTTPSOCSPVerifyTest,
9499 testing::ValuesIn(kOCSPVerifyData)); 9501 testing::ValuesIn(kOCSPVerifyData));
9502 #endif
9500 9503
9501 class HTTPSHardFailTest : public HTTPSOCSPTest { 9504 class HTTPSHardFailTest : public HTTPSOCSPTest {
9502 protected: 9505 protected:
9503 void SetupContext() override { 9506 void SetupContext() override {
9504 context_.set_ssl_config_service(new TestSSLConfigService( 9507 context_.set_ssl_config_service(new TestSSLConfigService(
9505 false /* check for EV */, false /* online revocation checking */, 9508 false /* check for EV */, false /* online revocation checking */,
9506 true /* require rev. checking for local 9509 true /* require rev. checking for local
9507 anchors */, 9510 anchors */,
9508 false /* token binding enabled */)); 9511 false /* token binding enabled */));
9509 } 9512 }
(...skipping 638 matching lines...) Expand 10 before | Expand all | Expand 10 after
10148 AddTestInterceptor()->set_main_intercept_job(std::move(job)); 10151 AddTestInterceptor()->set_main_intercept_job(std::move(job));
10149 10152
10150 req->Start(); 10153 req->Start();
10151 req->Cancel(); 10154 req->Cancel();
10152 base::RunLoop().RunUntilIdle(); 10155 base::RunLoop().RunUntilIdle();
10153 EXPECT_EQ(URLRequestStatus::CANCELED, req->status().status()); 10156 EXPECT_EQ(URLRequestStatus::CANCELED, req->status().status());
10154 EXPECT_EQ(0, d.received_redirect_count()); 10157 EXPECT_EQ(0, d.received_redirect_count());
10155 } 10158 }
10156 10159
10157 } // namespace net 10160 } // namespace net
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698