You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by nd...@apache.org on 2006/11/27 02:33:26 UTC
svn commit: r479488 -
/harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/BasicSwingTestCase.java
Author: ndbeyer
Date: Sun Nov 26 17:33:25 2006
New Revision: 479488
URL: http://svn.apache.org/viewvc?view=rev&rev=479488
Log:
Replace "Thread.stop" use with repeated thread interrupts.
Modified:
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/BasicSwingTestCase.java
Modified: harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/BasicSwingTestCase.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/BasicSwingTestCase.java?view=diff&rev=479488&r1=479487&r2=479488
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/BasicSwingTestCase.java (original)
+++ harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/BasicSwingTestCase.java Sun Nov 26 17:33:25 2006
@@ -140,10 +140,12 @@
});
thread.start();
thread.join(timeoutDelay);
- if (thread.isAlive()) {
- // TODO is this stop necessary?
- thread.stop();
- fail("Test interrupted due timeout");
+ int interruptAttempts = 0;
+ while (thread.isAlive()){
+ thread.interrupt();
+ if (interruptAttempts++ < 5) {
+ fail("Test interrupted due timeout");
+ }
}
if (exception[0] != null) {
rethrow(exception[0]);