issue testng-team#1400: don't add duplicate reporter

missedone · Apr 1, 2017 · 1595ce2 · 1595ce2
1 parent 76c52be
commit 1595ce2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -11,6 +11,7 @@ Fixed: GITHUB-1365: Be able to override default XML parser (@ChristiKh & Julien
 Fixed: GITHUB-1360: TestNG does not distinguish between methods of different priorities (Krishnan Mahadevan)
 Fixed: GITHUB-1144: Add Class and Constructor as legal native dependency injection (Guillaume Juillot)
 Fixed: GITHUB-1380: Circular dependencies may fail in parallel (Julien Herr)
+Fixed: GITHUB-1400: TestNG, Multiple duplicate listener warnings on implementing multiple listener interfaces (@bipo1980 & Nick Tan)
 
 6.11
 2017/02/27

diff --git a/src/main/java/org/testng/TestNG.java b/src/main/java/org/testng/TestNG.java
@@ -1394,7 +1394,9 @@ private SuiteRunner createSuiteRunner(XmlSuite xmlSuite) {
     }
 
     for (IReporter r : result.getReporters()) {
-      addListener(r);
+      if (!m_reporters.containsKey(r.getClass())) {
+        addListener(r);
+      }
     }
 
     for (IConfigurationListener cl : m_configuration.getConfigurationListeners()) {