mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/Popups
synced 2024-11-14 11:15:08 +00:00
95047ba360
Replace Istanbul with nyc, Istanbul's CLI. nyc appears to have some
bugs that this patch works around:
- When all files in the project are considered, not just those
imported via tests, the coverage rates and line numbers vary between
runs. This patch disables the `all` option for now and points to the
bug:
https://github.com/istanbuljs/nyc/issues/537#issuecomment-390814662.
- Source map line numbers appear to be incorrect except when `all` is
enabled and working correctly (see previous bullet).
- `sourceMap` must be disabled to avoid ENAMETOOLONG errors when nyc
tries to include them as encoded strings. The patch disables the
setting and points to: https://github.com/istanbuljs/nyc/issues/847.
Using babel-plugin-istanbul and source-map-support appears to have
no effect (the former in tests/node-qunit/run.js and .babelrc).
- CI fails with
`Error: EACCES: permission denied, mkdir '/nonexistent'`. Specify
`SPAWN_WRAP_SHIM_ROOT` instead of constructing a subdirectory from a
nonexistent home directory.
Bug: T196952
Bug: T193519
Change-Id: Idf2e3accd4a6277cbef91c1156fcd206c9e7d882
32 lines
1 KiB
JSON
32 lines
1 KiB
JSON
{
|
|
"//": "todo: enable caching. There are too many other issues to verify that",
|
|
"//": "caching works correctly.",
|
|
"cache": false,
|
|
|
|
"//": "todo: check coverage on all files not just those included in tests.",
|
|
"//": "Enable when",
|
|
"//": "https://github.com/istanbuljs/nyc/issues/537#issuecomment-390814662",
|
|
"//": "is fixed.",
|
|
"all": false,
|
|
|
|
"//": "Ignore files in vendor/, resources/, and elsewhere.",
|
|
"include": [ "src/**/*.js" ],
|
|
|
|
"//": "Set the coverage percentage by category thresholds.",
|
|
"statements": 90,
|
|
"branches": 85,
|
|
"functions": 85,
|
|
"lines": 90,
|
|
|
|
"//": "Fail if the coverage is below threshold.",
|
|
"check-coverage": true,
|
|
|
|
"//": "Work around source maps being included ",
|
|
"//": "https://github.com/istanbuljs/nyc/issues/847:",
|
|
"//": " Error: ENAMETOOLONG: name too long, open '.../vagrant/mediawiki/extensions/Popups/src/data:application/json;...'",
|
|
"//": "Unfortunately, the reported line numbers appear to be",
|
|
"//": "nondeterministic across runs when all is enabled and incorrect when",
|
|
"//": "disabled.",
|
|
"sourceMap": false
|
|
}
|