Fix for #3189 - fixed ignored testcases count

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,13 @@
+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,7 @@ 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 +103,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();
   }