Skip to content

Commit

Permalink
test: add test
Browse files Browse the repository at this point in the history
  • Loading branch information
hi-ogawa committed Oct 30, 2024
1 parent 9fec89c commit 199d958
Show file tree
Hide file tree
Showing 7 changed files with 26 additions and 14 deletions.
12 changes: 6 additions & 6 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

File renamed without changes.
1 change: 1 addition & 0 deletions test/core/deps/dep-cjs/esm-string.js
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
module.exports = { test: ' import.meta' }
8 changes: 8 additions & 0 deletions test/core/deps/dep-cjs/package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{
"name": "@vitest/test-dep-cjs",
"type": "commonjs",
"exports": {
"./esm-comment": "./esm-comment.js",
"./esm-string": "./esm-string.js"
}
}
5 changes: 0 additions & 5 deletions test/core/deps/dep-esm-comment/package.json

This file was deleted.

2 changes: 1 addition & 1 deletion test/core/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"@vitest/expect": "workspace:*",
"@vitest/mocker": "workspace:*",
"@vitest/runner": "workspace:*",
"@vitest/test-dep-esm-comment": "file:./deps/dep-esm-comment",
"@vitest/test-dep-cjs": "file:./deps/dep-cjs",
"@vitest/test-dep1": "file:./deps/dep1",
"@vitest/test-dep2": "file:./deps/dep2",
"@vitest/utils": "workspace:*",
Expand Down
12 changes: 10 additions & 2 deletions test/core/test/dual-package-hazard.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@ import * as dep1 from '@vitest/test-dep1'
import * as dep2 from '@vitest/test-dep2'

// @ts-expect-error no ts
import depEsmComment from '@vitest/test-dep-esm-comment'
import depEsmComment from '@vitest/test-dep-cjs/esm-comment'

// @ts-expect-error no ts
import depEsmString from '@vitest/test-dep-cjs/esm-string'

const require = createRequire(import.meta.url)

Expand All @@ -18,6 +21,11 @@ test('no dual package hazard by externalizing esm deps by default', async () =>
})

test('externalize cjs with esm comment', async () => {
const depEsmCommentRequire = require('@vitest/test-dep-esm-comment')
const depEsmCommentRequire = require('@vitest/test-dep-cjs/esm-comment')
expect(depEsmComment).toBe(depEsmCommentRequire)
})

test('externalize cjs with esm string', async () => {
const depEsmStringRequire = require('@vitest/test-dep-cjs/esm-string')
expect(depEsmString).toBe(depEsmStringRequire)
})

0 comments on commit 199d958

Please sign in to comment.