Skip to content

Commit e646785

Browse files
authored
test: better code coverage reports (#164)
istanbul wanted a `sourceRoot` property to map back the sourceMaps with. This makes `nyc merge` work. This means that all code coverage can now be merged into a single report. Also remove the lower line coverage and change the process chaining to `&&` so that CI will _always_ fail.
1 parent 159feee commit e646785

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

karma.conf.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ module.exports = function (config) {
2929
options: {
3030
configFile: 'tsconfig.module.json',
3131
compilerOptions: {
32-
rootDir: './'
32+
sourceRoot: 'modules/'
3333
}
3434
}
3535
}

package.json

+4-3
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,10 @@
1414
"build": "run-s build-*",
1515
"mocha": "mocha --require source-map-support/register --require ts-node/register --exclude 'modules/*-+(browser|backend)/test/*test.ts' modules/**/test/*test.ts",
1616
"karma": "karma start karma.conf.js",
17-
"coverage-browser": "npm run karma && nyc report --lines 70 -t .karma_output --exclude-after-remap false --check-coverage",
18-
"coverage-node": "nyc --lines 70 --require ts-node/register --check-coverage --instrument -e .ts --all -n 'modules/**/src/*.ts' -x 'modules/**/test/*.ts' -x 'modules/*-+(browser|backend)/**/*.ts' npm run mocha",
19-
"coverage": "run-s coverage-*",
17+
"coverage-browser": "npm run karma && nyc report -t .karma_output --check-coverage --extension .ts",
18+
"coverage-node": "nyc --require ts-node/register --instrument --extension .ts --all --check-coverage -n 'modules/**/src/*.ts' -x 'modules/**/test/*.ts' -x 'modules/*-+(browser|backend)/**/*.ts' npm run mocha",
19+
"coverage": "run-s coverage-* && nyc merge .karma_output .nyc_output/browser.json && nyc report --extension .ts --check-coverage",
20+
"report-coverage": "nyc report --extension .ts --check-coverage -r html",
2021
"test": "npm run lint && npm run build && npm run coverage",
2122
"integration-browser-decrypt": "npm run build; lerna run build_fixtures --stream --no-prefix -- -- decrypt -v $npm_package_config_localTestVectors --karma",
2223
"integration-browser-encrypt": "npm run build; lerna run build_fixtures --stream --no-prefix -- -- encrypt -m $npm_package_config_encryptManifestList -k $npm_package_config_encryptKeyManifest -o $npm_package_config_decryptOracle --karma",

0 commit comments

Comments
 (0)