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

Change generate_json_report to parse root <testsuites> tag #1279

Merged
merged 1 commit into from
Aug 23, 2019

Conversation

sean-smith
Copy link
Contributor

On August 15th, pytest released version 5.1.0 https://pypi.org/project/pytest/#history

This version adds a new root <testsuites> tag. See pytest-dev/pytest#5477

Now

<testsuites>
<testsuite errors="0" failures="18" hostname="3a266349df18" name="pytest" skipped="0" tests="30" time="13427.235" timestamp="2019-08-23T04:05:15.798369">
...

Previously

<testsuite errors="204" failures="9" name="pytest" skipped="0" tests="793" time="947565.5930000005">
...

This patch parses that tag properly.

Signed-off-by: Sean Smith [email protected]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

On August 15th, `pytest` released version 5.1.0 https://pypi.org/project/pytest/#history

This version adds a new root `<testsuites>` tag. See pytest-dev/pytest#5477

```xml
<testsuites>
<testsuite errors="0" failures="18" hostname="3a266349df18" name="pytest" skipped="0" tests="30" time="13427.235" timestamp="2019-08-23T04:05:15.798369">
...
```

```xml
<testsuite errors="204" failures="9" name="pytest" skipped="0" tests="793" time="947565.5930000005">
...
```

This patch parses that tag properly.

Signed-off-by: Sean Smith <[email protected]>
@sean-smith sean-smith merged commit 5d3cdd6 into aws:develop Aug 23, 2019
@sean-smith sean-smith deleted the fix-test-merge branch August 23, 2019 19:23
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.

2 participants