review feedback

This commit is contained in:
Max Schmitt 2024-09-19 16:28:16 +02:00
parent c83ba0c8d4
commit 0dccf16658
2 changed files with 17 additions and 37 deletions

View file

@ -151,7 +151,8 @@ test('should not transform external', async ({ runInlineTest }) => {
expect(result.output).toMatch(/(Cannot use import statement outside a module|require\(\) of ES Module .* not supported.)/); expect(result.output).toMatch(/(Cannot use import statement outside a module|require\(\) of ES Module .* not supported.)/);
}); });
test('should support import assertions', { for (const type of ['module', undefined]) {
test(`should support import assertions with type=${type} in the package.json`, {
annotation: { annotation: {
type: 'issue', type: 'issue',
description: 'https://github.com/microsoft/playwright/issues/32659' description: 'https://github.com/microsoft/playwright/issues/32659'
@ -163,8 +164,8 @@ test('should support import assertions', {
console.log('imported value: ' + packageJSON.foo); console.log('imported value: ' + packageJSON.foo);
export default { }; export default { };
`, `,
'package.json': JSON.stringify({ foo: 'bar' }), 'package.json': JSON.stringify({ foo: 'bar', type }),
'a.esm.test.ts': ` 'a.test.ts': `
import { test, expect } from '@playwright/test'; import { test, expect } from '@playwright/test';
test('check project name', ({}, testInfo) => { test('check project name', ({}, testInfo) => {
@ -177,3 +178,4 @@ test('should support import assertions', {
expect(result.passed).toBe(1); expect(result.passed).toBe(1);
expect(result.stdout).toContain('imported value: bar'); expect(result.stdout).toContain('imported value: bar');
}); });
}

View file

@ -35,28 +35,6 @@ test('should load nested as esm when package.json has type module', async ({ run
expect(result.passed).toBe(1); 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 }) => { test('should support import attributes', async ({ runInlineTest }) => {
const result = await runInlineTest({ const result = await runInlineTest({
'playwright.config.ts': ` 'playwright.config.ts': `