From 0dccf1665882708087d9b91823b805ee579d04fd Mon Sep 17 00:00:00 2001
From: Max Schmitt <max@schmitt.mx>
Date: Thu, 19 Sep 2024 16:28:16 +0200
Subject: [PATCH] review feedback

---
 tests/playwright-test/babel.spec.ts | 32 +++++++++++++++--------------
 tests/playwright-test/esm.spec.ts   | 22 --------------------
 2 files changed, 17 insertions(+), 37 deletions(-)

diff --git a/tests/playwright-test/babel.spec.ts b/tests/playwright-test/babel.spec.ts
index eae3d62e65..517d813bc8 100644
--- a/tests/playwright-test/babel.spec.ts
+++ b/tests/playwright-test/babel.spec.ts
@@ -151,29 +151,31 @@ test('should not transform external', async ({ runInlineTest }) => {
   expect(result.output).toMatch(/(Cannot use import statement outside a module|require\(\) of ES Module .* not supported.)/);
 });
 
-test('should support import assertions', {
-  annotation: {
-    type: 'issue',
-    description: 'https://github.com/microsoft/playwright/issues/32659'
-  }
-}, async ({ runInlineTest }) => {
-  const result = await runInlineTest({
-    'playwright.config.ts': `
+for (const type of ['module', undefined]) {
+  test(`should support import assertions with type=${type} in the package.json`, {
+    annotation: {
+      type: 'issue',
+      description: 'https://github.com/microsoft/playwright/issues/32659'
+    }
+  }, async ({ runInlineTest }) => {
+    const result = await runInlineTest({
+      'playwright.config.ts': `
       import packageJSON from './package.json' assert { type: 'json' };
       console.log('imported value: ' + packageJSON.foo);
       export default { };
     `,
-    'package.json': JSON.stringify({ foo: 'bar' }),
-    'a.esm.test.ts': `
+      'package.json': JSON.stringify({ foo: 'bar', type }),
+      'a.test.ts': `
       import { test, expect } from '@playwright/test';
 
       test('check project name', ({}, testInfo) => {
         expect(1).toBe(1);
       });
     `
-  });
+    });
 
-  expect(result.exitCode).toBe(0);
-  expect(result.passed).toBe(1);
-  expect(result.stdout).toContain('imported value: bar');
-});
\ No newline at end of file
+    expect(result.exitCode).toBe(0);
+    expect(result.passed).toBe(1);
+    expect(result.stdout).toContain('imported value: bar');
+  });
+}
diff --git a/tests/playwright-test/esm.spec.ts b/tests/playwright-test/esm.spec.ts
index 1aa89000e5..ce75fca767 100644
--- a/tests/playwright-test/esm.spec.ts
+++ b/tests/playwright-test/esm.spec.ts
@@ -35,28 +35,6 @@ test('should load nested as esm when package.json has type module', async ({ run
   expect(result.passed).toBe(1);
 });
 
-test('should support import assertions', async ({ runInlineTest }) => {
-  const result = await runInlineTest({
-    'playwright.config.ts': `
-      import packageJSON from './package.json' assert { type: 'json' };
-      console.log('imported value: ' + packageJSON.foo);
-      export default { };
-    `,
-    'package.json': JSON.stringify({ type: 'module', foo: 'bar' }),
-    'a.esm.test.ts': `
-      import { test, expect } from '@playwright/test';
-
-      test('check project name', ({}, testInfo) => {
-        expect(1).toBe(1);
-      });
-    `
-  });
-
-  expect(result.exitCode).toBe(0);
-  expect(result.passed).toBe(1);
-  expect(result.stdout).toContain('imported value: bar');
-});
-
 test('should support import attributes', async ({ runInlineTest }) => {
   const result = await runInlineTest({
     'playwright.config.ts': `