log: Add reportErrorModally

Signed-off-by: Zixuan James Li <[email protected]>

Author: PIG208

lib/log.dart

@@ -36,6 +36,8 @@ bool debugLog(String message) {
 // `null` for the `message` parameter and promptly dismiss the reported errors.
 typedef ReportErrorCancellablyCallback = void Function(String? message, {String? details});
 
+typedef ReportErrorCallback = void Function(String title, {String? message});
+
 /// Show the user an error message, without requiring them to interact with it.
 ///
 /// Typically this shows a [SnackBar] containing the message.
@@ -53,10 +55,25 @@ typedef ReportErrorCancellablyCallback = void Function(String? message, {String?
 // the app.
 ReportErrorCancellablyCallback reportErrorToUserBriefly = defaultReportErrorToUserBriefly;
 
+/// Show the user a dismissable error message in a modal popup.
+///
+/// Typically this shows an [AlertDialog] with `title` as the title, `message`
+/// as the body.  If called before the app's widget tree is ready
+/// (see [ZulipApp.ready]), then we give up on showing the message to the user,
+/// and just log the message to the console.
+// This gets set in [ZulipApp].  We need this indirection to keep `lib/log.dart`
+// from importing widget code, because the file is a dependency for the rest of
+// the app.
+ReportErrorCallback reportErrorToUserModally = defaultReportErrorToUserModally;
+
 void defaultReportErrorToUserBriefly(String? message, {String? details}) {
   _reportErrorToConsole(message, details);
 }
 
+void defaultReportErrorToUserModally(String title, {String? message}) {
+  _reportErrorToConsole(title, message);
+}
+
 void _reportErrorToConsole(String? message, String? details) {
   // Error dismissing is a no-op for the console.
   if (message == null) return;

lib/widgets/app.dart

@@ -85,6 +85,7 @@ class ZulipApp extends StatefulWidget {
   static void debugReset() {
     _snackBarCount = 0;
     reportErrorToUserBriefly = defaultReportErrorToUserBriefly;
+    reportErrorToUserModally = defaultReportErrorToUserModally;
     _ready.dispose();
     _ready = ValueNotifier(false);
   }
@@ -128,10 +129,21 @@ class ZulipApp extends StatefulWidget {
     newSnackBar.closed.whenComplete(() => _snackBarCount--);
   }
 
+  /// The callback we normally use as [reportErrorToUserModally].
+  static void _reportErrorToUserModally(String title, {String? message}) {
+    assert(_ready.value);
+
+    showErrorDialog(
+      context: navigatorKey.currentContext!,
+      title: title,
+      message: message);
+  }
+
   void _declareReady() {
     assert(navigatorKey.currentContext != null);
     _ready.value = true;
     reportErrorToUserBriefly = _reportErrorToUserBriefly;
+    reportErrorToUserModally = _reportErrorToUserModally;
   }
 
   @override

test/widgets/app_test.dart

@@ -363,5 +363,24 @@ void main() {
       await tester.pumpAndSettle();
       check(findSnackBarByText('unrelated').evaluate()).single;
     });
+
+    testWidgets('reportErrorToUserModally', (tester) async {
+      addTearDown(testBinding.reset);
+      await tester.pumpWidget(const ZulipApp());
+      const title = 'test title';
+      const message = 'test message';
+
+      // Prior to app startup, reportErrorToUserModally only logs.
+      reportErrorToUserModally(title, message: message);
+      check(ZulipApp.ready).value.isFalse();
+      await tester.pump();
+      checkNoErrorDialog(tester);
+
+      check(ZulipApp.ready).value.isTrue();
+      // After app startup, reportErrorToUserModally displays an [AlertDialog].
+      reportErrorToUserModally(title, message: message);
+      await tester.pump();
+      checkErrorDialog(tester, expectedTitle: title, expectedMessage: message);
+    });
   });
 }