Clarify reconnect behavior.

cojen · Oct 6, 2024 · 9a5f31b · 9a5f31b
1 parent 91a1ce4
commit 9a5f31b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/main/java/org/cojen/dirmi/Session.java b/src/main/java/org/cojen/dirmi/Session.java
@@ -158,7 +158,8 @@ void connected(SocketAddress localAddr, SocketAddress remoteAddr,
     void addStateListener(BiPredicate<Session<?>, Throwable> listener);
 
     /**
-     * Closes all connections and initiates a reconnect.
+     * Closes all connections and initiates a reconnect. Operation has no effect if the session
+     * is closed.
      */
     void reconnect();
 

diff --git a/src/test/java/org/cojen/dirmi/RestorableTest.java b/src/test/java/org/cojen/dirmi/RestorableTest.java
@@ -210,6 +210,16 @@ public boolean test(Session<?> session, Throwable ex) {
         }
 
         assertTrue(listener.reconnected);
+
+        mSession.close();
+
+        mSession.reconnect();
+
+        try {
+            r1x.echo("world");
+            fail();
+        } catch (ClosedException e) {
+        }
     }
 
     @Test