feat: fixing for Flutter 3.13.0 (#51)

* feat: fixing for Flutter 3.13.0

* fix workflows

* reverting flutter constraint

* Apply suggestions from code review

Co-authored-by: Jochum van der Ploeg <[email protected]>

---------

Co-authored-by: Jochum van der Ploeg <[email protected]>

Author: erickzanardo

.github/workflows/example.yaml

@@ -14,10 +14,10 @@ on:
 
 jobs:
   semantic_pull_request:
-    uses: VeryGoodOpenSource/very_good_workflows/.github/workflows/semantic_pull_request.yml@v1.11.0
+    uses: VeryGoodOpenSource/very_good_workflows/.github/workflows/semantic_pull_request.yml@v1
 
   build:
-    uses: VeryGoodOpenSource/very_good_workflows/.github/workflows/flutter_package.yml@v1.11.0
+    uses: VeryGoodOpenSource/very_good_workflows/.github/workflows/flutter_package.yml@v1
     with:
       flutter_channel: stable
       working_directory: example

lib/src/mock_navigator.dart

@@ -1,6 +1,27 @@
 import 'package:flutter/material.dart';
 import 'package:mocktail/mocktail.dart';
 
+class _MockMaterialPageRoute extends MaterialPageRoute<void> {
+  _MockMaterialPageRoute({required super.builder});
+
+  void hackOverlays() {
+    for (var i = 0; i < overlayEntries.length; i++) {
+      final state = OverlayState();
+      final entry = OverlayEntry(builder: (_) => const SizedBox());
+      try {
+        // We need to call insert since that is the only way to populate the
+        // `_overlay` field in the entry. But that method calls a setState,
+        // which will fail since we are not in a widget tree.
+        //
+        // By the time the setState is called, the attribute is already set
+        // so we just ignore the error and the hack will do its job.
+        state.insert(entry);
+      } catch (_) {}
+      overlayEntries[i] = entry;
+    }
+  }
+}
+
 class _FakeRoute<T> extends Fake implements Route<T> {}
 
 /// {@template mock_navigator_provider}
@@ -29,7 +50,13 @@ class MockNavigatorProvider extends Navigator {
 
   @override
   RouteFactory? get onGenerateRoute {
-    return (_) => MaterialPageRoute<dynamic>(builder: (_) => child);
+    return (_) {
+      final route = _MockMaterialPageRoute(builder: (_) => child);
+
+      navigator._routes.add(route);
+
+      return route;
+    };
   }
 }
 
@@ -48,6 +75,8 @@ class MockNavigator extends Mock
     registerFallbackValue(_FakeRoute<String>());
     registerFallbackValue(_FakeRoute<num>());
   }
+
+  final _routes = <_MockMaterialPageRoute>[];
 }
 
 /// A mixin necessary when implementing a [MockNavigator].
@@ -74,6 +103,15 @@ class _MockNavigatorState extends NavigatorState {
   @override
   Widget build(BuildContext context) => _child;
 
+  @override
+  void dispose() {
+    for (final route in _navigator._routes) {
+      route.hackOverlays();
+    }
+
+    super.dispose();
+  }
+
   @override
   Future<T?> push<T extends Object?>(Route<T> route) {
     return _navigator.push<T>(route);