[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 <tibo...@lycos.com> Authored: Sat Jul 25 03:30:55 2015 +0200 Committer: Tibor17 <tibo...@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 ); } }
