Updated Branches: refs/heads/master b80783fa9 -> 5b56fe700
[SUREFIRE-937] Intermittent incorrect testcount in parallel junit provider Project: http://git-wip-us.apache.org/repos/asf/maven-surefire/repo Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/commit/5b56fe70 Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/5b56fe70 Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/5b56fe70 Branch: refs/heads/master Commit: 5b56fe7002eb0671ef2035cd39adfb7a89bba5a5 Parents: b80783f Author: Kristian Rosenvold <krosenv...@apache.org> Authored: Sun Dec 16 16:52:39 2012 +0100 Committer: Kristian Rosenvold <krosenv...@apache.org> Committed: Sun Dec 16 17:08:52 2012 +0100 ---------------------------------------------------------------------- .../junitcore/ConcurrentReporterManager.java | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/5b56fe70/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentReporterManager.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentReporterManager.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentReporterManager.java index 9429310..777f50b 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentReporterManager.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentReporterManager.java @@ -71,6 +71,7 @@ public abstract class ConcurrentReporterManager { testSet.replay( reporterManager ); } + detachTestMethodFromThread(); reporterManagerThreadLocal.remove(); } @@ -81,6 +82,7 @@ public abstract class ConcurrentReporterManager { testMethod.testFailure( failure ); } + detachTestMethodFromThread(); } public void testError( ReportEntry failure ) @@ -90,6 +92,7 @@ public abstract class ConcurrentReporterManager { testMethod.testError( failure ); } + detachTestMethodFromThread(); } public void testSkipped( ReportEntry description ) @@ -98,6 +101,7 @@ public abstract class ConcurrentReporterManager TestMethod testMethod = getTestSet( description ).createTestMethod( description ); testMethod.testIgnored( description ); testSet.incrementFinishedTests( getRunListener(), reportImmediately ); + detachTestMethodFromThread(); } public void testAssumptionFailure( ReportEntry failure ) @@ -107,6 +111,7 @@ public abstract class ConcurrentReporterManager { testMethod.testIgnored( failure ); } + detachTestMethodFromThread(); } public void testStarting( ReportEntry description )