Skip to content

Commit 69298d3

Browse files
committed
test: formatting skip messages for TAP parsing
This patch makes the skip messages consistent so that the TAP plugin in CI can parse the messages properly. The format will be 1..0 # Skipped: [Actual reason why the test is skipped] PR-URL: #2109 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
1 parent 6b85d5a commit 69298d3

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+62
-62
lines changed

test/parallel/test-cluster-bind-privileged-port.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ var cluster = require('cluster');
55
var net = require('net');
66

77
if (process.platform === 'win32') {
8-
console.log('Skipping test, not reliable on Windows.');
8+
console.log('1..0 # Skipped: not reliable on Windows.');
99
process.exit(0);
1010
}
1111

test/parallel/test-cluster-disconnect-unshared-udp.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22
if (process.platform === 'win32') {
3-
console.log('skipping test on windows, where clustered dgram is ENOTSUP');
3+
console.log('1..0 # Skipped: on windows, because clustered dgram is ENOTSUP');
44
process.exit(0);
55
}
66

test/parallel/test-cluster-shared-handle-bind-privileged-port.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,12 @@ var cluster = require('cluster');
55
var net = require('net');
66

77
if (process.platform === 'win32') {
8-
console.log('Skipping test, not reliable on Windows.');
8+
console.log('1..0 # Skipped: not reliable on Windows');
99
process.exit(0);
1010
}
1111

1212
if (process.getuid() === 0) {
13-
console.log('Do not run this test as root.');
13+
console.log('1..0 # Skipped: as this test should not be run as `root`');
1414
process.exit(0);
1515
}
1616

test/parallel/test-crypto-authenticated.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ for (var i in TEST_CASES) {
5656
var test = TEST_CASES[i];
5757

5858
if (ciphers.indexOf(test.algo) == -1) {
59-
console.log('skipping unsupported ' + test.algo + ' test');
59+
console.log('1..0 # Skipped: unsupported ' + test.algo + ' test');
6060
continue;
6161
}
6262

test/parallel/test-dgram-bind-default-address.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ dgram.createSocket('udp4').bind(common.PORT + 0, common.mustCall(function() {
1616
}));
1717

1818
if (!common.hasIPv6) {
19-
console.error('Skipping udp6 part of test, no IPv6 support');
19+
console.log('1..0 # Skipped: udp6 part of test, because no IPv6 support');
2020
return;
2121
}
2222

test/parallel/test-dgram-empty-packet.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ var client;
99
var timer;
1010

1111
if (process.platform === 'darwin') {
12-
console.error('Test is disabled due to 17894467 Apple bug');
12+
console.log('1..0 # Skipped: because of 17894467 Apple bug');
1313
return;
1414
}
1515

test/parallel/test-dgram-send-empty-buffer.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ var callbacks = 0;
88
var client, timer, buf;
99

1010
if (process.platform === 'darwin') {
11-
console.error('Test is disabled due to 17894467 Apple bug');
11+
console.log('1..0 # Skipped: because of 17894467 Apple bug');
1212
return;
1313
}
1414

test/parallel/test-fs-readfile-error.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ var path = require('path');
77
// `fs.readFile('/')` does not fail on FreeBSD, because you can open and read
88
// the directory there.
99
if (process.platform === 'freebsd') {
10-
console.error('Skipping test, platform not supported.');
10+
console.log('1..0 # Skipped: platform not supported.');
1111
process.exit();
1212
}
1313

test/parallel/test-fs-readfile-pipe-large.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ var path = require('path');
66
// simulate `cat readfile.js | node readfile.js`
77

88
if (process.platform === 'win32') {
9-
console.error('No /dev/stdin on windows. Skipping test.');
9+
console.log('1..0 # Skipped: No /dev/stdin on windows.');
1010
process.exit();
1111
}
1212

test/parallel/test-fs-readfile-pipe.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ var assert = require('assert');
55
// simulate `cat readfile.js | node readfile.js`
66

77
if (process.platform === 'win32') {
8-
console.error('No /dev/stdin on windows. Skipping test.');
8+
console.log('1..0 # Skipped: No /dev/stdin on windows.');
99
process.exit();
1010
}
1111

0 commit comments

Comments
 (0)