Skip to content

Commit a93fa90

Browse files
DRIVERS-3092 Check for timeout in tailable-awaitData unified spec tests (#1820)
1 parent 25ebca1 commit a93fa90

File tree

2 files changed

+136
-11
lines changed

2 files changed

+136
-11
lines changed

source/client-side-operations-timeout/tests/tailable-awaitData.json

Lines changed: 78 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

source/client-side-operations-timeout/tests/tailable-awaitData.yml

Lines changed: 58 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ tests:
4848
expectError:
4949
isClientError: true
5050

51-
- description: "error if maxAwaitTimeMS is greater than timeoutMS"
51+
- description: "error on find if maxAwaitTimeMS is greater than timeoutMS"
5252
operations:
5353
- name: find
5454
object: *collection
@@ -59,8 +59,33 @@ tests:
5959
maxAwaitTimeMS: 10
6060
expectError:
6161
isClientError: true
62+
isTimeoutError: false
6263

63-
- description: "error if maxAwaitTimeMS is equal to timeoutMS"
64+
- description: "error on aggregate if maxAwaitTimeMS is greater than timeoutMS"
65+
operations:
66+
- name: aggregate
67+
object: *collection
68+
arguments:
69+
pipeline: []
70+
timeoutMS: 5
71+
maxAwaitTimeMS: 10
72+
expectError:
73+
isClientError: true
74+
isTimeoutError: false
75+
76+
- description: "error on watch if maxAwaitTimeMS is greater than timeoutMS"
77+
operations:
78+
- name: createChangeStream
79+
object: *collection
80+
arguments:
81+
pipeline: []
82+
timeoutMS: 5
83+
maxAwaitTimeMS: 10
84+
expectError:
85+
isClientError: true
86+
isTimeoutError: false
87+
88+
- description: "error on find if maxAwaitTimeMS is equal to timeoutMS"
6489
operations:
6590
- name: find
6691
object: *collection
@@ -71,6 +96,31 @@ tests:
7196
maxAwaitTimeMS: 5
7297
expectError:
7398
isClientError: true
99+
isTimeoutError: false
100+
101+
- description: "error on aggregate if maxAwaitTimeMS is equal to timeoutMS"
102+
operations:
103+
- name: aggregate
104+
object: *collection
105+
arguments:
106+
pipeline: []
107+
timeoutMS: 5
108+
maxAwaitTimeMS: 5
109+
expectError:
110+
isClientError: true
111+
isTimeoutError: false
112+
113+
- description: "error on watch if maxAwaitTimeMS is equal to timeoutMS"
114+
operations:
115+
- name: createChangeStream
116+
object: *collection
117+
arguments:
118+
pipeline: []
119+
timeoutMS: 5
120+
maxAwaitTimeMS: 5
121+
expectError:
122+
isClientError: true
123+
isTimeoutError: false
74124

75125
- description: "timeoutMS applied to find"
76126
operations:
@@ -103,7 +153,7 @@ tests:
103153
tailable: true
104154
awaitData: true
105155
maxTimeMS: { $$exists: true }
106-
156+
107157
# If maxAwaitTimeMS is not set, timeoutMS should be refreshed for the getMore and the getMore should not have a
108158
# maxTimeMS field.
109159
- description: "timeoutMS is refreshed for getMore if maxAwaitTimeMS is not set"
@@ -271,7 +321,7 @@ tests:
271321
saveResultAsEntity: &tailableCursor tailableCursor
272322
- name: iterateOnce
273323
object: *tailableCursor
274-
- name: iterateUntilDocumentOrError
324+
- name: iterateUntilDocumentOrError
275325
object: *tailableCursor
276326
expectError:
277327
isTimeoutError: true
@@ -283,12 +333,12 @@ tests:
283333
commandName: find
284334
databaseName: *databaseName
285335
- commandStartedEvent:
286-
commandName: getMore
336+
commandName: getMore
287337
databaseName: *databaseName
288338
command:
289339
maxTimeMS: { $$lte: 100 }
290340
- commandStartedEvent:
291-
commandName: getMore
341+
commandName: getMore
292342
databaseName: *databaseName
293343
command:
294344
maxTimeMS: { $$lte: 70 }
@@ -316,7 +366,8 @@ tests:
316366
commandName: find
317367
databaseName: *databaseName
318368
- commandStartedEvent:
319-
commandName: getMore
369+
commandName: getMore
320370
databaseName: *databaseName
321371
command:
322372
maxTimeMS: { $$lte: 100 }
373+

0 commit comments

Comments
 (0)