-
Type:
Task
-
Resolution: Unresolved
-
Priority:
Unknown
-
None
-
Affects Version/s: None
-
Component/s: None
-
None
Context
`test_retryable_reads` contains references to `client.t.t` as shorthand for `client.database.collection` or similar syntax referring to a collection for testing purposes. `t.t` does not clearly communicate the intent to test a collection and relies on knowledge of MongoDB functionality (admittedly basic). `test.test` is not much better. Propose standardizing on `db.coll` or some other agreed upon standard.
| test.test | 141 |
| pymongo_test.test | 114 |
| db.test | 49 |
| t.t | 24 |
| db.collection | 24 |
| test_lazy_connect_w0.test | 12 |
| db.t | 12 |
| db.decryption_events | 12 |
| db.csfle | 12 |
| test.collection | 8 |
| pymongo_test.collection2 | 8 |
| ldap.test | 8 |
| db.qe | 8 |
| db.no_schema | 8 |
| db.coll | 8 |
| db.sales | 6 |
| pymongo_test.testcoll | 4 |
| db.view | 4 |
| db.qe2 | 4 |
| db.explicit_encryption | 4 |
| db.csfle2 | 4 |
| config.shards | 4 |
| test.items | 2 |
| test_session_gc.test | 2 |
| pymongo_test2.test | 2 |
| pymongo_test.test_unacked_writes | 2 |
| pymongo_test.test_raw | 2 |
| pymongo_test.ssl_test | 2 |
| pymongo_test_mike.test | 2 |
| pymongo_test_bernie.test | 2 |
| local.oplog | 2 |
| keyvault.datakeys | 2 |
| db.no_schema2 | 2 |
| dat.dat | 2 |
| aws.test | 1 |
| perfest.corpus | 1 |
Definition of done
Standardize on referring to test collections as `db.coll` or some other agreed upon standard.
Pitfalls
Sometimes choosing names specific to the test is desirable e.g. `encrypted_database.encrypted_collection`. Those cases could arguably benefit by ignoring the proposed standard.
- is related to
-
-
- Ready for Work
-