From b29e65be16f82feae698dbc418af6134581bb62a Mon Sep 17 00:00:00 2001
From: Mouad Debbar <mouad.debbar@gmail.com>
Date: Wed, 10 Mar 2021 15:50:02 -0800
Subject: [PATCH] [web] Add few awaits in history tests (flutter/engine#24934)

---
 .../src/flutter/lib/web_ui/test/engine/history_test.dart | 3 ---
 engine/src/flutter/lib/web_ui/test/window_test.dart      | 9 ++++-----
 2 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/engine/src/flutter/lib/web_ui/test/engine/history_test.dart b/engine/src/flutter/lib/web_ui/test/engine/history_test.dart
index 85dd0356568..e1a0140b634 100644
--- a/engine/src/flutter/lib/web_ui/test/engine/history_test.dart
+++ b/engine/src/flutter/lib/web_ui/test/engine/history_test.dart
@@ -8,7 +8,6 @@
 
 import 'dart:async';
 import 'dart:html' as html;
-import 'dart:typed_data';
 
 import 'package:test/bootstrap/browser.dart';
 import 'package:test/test.dart';
@@ -32,8 +31,6 @@ const Map<String, bool> flutterState = <String, bool>{'flutter': true};
 
 const MethodCodec codec = JSONMethodCodec();
 
-void emptyCallback(ByteData date) {}
-
 void main() {
   internalBootstrapBrowserTest(() => testMain);
 }
diff --git a/engine/src/flutter/lib/web_ui/test/window_test.dart b/engine/src/flutter/lib/web_ui/test/window_test.dart
index a7c4f3c1e78..4cb6e96b8f2 100644
--- a/engine/src/flutter/lib/web_ui/test/window_test.dart
+++ b/engine/src/flutter/lib/web_ui/test/window_test.dart
@@ -6,7 +6,6 @@
 import 'dart:async';
 import 'dart:html' as html;
 import 'dart:js_util' as js_util;
-import 'dart:typed_data';
 
 import 'package:test/bootstrap/browser.dart';
 import 'package:test/test.dart';
@@ -18,8 +17,6 @@ import 'matchers.dart';
 
 const MethodCodec codec = JSONMethodCodec();
 
-void emptyCallback(ByteData data) {}
-
 void main() {
   internalBootstrapBrowserTest(() => testMain);
 }
@@ -57,13 +54,15 @@ void testMain() {
     // Reading it multiple times should return the same value.
     expect(window.defaultRouteName, '/initial');
     expect(window.defaultRouteName, '/initial');
+
+    Completer<void> callback = Completer<void>();
     window.sendPlatformMessage(
       'flutter/navigation',
       JSONMethodCodec().encodeMethodCall(MethodCall(
         'routeUpdated',
         <String, dynamic>{'routeName': '/bar'},
       )),
-      emptyCallback,
+      (_) { callback.complete(); },
     );
     // After a navigation platform message, [window.defaultRouteName] should
     // reset to "/".
@@ -192,7 +191,7 @@ void testMain() {
     expect(window.browserHistory.currentPath, '/');
 
     // Perform some navigation operations.
-    routeInformationUpdated('/foo/bar', null);
+    await routeInformationUpdated('/foo/bar', null);
     // Path should not be updated because URL strategy is disabled.
     expect(window.browserHistory.currentPath, '/');
   });