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.

Sign up for GitHub

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

Jump to bottom

xUnit report is skipping <testresult> for pending tests #1050

Closed
tkmunzwa opened this issue Nov 29, 2013 · 1 comment · Fixed by #1051
Closed

xUnit report is skipping <testresult> for pending tests #1050

tkmunzwa opened this issue Nov 29, 2013 · 1 comment · Fixed by #1051

Comments

@tkmunzwa
Copy link

The reporter for xUnit skips pending test. For example, a testsuite with a single pending tests results in:

<testsuite name="Mocha Tests" tests="1"  skipped="1" timestamp="Fri, 29 Nov 2013 15:51:33 GMT" time="4.779">
</testsuite>

instead of:

<testsuite name="Mocha Tests" tests="1"  skipped="1" timestamp="Fri, 29 Nov 2013 15:51:33 GMT" time="4.779">
<testcase classname="example" name="should be skipped" time="NaN"><skipped/></testcase>
</testsuite>

This leads to Jenkins not showing the skipped/pending tests on the list of tests.

The JUnit XSD that Jenkins uses can be found at https://svn.jenkins-ci.org/trunk/hudson/dtkit/dtkit-format/dtkit-junit-model/src/main/resources/com/thalesgroup/dtkit/junit/model/xsd/junit-4.xsd
tkmunzwa pushed a commit to tkmunzwa/mocha that referenced this issue Nov 29, 2013
@noirlab
Add pending tests to test result data. Issue mochajs#1050
4ac4be4 
Resolves mochajs#1050

Adds test result for pending tests to results array. Pending tests will be
 are then reported individually as <testcase name="test1"></skipped></testcase>
@tkmunzwa tkmunzwa mentioned this issue Nov 29, 2013
Merged
@travisjeffery
Copy link
Contributor

closing this to discuss in the pr

travisjeffery added a commit that referenced this issue Nov 30, 2013
@travisjeffery
Merge pull request #1051 from tkmunzwa/master
2138dab 
Add <testcase></skipped><testcase> for pending tests to xunit reporter

* tkmunzwa/master:
  Add pending tests to test result data. Issue #1050
@snyk-bot snyk-bot mentioned this issue Mar 17, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add <testcase></skipped><testcase> for pending tests to xunit reporter tkmunzwa/mocha
2 participants
@travisjeffery @tkmunzwa