Fix testng verbose

Signed-off-by: Olivier Lamy <[email protected]>

Author: olamy

maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java

@@ -1436,6 +1436,10 @@ private void convertTestNGParameters() throws MojoExecutionException {
         if (this.getParallel() != null) {
             getProperties().setProperty(ProviderParameterNames.PARALLEL_PROP, this.getParallel());
         }
+
+        Optional.ofNullable(getProperties().get("surefire.testng.verbose"))
+                .ifPresent(s -> getProperties().setProperty("testng.verbose", (String) s));
+
         convertGroupParameters();
 
         if (this.getThreadCount() > 0) {

surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgReportTestIT.java

@@ -47,16 +47,16 @@ public void shouldNotBeVerbose() throws Exception {
                 .sysProp("testNgVersion", "6.14.3")
                 .executeTest()
                 .verifyErrorFreeLog()
-                .assertThatLogLine(containsString("[Parser] Running:"), is(0));
+                .assertThatLogLine(containsString("===== Invoked methods"), is(0));
     }
 
     @Test
     public void shouldBeVerbose() throws Exception {
         unpack("/testng-simple")
                 .sysProp("testNgVersion", "6.14.3")
-                .sysProp("surefire.testng.verbose", "10")
+                .sysProp("surefire.testng.verbose", "15")
                 .executeTest()
                 .verifyErrorFreeLog()
-                .assertThatLogLine(containsString("[Parser] Running:"), is(1));
+                .assertThatLogLine(containsString("===== Invoked methods"), is(1));
     }
 }