[SERVER-76964] Make CollatorFactoryMock accept more collations Created: 09/May/23 Updated: 14/Jun/23 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Jason Zhang | Assignee: | Backlog - Query Execution |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Assigned Teams: |
Query Execution
|
||||||||
| Participants: | |||||||||
| Description |
|
Currently the CollatorFactoryMock we use in unittests (like in catalog_cache_test_fixture.cpp, can only set the collection default collation to either the simple collation or the kReverseString collation. This makes collation unittesting much more challenging since all calls to parse a collation BSONObj, will result in the kReverseString collation. I think for more robust unittesting, it's necessary to at least support the three basic types outlined in collator_interface_mock.h. |