search

Codeception / robo-paracept

Robo tasks for Codeception tests parallel execution
MIT License
57 stars 59 forks source link

Max execution time #120

Closed ccsuperstar closed 1 year ago

ccsuperstar commented 1 year ago

Related to issue #119

The html reports with Codeception 4 have an execution time format of the form 198.165s The html reports with Codeception 5 have an execution time format of the form 22:45.165

The regex matches both versions

Added maxSuiteTime support for the final xml report (the execution time of a suite is the longest run time among all parallel runs) max($this->suiteDuration[$suiteName])

Added maxTime support for the final html report (the total execution time is the longest run time among all parallel runs) $this->executionTime[] = (string)$matches['timesum']; $executionTime = max($this->executionTime);

I am looking for feedbacks, I think everything can be improved!

vansari commented 1 year ago

@ccsuperstar Thank you for your contribution. Please do not forget to write a test for your changes.

ccsuperstar commented 1 year ago

@ccsuperstar Thank you for your contribution. Please do not forget to write a test for your changes.

yes I will do it, then I guess I have to do tests for codeception 5 and for the previous versions