Skip to content

Commit d039c21

Browse files
fix lint and add ticket to skip reason
1 parent fd3afe8 commit d039c21

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

test/unit/assorted/polling_srv_records_for_mongos_discovery.prose.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,12 +46,13 @@ interface ShardedClusterMocks {
4646
// TODO(NODE-3773): Implement tests 6-9
4747
describe('Polling Srv Records for Mongos Discovery', () => {
4848
beforeEach(function () {
49+
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
4950
const test = this.currentTest!;
5051

5152
const { major } = coerce(process.version);
5253
test.skipReason =
5354
major === 18 || major === 20
54-
? 'TODO(NODE-xxxx): fix failing unit tests on Node18'
55+
? 'TODO(NODE-5666): fix failing unit tests on Node18'
5556
: undefined;
5657

5758
if (test.skipReason) this.skip();

test/unit/connection_string.spec.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ describe('Connection String spec tests', function () {
1515
const suites = loadSpecTests('connection-string');
1616

1717
beforeEach(function () {
18+
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
1819
const test = this.currentTest!;
1920

2021
const { major } = coerce(process.version);
@@ -24,7 +25,7 @@ describe('Connection String spec tests', function () {
2425
];
2526
test.skipReason =
2627
major === 20 && skippedTests.includes(test.title)
27-
? 'TODO(NODE-xxxx): fix failing unit tests on Node18'
28+
? 'TODO(NODE-5666): fix failing unit tests on Node18'
2829
: undefined;
2930

3031
if (test.skipReason) this.skip();

test/unit/sdam/monitor.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ describe('monitoring', function () {
3131
let mockServer;
3232

3333
beforeEach(function () {
34+
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
3435
const test = this.currentTest!;
3536

3637
const { major } = coerce(process.version);
@@ -43,7 +44,7 @@ describe('monitoring', function () {
4344
];
4445
test.skipReason =
4546
(major === 18 || major === 20) && failingTests.includes(test.title)
46-
? 'TODO(NODE-xxxx): fix failing unit tests on Node18'
47+
? 'TODO(NODE-5666): fix failing unit tests on Node18'
4748
: undefined;
4849

4950
if (test.skipReason) this.skip();

0 commit comments

Comments
 (0)