- build: ci: add server name to test name

- now maven will display the server name on test
- remove/cleanup display name for servertest: hide arguments

Author: jknack

pom.xml

@@ -1409,6 +1409,14 @@
             <jooby.maven.build>true</jooby.maven.build>
           </systemPropertyVariables>
           <argLine>@{argLine}</argLine>
+          <statelessTestsetReporter implementation="org.apache.maven.plugin.surefire.extensions.junit5.JUnit5Xml30StatelessReporter">
+            <disable>false</disable>
+            <version>3.0</version>
+            <usePhrasedFileName>false</usePhrasedFileName>
+            <usePhrasedTestSuiteClassName>true</usePhrasedTestSuiteClassName>
+            <usePhrasedTestCaseClassName>true</usePhrasedTestCaseClassName>
+            <usePhrasedTestCaseMethodName>true</usePhrasedTestCaseMethodName>
+          </statelessTestsetReporter>
         </configuration>
       </plugin>
     </plugins>

tests/src/test/java/io/jooby/i3500/WidgetService.java

@@ -18,7 +18,6 @@ public WidgetService() {
         "/api/widgets1",
         ctx -> {
           Widget widget = ctx.body().to(Widget.class);
-          System.out.println("Created " + widget);
           return ctx.send(StatusCode.CREATED);
         });
 

tests/src/test/java/io/jooby/junit/CleanMethodNameGenerator.java

@@ -0,0 +1,27 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package io.jooby.junit;
+
+import java.lang.reflect.Method;
+import java.util.List;
+import java.util.stream.Stream;
+
+import org.junit.jupiter.api.DisplayNameGenerator;
+
+public class CleanMethodNameGenerator extends DisplayNameGenerator.Standard {
+  @Override
+  public String generateDisplayNameForMethod(
+      List<Class<?>> enclosingInstanceTypes, Class<?> testClass, Method testMethod) {
+    // remove (ServerTestRunner) from test name:
+    var args =
+        Stream.of(testMethod.getParameters())
+            .filter(param -> !param.getType().equals(ServerTestRunner.class))
+            .toList();
+    return args.isEmpty()
+        ? testMethod.getName()
+        : super.generateDisplayNameForMethod(enclosingInstanceTypes, testClass, testMethod);
+  }
+}

tests/src/test/java/io/jooby/junit/ServerExtensionImpl.java

@@ -117,7 +117,7 @@ private TestTemplateInvocationContext invocationContext(ServerInfo serverInfo) {
     return new TestTemplateInvocationContext() {
       @Override
       public String getDisplayName(int invocationIndex) {
-        return serverInfo.description;
+        return isMavenBuild() ? "(" + serverInfo.description + ")" : serverInfo.description;
       }
 
       @Override
@@ -138,4 +138,8 @@ private static String displayName(Class server, ExecutionMode mode, int i, int t
     }
     return displayName.toString();
   }
+
+  static boolean isMavenBuild() {
+    return !System.getProperty("surefire.real.class.path", "").isEmpty();
+  }
 }

tests/src/test/java/io/jooby/junit/ServerTestRunner.java

@@ -91,8 +91,7 @@ public void ready(SneakyThrows.Consumer2<WebClient, WebClient> onReady) {
       System.setProperty("___server_name__", server.getName());
       var app = provider.get();
       // Reduce log from maven build:
-      var mavenBuild = System.getProperty("surefire.real.class.path", "").length() > 0;
-      if (mavenBuild) {
+      if (ServerExtensionImpl.isMavenBuild()) {
         applogger = app.getClass().getName();
         app.setStartupSummary(List.of(StartupSummary.NONE));
         app.error(

tests/src/test/resources/junit-platform.properties

@@ -0,0 +1 @@
+junit.jupiter.displayname.generator.default=io.jooby.junit.CleanMethodNameGenerator