-
Notifications
You must be signed in to change notification settings - Fork 233
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
Update documentation to contain start time of test instead of end time #765
Comments
The example in the documentation is just that, an example. If we change it, then someone will come along and want the end time - and around we go. I think your solution looks good, if it achieves what you're looking for. 👍 |
@BeyondEvil Yes I understand that. My expectation for the time column was just wrong and I only noticed it when I saw that the duration added did not match the next timestamp. |
I'll accept a PR with that change. :) |
I added a time column as explained in the user documentation but this adds the end timestamp.
The following seems to achieve more what I want. Some feedback would be great and maybe it is worth updating the documentation?
I am still a bit unsure about the different phases of a test like setup, call and teardown. Maybe this needs extra handling?
The text was updated successfully, but these errors were encountered: