Skip to content

Commit

Permalink
test: add trailing commas in test/message
Browse files Browse the repository at this point in the history
PR-URL: #46372
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Geoffrey Booth <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
aduh95 authored and juanarbol committed Mar 5, 2023
1 parent ed564a9 commit f2ebe66
Show file tree
Hide file tree
Showing 10 changed files with 15 additions and 16 deletions.
1 change: 0 additions & 1 deletion test/.eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ overrides:
- internet/*.js
- js-native-api/*/*.js
- known_issues/*.js
- message/*.js
- node-api/*/*.js
- parallel/*.js
- parallel/*.mjs
Expand Down
2 changes: 1 addition & 1 deletion test/message/async_error_eval_cjs.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ main();
'-e',
main,
], {
env: { ...process.env }
env: { ...process.env },
});

if (child.status !== 0) {
Expand Down
2 changes: 1 addition & 1 deletion test/message/async_error_eval_esm.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ main();
'-e',
main,
], {
env: { ...process.env }
env: { ...process.env },
});

if (child.status !== 0) {
Expand Down
2 changes: 1 addition & 1 deletion test/message/console_low_stack_space.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ const consoleDescriptor = Object.getOwnPropertyDescriptor(global, 'console');
Object.defineProperty(global, 'console', {
configurable: true,
writable: true,
value: {}
value: {},
});

require('../common');
Expand Down
4 changes: 2 additions & 2 deletions test/message/test_runner_describe_it.js
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@ it('custom inspect symbol fail', () => {
[util.inspect.custom]() {
return 'customized';
},
foo: 1
foo: 1,
};

throw obj;
Expand All @@ -281,7 +281,7 @@ it('custom inspect symbol that throws fail', () => {
[util.inspect.custom]() {
throw new Error('bad-inspect');
},
foo: 1
foo: 1,
};

throw obj;
Expand Down
4 changes: 2 additions & 2 deletions test/message/test_runner_output.js
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,7 @@ test('custom inspect symbol fail', () => {
[util.inspect.custom]() {
return 'customized';
},
foo: 1
foo: 1,
};

throw obj;
Expand All @@ -314,7 +314,7 @@ test('custom inspect symbol that throws fail', () => {
[util.inspect.custom]() {
throw new Error('bad-inspect');
},
foo: 1
foo: 1,
};

throw obj;
Expand Down
2 changes: 1 addition & 1 deletion test/message/test_runner_test_name_pattern.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ test('top level test enabled', common.mustCall(async (t) => {
t.afterEach(common.mustCall());
await t.test(
'nested test runs because name includes PATTERN',
common.mustCall()
common.mustCall(),
);
}));

Expand Down
6 changes: 3 additions & 3 deletions test/message/util-inspect-error-cause.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,11 @@ const cause5 = new Error('Object cause', {
message: 'Unique',
name: 'Error',
stack: 'Error: Unique\n' +
' at Module._compile (node:internal/modules/cjs/loader:827:30)'
}
' at Module._compile (node:internal/modules/cjs/loader:827:30)',
},
});
const cause6 = new Error('undefined cause', {
cause: undefined
cause: undefined,
});

console.log(cause4);
Expand Down
4 changes: 2 additions & 2 deletions test/message/vm_dont_display_runtime_error.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ console.error('beginning');
try {
vm.runInThisContext('throw new Error("boo!")', {
filename: 'test.vm',
displayErrors: false
displayErrors: false,
});
} catch {
// Continue regardless of error.
Expand All @@ -40,7 +40,7 @@ console.error('middle');

vm.runInThisContext('throw new Error("boo!")', {
filename: 'test.vm',
displayErrors: false
displayErrors: false,
});

console.error('end');
4 changes: 2 additions & 2 deletions test/message/vm_dont_display_syntax_error.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ console.error('beginning');
try {
vm.runInThisContext('var 5;', {
filename: 'test.vm',
displayErrors: false
displayErrors: false,
});
} catch {
// Continue regardless of error.
Expand All @@ -40,7 +40,7 @@ console.error('middle');

vm.runInThisContext('var 5;', {
filename: 'test.vm',
displayErrors: false
displayErrors: false,
});

console.error('end');

0 comments on commit f2ebe66

Please sign in to comment.