store: Reoprt non-transient polling errors.

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

Author: PIG208

lib/model/store.dart

@@ -801,6 +801,7 @@ class UpdateMachine {
             assert(debugLog('Error polling event queue for $store: $e\n'
                 'Backing off and retrying even though may be hopeless…'));
             // TODO tell user on non-transient error in polling
+            reportErrorToUserInDialog('Error loading server data. Will retry: $e');
             await (backoffMachine ??= BackoffMachine()).wait();
             assert(debugLog('… Backoff wait complete, retrying poll.'));
             continue;

test/model/store_test.dart

@@ -8,6 +8,7 @@ import 'package:zulip/api/model/events.dart';
 import 'package:zulip/api/model/model.dart';
 import 'package:zulip/api/route/events.dart';
 import 'package:zulip/api/route/messages.dart';
+import 'package:zulip/log.dart';
 import 'package:zulip/model/store.dart';
 import 'package:zulip/notifications/receive.dart';
 
@@ -427,7 +428,9 @@ void main() {
       check(store.userSettings!.twentyFourHourTime).isTrue();
     }));
 
-    void checkRetry(String description, void Function() prepareError) {
+    void checkRetry(String description, void Function() prepareError, {
+      String? errorMessage,
+    }) {
       test(description, () {
         awaitFakeAsync((async) async {
           await prepareStore(lastEventId: 1);
@@ -438,7 +441,13 @@ void main() {
           // Make the request, inducing an error in it.
           prepareError();
           updateMachine.debugAdvanceLoop();
+          check(debugLastReportedError).isNull();
           async.elapse(Duration.zero);
+          if (errorMessage == null) {
+            check(takeLastReportedError()).isNull();
+          } else {
+            check(takeLastReportedError()).isNotNull().contains(errorMessage);
+          }
           checkLastRequest(lastEventId: 1);
           check(store).isLoading.isTrue();
 
@@ -470,10 +479,12 @@ void main() {
 
       checkRetry('retries on ZulipApiException',
         () => connection.prepare(httpStatus: 400, json: {
-          'result': 'error', 'code': 'BAD_REQUEST', 'msg': 'Bad request'}));
+          'result': 'error', 'code': 'BAD_REQUEST', 'msg': 'Bad request'}),
+        errorMessage: 'Error loading server data. Will retry');
 
       checkRetry('retries on MalformedServerResponseException',
-        () => connection.prepare(httpStatus: 200, body: 'nonsense'));
+        () => connection.prepare(httpStatus: 200, body: 'nonsense'),
+        errorMessage: 'Error loading server data. Will retry');
     });
   });