-
Notifications
You must be signed in to change notification settings - Fork 55
Make TestCase support Properties tags #171
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
base: master
Are you sure you want to change the base?
Conversation
|
@weiwei - I'm interested in your thoughts on this PR. Feedback is welcome, I'd be happy to update it to your preferences. |
|
Hey, sorry for getting back late. pytest compatible schema is added in |
|
@weiwei - Looks like xunit2 didn't have Properties in test cases. I've moved the implementation there updated/extended the README with a short code example. |
EnricoMi
left a comment
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.
Thanks for the contribution. Some minor remarks.
eap
left a comment
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.
Thanks for the comments. I've updated the PR with your suggestions.
pytest and ctest both support adding properites to individual test cases (as noted in the attached issue). This PR updates the
propertieselement parsing so that it now works with both test cases and test suites.Fixes: #116