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

testing: initial implementation of continuous run (#170770) #1

Merged
merged 1 commit into from
Jan 8, 2023

Conversation

humsarika
Copy link
Owner

First pass of microsoft#134941

The UI is pretty minimal so far, just a new icon (though I have a proposal for some changes in the UX channel.)

If there's more that one profile that supports continuous test runs, then we open a quickpick and ask the user which ones they want to use.

The data flow is then fairly simple. We call the profile.runHandler with the appropriate request, but just doesn't "track" it like we do for normal test runs. Then, any createTestRun calls are tracked as ordinary extension-triggered test runs.

Currently we don't do anything to associate these new test runs with the fact that they were from a specific auto-run request, but that could be added if need to in the future.

First pass of #134941

The UI is pretty minimal so far, just a new icon (though I have a
proposal for some changes in the UX channel.)

If there's more that one profile that supports continuous test runs,
then we open a quickpick and ask the user which ones they want to use.

The data flow is then fairly simple. We call the `profile.runHandler`
with the appropriate request, but just doesn't "track" it like we do for
normal test runs. Then, any `createTestRun` calls are tracked as
ordinary extension-triggered test runs.

Currently we don't do anything to associate these new test runs with the
fact that they were from a specific auto-run request, but that could be
added if need to in the future.
@humsarika humsarika merged commit 6134fed into humsarika:main Jan 8, 2023
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 this pull request may close these issues.

2 participants