Use Navigator.of(context) inside showAlertDialog so MaterialApp.route…

…r is not needed in tests # Conflicts: # ecommerce_app/lib/src/common_widgets/alert_dialogs.dart
bizz84 · Dec 20, 2023 · 820bba9 · 820bba9
1 parent 6378b70
commit 820bba9
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/ecommerce_app/lib/src/common_widgets/alert_dialogs.dart b/ecommerce_app/lib/src/common_widgets/alert_dialogs.dart
@@ -4,7 +4,6 @@ import 'package:ecommerce_app/src/localization/string_hardcoded.dart';
 import 'package:flutter/cupertino.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
-import 'package:go_router/go_router.dart';
 
 /// Generic function to show a platform-aware Material or Cupertino dialog
 Future<bool?> showAlertDialog({
@@ -28,22 +27,22 @@ Future<bool?> showAlertDialog({
               if (cancelActionText != null)
                 TextButton(
                   child: Text(cancelActionText),
-                  onPressed: () => context.pop(false),
+                  onPressed: () => Navigator.of(context).pop(false),
                 ),
               TextButton(
                 child: Text(defaultActionText),
-                onPressed: () => context.pop(true),
+                onPressed: () => Navigator.of(context).pop(true),
               ),
             ]
           : <Widget>[
               if (cancelActionText != null)
                 CupertinoDialogAction(
                   child: Text(cancelActionText),
-                  onPressed: () => context.pop(false),
+                  onPressed: () => Navigator.of(context).pop(false),
                 ),
               CupertinoDialogAction(
                 child: Text(defaultActionText),
-                onPressed: () => context.pop(true),
+                onPressed: () => Navigator.of(context).pop(true),
               ),
             ],
     ),