From 52694692f09027d54763b167604eda38666f6581 Mon Sep 17 00:00:00 2001 From: Yury Semikhatsky Date: Wed, 5 Feb 2025 12:06:39 -0800 Subject: [PATCH] use ; --- .github/actions/run-test/action.yml | 40 ++++++++++++----------------- tests/page/page-leaks.spec.ts | 2 -- 2 files changed, 16 insertions(+), 26 deletions(-) diff --git a/.github/actions/run-test/action.yml b/.github/actions/run-test/action.yml index c01987c22d..051ae3539b 100644 --- a/.github/actions/run-test/action.yml +++ b/.github/actions/run-test/action.yml @@ -36,29 +36,23 @@ runs: with: node-version: ${{ inputs.node-version }} - uses: ./.github/actions/enable-microphone-access - - run: | - { - echo "::group::npm ci" - npm ci - echo "::endgroup::" - } + - run: + echo "::group::npm ci"; + npm ci; + echo "::endgroup::"; shell: bash env: DEBUG: pw:install PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: '1' - - run: | - { - echo "::group::npm run build" - npm run build - echo "::endgroup::" - } + - run: + echo "::group::npm run build"; + npm run build; + echo "::endgroup::"; shell: bash - - run: | - { - echo "::group::npx playwright install --with-deps" - npx playwright install --with-deps ${{ inputs.browsers-to-install }} - echo "::endgroup::" - } + - run: + echo "::group::npx playwright install --with-deps"; + npx playwright install --with-deps ${{ inputs.browsers-to-install }}; + echo "::endgroup::"; shell: bash - name: Run tests if: inputs.shell == 'bash' @@ -84,12 +78,10 @@ runs: client-id: ${{ inputs.flakiness-client-id }} tenant-id: ${{ inputs.flakiness-tenant-id }} subscription-id: ${{ inputs.flakiness-subscription-id }} - - run: | - { - echo "::group::./utils/upload_flakiness_dashboard.sh" - ./utils/upload_flakiness_dashboard.sh ./test-results/report.json - echo "::endgroup::" - } + - run: + echo "::group::./utils/upload_flakiness_dashboard.sh"; + ./utils/upload_flakiness_dashboard.sh ./test-results/report.json; + echo "::endgroup::"; if: ${{ !cancelled() }} shell: bash - name: Upload blob report diff --git a/tests/page/page-leaks.spec.ts b/tests/page/page-leaks.spec.ts index f243f68634..abd33e651e 100644 --- a/tests/page/page-leaks.spec.ts +++ b/tests/page/page-leaks.spec.ts @@ -80,7 +80,6 @@ test.afterEach(() => { }); test('click should not leak', async ({ page, toImpl }) => { - const start = Date.now(); await page.setContent(` @@ -103,7 +102,6 @@ test('click should not leak', async ({ page, toImpl }) => { expect(leakedJSHandles()).toBeFalsy(); await checkWeakRefs(toImpl(page), 2, 25); - console.log('click should not leak', Date.now() - start); }); test('fill should not leak', async ({ page, mode, toImpl }) => {