From 2b9285c5d0d432e1bcb758101d64f0a37c2282e2 Mon Sep 17 00:00:00 2001 From: Hiroshi Ogawa <4232207+hi-ogawa@users.noreply.github.com> Date: Wed, 21 Oct 2026 18:32:27 +0900 Subject: [PATCH] test: merge retry and repeat failure screenshot tests Co-Authored-By: Claude Opus 5.5 --- test/browser/specs/failure-screenshot.test.ts | 34 ++----------------- 1 file changed, 2 insertions(+), 32 deletions(-) diff --git a/test/browser/specs/failure-screenshot.test.ts b/test/browser/specs/failure-screenshot.test.ts index b8afbd695..93bec9035 100644 --- a/test/browser/specs/failure-screenshot.test.ts +++ b/test/browser/specs/failure-screenshot.test.ts @@ -108,7 +108,7 @@ describe('failure screenshots', () => { }) }) - describe('retried test', () => { + describe('retry and repeats', () => { function summarize(stderr: string, root: string) { return stderr .split('\n') @@ -122,37 +122,7 @@ describe('failure screenshots', () => { return paths.map((path) => readFileSync(resolve(path)).toString('base64')) } - test('saves and reports one screenshot per retry', async () => { - const { stderr, root } = await runBrowserTests({ - [testFilename]: /* ts */ ` - import { expect, test } from 'vitest' - import { page } from 'vitest/browser' - - let attempt = 0 - test('retried', { retry: 2 }, async () => { - attempt++ - document.body.innerHTML = \`\` - await page.getByText(\`attempt \${attempt}\`).click() - expect(attempt).toBe(10) - }) - `, - }) - - expect(summarize(stderr, root)).toMatchInlineSnapshot(` - " FAIL |chromium| basic.test.ts > retried - AssertionError: expected 1 to be 10 // Object.is equality - - /.vitest/attachments/failure-screenshots/basic.test.ts/retried-0-0.png - FAIL |chromium| basic.test.ts > retried - AssertionError: expected 2 to be 10 // Object.is equality - - /.vitest/attachments/failure-screenshots/basic.test.ts/retried-0-1.png - FAIL |chromium| basic.test.ts > retried - AssertionError: expected 3 to be 10 // Object.is equality - - /.vitest/attachments/failure-screenshots/basic.test.ts/retried-0-2.png" - `) - expect(new Set(readScreenshots(stderr)).size).toBe(3) - }) - - test('saves and reports one screenshot per repeat', async () => { + test('saves and reports one screenshot per attempt', async () => { const { stderr, root } = await runBrowserTests({ [testFilename]: /* ts */ ` import { expect, test } from 'vitest' -- 2.54.0