[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:
Related
related to SERVER-49526 configsvrReshardCollection should cal... Closed
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.

Generated at Thu Feb 08 05:31:17 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.