[SERVER-57087] Write unit tests for the AsyncRequestsSender Created: 20/May/21 Updated: 06/Dec/22 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Matthew Saltz (Inactive) | Assignee: | Backlog - Service Architecture |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | sa-remove-fv-backlog-22 | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Assigned Teams: |
Service Arch
|
||||||||
| Participants: | |||||||||
| Linked BF Score: | 0 | ||||||||
| Description |
|
There are no unit tests for the AsyncRequestsSender class, which makes it very hard to make changes comfortably. When I tried doing a small refactor it would break things in random java script tests in ways that were very hard to track down. We should first write comprehensive unit tests for the existing behavior to make it possible to refactor it more comfortably |