search

extent-framework / extentreports-csharp

Extent Reporting Library, .NET
http://extentreports.com
Apache License 2.0
49 stars 40 forks source link

Time Taken Badge on Report Says "secs" instead of "ms" #194

Closed bryanh-bloomerang closed 7 months ago

bryanh-bloomerang commented 8 months ago

I've recently started using extent reports and noticed that the time taken badge on a tests is showing "secs". I believe it should say "ms". Given the screenshot below, the start time of this particular test is at 12:39:38 PM and the end time is 12:39:43 PM. The time badge says "5119.81 secs". Given the start and end time and know how long this test generally takes, the value provided is in milliseconds not seconds, so the label should be "ms".

Screenshot 2023-10-17 at 1 41 52 PM
sudheshg commented 8 months ago

@bryanh-bloomerang PR has been raised. https://github.com/extent-framework/extentreports-csharp/pull/195

joegardiner commented 8 months ago

It would be good if this would display as ms if below 1s, and seconds above 1s

anshooarora commented 7 months ago

Closing since PR is now merged.

For the feature request toggling between ms/s, please see https://github.com/extent-framework/extentreports-csharp/issues/204