Skip to content

Commit

Permalink
8307411: Test java/foreign/channels/TestAsyncSocketChannels.java fail…
Browse files Browse the repository at this point in the history
…ed: IllegalStateException: Already closed

Reviewed-by: jvernee
  • Loading branch information
mcimadamore committed May 9, 2023
1 parent 7a3bea1 commit f92d095
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions test/jdk/java/foreign/channels/TestAsyncSocketChannels.java
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,6 @@ public void testCloseWithOutstandingRead(Supplier<Arena> arenaSupplier, int time
ioOp.accept(handler);
assertFalse(handler.isDone());
assertTrue(drop.scope().isAlive());
assertMessage(expectThrows(ISE, () -> drop.close()), "Session is acquired by");

// write to allow the blocking read complete, which will
// in turn unlock the session and allow it to be closed.
Expand Down Expand Up @@ -275,7 +274,6 @@ public void completed(Long result, Void att) {
// give time for socket buffer to fill up.
awaitNoFurtherWrites(bytesWritten);

assertMessage(expectThrows(ISE, () -> drop.close()), "Session is acquired by");
assertTrue(drop.scope().isAlive());

// signal handler to stop further writing
Expand Down

1 comment on commit f92d095

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.