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

Time parsing is incorrect #141

Open
asaf-kali opened this issue Nov 21, 2024 · 0 comments
Open

Time parsing is incorrect #141

asaf-kali opened this issue Nov 21, 2024 · 0 comments

Comments

@asaf-kali
Copy link

In my company, we run tests in parallel.

Pytest exports the test suite with a (correct) time tag:

<?xml version="1.0" encoding="utf-8"?>
<testsuites>
    <testsuite name="pytest" errors="0" failures="14" skipped="0" tests="76" time="4005.821" timestamp="...." hostname="....">
        <testcase classname="...." name="...." time="661.025"/>

Indeed, tests took about an hour.

The time provided by TestSuite.time was 23406 seconds, which is the accumeltated time of all the tests.
It seems the logic in TestSuite.update_statistics does not take into account such cases, so I suggest the following logic change:
If time tag already exists on the element, skip the sum logic, and use the tag value as value for time.

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

No branches or pull requests

1 participant