-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Updated xUnit test reporter #809
Updated xUnit test reporter #809
Conversation
Added check if test.err exists and changed test for whether a test has passed or failed. Tested both a test that should pass and a test that should fail. Both printed correctly.
@@ -98,7 +98,7 @@ function XUnit(options) { | |||
parentSuite.pending++; | |||
test.id = ++id; | |||
test.time = new Date() - tests[test.pid][0].end, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you replace the ,
with an ;
and remove the indentation below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sixsixtrample I still see
test.time = new Date() - tests[test.pid][0].end,
but it should be
test.time = new Date() - tests[test.pid][0].end;
@sixsixtrample One comment on something that is not related to this but would be great if we can fix this as well. After that it is good to go |
removed indent and changed , to ;
@@ -108,7 +108,7 @@ function XUnit(options) { | |||
parentSuite.passes++; | |||
test.id = ++id; | |||
test.time = new Date() - tests[test.pid][0].end, | |||
test.timestamp = tests[test.pid][0].end = new Date(); | |||
test.timestamp = tests[test.pid][0].end = new Date(); | |||
test.runnerData = runnerData[test.pid]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please also change it here
Corrected , and ;
Ok, I believe they are all fixed. |
@sixsixtrample please have a look at the Files Change tab. Have you missed to push to Github? |
It says I have no uncommitted changes. And I pushed it to my fork. Is there something else I need to do? |
Updated xUnit test reporter
Thanks! |
Published with |
Added check if test.err exists and changed test for whether a test has
passed or failed. Tested both a test that should pass and a test that
should fail. Both printed correctly.