[maven-surefire] branch master updated: CheckTestNgSuiteXmlIT should not be ignored

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[maven-surefire] branch master updated: CheckTestNgSuiteXmlIT should not be ignored

Tibor Digana
This is an automated email from the ASF dual-hosted git repository.

tibordigana pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git


The following commit(s) were added to refs/heads/master by this push:
     new 25fadfc  CheckTestNgSuiteXmlIT should not be ignored
25fadfc is described below

commit 25fadfca1c9aeaf2fd0a2c209484800dcaea4024
Author: Tibor17 <[hidden email]>
AuthorDate: Fri Oct 5 22:36:06 2018 +0200

    CheckTestNgSuiteXmlIT should not be ignored
---
 .../apache/maven/surefire/its/CheckTestNgSuiteXmlIT.java   | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

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 2e56dc9..d4b26e8 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
@@ -21,7 +21,6 @@ package org.apache.maven.surefire.its;
 
 import org.apache.maven.surefire.its.fixture.SurefireJUnit4IntegrationTestCase;
 import org.apache.maven.surefire.its.fixture.SurefireLauncher;
-import org.junit.Ignore;
 import org.junit.Test;
 
 /**
@@ -35,20 +34,25 @@ public class CheckTestNgSuiteXmlIT
     @Test
     public void suiteXml()
     {
-        unpack().executeTest().verifyErrorFree( 2 );
+        unpack().executeTest()
+                .verifyErrorFree( 2 );
     }
 
     @Test
-    @Ignore( "Fails - see SUREFIRE-1123" )
     public void suiteXmlForkModeAlways()
     {
-        unpack().forkAlways().executeTest().verifyErrorFree( 2 );
+        unpack().forkAlways()
+                .executeTest()
+                .verifyTextInLog( "Tests run: 2, Failures: 0, Errors: 0, Skipped: 0" );
     }
 
     @Test
     public void suiteXmlForkCountTwoReuse()
     {
-        unpack().forkCount( 2 ).reuseForks( true ).executeTest().verifyErrorFree( 2 );
+        unpack().forkCount( 2 )
+                .reuseForks( true )
+                .executeTest()
+                .verifyErrorFree( 2 );
     }
 
     private SurefireLauncher unpack()