You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ti...@apache.org on 2015/09/06 22:58:35 UTC

[04/17] maven-surefire git commit: [SUREFIRE] MasterProcessReader Thread should be quite after throwing exception

[SUREFIRE] MasterProcessReader Thread should be quite after throwing exception


Project: http://git-wip-us.apache.org/repos/asf/maven-surefire/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/commit/8e37515a
Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/8e37515a
Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/8e37515a

Branch: refs/heads/master
Commit: 8e37515a06a804a154713daf0b0b53dc319a6e0a
Parents: 67af00f
Author: Tibor17 <ti...@lycos.com>
Authored: Sat Jul 25 03:30:55 2015 +0200
Committer: Tibor17 <ti...@lycos.com>
Committed: Sun Sep 6 22:57:55 2015 +0200

----------------------------------------------------------------------
 .../org/apache/maven/surefire/booter/MasterProcessReader.java  | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/8e37515a/surefire-booter/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java
----------------------------------------------------------------------
diff --git a/surefire-booter/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java b/surefire-booter/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java
index 1d86167..2dde2e1 100644
--- a/surefire-booter/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java
+++ b/surefire-booter/src/main/java/org/apache/maven/surefire/booter/MasterProcessReader.java
@@ -228,10 +228,12 @@ public final class MasterProcessReader
             }
             catch ( IOException e )
             {
+                MasterProcessReader.this.state.set( TERMINATED );
+            }
+            finally
+            {
                 // ensure fail-safe iterator as well as safe to finish in for-each loop using ClassesIterator
                 insert( new Command( TEST_SET_FINISHED ) );
-                // and let us know what has happened with the stream
-                throw new IllegalStateException( e.getLocalizedMessage(), e );
             }
         }