[SERVER-53572] verifyTemporaryReshardingChunksMatchExpected in reshard_collection_basic.js does not assert the temporary chunk state Created: 05/Jan/21 Updated: 29/Oct/23 Resolved: 06/May/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 5.0.0-rc0 |
| Type: | Bug | Priority: | Minor - P4 |
| Reporter: | Lamont Nelson | Assignee: | Janna Golden |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Operating System: | ALL | ||||||||
| Sprint: | Sharding 2021-05-17 | ||||||||
| Participants: | |||||||||
| Story Points: | 1 | ||||||||
| Description |
|
verifyTemporaryReshardingChunksMatchExpected uses the 'size' property name instead of 'length' to check the size of an array. This causes the equality assertions to be no-ops since "undefined == undefined" and "0 < undefined" evaluates to true and false respectively. As it currently stands, fixing the 'size' issue causes the test to fail since the temp collection is renamed once resharding is successful. The test should either verify this using synchronization or only verify the properties of the output collection of the resharding command. |
| Comments |
| Comment by Janna Golden [ 06/May/21 ] |
|
This was fixed as a part of SERVER-49526. |