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

Add timing information #49

Closed
mmiller-max opened this issue Oct 29, 2020 · 2 comments · Fixed by #54
Closed

Add timing information #49

mmiller-max opened this issue Oct 29, 2020 · 2 comments · Fixed by #54

Comments

@mmiller-max
Copy link
Member

Both testcase and testsuite attributes require a time attribute, and testsuite elements additionally require a timestamp attribute.

I don't think it will be possible (without it being super complicated) to time individual testcases (@test, @test_throws etc)

@mmiller-max mmiller-max mentioned this issue Oct 29, 2020
17 tasks
@oxinabox
Copy link
Member

We might be able to time indivual testcastes by:

  • Given each ReferenceTestSuite a last_record_time set to now() when created, and whenever record is called.
  • each @test will call record whenever it completes. So we just assign all time between last_record_time and it's calling record to that testcase

record(ts::ReportingTestSet, t) = (push!(ts.results, t); t)

@mmiller-max
Copy link
Member Author

Oh yeah that's a great idea, nice one. I'll give it a go on the branch I'm working on

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

Successfully merging a pull request may close this issue.

2 participants