Skip to content

Commit

Permalink
Re-enable SCTAuditingHandlerTest.DisableReporting test
Browse files Browse the repository at this point in the history
After the fix for crbug.com/1344881 in crrev.com/c/3773419, this should
no longer be prone to failure.

Bug: 1341847
Change-Id: I5406a794499812973154611e416dd5af788b80dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3778745
Auto-Submit: Chris Thompson <cthomp@chromium.org>
Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org>
Reviewed-by: Joe DeBlasio <jdeblasio@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1026531}
  • Loading branch information
christhompson authored and Chromium LUCI CQ committed Jul 21, 2022
1 parent e3f2923 commit 5280804
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -221,8 +221,7 @@ void MakeTestSCTAndStatus(
}

// Tests that if reporting is disabled, reports are not created.
// crbug.com/1341847 Disable the test due to flakiness
TEST_F(SCTAuditingHandlerTest, DISABLED_DisableReporting) {
TEST_F(SCTAuditingHandlerTest, DisableReporting) {
// Create a report which would normally trigger a send.
const net::HostPortPair host_port_pair("example.com", 443);
net::SignedCertificateTimestampAndStatusList sct_list;
Expand Down

0 comments on commit 5280804

Please sign in to comment.