svn commit: r649257 - /mina/trunk/core/src/test/java/org/apache/mina/transport/AbstractConnectorTest.java

[sj...@apache.org]

Author: sjlee
Date: Thu Apr 17 13:05:06 2008
New Revision: 649257

URL: http://svn.apache.org/viewvc?rev=649257&view=rev
Log:
DIRMINA-571

Modified the assert to check the correct condition, as sessionOpened() is no longer guaranteed to be called before the connect future completes.

Modified:
    mina/trunk/core/src/test/java/org/apache/mina/transport/AbstractConnectorTest.java

Modified: mina/trunk/core/src/test/java/org/apache/mina/transport/AbstractConnectorTest.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/test/java/org/apache/mina/transport/AbstractConnectorTest.java?rev=649257&r1=649256&r2=649257&view=diff
==============================================================================
--- mina/trunk/core/src/test/java/org/apache/mina/transport/AbstractConnectorTest.java (original)
+++ mina/trunk/core/src/test/java/org/apache/mina/transport/AbstractConnectorTest.java Thu Apr 17 13:05:06 2008
@@ -22,6 +22,7 @@
 import java.net.InetSocketAddress;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
+import java.util.regex.Pattern;
 
 import junit.framework.Assert;
 import junit.framework.TestCase;
@@ -76,7 +77,9 @@
             future.awaitUninterruptibly();
             buf.append("3");
             future.getSession().close();
-            Assert.assertEquals("123", buf.toString());
+            // sessionCreated() will fire before the connect future completes
+            // but sessionOpened() may not
+            Assert.assertTrue(Pattern.matches("12?32?", buf.toString()));
         } finally {
             acceptor.dispose();
         }