From 89a85b428d6edd86cf0d02371b11fbc85abc9dc1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dar=C3=ADo=20Kondratiuk?= Date: Tue, 25 May 2021 04:14:44 -0300 Subject: [PATCH] test: reorder describes in JS coverage spec (#7075) Just some code reorder. We had a describe between it calls. I'm moving that describe to the end Co-authored-by: Jack Franklin --- test/coverage.spec.ts | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/test/coverage.spec.ts b/test/coverage.spec.ts index b2a8730f..a5ff0dc0 100644 --- a/test/coverage.spec.ts +++ b/test/coverage.spec.ts @@ -125,6 +125,17 @@ describe('Coverage specs', function () { JSON.stringify(coverage, null, 2).replace(/:\d{4}\//g, ':/') ).toBeGolden('jscoverage-involved.txt'); }); + // @see https://crbug.com/990945 + xit('should not hang when there is a debugger statement', async () => { + const { page, server } = getTestState(); + + await page.coverage.startJSCoverage(); + await page.goto(server.EMPTY_PAGE); + await page.evaluate(() => { + debugger; // eslint-disable-line no-debugger + }); + await page.coverage.stopJSCoverage(); + }); describe('resetOnNavigation', function () { it('should report scripts across navigations when disabled', async () => { const { page, server } = getTestState(); @@ -146,17 +157,6 @@ describe('Coverage specs', function () { expect(coverage.length).toBe(0); }); }); - // @see https://crbug.com/990945 - xit('should not hang when there is a debugger statement', async () => { - const { page, server } = getTestState(); - - await page.coverage.startJSCoverage(); - await page.goto(server.EMPTY_PAGE); - await page.evaluate(() => { - debugger; // eslint-disable-line no-debugger - }); - await page.coverage.stopJSCoverage(); - }); }); describeChromeOnly('CSSCoverage', function () {