Skip to content

Commit 44548b7

Browse files
authored
chore: update dependencies (#1045)
Most notably `tap`.
1 parent 92ee01f commit 44548b7

File tree

4 files changed

+53
-26
lines changed

4 files changed

+53
-26
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,6 @@ node_modules/
22
test/scaffold/*/node_modules
33
test/.*
44
coverage/
5+
/.tap
56
.nyc_output/
67
.eslintcache

package.json

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
],
1717
"scripts": {
1818
"lint": "eslint bin/* lib/ test/ --cache",
19-
"tap": "c8 --reporter lcov tap --no-coverage \"test/**/test-*.js\"",
19+
"tap": "tap run \"test/**/test-*.js\"",
2020
"test": "npm run lint && npm run tap"
2121
},
2222
"author": "James M Snell <jasnell@gmail.com> (http://jasnell.me)",
@@ -31,40 +31,39 @@
3131
"license": "MIT",
3232
"dependencies": {
3333
"async": "^3.2.5",
34-
"bl": "^6.0.9",
34+
"bl": "^6.0.10",
3535
"chalk": "^5.3.0",
3636
"columnify": "^1.6.0",
3737
"execa": "^8.0.1",
3838
"lodash": "^4.17.21",
3939
"normalize-git-url": "^3.0.2",
4040
"npm-package-arg": "^11.0.1",
4141
"npm-which": "^3.0.1",
42-
"pnpm": "^8.13.1",
42+
"pnpm": "^8.14.0",
4343
"read-package-json": "^7.0.0",
4444
"root-check": "^2.0.0",
4545
"semver": "^7.5.4",
4646
"strip-ansi": "^7.1.0",
4747
"supports-color": "^9.4.0",
4848
"tar": "^6.2.0",
4949
"uid-number": "0.0.6",
50-
"undici": "^6.2.1",
50+
"undici": "^6.3.0",
5151
"which": "^4.0.0",
5252
"winston": "^3.11.0",
53-
"xml-sanitizer": "^1.2.9",
53+
"xml-sanitizer": "^2.0.0",
5454
"xmlbuilder": "^15.1.1",
5555
"yargs": "^17.7.2",
5656
"yarn": "^1.22.21"
5757
},
5858
"devDependencies": {
5959
"ansi-regex": "^6.0.1",
60-
"c8": "^8.0.1",
6160
"eslint": "^8.56.0",
6261
"eslint-config-prettier": "^9.1.0",
63-
"eslint-plugin-prettier": "^5.1.2",
62+
"eslint-plugin-prettier": "^5.1.3",
6463
"prettier": "^3.1.1",
6564
"string-to-stream": "^3.0.1",
66-
"tap": "^16.3.8",
67-
"tap-parser": "^11.0.1",
65+
"tap": "^18.6.1",
66+
"tap-parser": "^15.3.1",
6867
"xml2js": "^0.6.2"
6968
},
7069
"prettier": {
@@ -82,7 +81,7 @@
8281
},
8382
"tap": {
8483
"timeout": 480,
85-
"check-coverage": false
84+
"allow-incomplete-coverage": true
8685
},
8786
"engines": {
8887
"node": "18.x || 20.x || 21.x"

test/fixtures/parsed-tap.json

Lines changed: 38 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,52 @@
11
{
2-
"bailout": false,
2+
"ok": false,
33
"count": 3,
4+
"pass": 2,
45
"fail": 1,
6+
"bailout": false,
7+
"todo": 0,
8+
"skip": 1,
59
"failures": [
610
{
7-
"diag": {
8-
"duration_ms": 50
9-
},
10-
"id": 3,
11+
"ok": false,
1112
"name": "iFail v3.0.1 I dun wurk",
12-
"fullname": "",
13-
"ok": false
13+
"id": 3,
14+
"buffered": false,
15+
"tapError": null,
16+
"skip": false,
17+
"todo": false,
18+
"previous": null,
19+
"plan": null,
20+
"diag": {},
21+
"time": 50,
22+
"fullname": "iFail v3.0.1 I dun wurk",
23+
"closingTestPoint": false
1424
}
1525
],
16-
"ok": false,
17-
"pass": 2,
26+
"time": null,
1827
"plan": {
19-
"comment": "",
2028
"start": 1,
29+
"end": 3,
2130
"skipAll": false,
2231
"skipReason": "",
23-
"end": 3
32+
"comment": ""
2433
},
25-
"skip": 1,
26-
"todo": 0,
27-
"time": null
34+
"skips": [
35+
{
36+
"ok": true,
37+
"name": "iFlakyFail v3.3.3",
38+
"id": 2,
39+
"buffered": false,
40+
"tapError": null,
41+
"skip": "I dun wurk",
42+
"todo": false,
43+
"previous": null,
44+
"plan": null,
45+
"diag": {},
46+
"time": 50,
47+
"fullname": "iFlakyFail v3.3.3",
48+
"closingTestPoint": false
49+
}
50+
],
51+
"todos": []
2852
}

test/reporter/test-reporter-tap.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { tmpdir } from 'os';
66
import { fileURLToPath } from 'url';
77

88
import tap from 'tap';
9-
import Parser from 'tap-parser';
9+
import { Parser } from 'tap-parser';
1010
import str from 'string-to-stream';
1111

1212
import tapReporter from '../../lib/reporter/tap.js';
@@ -92,7 +92,10 @@ test('reporter.tap(): parser', (t) => {
9292

9393
tapReporter(logger, failingInput);
9494
const p = new Parser((results) => {
95-
t.same(results, tapParserExpected),
95+
// `results` contains JS classes that are not considered same as the
96+
// plain objects loaded from the JSON file.
97+
const plainResults = JSON.parse(JSON.stringify(results));
98+
t.same(plainResults, tapParserExpected),
9699
'the tap parser should correctly' + ' parse the tap file';
97100
t.end();
98101
});

0 commit comments

Comments
 (0)