Actions
Bug #13130
closedRemove unstable test
Start date:
10/21/2024
Due date:
% Done:
100%
Estimated time:
Description
In #11705 the AuthThrottle class was introduced. To formally verify its functionality, some test for this class were added as well.
Some of these tests, however, result in unstable behavior. They test whether the AuthThrottle::delayForNextAttempt() does indeed result in a valid countdown that decreases as time goes on. This verification uses std::this_thread::sleep_for. This is NOT a stable API that is useful for test, in case the tests depend on timing. It is only guaranteed that the thread will sleep for AT LEAST the specified amount of time, not EXACTLY this amount. Depending on the platform and its scheduler this proved to be unstable.
The tests will simply be removed.
Updated by Matthias Van Ceulebroeck about 1 month ago
- Status changed from InProgress to Review
- Assignee deleted (
Matthias Van Ceulebroeck)
Updated by Matthias Van Ceulebroeck about 1 month ago
- Status changed from Review to Implemented @Emweb
- Assignee set to Matthias Van Ceulebroeck
- % Done changed from 0 to 100
Updated by Matthias Van Ceulebroeck about 1 month ago
- Status changed from Implemented @Emweb to Implemented @Test
Updated by Matthias Van Ceulebroeck 18 days ago
- Status changed from Implemented @Test to Closed
Actions