Skip to content

Commit 04dc099

Browse files
committed
Revert "test(NODE-6862): fix flaky CSOT timeout spec tests"
This reverts commit 7d2db35721b7d6567ef7030abaa3fb69665c0778.
1 parent 18c24fb commit 04dc099

File tree

3 files changed

+7
-13
lines changed

3 files changed

+7
-13
lines changed

.evergreen/run-tests.sh

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

src/cmap/connection.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -316,7 +316,6 @@ 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);
320319
this.cleanup(new MongoNetworkTimeoutError(message, { beforeHandshake }));
321320
}, 1).unref(); // No need for this timer to hold the event loop open
322321
}
@@ -450,12 +449,6 @@ export class Connection extends TypedEventEmitter<ConnectionEvents> {
450449
options.socketTimeoutMS ??
451450
options?.timeoutContext?.getSocketTimeoutMS() ??
452451
this.socketTimeoutMS;
453-
console.log(
454-
'setTimeout',
455-
options.socketTimeoutMS,
456-
options.timeoutContext?.getSocketTimeoutMS(),
457-
this.socketTimeoutMS
458-
);
459452
this.socket.setTimeout(timeout);
460453

461454
try {

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

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,14 @@ 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'
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'
2428
};
2529

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

0 commit comments

Comments
 (0)