Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(esm): make sure import from './foo.js' is supported #13137

Merged
merged 1 commit into from Mar 28, 2022

Conversation

dgozman
Copy link
Contributor

@dgozman dgozman commented Mar 28, 2022

Drive-by: migrate all @esm tests to esm.spec.ts.

References #13078.

@dgozman dgozman changed the title test: move @esm tests to a single file fix(esm): make sure import from './foo.js' is supported Mar 28, 2022
Drive-by: migrate all @esm tests to esm.spec.ts.
@dgozman dgozman merged commit 9420a53 into microsoft:main Mar 28, 2022
dgozman added a commit to dgozman/playwright that referenced this pull request Mar 29, 2022
…js' is supported

Drive-by: migrate all @esm tests to esm.spec.ts.
dgozman added a commit that referenced this pull request Mar 29, 2022
…pported (#13160)

Drive-by: migrate all @esm tests to esm.spec.ts.
@frederikhors
Copy link

I'm having this issue using both 1.20.1 and next:

image

Only a VSCode issue. The tests are working.

If I use ./constants (without .ts) I get error in pw at runtime: [ERR_MODULE_NOT_FOUND] using PW_EXPERIMENTAL_TS_ESM=1 playwright test.

How can I fix?

liuxingbaoyu pushed a commit to liuxingbaoyu/playwright that referenced this pull request Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants