Skip to content

Commit

Permalink
#702: TestTermChaining: add check for single rank test. TODO: if unif…
Browse files Browse the repository at this point in the history
…orm bcast/send is added add extra checks for combining those in the chainset.
  • Loading branch information
nmm0 committed Jul 28, 2020
1 parent d931789 commit 4480bba
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions tests/unit/termination/test_term_chaining.cc
Original file line number Diff line number Diff line change
Expand Up @@ -193,13 +193,6 @@ struct TestTermChaining : TestParallelHarness {
vt::theMsg()->popEpoch(epoch2);
vt::theTerm()->finishedEpoch(epoch2);

EpochType epoch3 = theTerm()->makeEpochRooted();
vt::theMsg()->pushEpoch(epoch3);
auto msg3 = makeMessage<TestMsg>();
chain.add(epoch3, theMsg()->broadcastMsg<TestMsg, test_handler_bcast>(msg3.get()));
vt::theMsg()->popEpoch(epoch3);
vt::theTerm()->finishedEpoch(epoch3);

chain.done();
}
};
Expand Down Expand Up @@ -267,7 +260,7 @@ TEST_F(TestTermChaining, test_termination_chaining_collective_1) {
theTerm()->finishedEpoch(epoch);
theMsg()->popEpoch(epoch);
vt::runSchedulerThrough(epoch);
// EXPECT_EQ(handler_count, 3);
EXPECT_EQ(handler_count, 2);
}
}

Expand Down

0 comments on commit 4480bba

Please sign in to comment.