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

Process test framework name for results in XUnit format (fixes #576) #594

Merged
merged 2 commits into from
Sep 6, 2017

Conversation

eroshenkoam
Copy link
Member

//: # (
. Thank you so much for sending us a pull request!
.
. Make sure you have a clear name for your pull request.
. The name should start with a capital letter and no dot is required in the end of the sentence.
. To link the request with isses use the following notation: (fixes #123, fixes #321)
.
. An example of good pull request names:
. - Add Russian translation (fixes #123)
. - Add an ability to disable default plugins
. - Support emoji in test descriptions
)

Context

Checklist

@baev baev changed the title #576: process test framework from xunit Process test framework name for results in XUnit format (fixes #576) Sep 6, 2017
@@ -121,6 +124,7 @@ private void parseTest(final XmlElement testElement,
getStatusDetails(testElement).ifPresent(result::setStatusDetails);
getParameters(testElement).ifPresent(result::setParameters);

result.addLabelIfNotExists(FRAMEWORK, framework);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

possible null value label?

@baev baev merged commit 7277ea8 into master Sep 6, 2017
@baev baev deleted the xunit-framework branch September 6, 2017 15:51
@baev baev removed the work:review label Sep 6, 2017
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 this pull request may close these issues.

Support tab separated attachments
2 participants