[SERVER-65486] Run unittests with feature flags on all-feature-flag variants Created: 12/Apr/22 Updated: 17/Jul/23 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Spencer Jackson | Assignee: | Backlog - Replication Team |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | pm-2821-quick-wins | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Assigned Teams: |
Replication
|
||||||||||||||||
| Operating System: | ALL | ||||||||||||||||
| Participants: | |||||||||||||||||
| Description |
|
The all-feature-flag variants do not currently run unittests. This means that it's very difficult to write unittests for new features, or verify whether tests will pass when flags are enabled by default. We should make it possible to run unittests with feature flags enabled. We should then make sure the all-feature-flag variants run these tests with the flags enabled. |
| Comments |
| Comment by Max Hirschhorn [ 12/Apr/22 ] |
|
C++ unit tests intentionally are not run with different feature flags because the expectation is C++ unit tests are generally testing a layer of the system lower than where the feature flag is consulted. Of course such a layering isn't always possible with how the system operates in practice. |