merlimat opened a new pull request #6295: [C++] Improved time check expectation on BackoffTest.firstBackoffTimerTest URL: https://github.com/apache/pulsar/pull/6295 ### Motivation The test succeeds only if the timing is correct within a 10 millis interval. This is too strict and it can easily fail on slower build nodes. ``` [ RUN ] BackoffTest.firstBackoffTimerTest 488 /pulsar/pulsar-client-cpp/tests/BackoffTest.cc:62: Failure 489 Value of: diffBackOffTime >= milliseconds(300) && diffBackOffTime < milliseconds(310) 490 Actual: false 491 Expected: true 492 [ FAILED ] BackoffTest.firstBackoffTimerTest (312 ms) 493 [----------] 1 test from BackoffTest (312 ms total) 494 [----------] Global test environment tear-down 495 [==========] 1 test from 1 test case ran. (312 ms total) 496 [ PASSED ] 0 tests. 497 [ FAILED ] 1 test, listed below: 498 [ FAILED ] BackoffTest.firstBackoffTimerTest 499 1 FAILED TEST ```
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
