[material_ui] Enable bottom_navigation_bar_test (#12005)

Port over `bottom_navigation_bar_test` from
flutter/flutter#184279
* Ports over `feedback_tester.dart`
* Ports over `bottom_navigation_bar_test` 

As part of this change, moved `_findByTooltip` (originally defined in
`navigation_bar_test`) into new shared `test/finders.dart` (see
flutter/flutter#186966 for details).

Work towards flutter/flutter#182636 and
flutter/flutter#188395

## Pre-Review Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [AI contribution guidelines] and understand my
responsibilities, or I am not using AI tools.
- [x] I read the [Tree Hygiene] page, which explains my
responsibilities.
- [x] I read and followed the [relevant style guides] and ran [the
auto-formatter].
- [x] I signed the [CLA].
- [x] The title of the PR starts with the name of the package surrounded
by square brackets, e.g. `[shared_preferences]`
- [x] I [linked to at least one issue that this PR fixes] in the
description above.
- [x] I followed [the version and CHANGELOG instructions], using
[semantic versioning] and the [repository CHANGELOG style], or I have
commented below to indicate which documented exception this PR falls
under[^1].
- [x] I updated/added any relevant documentation (doc comments with
`///`).
- [x] I added new tests to check the change I am making, or I have
commented below to indicate which [test exemption] this PR falls
under[^1].
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

**Note**: The Flutter team is currently trialing the use of [Gemini Code
Assist for
GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code).
Comments from the `gemini-code-assist` bot should not be taken as
authoritative feedback from the Flutter team. If you find its comments
useful you can update your code accordingly, but if you are unsure or
disagree with the feedback, please feel free to wait for a Flutter team
member's review for guidance on which automated comments should be
addressed.

[^1]: Regular contributors who have demonstrated familiarity with the
repository guidelines only need to comment if the PR is not
auto-exempted by repo tooling.

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/packages/blob/main/CONTRIBUTING.md
[AI contribution guidelines]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#ai-contribution-guidelines
[Tree Hygiene]:
https://github.com/flutter/flutter/blob/master/docs/contributing/Tree-hygiene.md
[relevant style guides]:
https://github.com/flutter/packages/blob/main/CONTRIBUTING.md#style
[the auto-formatter]:
https://github.com/flutter/packages/blob/main/script/tool/README.md#format-code
[CLA]: https://cla.developers.google.com/
[Discord]:
https://github.com/flutter/flutter/blob/master/docs/contributing/Chat.md
[linked to at least one issue that this PR fixes]:
https://github.com/flutter/flutter/blob/master/docs/contributing/Tree-hygiene.md#overview
[the version and CHANGELOG instructions]:
https://github.com/flutter/flutter/blob/master/docs/ecosystem/contributing/README.md#version-and-changelog-updates
[semantic versioning]:
https://dart.dev/tools/pub/versioning#semantic-versions
[repository CHANGELOG style]:
https://github.com/flutter/flutter/blob/master/docs/ecosystem/contributing/README.md#changelog-style
[test exemption]:
https://github.com/flutter/flutter/blob/master/docs/contributing/Tree-hygiene.md#tests

Author: elliette

…ed_tests/bottom_navigation_bar_test.dart …_ui/test/bottom_navigation_bar_test.dartpackages/material_ui/temporarily_disabled_tests/bottom_navigation_bar_test.dart renamed to packages/material_ui/test/bottom_navigation_bar_test.dart packages/material_ui/temporarily_disabled_tests/bottom_navigation_bar_test.dart renamed to packages/material_ui/test/bottom_navigation_bar_test.dart

@@ -2,9 +2,6 @@
 // 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.',
-)
 // This file is run as part of a reduced test set in CI on Mac and Windows
 // machines.
 @Tags(<String>['reduced-test-set'])
@@ -13,13 +10,14 @@ library;
 import 'dart:math';
 import 'dart:ui';
 
-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 'package:vector_math/vector_math_64.dart' show Vector3;
 
-import '../widgets/feedback_tester.dart';
-import '../widgets/semantics_tester.dart';
+import 'feedback_tester.dart';
+import 'finders.dart';
+import 'semantics_tester.dart';
 
 void main() {
   testWidgets('BottomNavigationBar callback test', (WidgetTester tester) async {
@@ -1716,15 +1714,15 @@ void main() {
 
     expect(find.text('A'), findsOneWidget);
     await tester.longPress(find.text('A'));
-    expect(find.byTooltip('A tooltip'), findsOneWidget);
+    expect(findByTooltip('A tooltip'), findsOneWidget);
 
     expect(find.text('B'), findsOneWidget);
     await tester.longPress(find.text('B'));
-    expect(find.byTooltip('B'), findsNothing);
+    expect(findByTooltip('B'), findsNothing);
 
     expect(find.text('C'), findsOneWidget);
     await tester.longPress(find.text('C'));
-    expect(find.byTooltip('C'), findsNothing);
+    expect(findByTooltip('C'), findsNothing);
   });
 
   testWidgets('BottomNavigationBar limits width of tiles with long labels', (

packages/material_ui/test/feedback_tester.dart

@@ -0,0 +1,51 @@
+// Copyright 2013 The Flutter Authors
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+import 'package:flutter/services.dart';
+import 'package:flutter_test/flutter_test.dart';
+
+/// Tracks how often feedback has been requested since its instantiation.
+///
+/// It replaces the MockMethodCallHandler of [SystemChannels.platform] and
+/// cannot be used in combination with other classes that do the same.
+class FeedbackTester {
+  FeedbackTester() {
+    TestDefaultBinaryMessengerBinding.instance.defaultBinaryMessenger.setMockMethodCallHandler(
+      SystemChannels.platform,
+      _handler,
+    );
+  }
+
+  /// Number of times haptic feedback was requested (vibration).
+  int get hapticCount => _hapticCount;
+  int _hapticCount = 0;
+
+  /// Number of times the click sound was requested to play.
+  int get clickSoundCount => _clickSoundCount;
+  int _clickSoundCount = 0;
+
+  Future<void> _handler(MethodCall methodCall) async {
+    if (methodCall.method == 'HapticFeedback.vibrate') {
+      _hapticCount++;
+    }
+    if (methodCall.method == 'SystemSound.play' &&
+        methodCall.arguments == SystemSoundType.click.toString()) {
+      _clickSoundCount++;
+    }
+  }
+
+  /// Stops tracking.
+  void dispose() {
+    assert(
+      TestDefaultBinaryMessengerBinding.instance.defaultBinaryMessenger.checkMockMessageHandler(
+        SystemChannels.platform.name,
+        _handler,
+      ),
+    );
+    TestDefaultBinaryMessengerBinding.instance.defaultBinaryMessenger.setMockMethodCallHandler(
+      SystemChannels.platform,
+      null,
+    );
+  }
+}

packages/material_ui/test/finders.dart

@@ -0,0 +1,43 @@
+// Copyright 2013 The Flutter Authors
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+import 'package:flutter_test/flutter_test.dart';
+import 'package:material_ui/material_ui.dart';
+
+/// Finds [RawTooltip] or [Tooltip] widgets with the given `message`.
+///
+/// ## Sample code
+///
+/// ```dart
+/// expect(find.byTooltip('Back'), findsOneWidget);
+/// expect(find.byTooltip(RegExp('Back.*')), findsNWidgets(2));
+/// ```
+///
+/// If the `skipOffstage` argument is true (the default), then this skips
+/// nodes that are [Offstage] or that are from inactive [Route]s.
+///
+/// This was copied from flutter_test, which uses flutter/material.dart.
+///
+// TODO(justinmc): Port flutter_test to material_ui, then delete this method and
+// use that one. See https://github.com/flutter/flutter/issues/186966
+Finder findByTooltip(Pattern message, {bool skipOffstage = true}) {
+  return find.byWidgetPredicate((Widget widget) {
+    // Compare RawTooltip's semantics tooltip with the given message.
+    // However, Tooltip's message needs to be checked directly if:
+    // 1. Tooltip.excludeFromSemantics is true, since in this case Tooltip
+    //    provides no semantics tooltip to the underlying RawTooltip.
+    // 2. Tooltip.message and Tooltip.richMessage are empty, since in this
+    //    case no RawTooltip is created.
+    if (widget is Tooltip) {
+      final String tooltipMessage = widget.message ?? widget.richMessage!.toPlainText();
+      if ((widget.excludeFromSemantics ?? false) || tooltipMessage.isEmpty) {
+        return message is RegExp ? message.hasMatch(tooltipMessage) : tooltipMessage == message;
+      }
+    }
+    return widget is RawTooltip &&
+        (message is RegExp
+            ? message.hasMatch(widget.semanticsTooltip ?? '')
+            : widget.semanticsTooltip == message);
+  }, skipOffstage: skipOffstage);
+}

packages/material_ui/test/navigation_bar_test.dart

@@ -14,6 +14,7 @@ import 'package:flutter/gestures.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:material_ui/material_ui.dart';
+import 'finders.dart';
 
 void main() {
   testWidgets('Navigation bar updates destinations when tapped', (WidgetTester tester) async {
@@ -440,15 +441,15 @@ void main() {
 
     expect(find.text('A'), findsOneWidget);
     await tester.longPress(find.text('A'));
-    expect(_findByTooltip('A tooltip'), findsOneWidget);
+    expect(findByTooltip('A tooltip'), findsOneWidget);
 
     expect(find.text('B'), findsOneWidget);
     await tester.longPress(find.text('B'));
-    expect(_findByTooltip('B'), findsOneWidget);
+    expect(findByTooltip('B'), findsOneWidget);
 
     expect(find.text('C'), findsOneWidget);
     await tester.longPress(find.text('C'));
-    expect(_findByTooltip('C'), findsNothing);
+    expect(findByTooltip('C'), findsNothing);
   });
 
   testWidgets('Navigation bar semantics', (WidgetTester tester) async {
@@ -1814,40 +1815,3 @@ TextStyle _getLabelStyle(WidgetTester tester, String text) {
       .text
       .style!;
 }
-
-/// Finds [RawTooltip] or [Tooltip] widgets with the given `message`.
-///
-/// ## Sample code
-///
-/// ```dart
-/// expect(find.byTooltip('Back'), findsOneWidget);
-/// expect(find.byTooltip(RegExp('Back.*')), findsNWidgets(2));
-/// ```
-///
-/// If the `skipOffstage` argument is true (the default), then this skips
-/// nodes that are [Offstage] or that are from inactive [Route]s.
-///
-/// This was copied from flutter_test, which uses flutter/material.dart.
-// TODO(justinmc): Port flutter_test to material_ui, then delete this method and
-// use that one.
-Finder _findByTooltip(Pattern message, {bool skipOffstage = true}) {
-  return find.byWidgetPredicate((Widget widget) {
-    // Compare RawTooltip's semantics tooltip with the given message.
-    // However, Tooltip's message needs to be checked directly if:
-    // 1. Tooltip.excludeFromSemantics is true, since in this case Tooltip
-    //    provides no semantics tooltip to the underlying RawTooltip.
-    // 2. Tooltip.message and Tooltip.richMessage are empty, since in this
-    //    case no RawTooltip is created.
-    if (widget is Tooltip) {
-      //if (widget.runtimeType == 'Tooltip') {
-      final String tooltipMessage = widget.message ?? widget.richMessage!.toPlainText();
-      if ((widget.excludeFromSemantics ?? false) || tooltipMessage.isEmpty) {
-        return message is RegExp ? message.hasMatch(tooltipMessage) : tooltipMessage == message;
-      }
-    }
-    return widget is RawTooltip &&
-        (message is RegExp
-            ? message.hasMatch(widget.semanticsTooltip ?? '')
-            : widget.semanticsTooltip == message);
-  }, skipOffstage: skipOffstage);
-}