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: lucy
Backport-of: f92d095e164bd79b452586e49e166d1ba392632f
  • Loading branch information
GoeLin committed Apr 4, 2024
1 parent e913fa7 commit 9a0e875
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 @@ -221,7 +221,6 @@ public void testCloseWithOutstandingRead(Supplier<ResourceScope> scopeSupplier,
ioOp.accept(handler);
assertFalse(handler.isDone());
assertTrue(scope.isAlive());
assertMessage(expectThrows(ISE, () -> scope.close()), "Scope is acquired by");

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

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

// signal handler to stop further writing
Expand Down

1 comment on commit 9a0e875

@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.