Github user kl0u commented on a diff in the pull request:
    --- Diff: 
    @@ -119,6 +121,11 @@ public void run() {
    +           Throwable exception = error.get();
    +           if (exception != null) {
    +                   throw new AssertionError(exception);
    +           }
    --- End diff --
    Is this any different from using `Assert.assertNull(error.get())`? Not that 
it makes any difference but just to know why you picked throwing the exception 
here. Personally I think that the assertNull seems more explicit when you read 
the code and it is less verbose.


Reply via email to