[GitHub] maven-surefire pull request #168: [SUREFIRE-1424] javax.transaction.Transact...

[eolivelli <gi...@git.apache.org>]

Github user eolivelli commented on a diff in the pull request:

    https://github.com/apache/maven-surefire/pull/168#discussion_r152835233
  
    --- Diff: surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/Java9FullApiIT.java ---
    @@ -78,12 +83,13 @@ public void shouldLoadMultipleJavaModules_ToolchainsXML() throws IOException
                                                 .addGoal( "--toolchains" )
                                                 .addGoal( System.getProperty( "maven.toolchains.file" ) )
                                                 .execute( "verify" )
    -                                            .verifyErrorFree( 2 );
    +                                            .verifyErrorFree( 1 );
    --- End diff --
    
    @Tibor17 do you think it has some value to have an IT for corba ? I think that the given one is already a good proof, it tests other more important modules.
    Tell me, if you want I will clone this IT test only of corba


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org