devops: properly print group boundaries in GitHub actions

This commit is contained in:
Yury Semikhatsky 2025-02-05 11:55:34 -08:00
parent 4b64c47a25
commit 707c64d4b0
2 changed files with 22 additions and 12 deletions

View file

@ -37,22 +37,28 @@ runs:
node-version: ${{ inputs.node-version }} node-version: ${{ inputs.node-version }}
- uses: ./.github/actions/enable-microphone-access - uses: ./.github/actions/enable-microphone-access
- run: | - run: |
echo "::group::npm ci" {
npm ci echo "::group::npm ci"
echo "::endgroup::" npm ci
echo "::endgroup::"
}
shell: bash shell: bash
env: env:
DEBUG: pw:install DEBUG: pw:install
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: '1' PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: '1'
- run: | - run: |
echo "::group::npm run build" {
npm run build echo "::group::npm run build"
echo "::endgroup::" npm run build
echo "::endgroup::"
}
shell: bash shell: bash
- run: | - run: |
echo "::group::npx playwright install --with-deps" {
npx playwright install --with-deps ${{ inputs.browsers-to-install }} echo "::group::npx playwright install --with-deps"
echo "::endgroup::" npx playwright install --with-deps ${{ inputs.browsers-to-install }}
echo "::endgroup::"
}
shell: bash shell: bash
- name: Run tests - name: Run tests
if: inputs.shell == 'bash' if: inputs.shell == 'bash'
@ -79,9 +85,11 @@ runs:
tenant-id: ${{ inputs.flakiness-tenant-id }} tenant-id: ${{ inputs.flakiness-tenant-id }}
subscription-id: ${{ inputs.flakiness-subscription-id }} subscription-id: ${{ inputs.flakiness-subscription-id }}
- run: | - run: |
echo "::group::./utils/upload_flakiness_dashboard.sh" {
./utils/upload_flakiness_dashboard.sh ./test-results/report.json echo "::group::./utils/upload_flakiness_dashboard.sh"
echo "::endgroup::" ./utils/upload_flakiness_dashboard.sh ./test-results/report.json
echo "::endgroup::"
}
if: ${{ !cancelled() }} if: ${{ !cancelled() }}
shell: bash shell: bash
- name: Upload blob report - name: Upload blob report

View file

@ -80,6 +80,7 @@ test.afterEach(() => {
}); });
test('click should not leak', async ({ page, toImpl }) => { test('click should not leak', async ({ page, toImpl }) => {
const start = Date.now();
await page.setContent(` await page.setContent(`
<button>static button 1</button> <button>static button 1</button>
<button>static button 2</button> <button>static button 2</button>
@ -102,6 +103,7 @@ test('click should not leak', async ({ page, toImpl }) => {
expect(leakedJSHandles()).toBeFalsy(); expect(leakedJSHandles()).toBeFalsy();
await checkWeakRefs(toImpl(page), 2, 25); await checkWeakRefs(toImpl(page), 2, 25);
console.log('click should not leak', Date.now() - start);
}); });
test('fill should not leak', async ({ page, mode, toImpl }) => { test('fill should not leak', async ({ page, mode, toImpl }) => {