Skip to content

[flutter_tools] deflake pause while hot reload test#51608

Merged
jonahwilliams merged 1 commit intoflutter:masterfrom
jonahwilliams:deflake_hot_reload_test
Feb 28, 2020
Merged

[flutter_tools] deflake pause while hot reload test#51608
jonahwilliams merged 1 commit intoflutter:masterfrom
jonahwilliams:deflake_hot_reload_test

Conversation

@jonahwilliams
Copy link
Copy Markdown
Contributor

Description

It's not clear what value the tick # synchronization has for this test, since we can independently determine if the Isolate is paused using the VM service. On slower systems it seems like the pause may occur before the isolate can print.

Fixes #51526

@fluttergithubbot fluttergithubbot added the tool Affects the "flutter" command-line tool. See also t: labels. label Feb 28, 2020
Copy link
Copy Markdown
Member

@jmagman jmagman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

@jonahwilliams
Copy link
Copy Markdown
Contributor Author

Landing on red to fix build

@jonahwilliams jonahwilliams merged commit 4e5cb2b into flutter:master Feb 28, 2020
@jonahwilliams jonahwilliams deleted the deflake_hot_reload_test branch February 28, 2020 02:30
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test/integration.shard/hot_reload_test.dart: hot reload doesn't reassemble if paused is flaky again

4 participants