[SERVER-76587] Replace "test" collection names with unique names in tests Created: 27/Apr/23  Updated: 29/Oct/23  Resolved: 09/Oct/23

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

Type: Improvement Priority: Major - P3
Reporter: Jordi Olivares Provencio Assignee: David Dominguez Sal
Resolution: Fixed Votes: 0
Labels: neweng
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
Assigned Teams:
Storage Execution
Backwards Compatibility: Fully Compatible
Sprint: Execution EMEA Team 2023-10-16
Participants:
Linked BF Score: 5

 Description   

Searching for the following two strings in tests:

  • createCollection("test"
  • getCollection("test"

Yields the following list of tests making usage of the same collection name:

  • jstests/aggregation/expressions/expression_mod.js
  • jstests/aggregation/expressions/ts_second_increment.js
  • jstests/aggregation/variables/layered_variables.js
  • jstests/auth/lib/commands_lib.js
  • jstests/core/administrative/profile/profile_agg.js
  • jstests/core/administrative/profile/profile_count.js
  • jstests/core/administrative/profile/profile_delete.js
  • jstests/core/administrative/profile/profile_distinct.js
  • jstests/core/administrative/profile/profile_find.js
  • jstests/core/administrative/profile/profile_findandmodify.js
  • jstests/core/administrative/profile/profile_getmore.js
  • jstests/core/administrative/profile/profile_insert.js
  • jstests/core/administrative/profile/profile_mapreduce.js
  • jstests/core/administrative/profile/profile_update.js
  • jstests/core/api/api_params_getmore.js
  • jstests/core/api/apitest_db.js
  • jstests/core/query/agg_hint.js
  • jstests/core/sbe/from_plan_cache_flag.js
  • jstests/core/version_api_v1_command_coverage.js
  • jstests/fle2/implicit_schema_validation.js
  • jstests/noPassthrough/json_schema_validator_logging.js
  • jstests/noPassthrough/nested_code_with_scope.js
  • jstests/noPassthrough/profile_interrupted_op.js
  • jstests/noPassthrough/profile_write_conflict.js
  • jstests/noPassthrough/reconfig_restarts_collection_scan.js
  • jstests/noPassthrough/set_user_write_block_mode.js
  • jstests/noPassthrough/use_disk.js
  • jstests/noPassthrough/validate_db_metadata_limits.js
  • jstests/noPassthroughWithMongod/randomized_mixed_type_bug.js
  • jstests/replsets/create_drop_database_different_casing.js
  • jstests/replsets/kill_ttl_on_stepdown.js

We've already seen errors where two tests running in parallel within the same test instance can fail due to one dropping the collection used by the other. We should strive to make these collection names unique if possible.



 Comments   
Comment by Githook User [ 06/Oct/23 ]

Author:

{'name': 'David Dominguez-Sal', 'email': 'david.dominguez@mongodb.com', 'username': ''}

Message: SERVER-76587 Renamed collections named test as the filename where they are implemented
Branch: master
https://github.com/mongodb/mongo/commit/55a72762e90ed1c9e7008164c207b8a5254b7b69

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