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
Jenkins fixes #20
Jenkins fixes #20
Conversation
Jenkins uses "." for hierarchy.
Apparently the "tests" property of a "testsuite" is required: https://github.com/jenkinsci/xunit-plugin/blob/14c6e39c38408b9ed6280361484a13c6f5becca7/src/main/resources/org/jenkinsci/plugins/xunit/types/model/xsd/junit-10.xsd#L102
- used for including parent group names - hence create appropriate "classname" for each "testcase". (turns out "classname" is an required field) - in additions, add type signature to the `runTest` function to help understanding the application.
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.
looks good to me.
@@ -130,11 +131,14 @@ antXMLRunner = Tasty.TestReporter optionDescription runner | |||
Const summary <$ State.modify (+ 1) | |||
|
|||
runGroup groupName children = Tasty.Traversal $ Functor.Compose $ do | |||
Const soFar <- Reader.withReaderT (++ [groupName]) $ Functor.getCompose $ Tasty.getTraversal children | |||
Const soFar <- Reader.local (groupName :) $ Functor.getCompose $ Tasty.getTraversal children |
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.
nit:
pre-append groupName then reverse
seems a bit waist rather than append to 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.
Appending to a list is generally frowned upon for its computational complexity, but in this case it probably wouldn't be noticeable.
@ocharles just in case he missed.. |
Thanks for the reminder! |
Released in |
Two fixes for Jenkins, see the discussion at a01df06#commitcomment-21211275