Skip to content

Commit 2f5440d

Browse files
chore: apply Neal's suggestions
Co-authored-by: Neal Beeken <[email protected]>
1 parent dc584d1 commit 2f5440d

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

test/integration/load-balancers/load_balancers.spec.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const filter = ({ description }) => {
99
// a getMore before the killCursors even though the stream is immediately
1010
// closed.
1111
// TODO(NODE-3970): implement and reference a node specific integration test for this
12-
return 'TODO(NODE-3970)';
12+
return 'TODO(NODE-3970): implement and reference a node specific integration test for this';
1313
}
1414

1515
if (
@@ -22,7 +22,7 @@ const filter = ({ description }) => {
2222
// TODO(DRIVERS-1847): The following three tests are skipped pending a decision made on DRIVERS-1847,
2323
// since pinning the connection on any getMore error is very awkward in node and likely results
2424
// in sub-optimal pinning.
25-
return 'TODO(DRIVERS-1847)';
25+
return 'TODO(DRIVERS-1847): Skipped pending a decision made on DRIVERS-1847'
2626
}
2727

2828
if (description === 'errors during the initial connection hello are ignored') {

test/integration/unified-test-format/unified_test_format.spec.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,21 @@ const filter: TestFilter = ({ description }) => {
66
if (description === 'unpin after transient error within a transaction and commit') {
77
// OLD COMMENT: commitTransaction retry seems to be swallowed by mongos in this case
88
// TODO(NODE-3943):
9-
return `TODO(NODE-3943)`;
9+
return `TODO(NODE-3943): commitTransaction retry seems to be swallowed by mongos in this case`;
1010
}
1111

1212
if (description === 'Client side error in command starting transaction') {
1313
// TODO(NODE-2034): Will be implemented as part of NODE-2034
14-
return 'TODO(NODE-2034)';
14+
return 'TODO(NODE-2034): Specify effect of client-side errors on in-progress transactions';
1515
}
1616

1717
if (description === 'Dirty explicit session is discarded') {
1818
// TODO(NODE-3951): investigate why this is failing while the legacy version is passing
19-
return 'TODO(NODE-3951)';
19+
return 'TODO(NODE-3951): investigate why this is failing while the legacy version is passing';
2020
}
2121

2222
if (description === 'A successful find event with a getmore and the server kills the cursor') {
23-
return 'TODO(NODE-3308)';
23+
return 'TODO(NODE-3308): failures due unnecessary getMore and killCursors calls in 5.0';
2424
}
2525

2626
if (

0 commit comments

Comments
 (0)