[SERVER-45303] Unify skip lists for all commands tests Created: 26/Dec/19  Updated: 08/Jan/24  Resolved: 03/Jan/20

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Major - P3
Reporter: Benjamin Caimano (Inactive) Assignee: [DO NOT USE] Backlog - Sharding Team
Resolution: Won't Fix Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Assigned Teams:
Sharding
Participants:

 Description   

The following jstests all contain lists of commands to skip while they iterate through listCommands:

  • jstests/core/views/views_all_commands.js
  • jstests/sharding/database_versioning_all_commands.js
  • jstests/sharding/safe_secondary_reads_drop_recreate.js
  • jstests/sharding/safe_secondary_reads_single_migration_suspend_range_deletion.js
  • jstests/sharding/safe_secondary_reads_single_migration_waitForDelete.js
  • jstests/sharding/track_unsharded_collections_check_shard_version.js

If we make a map of commands in a file (ala jstests/lib/commands.js), then any author of a new command or a new listCommands using test can use that map and only touch one additional file instead of N tests x M commands.



 Comments   
Comment by Ratika Gandhi [ 03/Jan/20 ]

cost vs benefit does not justify the work. 

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