Skip to content

Commit

Permalink
Merge bitcoin#18211: test: Disable mockforward scheduler unit test fo…
Browse files Browse the repository at this point in the history
…r now

fab2527 test: Disable mockforward scheduler unit test for now (MarcoFalke)

Pull request description:

  This should be a workaround to fix bitcoin#18174 in the short run and buy us more time to investigate the issue while ci runs are green again 🙏

ACKs for top commit:
  fanquake:
    ACK fab2527 - be good to get Travis back.
  laanwj:
    ACK fab2527

Tree-SHA512: 027e86b3dfec203a464e5bf528e9933c208c36633c2d4bfcdbc10da1799637a5d6ea0a63af33a4174fb1ad7115df631a4cb838f56e31f4cbd15498e1e9fdf9cc
  • Loading branch information
fanquake committed Feb 27, 2020
2 parents e5d47ed + fab2527 commit 4502ed7
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/test/scheduler_tests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,7 @@ BOOST_AUTO_TEST_CASE(singlethreadedscheduler_ordered)
BOOST_CHECK_EQUAL(counter2, 100);
}

/* disabled for now. See discussion in https://github.com/bitcoin/bitcoin/pull/18174
BOOST_AUTO_TEST_CASE(mockforward)
{
CScheduler scheduler;
Expand Down Expand Up @@ -193,5 +194,6 @@ BOOST_AUTO_TEST_CASE(mockforward)
// should be between 2 & 3 minutes from now
BOOST_CHECK(delta > 2*60 && delta < 3*60);
}
*/

BOOST_AUTO_TEST_SUITE_END()

0 comments on commit 4502ed7

Please sign in to comment.