Github user Tibor17 commented on the pull request:
https://github.com/apache/maven-surefire/pull/96#issuecomment-142758232
@lamyaa
Did you have time to run this test using the fix we talked about in
`SUREFIRE-1144`?
Is the execution time calculation broken ?
If it is not broken, we can continue with this. I would rather talk with
Andreas before making any conclusion in this PR. First of all I will test rerun
together with HTML repor. Having `time` in partial executions within testcase
is maybe releval statistics. The only problem is with `failure` because it does
not have `time` in testcase without rerun. There is also another problem with
`time` in `failure` when JUnit tests use `ErrorCollector` which reports
multiple failures in testcase which needs to be tested in current release
2.18.1. The problem is that the time of such failures is not aggregated in
reality and thus individual times match the sum and the individual times are
not necessary. I guess the only failures and rerunFailure elements may have
`time` if and only if rerun is applied in such testcase.
Currently this decision is not trivial for me to make because I need to
make more investigations of current status.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
