Unverified Commit a9275d21 authored by Danny Tuppeny's avatar Danny Tuppeny Committed by GitHub

Swap order of attach/stop in flutter_attach integration test (#21106)

This is a bit of a stab in the dark for a flake sometimes seen on bots (but I can't repro locally) #20822. Detaching from a Flutter app currently seems to terminate it, so it may be the cause of the "app not found" when trying to stop the app subsequently.

This change means we stop the app from the main process first, before terminating the attach process.
parent cf764e30
...@@ -27,8 +27,8 @@ void main() { ...@@ -27,8 +27,8 @@ void main() {
// We can't call stop() on both of these because they'll both try to stop the // We can't call stop() on both of these because they'll both try to stop the
// same app. Just quit the attach process and then send a stop to the original // same app. Just quit the attach process and then send a stop to the original
// process. // process.
await _flutterAttach.quit();
await _flutterRun.stop(); await _flutterRun.stop();
await _flutterAttach.quit();
tryToDelete(tempDir); tryToDelete(tempDir);
}); });
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment