From 2dbf594fde915f531bfad55597796cb36cf06e4f Mon Sep 17 00:00:00 2001 From: Michael Goderbauer Date: Mon, 17 Jul 2023 15:05:37 -0700 Subject: [PATCH] Resolve TODOs in channels integration test (#130745) They were helpful in diagnosing the flake in https://github.com/flutter/flutter/issues/116663, but now that that's resolved they have outlived their usefulness. Let's remove 'em! --- .../channels/integration_test/main_test.dart | 7 ------- 1 file changed, 7 deletions(-) diff --git a/dev/integration_tests/channels/integration_test/main_test.dart b/dev/integration_tests/channels/integration_test/main_test.dart index 2ec03ce92e..438ad8aa23 100644 --- a/dev/integration_tests/channels/integration_test/main_test.dart +++ b/dev/integration_tests/channels/integration_test/main_test.dart @@ -3,7 +3,6 @@ // found in the LICENSE file. import 'package:channels/main.dart'; -import 'package:flutter/gestures.dart'; import 'package:flutter/widgets.dart'; import 'package:flutter_test/flutter_test.dart'; @@ -14,9 +13,6 @@ String getStatus(WidgetTester tester) => tester.widget(statusField).data!; void main() { testWidgets('step through', (WidgetTester tester) async { - // TODO(goderbauer): Remove this once https://github.com/flutter/flutter/issues/116663 is diagnosed. - debugPrintHitTestResults = true; - await tester.pumpWidget(const TestApp()); await tester.pumpAndSettle(); @@ -37,9 +33,6 @@ void main() { } } - // TODO(goderbauer): Remove this once https://github.com/flutter/flutter/issues/116663 is diagnosed. - debugPrintHitTestResults = false; - final String status = getStatus(tester); if (status != 'complete') { fail('Failed at step $step with status $status');