1. Create ExtentReport node test support TestNG data provider scenario

2. Release TestNG listener add by end user, end user can custom his listener and then disable the default listener and add his custom listeners

Author: ansonliao

src/main/java/com/github/ansonliao/selenium/parallel/SeleniumParallelTestListener.java

@@ -49,7 +49,8 @@ public void beforeInvocation(IInvokedMethod iInvokedMethod, ITestResult iTestRes
         }
 
         ExtentTestManager.createTest(method, browserName,
-                AuthorUtils.getMethodAuthors(method), groups);
+                AuthorUtils.getMethodAuthors(method), groups,
+                iTestResult.getParameters());
 
         // open url if URL annotation had value
         Optional.ofNullable(method.getAnnotation(URL.class))

src/main/java/com/github/ansonliao/selenium/report/factory/ExtentTestManager.java

@@ -4,14 +4,20 @@
 import com.aventstack.extentreports.ExtentTest;
 import com.aventstack.extentreports.Status;
 import com.github.ansonliao.selenium.annotations.Description;
+import com.google.common.base.Strings;
+import com.google.common.collect.Lists;
 import com.google.common.collect.Maps;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.testng.Reporter;
 
 import java.lang.reflect.Method;
 import java.util.List;
 import java.util.Map;
+import java.util.stream.Collectors;
 
 public class ExtentTestManager {
+    private static final Logger sfl4jLogger = LoggerFactory.getLogger(ExtentTestManager.class);
     public static ThreadLocal<ExtentTest> extentTests = new ThreadLocal<>();
     public static ExtentReports extentReport = ExtentManager.getExtentReports();
     private static ExtentTest extentTest;
@@ -37,7 +43,7 @@ public synchronized static ExtentTest createTest(String name, String description
     }
 
     public synchronized static ExtentTest createTest(Method method, String browserName,
-                                                     List<String> authors, List<String> groups) {
+                                                     List<String> authors, List<String> groups, Object... parameters) {
         ExtentTest test;
         String description;
         Class clazz = method.getDeclaringClass();
@@ -65,7 +71,19 @@ public synchronized static ExtentTest createTest(Method method, String browserNa
             childTests.put(childNodeKey, test);
         }
 
-        test = childTests.get(childNodeKey).createNode(browserName);
+        String extTestNodeName = browserName;
+        if (parameters != null && parameters.length != 0) {
+            String paramStr = Lists.newArrayList(parameters).parallelStream()
+                    .map(param -> {
+                        return param instanceof String
+                                ? "\"".concat(param.toString()).concat("\"")
+                                : String.valueOf(param);
+                    }).collect(Collectors.joining(", "));
+            extTestNodeName = extTestNodeName.concat("(").concat(paramStr).concat(")");
+        }
+
+        sfl4jLogger.info("Create ExtentReport test node: {}", extTestNodeName);
+        test = childTests.get(childNodeKey).createNode(extTestNodeName);
         if (groups != null && groups.size() > 0) {
             for (String group : groups) {
                 test.assignCategory(group);

src/main/java/com/github/ansonliao/selenium/testng/XmlSuiteBuilder.java

@@ -1,9 +1,12 @@
 package com.github.ansonliao.selenium.testng;
 
+import com.github.ansonliao.selenium.utils.TestNGListenerUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.testng.xml.XmlSuite;
 
+import java.util.List;
+
 public class XmlSuiteBuilder {
     private static final Logger logger = LoggerFactory.getLogger(XmlSuiteBuilder.class);
 
@@ -13,14 +16,19 @@ public static XmlSuite build() {
         xmlSuite.setParallel(XmlSuite.ParallelMode.TESTS);
         xmlSuite.setPreserveOrder(false);
         xmlSuite.setThreadCount(Runtime.getRuntime().availableProcessors());
-        xmlSuite.addListener(
-                "com.github.ansonliao.selenium.testng.TestResultListener");
-        //xmlSuite.addListener(
-        //        "com.github.ansonliao.selenium.testng.RetryListener");
-        xmlSuite.addListener(
-                "com.github.ansonliao.selenium.parallel.SeleniumParallelTestListener");
         xmlSuite.setVerbose(2);
 
+        // listeners
+        List<String> listeners = TestNGListenerUtils.getDefaultTestNGListners();
+        if (listeners.size() != 0) {
+            logger.info("TestNG Listeners found: {}", listeners);
+            listeners.forEach(xmlSuite::addListener);
+        } else {
+            List<String> defaultListeners = TestNGListenerUtils.getDefaultTestNGListners();
+            logger.info("No TestNG Listener found, add default listeners: ", defaultListeners);
+            defaultListeners.forEach(xmlSuite::addListener);
+        }
+
         XmlTestBuilder.setXmlSuite(xmlSuite);
         XmlTestBuilder.build();
         xmlSuite = XmlTestBuilder.getXmlSuite();

src/main/java/com/github/ansonliao/selenium/utils/TestNGListenerUtils.java

@@ -0,0 +1,43 @@
+package com.github.ansonliao.selenium.utils;
+
+
+import com.google.common.collect.Lists;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.testng.annotations.Test;
+import org.testng.util.Strings;
+
+import java.util.Arrays;
+import java.util.List;
+import java.util.stream.Collectors;
+
+public class TestNGListenerUtils {
+    private final Logger logger = LoggerFactory.getLogger(TestNGListenerUtils.class);
+    private static final String DEFAULT_TESTNG_LISTENER_KEY = "defaultTestNGListeners";
+    private static final String TESTNG_LISTENER_KEY = "testngListeners";
+
+    public static List<String> getDefaultTestNGListners() {
+        if (Strings.isNotNullAndNotEmpty(SEConfig.getString(DEFAULT_TESTNG_LISTENER_KEY))) {
+            return Arrays.stream(SEConfig.getString(DEFAULT_TESTNG_LISTENER_KEY).split(","))
+                    .map(String::trim).distinct().collect(Collectors.toList());
+        } else {
+            return Lists.newArrayList();
+        }
+    }
+
+    public static List<String> getTestNGListeners() {
+        if (Strings.isNotNullAndNotEmpty(SEConfig.getString(TESTNG_LISTENER_KEY))) {
+            return Arrays.stream(SEConfig.getString(TESTNG_LISTENER_KEY).split(","))
+                    .map(String::trim).distinct().collect(Collectors.toList());
+        } else {
+            return Lists.newArrayList();
+        }
+    }
+
+    @Test
+    public void f1() {
+        getDefaultTestNGListners().forEach(System.out::println);
+        getTestNGListeners().forEach(System.out::println);
+    }
+
+}

src/main/resources/seleniumextensions.properties

@@ -10,6 +10,9 @@ testingBrowserNames=
 testingTestGroups=
 testingTestNGClasses=
 
+testngListeners=
+defaultTestNGListeners=com.github.ansonliao.selenium.testng.TestResultListener, com.github.ansonliao.selenium.parallel.SeleniumParallelTestListener
+
 wdParameterKey.chrome="webdriver.chrome.driver"
 wdParameterKey.firefox="webdriver.gecko.driver"
 wdParameterKey.phantomjs="phantomjs.binary.path"