Skip to content

Add Dart: Clear Test Results command#3053

Merged
DanTup merged 6 commits intoDart-Code:masterfrom
BootBlock:clear-test-results
Jan 7, 2021
Merged

Add Dart: Clear Test Results command#3053
DanTup merged 6 commits intoDart-Code:masterfrom
BootBlock:clear-test-results

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 An enhancement or improvement that should be listed in release notes but is not a bug fix. 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 An enhancement or improvement that should be listed in release notes but is not a bug fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a command to "Clear Test Results"

2 participants