Skip to content

Commit

Permalink
Merge pull request #660 from apache/feature/659-fix-flaky-scheduled-e…
Browse files Browse the repository at this point in the history
…vent-tests

#659: Relax timing on scheduled event tests for local testing
  • Loading branch information
pnoltes committed Sep 27, 2023
2 parents dcd8157 + fc46e3b commit dfe99e7
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions libs/framework/gtest/src/ScheduledEventTestSuite.cc
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,10 @@

class ScheduledEventTestSuite : public ::testing::Test {
public:
#if defined(__APPLE__) && defined(TESTING_ON_CI)
const int ALLOWED_ERROR_MARGIN_IN_MS = 1500;
#elif TESTING_ON_CI
const int ALLOWED_ERROR_MARGIN_IN_MS = 1000;
#if defined(__APPLE__) || defined(TESTING_ON_CI)
const int ALLOWED_ERROR_MARGIN_IN_MS = 2000;
#else
const int ALLOWED_ERROR_MARGIN_IN_MS = 100;
const int ALLOWED_ERROR_MARGIN_IN_MS = 1000;
#endif

const double ALLOWED_PROCESSING_TIME_IN_SECONDS = 0.1;
Expand Down

0 comments on commit dfe99e7

Please sign in to comment.