Skip to content

Commit

Permalink
Fix test race introduced by 22dffdf (java-native-access#267)
Browse files Browse the repository at this point in the history
Motivation:

22dffdf added a testcase which had a race.

Modifications:

Remove the isDone() check as it is racy

Result:

Fix test-failure which can happen due the race
  • Loading branch information
normanmaurer authored May 5, 2021
1 parent 22dffdf commit 43d8d31
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,6 @@ public void exceptionCaught(ChannelHandlerContext ctx, Throwable cause) {
clientErrorRef.set(cause);
}
}).get();
assertFalse(writePromise.isDone());

// Let's trigger the reads. This will ensure we will consume the data and the remote peer
// should be notified that it can write more data.
Expand Down

0 comments on commit 43d8d31

Please sign in to comment.