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 Dart: Clear Test Results command #3053

Merged
merged 6 commits into from Jan 7, 2021
Merged

Add Dart: Clear Test Results command #3053

merged 6 commits into from Jan 7, 2021

Conversation

BootBlock
Copy link
Contributor

Fixes #3049.

Places the `clearAllResults` function into the Model it mutates.
Adds a test case for `clearAllResults`.
The user-exposed Clear Test Results command is not available during an active debug session as there is the potential for a race condition; initial testing shows this isn't the case, but it hasn't been thoroughly investigated as of yet.
@BootBlock BootBlock requested a review from DanTup January 6, 2021 18:42
@DanTup DanTup added this to the v3.19.0 milestone Jan 7, 2021
@DanTup DanTup added in commands Relates to commands (usually invoked from the command Palette) in testing Relates to test execution of Dart/Flutter tests for end users is enhancement labels Jan 7, 2021
@DanTup DanTup merged commit 467136a into Dart-Code:master Jan 7, 2021
@BootBlock BootBlock deleted the clear-test-results branch January 8, 2021 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in commands Relates to commands (usually invoked from the command Palette) in testing Relates to test execution of Dart/Flutter tests for end users is enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a command to "Clear Test Results"
2 participants