[GODRIVER-970] Amend change stream missing resume token tests for wire version 8+ Created: 17/Apr/19 Updated: 28/Oct/23 Resolved: 11/Jun/19 |
|
| Status: | Closed |
| Project: | Go Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.1.0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Ian Whalen (Inactive) | Assignee: | Unassigned |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Epic Link: | Godriver 4.2 Support | ||||||||
| Description |
|
See |
| Comments |
| Comment by Emmanuel Eppinger (Inactive) [ 11/Jun/19 ] |
|
There has not been any agreement on whether we need to check for specific error or just that an error was returned. Can revisit if needed |
| Comment by Divjot Arora (Inactive) [ 03/Jun/19 ] |
|
InĀ https://github.com/mongodb/mongo-go-driver/blob/master/mongo/change_stream_test.go#L352, we check that there was an error, not what kind of error it was. Do we need to make any changes for this ticket? If we need to ensure that wire version < 8 returns a client-side error and wire version >= 8 returns a server-side error, we could add an assertion to check if we get mongo.ErrMissingResumeToken or a mongo.CommandError. |