Skip to content

Commit

Permalink
add missing node-serialize builtin plugin dep
Browse files Browse the repository at this point in the history
Also, add a test to prevent this in the future.

Fix: #914
  • Loading branch information
isaacs committed Sep 23, 2023
1 parent 853dd27 commit b723522
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 0 deletions.
1 change: 1 addition & 0 deletions package-lock.json

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

1 change: 1 addition & 0 deletions src/tap/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@
"@tapjs/fixture": "1.0.4",
"@tapjs/intercept": "1.1.0",
"@tapjs/mock": "1.0.5",
"@tapjs/node-serialize": "1.0.0",
"@tapjs/run": "1.1.0",
"@tapjs/snapshot": "1.0.4",
"@tapjs/spawn": "1.0.4",
Expand Down
13 changes: 13 additions & 0 deletions src/tap/test/builtins.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// ensure that all the builtin plugins are deps of the tap pkg
import t from '../dist/esm/index.js'

import { defaultPlugins } from '@tapjs/test'
import { readFileSync } from 'fs'
import { fileURLToPath } from 'url'

const pj = fileURLToPath(new URL('../package.json', import.meta.url))
const { dependencies } = JSON.parse(readFileSync(pj, 'utf8')) as {
dependencies: Record<string, string>
}

t.match(new Set(Object.keys(dependencies)), new Set(defaultPlugins))

0 comments on commit b723522

Please sign in to comment.