You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@river.apache.org by pe...@apache.org on 2015/12/11 03:58:39 UTC

svn commit: r1719247 - /river/jtsk/trunk/test/src/org/apache/river/jeri/internal/mux/MuxStartTimeoutTest.java

Author: peter_firmstone
Date: Fri Dec 11 02:58:39 2015
New Revision: 1719247

URL: http://svn.apache.org/viewvc?rev=1719247&view=rev
Log:
Ensure that finish is called under all conditions in MuxStartTimeoutTest, as test failures on jenkins are indicating that it isn't.

Modified:
    river/jtsk/trunk/test/src/org/apache/river/jeri/internal/mux/MuxStartTimeoutTest.java

Modified: river/jtsk/trunk/test/src/org/apache/river/jeri/internal/mux/MuxStartTimeoutTest.java
URL: http://svn.apache.org/viewvc/river/jtsk/trunk/test/src/org/apache/river/jeri/internal/mux/MuxStartTimeoutTest.java?rev=1719247&r1=1719246&r2=1719247&view=diff
==============================================================================
--- river/jtsk/trunk/test/src/org/apache/river/jeri/internal/mux/MuxStartTimeoutTest.java (original)
+++ river/jtsk/trunk/test/src/org/apache/river/jeri/internal/mux/MuxStartTimeoutTest.java Fri Dec 11 02:58:39 2015
@@ -67,8 +67,9 @@ public class MuxStartTimeoutTest {
                         succeeded.set(true);
                     } catch (Throwable e) {
                         failed.set(true);
+                    } finally {
+                        finished.set(true);
                     }
-                    finished.set(true);
                 }
             });
             t.start();