Skip to content

Format test report times to adjust for ms

Scott Hampton requested to merge 213101-junit-report-times into master

What does this MR do?

Related to #213101 (closed)

This is fixing an issue when our test report duration time is less than a second and it shows 00:00:00.

In this solution, we show the time in milliseconds if the time is less than a second, otherwise we show the time in seconds. We also fix the decimal point to 2 digits to keep things clean.

Screenshots

Before After
Screen_Shot_2020-08-17_at_1.21.39_PM Screen_Shot_2020-08-18_at_9.48.26_AM
Screen_Shot_2020-08-17_at_1.21.56_PM Screen_Shot_2020-08-18_at_9.48.51_AM
Screen_Shot_2020-08-17_at_1.21.48_PM Screen_Shot_2020-08-18_at_9.48.41_AM

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Scott Hampton

Merge request reports