Fix for #3189 - fixed ignored testcases count (#3190)

* Fix for #3189 - fixed ignored testcases count

---------

Co-authored-by: Kanaduchi <[email protected]>

Author: Kanaduchi

CHANGES.txt

@@ -5,6 +5,7 @@ Fixed: GITHUB-3028: Execution stalls when using "use-global-thread-pool" (Krishn
 Fixed: GITHUB-3122: Update JCommander to 1.83 (Antoine Dessaigne)
 Fixed: GITHUB-3135: assertEquals on arrays - Failure message is missing information about the array index when an array element is unexpectedly null or non-null (Albert Choi)
 Fixed: GITHUB-3140: assertEqualsDeep on Sets - Deep comparison was using the wrong expected value
+Fixed: GITHUB-3189: Incorrect number of ignored tests displayed in the XML results
 
 7.10.2
 Fixed: GITHUB-3117: ListenerComparator doesn't work (Krishnan Mahadevan)

testng-core/src/main/java/org/testng/reporters/AbstractXmlReporter.java

@@ -228,7 +228,7 @@ public void add(Count count) {
       this.failed += count.failed;
       this.skipped += count.skipped;
       this.retried += count.retried;
-      this.ignored += count.retried;
+      this.ignored += count.ignored;
     }
 
     private Count(Builder builder) {

testng-core/src/test/java/test/reports/SimpleIgnoredSample.java

@@ -0,0 +1,11 @@
+package test.reports;
+
+import org.testng.annotations.Ignore;
+import org.testng.annotations.Test;
+
+public class SimpleIgnoredSample {
+
+  @Test
+  @Ignore
+  public void ignored() {}
+}

testng-core/src/test/java/test/reports/XmlReporterTest.java

@@ -93,7 +93,12 @@ public void ensureReportGenerationWhenTestMethodIsWrappedWithWrappedTestNGMethod
   @Test(description = "GITHUB-2886")
   public void ensureConfigurationMethodsAreNotCountedAsSkippedInXmlReports() throws Exception {
     File file =
-        runTest(RuntimeBehavior.FILE_NAME, null, JekyllTestSample.class, HydeTestSample.class);
+        runTest(
+            RuntimeBehavior.FILE_NAME,
+            null,
+            JekyllTestSample.class,
+            HydeTestSample.class,
+            SimpleIgnoredSample.class);
     DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
     DocumentBuilder builder = factory.newDocumentBuilder();
     Document doc = builder.parse(file);
@@ -103,8 +108,8 @@ public void ensureConfigurationMethodsAreNotCountedAsSkippedInXmlReports() throw
     int total = Integer.parseInt(node.getAttributes().getNamedItem("total").getNodeValue());
     int passed = Integer.parseInt(node.getAttributes().getNamedItem("passed").getNodeValue());
     int failed = Integer.parseInt(node.getAttributes().getNamedItem("failed").getNodeValue());
-    assertThat(ignored).isZero();
-    assertThat(total).isEqualTo(2);
+    assertThat(ignored).isEqualTo(1);
+    assertThat(total).isEqualTo(3);
     assertThat(passed).isEqualTo(2);
     assertThat(failed).isZero();
   }