-
Notifications
You must be signed in to change notification settings - Fork 14
Open
Description
Due to the fact that extension added --test-reporter without --test-reporter-destination here
nodejs-testing/src/runner-worker.ts
Line 130 in ee39d46
| args.push("--test-reporter", pathToFileURL(join(__dirname, "runner-loader.js")).toString()); |
I could not add my own reporters
so I have to add one additional --test-reporter-destination flag but it was not outlined nowhere in the documentation.
"nodejs-testing.extensions": [
{
"extensions": [
"mjs",
"cjs",
"js"
],
"parameters": [
"--experimental-require-module",
"--experimental-test-module-mocks",
"--experimental-test-coverage",
"--test-reporter-destination=coverage/lcov.info", // <-- this like a compensation
"--test-reporter-destination=coverage/lcov.info",
"--test-reporter=lcov",
"--test-reporter-destination=stdout",
"--test-reporter=spec"
]
}
],
Also I noted that by default node test runner uses "--experimental-test-snapshots" by default. I think that should be configurable rather than required flag
node --experimental-test-snapshots" "--test-reporter" "file:///.vscode/extensions/connor4312.nodejs-testing-1.7.0/out/runner-loader.js"Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels