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.

Sign up for GitHub

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

Jump to bottom

Create events to notify that the test execution has been completed #36

Closed
mandel-macaque opened this issue Apr 16, 2020 · 0 comments · Fixed by #46
Closed

Create events to notify that the test execution has been completed #36

mandel-macaque opened this issue Apr 16, 2020 · 0 comments · Fixed by #46
Assignees
@mandel-macaque
Labels
test-runners TestRunner area

Comments

@mandel-macaque
Copy link
Member

Add events in the runners to notify the callers that the tests started and completed so that they can react to them. This follows the comment here: dotnet/runtime#34976 (comment)
@mandel-macaque mandel-macaque added the test-runners TestRunner area label Apr 16, 2020
@mandel-macaque mandel-macaque self-assigned this Apr 16, 2020
@mandel-macaque mandel-macaque mentioned this issue Apr 16, 2020
Merged
mandel-macaque added a commit to mandel-macaque/xharness that referenced this issue Apr 17, 2020
@mandel-macaque
[Runners] Add events to notify that a test run started and completed.
47c00e5 
Add two events that notify when the runner started and when it
completed. The completed event will provide information of the results
of the run.

Later we can look at adding event for each of the tests starting and
completing, which is possible, but we are not going to add events that
we have not been asked to add. KISS

fixes: dotnet#36
@mandel-macaque mandel-macaque mentioned this issue Apr 17, 2020
Merged
mandel-macaque added a commit that referenced this issue Apr 20, 2020
@mandel-macaque
[Runners] Add events to notify that a test run started and completed. (…
bb227b5 
…#46)

Add two events that notify when the runner started and when it
completed. The completed event will provide information of the results
of the run.

Later we can look at adding event for each of the tests starting and
completing, which is possible, but we are not going to add events that
we have not been asked to add. KISS

fixes: #36
@premun premun changed the title [Runners] Create events to notify that the test execution has been completed. Create events to notify that the test execution has been completed Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mandel-macaque mandel-macaque

Labels
test-runners TestRunner area
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Runners] Add events to notify that a test run started and completed. mandel-macaque/xharness
1 participant
@mandel-macaque