[SERVER-34678] Modify list_all_local_cursors.js test to make failure easier to diagnose. Created: 25/Apr/18 Updated: 29/Oct/23 Resolved: 25/Apr/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Querying |
| Affects Version/s: | None |
| Fix Version/s: | 4.0.0-rc0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | David Storch | Assignee: | David Storch |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Sprint: | Query 2018-05-07 | ||||
| Participants: | |||||
| Linked BF Score: | 16 | ||||
| Description |
|
We have observed the following assertion fail in our continuous integration testing: Specifically, the test failed because the count was 1 instead of 2. The assertion is written in such a way that we can't tell which of the two cursors wasn't found; we also can't tell if the cursor wasn't found at all, or whether one of the cursors wasn't found because the logical session ID did not match. We should rewrite this assertion in order to make these assertions separately, in order to make test failures easier to diagnose. |
| Comments |
| Comment by Githook User [ 25/Apr/18 ] |
|
Author: {'email': 'david.storch@10gen.com', 'username': 'dstorch', 'name': 'David Storch'}Message: |