diff --git a/transport/src/main/java/io/netty/channel/nio/SelectedSelectionKeySet.java b/transport/src/main/java/io/netty/channel/nio/SelectedSelectionKeySet.java index 452f0f0d77..0c7a1cc1f5 100644 --- a/transport/src/main/java/io/netty/channel/nio/SelectedSelectionKeySet.java +++ b/transport/src/main/java/io/netty/channel/nio/SelectedSelectionKeySet.java @@ -44,6 +44,16 @@ final class SelectedSelectionKeySet extends AbstractSet { return true; } + @Override + public boolean remove(Object o) { + return false; + } + + @Override + public boolean contains(Object o) { + return false; + } + @Override public int size() { return size; diff --git a/transport/src/test/java/io/netty/channel/nio/SelectedSelectionKeySetTest.java b/transport/src/test/java/io/netty/channel/nio/SelectedSelectionKeySetTest.java index 8350640692..88bb7b0a29 100644 --- a/transport/src/test/java/io/netty/channel/nio/SelectedSelectionKeySetTest.java +++ b/transport/src/test/java/io/netty/channel/nio/SelectedSelectionKeySetTest.java @@ -97,8 +97,8 @@ public class SelectedSelectionKeySetTest { SelectedSelectionKeySet set = new SelectedSelectionKeySet(); assertTrue(set.add(mockKey)); assertTrue(set.add(mockKey2)); - assertTrue(set.contains(mockKey)); - assertTrue(set.contains(mockKey2)); + assertFalse(set.contains(mockKey)); + assertFalse(set.contains(mockKey2)); assertFalse(set.contains(mockKey3)); } @@ -106,12 +106,7 @@ public class SelectedSelectionKeySetTest { public void remove() { SelectedSelectionKeySet set = new SelectedSelectionKeySet(); assertTrue(set.add(mockKey)); + assertFalse(set.remove(mockKey)); assertFalse(set.remove(mockKey2)); - try { - set.remove(mockKey); - fail(); - } catch (UnsupportedOperationException expected) { - // expected - } } }