Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect number of ignored tests displayed in the XML results #3189

Closed
7 tasks
Kanaduchi opened this issue Nov 21, 2024 · 1 comment · Fixed by #3190
Closed
7 tasks

Incorrect number of ignored tests displayed in the XML results #3189

Kanaduchi opened this issue Nov 21, 2024 · 1 comment · Fixed by #3190
Milestone

Comments

@Kanaduchi
Copy link
Contributor

Kanaduchi commented Nov 21, 2024

TestNG Version

7.10.2

Expected behavior

The correct number of ignored tests should be displayed in the XML results

Actual behavior

Ignored statistic shows count of retried tests:
testng-core/src/main/java/org/testng/reporters/AbstractXmlReporter.java

public void add(Count count) {
      this.passed += count.passed;
      this.failed += count.failed;
      this.skipped += count.skipped;
      this.retried += count.retried;
      this.ignored += count.retried;
    }

Is the issue reproducible on runner?

  • Shell
  • Maven
  • Gradle
  • Ant
  • Eclipse
  • IntelliJ
  • NetBeans

Test case sample

@Test
@Ignore
   public void testIgnored() {
      System.out.println("Ignored testcase");
   }

Contribution guidelines

Incase you plan to raise a pull request to fix this issue, please make sure you refer our Contributing section for detailed set of steps.

Kanaduchi pushed a commit to Kanaduchi/testng that referenced this issue Nov 21, 2024
Kanaduchi pushed a commit to Kanaduchi/testng that referenced this issue Nov 21, 2024
@Kanaduchi
Copy link
Contributor Author

Fix is prepared in PR: #3190

krmahadevan pushed a commit that referenced this issue Nov 22, 2024
* Fix for #3189 - fixed ignored testcases count

---------

Co-authored-by: Kanaduchi <[email protected]>
@krmahadevan krmahadevan added this to the 7.11.0 milestone Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants