Skip to content

Commit

Permalink
Add sanity interruption assertion
Browse files Browse the repository at this point in the history
This commit adds an assertion to the TestSecureSM#testNoModifySibling
test that sanity checks that the second child was actually interrupted
and therefore actually attempted to interrupt the first child.
  • Loading branch information
jasontedor committed Oct 29, 2015
1 parent 80042ae commit 06817ea
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/java/org/elasticsearch/TestSecureSM.java
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,8 @@ public void run() {
t2.start();
t2.interrupt();
t2.join();
// sibling was not able to muck with its other sibling
// sibling attempted to but was not able to muck with its other sibling
assertTrue(interrupted2.get());
assertFalse(interrupted1.get());
// but we are the parent and can terminate
t1.interrupt();
Expand Down

0 comments on commit 06817ea

Please sign in to comment.