cherry-pick(#27103): fix: list tests only once (#27107)

Fixes #27087
microsoft · Sep 15, 2023 · 476b74f · 476b74f
1 parent 0861364
commit 476b74f
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/packages/playwright/src/runner/reporters.ts b/packages/playwright/src/runner/reporters.ts
@@ -104,4 +104,8 @@ class ListModeReporter extends EmptyReporter {
     // eslint-disable-next-line no-console
     console.error('\n' + formatError(error, false).message);
   }
+
+  override printsToStdio(): boolean {
+    return true;
+  }
 }
diff --git a/tests/playwright-test/list-mode.spec.ts b/tests/playwright-test/list-mode.spec.ts
@@ -196,3 +196,21 @@ test('should report errors with location', async ({ runInlineTest }) => {
     column: 9,
   });
 });
+
+test('should list tests once', async ({ runInlineTest }) => {
+  test.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/27087' });
+  const result = await runInlineTest({
+    'playwright.config.ts': `
+      module.exports = { };
+    `,
+    'a.test.js': `
+      const { test, expect } = require('@playwright/test');
+      test('test 1', ({}) => {});
+    `
+  }, { 'list': true });
+  expect(result.exitCode).toBe(0);
+  expect(result.output).toEqual(`Listing tests:
+  a.test.js:3:7 › test 1
+Total: 1 test in 1 file
+`);
+});