Skip to content

Commit 1670340

Browse files
Adam RaineDevtools-frontend LUCI CQ
authored andcommitted
[Lighthouse] Fix flaky e2e test
Fixed: 399896980 Change-Id: I6f178fa92225ddb7d5b1fd94b883fcbd356bf047 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6313499 Reviewed-by: Adriana Ixba <[email protected]> Commit-Queue: Adam Raine <[email protected]> Auto-Submit: Adam Raine <[email protected]> Commit-Queue: Adriana Ixba <[email protected]>
1 parent b600c7c commit 1670340

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/e2e/lighthouse/navigation_test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -179,8 +179,7 @@ describe('Navigation', function() {
179179
assert.strictEqual(await getServiceWorkerCount(), 0);
180180
});
181181

182-
// Flaky test
183-
it.skip('[crbug.com/399896980] successfully returns a Lighthouse report with DevTools throttling', async () => {
182+
it('successfully returns a Lighthouse report with DevTools throttling', async () => {
184183
await navigateToLighthouseTab('lighthouse/hello.html');
185184

186185
await setThrottlingMethod('devtools');
@@ -194,6 +193,7 @@ describe('Navigation', function() {
194193
// [crbug.com/1347220] DevTools throttling can force resources to load slow enough for these audits to fail sometimes.
195194
const flakyAudits = [
196195
'server-response-time',
196+
'document-latency-insight',
197197
'render-blocking-resources',
198198
'render-blocking-insight',
199199
'max-potential-fid',

0 commit comments

Comments
 (0)