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

Start datetime display doesn't match NUNIT start-time #68

Closed
msscribner opened this issue Oct 24, 2019 · 3 comments
Closed

Start datetime display doesn't match NUNIT start-time #68

msscribner opened this issue Oct 24, 2019 · 3 comments
Assignees
Labels

Comments

@msscribner
Copy link

Describe the bug
Looking at the NUNIT log, the tests ran from:
start-time="2019-10-07 02:51:57Z"
end-time="2019-10-07 07:10:49Z"
duration="15532.193087"

But the report is displaying:
Start datetime: 2019-10-06 22:51:58
Finish datetime: 2019-10-07 03:10:49
Duration: 04:18:51.00

Start datetime has the wrong day and time!

image

image

Expected behavior
Was expecting to see something along the lines of:
Start datetime: 2019-10-07 02:51:57
Finish datetime: 2019-10-07 07:10:49

The NuGet packages I'm using are the latest:

image

@elv1s42 elv1s42 self-assigned this Oct 25, 2019
@elv1s42 elv1s42 added the bug label Oct 25, 2019
@elv1s42
Copy link
Member

elv1s42 commented Oct 25, 2019

Hi @msscribner !

In Ghpr.NUnit there is a different approach to get test run start and end time. That's because in NUnit both start-time and end-time for each run and for each test have format yyyy-MM-dd HH:mm:ss.

But the bug may be related to timezone differences as NUnit is using UTC DateTime.

Can you please tell me what was the local timezone on the machine where the tests were running?
I can assume that you have ~4 hour difference with UTC time. If so then the bug should be easy to fix.

Thank you

@msscribner
Copy link
Author

Report is generated in Eastern Time Zone.... but I think the 4 hour difference is probably related to UTC time as you pointed out. I'll 'Close' issue.

elv1s42 added a commit that referenced this issue Oct 28, 2019
elv1s42 added a commit that referenced this issue Oct 28, 2019
@elv1s42
Copy link
Member

elv1s42 commented Nov 15, 2019

Released in v0.9.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants