twitter-archive#93 the session is closed when the channel is

src/main/java/com/cloudhopper/smpp/impl/DefaultSmppSession.java

@@ -708,6 +708,7 @@ public void fireChannelClosed() {
             // do nothing -- ignore it
             logger.debug("Unbind/close was requested, ignoring channelClosed event");
         } else {
+            state.set(STATE_CLOSED);
             this.sessionHandler.fireChannelUnexpectedlyClosed();
         }
     }

src/test/java/com/cloudhopper/smpp/impl/DefaultSmppSessionTest.java

@@ -629,6 +629,7 @@ public void closeDoesNotTriggerUnexpectedlyClosedEvent() throws Exception {
 
         Assert.assertEquals(0, sessionHandler.getClosedCount());
         Assert.assertEquals(false, session.isBound());
+        Assert.assertEquals(true, session.isClosed());
     }
 
 
@@ -650,6 +651,7 @@ public void unbindWithNoResponseDoesNotTriggerUnexpectedlyClosedEvent() throws E
 
         Assert.assertEquals(0, sessionHandler.getClosedCount());
         Assert.assertEquals(false, session.isBound());
+        Assert.assertEquals(true, session.isClosed());
     }
 
     @Test
@@ -676,6 +678,7 @@ public boolean process(SmppSimulatorSessionHandler session, Channel channel, Pdu
 
         Assert.assertEquals(0, sessionHandler.getClosedCount());
         Assert.assertEquals(false, session.isBound());
+        Assert.assertEquals(true, session.isClosed());
     }
 
     @Test
@@ -730,8 +733,9 @@ public void remoteCloseDoesTriggerUnexpectedlyClosedEvent() throws Exception {
         Thread.sleep(500);
 
         Assert.assertEquals(1, sessionHandler.getClosedCount());
-        // DEFAULT handling is that we don't do anything special with this...
-        Assert.assertEquals(true, session.isBound());
+        // the session is closed when the channel is
+        Assert.assertEquals(false, session.isBound());
+        Assert.assertEquals(true, session.isClosed());
 
         // unbind the session now -- this should work okay even though the channel is closed
         session.unbind(100);