[DRIVERS-2484] Introduce Load Balanced CMAP unit tests Created: 26/Oct/22 Updated: 03/Oct/23 |
|
| Status: | Backlog |
| Project: | Drivers |
| Component/s: | CMAP, Load Balancer |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor - P4 |
| Reporter: | Patrick Freed | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Driver Changes: | Needed |
| Description |
SummaryThere are currently no LB-specific CMAP unit tests. Many of the existing unit tests do work in LB mode, but some of them (pool-clear-min-size.yml for example) do not pass. We should update the unit test format to properly account for LB mode and introduce new, LB-mode specific unit tests to ensure full coverage. MotivationWho is the affected end user?Driver authors, and potentially users if the gap in test coverage results in bugs. How does this affect the end user?Gaps in test coverage may lead to bugs. Is this issue urgent?There are existing unified format tests that cover LB mode pools, so this isn't a blocker. That said, there could be gaps in the coverage that could be filled by unit tests. Is this ticket required by a downstream team?No Is this ticket only for tests?Yes |
| Comments |
| Comment by Bailey Pearson [ 25/Sep/23 ] |
|
We would like to audit the existing CMAP unit, integration and prose tests for gaps in testing coverage load balanced mode. If there are gaps in the existing coverage, we will and add new tests to the CMAP spec. This work may not result in any additional tests if there are no gaps in our existing coverage. |