[TestNG] Lookup test name for report first in 'testName' field inside 'Test' annotation

Author: rosolko

allure-testng/src/main/java/io/qameta/allure/testng/AllureTestNg.java

@@ -51,6 +51,7 @@
 import org.testng.ITestNGMethod;
 import org.testng.ITestResult;
 import org.testng.annotations.Parameters;
+import org.testng.annotations.Test;
 import org.testng.internal.ConstructorOrMethod;
 import org.testng.xml.XmlSuite;
 import org.testng.xml.XmlTest;
@@ -811,6 +812,7 @@ private List<Parameter> getParameters(final ITestContext context,
 
     private String getMethodName(final ITestNGMethod method) {
         return firstNonEmpty(
+                getTestNameFromAnnotation(method),
                 method.getDescription(),
                 method.getMethodName(),
                 getQualifiedName(method)).orElse("Unknown");
@@ -881,6 +883,25 @@ private static boolean isClassAvailableOnClasspath(final String clazz) {
         }
     }
 
+    private String getTestNameFromAnnotation(final ITestNGMethod iTestNGMethod) {
+        final ConstructorOrMethod constructorOrMethod = iTestNGMethod.getConstructorOrMethod();
+        if (Objects.isNull(constructorOrMethod)) {
+            return null;
+        }
+
+        final Method method = constructorOrMethod.getMethod();
+        if (Objects.isNull(method)) {
+            return null;
+        }
+
+        final Test annotation = method.getAnnotation(Test.class);
+        if (Objects.isNull(annotation)) {
+            return null;
+        }
+
+        return annotation.testName();
+    }
+
     /**
      * The stage of current result context.
      */

allure-testng/src/test/java/io/qameta/allure/testng/AllureTestNgTest.java

@@ -997,7 +997,11 @@ public void shouldProcessCyrillicDescriptions() {
 
         assertThat(results.getTestResults())
                 .extracting(TestResult::getName)
-                .containsExactlyInAnyOrder("Тест с описанием на русском языке");
+                .contains(
+                        "Тест с описанием на русском языке только в testName",
+                        "Тест с описанием на русском языке только в description",
+                        "Тест с описанием на русском языке и в testName"
+                );
     }
 
     @AllureFeatures.Fixtures

allure-testng/src/test/java/io/qameta/allure/testng/samples/CyrillicDescriptions.java

@@ -22,7 +22,16 @@
  */
 public class CyrillicDescriptions {
 
-    @Test(description = "Тест с описанием на русском языке")
+    @Test(testName = "Тест с описанием на русском языке только в testName")
+    public void testWithCyrillicTestName() throws Exception {
+    }
+
+    @Test(description = "Тест с описанием на русском языке только в description")
     public void testWithCyrillicDescription() throws Exception {
     }
+
+    @Test(testName = "Тест с описанием на русском языке и в testName",
+            description = "Тест с описанием на русском языке и в description")
+    public void testWithCyrillicTestNameAndDescription() throws Exception {
+    }
 }