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 2020/05/12 16:20:21 UTC
[maven-surefire] branch SUREFIRE-1788 updated (ab1b325 -> 75cd6d4)
This is an automated email from the ASF dual-hosted git repository.
tibordigana pushed a change to branch SUREFIRE-1788
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git.
discard ab1b325 other ITs should not be so sensitive to TestNG and Mockito Reflection [ERROR]s
omit c60df38 [SUREFIRE-1788] Unhandled native logs in SurefireForkChannel
omit 9e436fd [SUREFIRE-1788] Unhandled native logs in SurefireForkChannel
add 754e4ab [SUREFIRE-1791] Documentation and the integration test for Spock/Groovy with JUnit5
add f7c1d0d Verifier 1.7.2
add 882ae2c [jenkinsfile] workspace cleanup before/after run
add d5bbb3f [SUREFIRE-1787] Support multiple runners (JUnit4, TestNG, other) and their API in JUnit5 Provider
add d342819 [.asf.yaml] enabled all merge policies in enabled_merge_buttons
new c738826 [SUREFIRE-1788] Unhandled native logs in SurefireForkChannel
new 75cd6d4 other ITs should not be so sensitive to TestNG and Mockito Reflection [ERROR]s
This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version. This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:
* -- * -- B -- O -- O -- O (ab1b325)
\
N -- N -- N refs/heads/SUREFIRE-1788 (75cd6d4)
You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.
Any revisions marked "omit" are not gone; other references still
refer to them. Any revisions marked "discard" are gone forever.
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.asf.yaml | 8 +
Jenkinsfile | 52 +-
.../plugin/surefire/AbstractSurefireMojo.java | 140 ++---
.../surefire/SurefireDependencyResolver.java | 26 +-
.../plugin/surefire/AbstractSurefireMojoTest.java | 629 ++++++++++++---------
.../surefire/SurefireDependencyResolverTest.java | 9 +-
.../src/site/apt/examples/junit-platform.apt.vm | 241 +++++++-
.../src/site/apt/examples/spock.apt.vm | 174 ++++++
maven-surefire-plugin/src/site/site.xml | 1 +
pom.xml | 4 +-
.../surefire/its/fixture/HelperAssertions.java | 17 +-
...rectConcurrentTestCountIT.java => SpockIT.java} | 19 +-
.../surefire/its/jiras/Surefire1787JUnit5IT.java | 122 ++++
surefire-its/src/test/resources/junit-4-5/pom.xml | 166 ++++++
.../src/test/java/pkg/JUnit4Test.java} | 8 +-
.../junit-4-5/src/test/java/pkg/JUnit5Test.java | 10 +
.../{surefire-1727 => junit5-runner}/pom.xml | 44 +-
.../src/test/java/examples/RootTest.java | 10 +
.../src/test/java/examples/a/ATest.java | 10 +
.../src/test/java/examples/b/BTest.java | 10 +
.../src/test/java/pkg/JUnit5Tests.java | 15 +
.../{surefire-1727 => junit5-spock}/pom.xml | 32 +-
.../src/main/java/pkg/Calculator.java} | 9 +-
.../src/test/groovy/pkg/CalculatorTest.groovy | 26 +
.../src/test/resources/junit5-testng/pom.xml | 94 +++
.../src/test/java/pkg/JUnit5Test.java | 10 +
.../src/test/java/pkg/TestNGTest.java} | 8 +-
27 files changed, 1427 insertions(+), 467 deletions(-)
create mode 100644 maven-surefire-plugin/src/site/apt/examples/spock.apt.vm
copy surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/{Surefire1055CorrectConcurrentTestCountIT.java => SpockIT.java} (69%)
create mode 100644 surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1787JUnit5IT.java
create mode 100644 surefire-its/src/test/resources/junit-4-5/pom.xml
copy surefire-its/src/test/resources/{fail-fast-junit/src/test/java/pkg/ETest.java => junit-4-5/src/test/java/pkg/JUnit4Test.java} (52%)
create mode 100644 surefire-its/src/test/resources/junit-4-5/src/test/java/pkg/JUnit5Test.java
copy surefire-its/src/test/resources/{surefire-1727 => junit5-runner}/pom.xml (62%)
create mode 100644 surefire-its/src/test/resources/junit5-runner/src/test/java/examples/RootTest.java
create mode 100644 surefire-its/src/test/resources/junit5-runner/src/test/java/examples/a/ATest.java
create mode 100644 surefire-its/src/test/resources/junit5-runner/src/test/java/examples/b/BTest.java
create mode 100644 surefire-its/src/test/resources/junit5-runner/src/test/java/pkg/JUnit5Tests.java
copy surefire-its/src/test/resources/{surefire-1727 => junit5-spock}/pom.xml (73%)
copy surefire-its/src/test/resources/{surefire-1185/src/test/java/pkg/RunningTest.java => junit5-spock/src/main/java/pkg/Calculator.java} (90%)
create mode 100644 surefire-its/src/test/resources/junit5-spock/src/test/groovy/pkg/CalculatorTest.groovy
create mode 100644 surefire-its/src/test/resources/junit5-testng/pom.xml
create mode 100644 surefire-its/src/test/resources/junit5-testng/src/test/java/pkg/JUnit5Test.java
copy surefire-its/src/test/resources/{fail-fast-testng/src/test/java/pkg/ETest.java => junit5-testng/src/test/java/pkg/TestNGTest.java} (57%)
[maven-surefire] 01/02: [SUREFIRE-1788] Unhandled native logs in
SurefireForkChannel
Posted by ti...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
tibordigana pushed a commit to branch SUREFIRE-1788
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git
commit c73882605a9646ae788d7c590fc1c33dbdbc2952
Author: tibordigana <ti...@apache.org>
AuthorDate: Mon May 4 17:22:59 2020 +0200
[SUREFIRE-1788] Unhandled native logs in SurefireForkChannel
---
.../plugin/surefire/booterclient/ForkStarter.java | 6 ++---
.../output/NativeStdErrStreamConsumer.java | 14 +++++------
.../output/NativeStdOutStreamConsumer.java | 16 +++++++++----
.../surefire/extensions/LegacyForkChannel.java | 4 ++--
.../surefire/extensions/SurefireForkChannel.java | 13 +++++++---
.../maven/plugin/surefire/extensions/E2ETest.java | 7 +++++-
.../maven/surefire/extensions/ForkChannelTest.java | 15 ++++++++----
.../maven/surefire/extensions/ForkChannel.java | 10 ++------
.../surefire/extensions/StdOutStreamLine.java | 28 ----------------------
9 files changed, 52 insertions(+), 61 deletions(-)
diff --git a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
index 44c9f99..428f4e3 100644
--- a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
+++ b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
@@ -631,7 +631,7 @@ public class ForkStarter
DefaultReporterFactory reporter = forkClient.getDefaultReporterFactory();
currentForkClients.add( forkClient );
CountdownCloseable countdownCloseable =
- new CountdownCloseable( eventConsumer, 1 + ( forkChannel.useStdOut() ? 1 : 0 ) );
+ new CountdownCloseable( eventConsumer, forkChannel.getCountdownCloseablePermits() );
try ( CommandlineExecutor exec = new CommandlineExecutor( cli, countdownCloseable ) )
{
CommandlineStreams streams = exec.execute();
@@ -646,8 +646,8 @@ public class ForkStarter
out = forkChannel.bindEventHandler( eventConsumer, countdownCloseable, streams.getStdOutChannel() );
out.start();
- EventHandler<String> errConsumer = new NativeStdErrStreamConsumer( reporter );
- err = new LineConsumerThread( "fork-" + forkNumber + "-err-thread-", streams.getStdErrChannel(),
+ EventHandler<String> errConsumer = new NativeStdErrStreamConsumer( log );
+ err = new LineConsumerThread( "fork-" + forkNumber + "-err-thread", streams.getStdErrChannel(),
errConsumer, countdownCloseable );
err.start();
diff --git a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/NativeStdErrStreamConsumer.java b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/NativeStdErrStreamConsumer.java
index bc0bc7c..d7136ff 100644
--- a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/NativeStdErrStreamConsumer.java
+++ b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/NativeStdErrStreamConsumer.java
@@ -19,13 +19,14 @@ package org.apache.maven.plugin.surefire.booterclient.output;
* under the License.
*/
-import org.apache.maven.plugin.surefire.report.DefaultReporterFactory;
+import org.apache.maven.plugin.surefire.log.api.ConsoleLogger;
import org.apache.maven.surefire.extensions.EventHandler;
import javax.annotation.Nonnull;
/**
- * Used by forked JMV, see {@link org.apache.maven.plugin.surefire.booterclient.ForkStarter}.
+ * The error logger for the error stream of the forked JMV,
+ * see {@link org.apache.maven.plugin.surefire.booterclient.ForkStarter}.
*
* @author <a href="mailto:tibordigana@apache.org">Tibor Digana (tibor17)</a>
* @since 2.20
@@ -34,17 +35,16 @@ import javax.annotation.Nonnull;
public final class NativeStdErrStreamConsumer
implements EventHandler<String>
{
- private final DefaultReporterFactory defaultReporterFactory;
+ private final ConsoleLogger logger;
- public NativeStdErrStreamConsumer( DefaultReporterFactory defaultReporterFactory )
+ public NativeStdErrStreamConsumer( ConsoleLogger logger )
{
- this.defaultReporterFactory = defaultReporterFactory;
+ this.logger = logger;
}
@Override
public void handleEvent( @Nonnull String line )
{
- InPluginProcessDumpSingleton.getSingleton()
- .dumpStreamText( line, defaultReporterFactory.getReportsDirectory() );
+ logger.error( line );
}
}
diff --git a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/NativeStdOutStreamConsumer.java b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/NativeStdOutStreamConsumer.java
index 1f915ae..0d9cef2 100644
--- a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/NativeStdOutStreamConsumer.java
+++ b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/NativeStdOutStreamConsumer.java
@@ -20,13 +20,19 @@ package org.apache.maven.plugin.surefire.booterclient.output;
*/
import org.apache.maven.plugin.surefire.log.api.ConsoleLogger;
-import org.apache.maven.surefire.extensions.StdOutStreamLine;
+import org.apache.maven.surefire.extensions.EventHandler;
+
+import javax.annotation.Nonnull;
/**
+ * The output/INFO logger for the output stream of the forked JMV,
+ * see org.apache.maven.plugin.surefire.extensions.SurefireForkChannel.
*
+ * @author <a href="mailto:tibordigana@apache.org">Tibor Digana (tibor17)</a>
+ * @since 3.0.0-M5
*/
-public class NativeStdOutStreamConsumer
- implements StdOutStreamLine
+public final class NativeStdOutStreamConsumer
+ implements EventHandler<String>
{
private final ConsoleLogger logger;
@@ -36,8 +42,8 @@ public class NativeStdOutStreamConsumer
}
@Override
- public void handleLine( String line )
+ public void handleEvent( @Nonnull String event )
{
- logger.info( line );
+ logger.info( event );
}
}
diff --git a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/LegacyForkChannel.java b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/LegacyForkChannel.java
index e13846b..2edb824 100644
--- a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/LegacyForkChannel.java
+++ b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/LegacyForkChannel.java
@@ -58,9 +58,9 @@ final class LegacyForkChannel extends ForkChannel
}
@Override
- public boolean useStdOut()
+ public int getCountdownCloseablePermits()
{
- return true;
+ return 2;
}
@Override
diff --git a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/SurefireForkChannel.java b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/SurefireForkChannel.java
index 11aeb18..0cf7115 100644
--- a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/SurefireForkChannel.java
+++ b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/SurefireForkChannel.java
@@ -19,6 +19,7 @@ package org.apache.maven.plugin.surefire.extensions;
* under the License.
*/
+import org.apache.maven.plugin.surefire.booterclient.output.NativeStdOutStreamConsumer;
import org.apache.maven.surefire.eventapi.Event;
import org.apache.maven.surefire.extensions.CloseableDaemonThread;
import org.apache.maven.surefire.extensions.CommandReader;
@@ -26,6 +27,7 @@ import org.apache.maven.surefire.extensions.EventHandler;
import org.apache.maven.surefire.extensions.ForkChannel;
import org.apache.maven.surefire.extensions.ForkNodeArguments;
import org.apache.maven.surefire.extensions.util.CountdownCloseable;
+import org.apache.maven.surefire.extensions.util.LineConsumerThread;
import javax.annotation.Nonnull;
import java.io.Closeable;
@@ -75,6 +77,7 @@ final class SurefireForkChannel extends ForkChannel
private final String localHost;
private final int localPort;
private volatile AsynchronousSocketChannel worker;
+ private volatile LineConsumerThread out;
SurefireForkChannel( @Nonnull ForkNodeArguments arguments ) throws IOException
{
@@ -130,9 +133,9 @@ final class SurefireForkChannel extends ForkChannel
}
@Override
- public boolean useStdOut()
+ public int getCountdownCloseablePermits()
{
- return false;
+ return 3;
}
@Override
@@ -152,6 +155,10 @@ final class SurefireForkChannel extends ForkChannel
@Nonnull CountdownCloseable countdownCloseable,
ReadableByteChannel stdOut )
{
+ out = new LineConsumerThread( "fork-" + getArguments().getForkChannelId() + "-out-thread", stdOut,
+ new NativeStdOutStreamConsumer( getArguments().getConsoleLogger() ), countdownCloseable );
+ out.start();
+
ReadableByteChannel channel = newBufferedChannel( newInputStream( worker ) );
return new EventConsumerThread( "fork-" + getArguments().getForkChannelId() + "-event-thread", channel,
eventHandler, countdownCloseable, getArguments() );
@@ -161,7 +168,7 @@ final class SurefireForkChannel extends ForkChannel
public void close() throws IOException
{
//noinspection unused,EmptyTryBlock,EmptyTryBlock
- try ( Closeable c1 = worker; Closeable c2 = server )
+ try ( Closeable c1 = worker; Closeable c2 = server; Closeable c3 = out )
{
// only close all channels
}
diff --git a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/extensions/E2ETest.java b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/extensions/E2ETest.java
index 80a3474..5f63b9b 100644
--- a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/extensions/E2ETest.java
+++ b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/extensions/E2ETest.java
@@ -31,12 +31,15 @@ import org.junit.Test;
import javax.annotation.Nonnull;
import java.io.Closeable;
+import java.nio.ByteBuffer;
+import java.nio.channels.ReadableByteChannel;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.atomic.AtomicInteger;
import java.util.concurrent.atomic.AtomicLong;
import static org.fest.assertions.Assertions.assertThat;
+import static org.mockito.ArgumentMatchers.any;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
@@ -162,7 +165,9 @@ public class E2ETest
}
};
- server.bindEventHandler( h, new CountdownCloseable( c, 1 ), null )
+ ReadableByteChannel stdOut = mock( ReadableByteChannel.class );
+ when( stdOut.read( any( ByteBuffer.class ) ) ).thenReturn( -1 );
+ server.bindEventHandler( h, new CountdownCloseable( c, 1 ), stdOut )
.start();
assertThat( awaitHandlerFinished.await( 30L, TimeUnit.SECONDS ) )
diff --git a/maven-surefire-common/src/test/java/org/apache/maven/surefire/extensions/ForkChannelTest.java b/maven-surefire-common/src/test/java/org/apache/maven/surefire/extensions/ForkChannelTest.java
index 7b20825..070de1f 100644
--- a/maven-surefire-common/src/test/java/org/apache/maven/surefire/extensions/ForkChannelTest.java
+++ b/maven-surefire-common/src/test/java/org/apache/maven/surefire/extensions/ForkChannelTest.java
@@ -35,6 +35,8 @@ import java.io.File;
import java.io.IOException;
import java.net.Socket;
import java.net.URI;
+import java.nio.ByteBuffer;
+import java.nio.channels.ReadableByteChannel;
import java.util.Queue;
import java.util.concurrent.ConcurrentLinkedQueue;
import java.util.concurrent.CountDownLatch;
@@ -43,6 +45,9 @@ import java.util.concurrent.atomic.AtomicBoolean;
import static java.nio.charset.StandardCharsets.US_ASCII;
import static java.util.concurrent.TimeUnit.MILLISECONDS;
import static org.fest.assertions.Assertions.assertThat;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
/**
*
@@ -90,8 +95,8 @@ public class ForkChannelTest
assertThat( channel.getArguments().getForkChannelId() )
.isEqualTo( 1 );
- assertThat( channel.useStdOut() )
- .isFalse();
+ assertThat( channel.getCountdownCloseablePermits() )
+ .isEqualTo( 3 );
assertThat( channel.getForkNodeConnectionString() )
.startsWith( "tcp://127.0.0.1:" )
@@ -113,7 +118,7 @@ public class ForkChannelTest
isCloseableCalled.countDown();
}
};
- CountdownCloseable cc = new CountdownCloseable( closeable, 1 );
+ CountdownCloseable cc = new CountdownCloseable( closeable, 2 );
Consumer consumer = new Consumer();
Client client = new Client( uri.getPort() );
@@ -121,7 +126,9 @@ public class ForkChannelTest
channel.connectToClient();
channel.bindCommandReader( commandReader, null ).start();
- channel.bindEventHandler( consumer, cc, null ).start();
+ ReadableByteChannel stdOut = mock( ReadableByteChannel.class );
+ when( stdOut.read( any( ByteBuffer.class ) ) ).thenReturn( -1 );
+ channel.bindEventHandler( consumer, cc, stdOut ).start();
commandReader.noop();
diff --git a/surefire-extensions-api/src/main/java/org/apache/maven/surefire/extensions/ForkChannel.java b/surefire-extensions-api/src/main/java/org/apache/maven/surefire/extensions/ForkChannel.java
index dac54f2..4b24bb6 100644
--- a/surefire-extensions-api/src/main/java/org/apache/maven/surefire/extensions/ForkChannel.java
+++ b/surefire-extensions-api/src/main/java/org/apache/maven/surefire/extensions/ForkChannel.java
@@ -62,15 +62,9 @@ public abstract class ForkChannel implements Closeable
public abstract String getForkNodeConnectionString();
/**
- * Determines which one of the two <em>bindEventHandler-s</em> to call in <em>ForkStarter</em>.
- * Can be called anytime.
- *
- * @return If {@code true}, both {@link ReadableByteChannel} and {@link CountdownCloseable} must not be null
- * in {@link #bindEventHandler(EventHandler, CountdownCloseable, ReadableByteChannel)}. If {@code false} then
- * both {@link ReadableByteChannel} and {@link CountdownCloseable} have to be null
- * in {@link #bindEventHandler(EventHandler, CountdownCloseable, ReadableByteChannel)}.
+ * the permits in {@link CountdownCloseable}.
*/
- public abstract boolean useStdOut();
+ public abstract int getCountdownCloseablePermits();
/**
* Binds command handler to the channel.
diff --git a/surefire-extensions-api/src/main/java/org/apache/maven/surefire/extensions/StdOutStreamLine.java b/surefire-extensions-api/src/main/java/org/apache/maven/surefire/extensions/StdOutStreamLine.java
deleted file mode 100644
index f615764..0000000
--- a/surefire-extensions-api/src/main/java/org/apache/maven/surefire/extensions/StdOutStreamLine.java
+++ /dev/null
@@ -1,28 +0,0 @@
-package org.apache.maven.surefire.extensions;
-
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-/**
- * The line handler of forked process standard-output.
- */
-public interface StdOutStreamLine
-{
- void handleLine( String line );
-}
[maven-surefire] 02/02: other ITs should not be so sensitive to
TestNG and Mockito Reflection [ERROR]s
Posted by ti...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
tibordigana pushed a commit to branch SUREFIRE-1788
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git
commit 75cd6d4b0db92ff0d70c7548ad98e07cc2f6c4e9
Author: tibordigana <ti...@apache.org>
AuthorDate: Tue May 12 13:39:58 2020 +0200
other ITs should not be so sensitive to TestNG and Mockito Reflection [ERROR]s
---
.../surefire/its/CheckTestNgListenerReporterIT.java | 2 +-
.../maven/surefire/its/CheckTestNgSuiteXmlIT.java | 7 ++++---
.../maven/surefire/its/CheckTestNgVersionsIT.java | 2 +-
.../apache/maven/surefire/its/JUnitPlatformIT.java | 6 +++---
.../org/apache/maven/surefire/its/TwoTestCasesIT.java | 2 +-
.../Surefire1712ExtractedModulenameWithoutASMIT.java | 16 +++++++++-------
.../surefire/its/jiras/Surefire673MockitoIT.java | 6 ++++--
...urefire735ForkFailWithRedirectConsoleOutputIT.java | 19 ++++++++++++++++---
8 files changed, 39 insertions(+), 21 deletions(-)
diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgListenerReporterIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgListenerReporterIT.java
index 6221914..57c89b4 100644
--- a/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgListenerReporterIT.java
+++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgListenerReporterIT.java
@@ -103,7 +103,7 @@ public class CheckTestNgListenerReporterIT
}
launcher.executeTest()
- .verifyErrorFree( 1 )
+ .assertTestSuiteResults( 1, 0, 0, 0 )
.getTargetFile( "resultlistener-output.txt" ).assertFileExists()
.getTargetFile( "suitelistener-output.txt" ).assertFileExists()
.getTargetFile( "reporter-output.txt" ).assertFileExists();
diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgSuiteXmlIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgSuiteXmlIT.java
index d4b26e8..5d6cc81 100644
--- a/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgSuiteXmlIT.java
+++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgSuiteXmlIT.java
@@ -34,8 +34,9 @@ public class CheckTestNgSuiteXmlIT
@Test
public void suiteXml()
{
- unpack().executeTest()
- .verifyErrorFree( 2 );
+ unpack()
+ .executeTest()
+ .assertTestSuiteResults( 2, 0, 0, 0 );
}
@Test
@@ -52,7 +53,7 @@ public class CheckTestNgSuiteXmlIT
unpack().forkCount( 2 )
.reuseForks( true )
.executeTest()
- .verifyErrorFree( 2 );
+ .assertTestSuiteResults( 2, 0, 0, 0 );
}
private SurefireLauncher unpack()
diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgVersionsIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgVersionsIT.java
index 64235ee..9a0c1f7 100644
--- a/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgVersionsIT.java
+++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgVersionsIT.java
@@ -205,7 +205,7 @@ public class CheckTestNgVersionsIT
final OutputValidator outputValidator = launcher.executeTest();
- outputValidator.verifyErrorFreeLog().assertTestSuiteResults( 3, 0, 0, 0 );
+ outputValidator.assertTestSuiteResults( 3, 0, 0, 0 );
if ( validateRunOrder )
{
diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformIT.java
index 7a0754f..e6f5ba3 100644
--- a/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformIT.java
+++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformIT.java
@@ -78,7 +78,7 @@ public class JUnitPlatformIT
.sysProp( "junit5.version", junit5Version )
.sysProp( "jqwik.version", jqwikVersion )
.executeTest()
- .verifyErrorFree( 1 );
+ .assertTestSuiteResults( 1, 0, 0, 0 );
}
@Test
@@ -88,7 +88,7 @@ public class JUnitPlatformIT
.sysProp( "junit5.version", junit5Version )
.sysProp( "jqwik.version", jqwikVersion )
.executeTest()
- .verifyErrorFree( 1 );
+ .assertTestSuiteResults( 1, 0, 0, 0 );
}
@Test
@@ -99,7 +99,7 @@ public class JUnitPlatformIT
.sysProp( "junit5.version", junit5Version )
.sysProp( "jqwik.version", jqwikVersion )
.executeTest()
- .verifyErrorFree( 7 );
+ .assertTestSuiteResults( 7, 0, 0, 0 );
validator.getSurefireReportsFile( "TEST-junitplatformenginejupiter.BasicJupiterTest.xml", UTF_8 )
diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/TwoTestCasesIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/TwoTestCasesIT.java
index 23e6dd0..02458ed 100644
--- a/surefire-its/src/test/java/org/apache/maven/surefire/its/TwoTestCasesIT.java
+++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/TwoTestCasesIT.java
@@ -120,7 +120,7 @@ public class TwoTestCasesIT
.sysProp( "testNgVersion", "5.7" )
.sysProp( "testNgClassifier", "jdk15" )
.executeTest();
- outputValidator.verifyErrorFreeLog().assertTestSuiteResults( 2, 0, 0, 0 );
+ outputValidator.assertTestSuiteResults( 2, 0, 0, 0 );
List<ReportTestSuite> reports = HelperAssertions.extractReports( outputValidator.getBaseDir() );
Set<String> classNames = extractClassNames( reports );
assertContains( classNames, "testng.two.TestNGTestTwo" );
diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1712ExtractedModulenameWithoutASMIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1712ExtractedModulenameWithoutASMIT.java
index 92c56ad..ce8ce68 100644
--- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1712ExtractedModulenameWithoutASMIT.java
+++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1712ExtractedModulenameWithoutASMIT.java
@@ -22,7 +22,8 @@ package org.apache.maven.surefire.its.jiras;
import org.apache.maven.surefire.its.AbstractJigsawIT;
import org.junit.Test;
-import java.io.IOException;
+import static org.hamcrest.CoreMatchers.containsString;
+import static org.hamcrest.CoreMatchers.is;
/**
* See the JIRA https://issues.apache.org/jira/browse/SUREFIRE-1712
@@ -35,14 +36,15 @@ public class Surefire1712ExtractedModulenameWithoutASMIT
{
@Test
public void test()
- throws IOException
+ throws Exception
{
assumeJava9()
- .debugLogging()
- .executeTest()
- .assertTestSuiteResults( 1, 0, 0, 0 )
- .verifyErrorFreeLog()
- .verifyTextInLog( "main module descriptor name: wtf.g4s8.oot" );
+ .debugLogging()
+ .executeTest()
+ .assertTestSuiteResults( 1, 0, 0, 0 )
+ .assertThatLogLine( containsString( "Unsupported class file major version" ), is( 0 ) )
+ .assertThatLogLine( containsString( "at org.objectweb.asm.ClassReader.<init>" ), is( 0 ) )
+ .verifyTextInLog( "main module descriptor name: wtf.g4s8.oot" );
}
@Override
diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire673MockitoIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire673MockitoIT.java
index 7f3e2ca..74469c9 100644
--- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire673MockitoIT.java
+++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire673MockitoIT.java
@@ -33,6 +33,8 @@ public class Surefire673MockitoIT
@Test
public void testBuildFailingWhenErrors()
{
- unpack( "/surefire-673-mockito" ).executeTest().verifyErrorFreeLog();
+ unpack( "/surefire-673-mockito" )
+ .executeTest()
+ .assertTestSuiteResults( 1, 0, 0, 0 );
}
-}
\ No newline at end of file
+}
diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire735ForkFailWithRedirectConsoleOutputIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire735ForkFailWithRedirectConsoleOutputIT.java
index efbb0f7..a8515ac 100644
--- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire735ForkFailWithRedirectConsoleOutputIT.java
+++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire735ForkFailWithRedirectConsoleOutputIT.java
@@ -19,6 +19,7 @@ package org.apache.maven.surefire.its.jiras;
* under the License.
*/
+import org.apache.maven.it.VerificationException;
import org.apache.maven.surefire.its.fixture.OutputValidator;
import org.apache.maven.surefire.its.fixture.SurefireJUnit4IntegrationTestCase;
import org.apache.maven.surefire.its.fixture.SurefireLauncher;
@@ -26,8 +27,10 @@ import org.junit.Test;
import java.io.File;
import java.io.FilenameFilter;
+import java.util.Collection;
import static org.fest.assertions.Assertions.assertThat;
+import static org.hamcrest.CoreMatchers.containsString;
/**
* @author Kristian Rosenvold
@@ -37,14 +40,14 @@ public class Surefire735ForkFailWithRedirectConsoleOutputIT
{
@Test
- public void vmStartFail()
+ public void vmStartFail() throws VerificationException
{
OutputValidator outputValidator = unpack().failNever().executeTest();
assertJvmCrashed( outputValidator );
}
@Test
- public void vmStartFailShouldFailBuildk()
+ public void vmStartFailShouldFailBuildk() throws VerificationException
{
OutputValidator outputValidator = unpack().maven().withFailure().executeTest();
assertJvmCrashed( outputValidator );
@@ -55,8 +58,18 @@ public class Surefire735ForkFailWithRedirectConsoleOutputIT
return unpack( "fork-fail" );
}
- private static void assertJvmCrashed( OutputValidator outputValidator )
+ private static void assertJvmCrashed( OutputValidator outputValidator ) throws VerificationException
{
+ Collection<String> matchedLines =
+ outputValidator.loadLogLines( containsString( "Invalid maximum heap size: -Xmxxxx712743m" ) );
+ if ( !matchedLines.isEmpty() )
+ {
+ // the error line was printed in std/err by the JVM
+ return;
+ }
+
+ // the error line should be printed in std/out by the JVM if we use the process pipes
+ // then the ForkClient caught it and printed in the dump stream
File reportDir = outputValidator.getSurefireReportsDirectory();
String[] dumpFiles = reportDir.list( new FilenameFilter()
{