Skip to content

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