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

NUnit duration and TestRun status #63

Closed
siposaron opened this issue May 2, 2024 · 3 comments
Closed

NUnit duration and TestRun status #63

siposaron opened this issue May 2, 2024 · 3 comments

Comments

@siposaron
Copy link

https://docs.nunit.org/articles/nunit/technical-notes/usage/Test-Result-XML-Format.html

The NUnit v3 format does not have time property, but it has duration. Current version 0.1.14 does not parse the duration as per NUnit specifications.

i.e.
<test-run id="0" runstate="Runnable" testcasecount="40" result="Passed" total="6" passed="6" failed="0" warnings="0" inconclusive="0" skipped="0" asserts="10" engine-version="3.15" clr-version="4.0" start-time="2024-03-21 13:00:15Z" end-time="2024-03-21 13:00:42Z" duration="26.189586">

Another thing, the test-run Passed status is parsed to FAIL.

@bryanbcook
Copy link
Contributor

Confirmed regarding the duration property -- the nunit-v3.xml sample used in the test suite has some omissions. I'll investigate.

bryanbcook added a commit to bryanbcook/test-results-parser that referenced this issue May 28, 2024
@bryanbcook
Copy link
Contributor

There's a disconnect between the sample formats provided by the NUnit team and the actual output.

This document refers to the following sample files, which our tests are based on:

duration/time seems to be the most glaring difference and I can apply a fix between v2/v3 versions.

Good catch on the top-level test-run status. It's currently not being mapped which results in a FAIL.

bryanbcook added a commit to bryanbcook/test-results-parser that referenced this issue May 28, 2024
ASaiAnudeep pushed a commit that referenced this issue May 29, 2024
* Addressed time/duration mismatch for nunit3 #63

* Addressed unmapped nunit top-level status #63
@ASaiAnudeep
Copy link
Member

Fix released in - v0.1.18

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

No branches or pull requests

3 participants