From 2b003d79115decf672dd37e4be4aafcc1cf15f93 Mon Sep 17 00:00:00 2001 From: Elliott Brooks <21270878+elliette@users.noreply.github.com> Date: Thu, 25 Jun 2026 15:24:29 -0700 Subject: [PATCH 1/3] Enable popup_menu_test --- .../popup_menu_test.dart | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) rename packages/material_ui/{temporarily_disabled_tests => test}/popup_menu_test.dart (99%) diff --git a/packages/material_ui/temporarily_disabled_tests/popup_menu_test.dart b/packages/material_ui/test/popup_menu_test.dart similarity index 99% rename from packages/material_ui/temporarily_disabled_tests/popup_menu_test.dart rename to packages/material_ui/test/popup_menu_test.dart index 18e732565a9b..38b3cec8710f 100644 --- a/packages/material_ui/temporarily_disabled_tests/popup_menu_test.dart +++ b/packages/material_ui/test/popup_menu_test.dart @@ -2,18 +2,15 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. -@Skip( - 'This file is skipped due to a cross-import that needs to be fixed. Tracked in https://github.com/flutter/flutter/issues/177028.', -) import 'dart:ui'; import 'package:flutter/foundation.dart'; -import 'package:material_ui/material_ui.dart'; import 'package:flutter/rendering.dart'; import 'package:flutter_test/flutter_test.dart'; +import 'package:material_ui/material_ui.dart'; -import '../widgets/feedback_tester.dart'; -import '../widgets/semantics_tester.dart'; +import 'feedback_tester.dart'; +import 'semantics_tester.dart'; void main() { testWidgets('Navigator.push works within a PopupMenuButton', (WidgetTester tester) async { From 7c49a60b7e4e4d62c9fd99929bc7412e2c65a544 Mon Sep 17 00:00:00 2001 From: Elliott Brooks <21270878+elliette@users.noreply.github.com> Date: Fri, 26 Jun 2026 13:31:26 -0700 Subject: [PATCH 2/3] Add unawaited --- packages/material_ui/test/popup_menu_test.dart | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/material_ui/test/popup_menu_test.dart b/packages/material_ui/test/popup_menu_test.dart index 38b3cec8710f..a11713baaf76 100644 --- a/packages/material_ui/test/popup_menu_test.dart +++ b/packages/material_ui/test/popup_menu_test.dart @@ -2,6 +2,7 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. +import 'dart:async'; import 'dart:ui'; import 'package:flutter/foundation.dart'; @@ -3722,11 +3723,13 @@ void main() { final BuildContext context = tester.element(find.text('Go')); const exampleSetting = RouteSettings(name: 'simple'); - showMenu( + unawaited( + showMenu( context: context, position: RelativeRect.fill, items: const >[PopupMenuItem(child: Text('foo'))], routeSettings: exampleSetting, + ), ); await tester.pumpAndSettle(); From 450aeafea684e32ae543cec400e2c7e93b6d2d1c Mon Sep 17 00:00:00 2001 From: Elliott Brooks <21270878+elliette@users.noreply.github.com> Date: Fri, 26 Jun 2026 13:55:33 -0700 Subject: [PATCH 3/3] Format --- packages/material_ui/test/popup_menu_test.dart | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/material_ui/test/popup_menu_test.dart b/packages/material_ui/test/popup_menu_test.dart index a11713baaf76..994b48bc38a0 100644 --- a/packages/material_ui/test/popup_menu_test.dart +++ b/packages/material_ui/test/popup_menu_test.dart @@ -3725,10 +3725,10 @@ void main() { unawaited( showMenu( - context: context, - position: RelativeRect.fill, - items: const >[PopupMenuItem(child: Text('foo'))], - routeSettings: exampleSetting, + context: context, + position: RelativeRect.fill, + items: const >[PopupMenuItem(child: Text('foo'))], + routeSettings: exampleSetting, ), );