Skip to content

Commit 92fda43

Browse files
committed
test: enforce full coverage
Also, get Windows up to full coverage. PR-URL: #1598 Credit: @isaacs Close: #1598 Reviewed-by: @ruyadorno
1 parent 5c4d192 commit 92fda43

File tree

4 files changed

+4
-1
lines changed

4 files changed

+4
-1
lines changed

lib/utils/error-message.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ module.exports = (er) => {
9090
if (er.file === resolve(npm.prefix, 'package.json')) {
9191
const { isDiff } = require('parse-conflict-json')
9292
const txt = require('fs').readFileSync(er.file, 'utf8')
93+
.replace(/\r\n/g, '\n')
9394
if (isDiff(txt)) {
9495
detail.push([
9596
'',

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -231,6 +231,7 @@
231231
"tap": {
232232
"files": "test/{lib,bin}",
233233
"coverage-map": "test/coverage-map.js",
234+
"check-coverage": true,
234235
"esm": false,
235236
"timeout": 600
236237
},

test/lib/npm.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,7 @@ t.test('npm.load', t => {
129129
])
130130
logs.length = 0
131131

132+
bePosix()
132133
t.equal(resolve(npm.cache), resolve(CACHE), 'cache is cache')
133134
const newCache = t.testdir()
134135
npm.cache = newCache

test/lib/utils/error-message.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,7 @@ t.test('json parse', t => {
282282
process.argv = ['arg', 'v']
283283
t.matchSnapshot(errorMessage(Object.assign(new Error('not json'), {
284284
code: 'EJSONPARSE',
285-
file: `${dir}/package.json`
285+
file: resolve(dir, 'package.json')
286286
})))
287287
t.end()
288288
})

0 commit comments

Comments
 (0)