Skip to content

Commit 18c24fb

Browse files
committed
test(NODE-6862): fix flaky CSOT timeout spec tests
1 parent b2511f0 commit 18c24fb

File tree

3 files changed

+13
-7
lines changed

3 files changed

+13
-7
lines changed

.evergreen/run-tests.sh

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,4 +58,7 @@ export MONGODB_URI=${MONGODB_URI}
5858
export LOAD_BALANCER=${LOAD_BALANCER}
5959
export TEST_CSFLE=${TEST_CSFLE}
6060
export COMPRESSOR=${COMPRESSOR}
61-
npm run check:integration-coverage
61+
62+
for run in {1..100}; do
63+
npm run check:integration-coverage
64+
done

src/cmap/connection.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -316,6 +316,7 @@ export class Connection extends TypedEventEmitter<ConnectionEvents> {
316316
this.delayedTimeoutId = setTimeout(() => {
317317
const message = `connection ${this.id} to ${this.address} timed out`;
318318
const beforeHandshake = this.hello == null;
319+
console.log(this.socketTimeoutMS, this.hello, this.socket);
319320
this.cleanup(new MongoNetworkTimeoutError(message, { beforeHandshake }));
320321
}, 1).unref(); // No need for this timer to hold the event loop open
321322
}
@@ -449,6 +450,12 @@ export class Connection extends TypedEventEmitter<ConnectionEvents> {
449450
options.socketTimeoutMS ??
450451
options?.timeoutContext?.getSocketTimeoutMS() ??
451452
this.socketTimeoutMS;
453+
console.log(
454+
'setTimeout',
455+
options.socketTimeoutMS,
456+
options.timeoutContext?.getSocketTimeoutMS(),
457+
this.socketTimeoutMS
458+
);
452459
this.socket.setTimeout(timeout);
453460

454461
try {

test/integration/client-side-operations-timeout/client_side_operations_timeout.spec.test.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,14 +20,10 @@ const skippedTests = {
2020
'timeoutMS applies to full resume attempt in a next call': 'TODO(DRIVERS-3006)',
2121
'timeoutMS is refreshed for getMore if maxAwaitTimeMS is set': 'TODO(DRIVERS-3018)',
2222
'operation succeeds after one socket timeout - aggregate on collection':
23-
'TODO(NODE-6863): fix flaky test',
24-
'socketTimeoutMS is ignored if timeoutMS is set - dropIndex on collection':
25-
'TODO(NODE-6862): fix flaky test',
26-
'operation fails after two consecutive socket timeouts - aggregate on collection':
27-
'TODO(NODE-6867): fix flaky test'
23+
'TODO(NODE-6863): fix flaky test'
2824
};
2925

30-
describe('CSOT spec tests', function () {
26+
describe.only('CSOT spec tests', function () {
3127
const specs = loadSpecTests('client-side-operations-timeout');
3228
for (const spec of specs) {
3329
for (const test of spec.tests) {

0 commit comments

Comments
 (0)