[SUREFIRE-2152] Leave out suffix of templated tests, for StackTraceWriter

It seems cleaner to strip the suffix of templated tests again here,
since it is unrelated to the actual method signature, and might hence
just cause confusion in context of a StackTraceWriter.

Author: s-rwe

surefire-providers/surefire-junit-platform/src/main/java/org/apache/maven/surefire/junitplatform/RunListenerAdapter.java

@@ -255,8 +255,14 @@ private SimpleReportEntry createReportEntry( TestIdentifier testIdentifier,
         {
             methodText = null;
         }
-        StackTraceWriter stw =
-                testExecutionResult == null ? null : toStackTraceWriter( className, methodName, testExecutionResult );
+        String methodNameForSTW = methodName;
+        if ( methodNameForSTW != null && methodNameForSTW.contains( ")[" ) )
+        {
+            // don't pass suffix of templated-tests ("[1] ...") on to STW, as it's not part of the method signature
+            methodNameForSTW = methodNameForSTW.substring( 0, methodNameForSTW.indexOf( ")[" ) + 1 );
+        }
+        StackTraceWriter stw = testExecutionResult == null ? null
+            : toStackTraceWriter( className, methodNameForSTW, testExecutionResult );
         return new SimpleReportEntry( runMode, classMethodIndexer.indexClassMethod( className, methodName ), className,
             classText, methodName, methodText, stw, elapsedTime, reason, systemProperties );
     }