login [nfc]: Pass GlobalStore to logOutAccount

This allows us to call it from model code when GlobalStore is available.

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

Author: PIG208

lib/widgets/actions.dart

@@ -20,9 +20,7 @@ import '../notifications/receive.dart';
 import 'dialog.dart';
 import 'store.dart';
 
-Future<void> logOutAccount(BuildContext context, int accountId) async {
-  final globalStore = GlobalStoreWidget.of(context);
-
+Future<void> logOutAccount(GlobalStore globalStore, int accountId) async {
   final account = globalStore.getAccount(accountId);
   if (account == null) return; // TODO(log)
 

lib/widgets/app.dart

@@ -265,7 +265,7 @@ class ChooseAccountPage extends StatelessWidget {
                   actionButtonText: zulipLocalizations.logOutConfirmationDialogConfirmButton,
                   onActionButtonPress: () {
                     // TODO error handling if db write fails?
-                    logOutAccount(context, accountId);
+                    logOutAccount(GlobalStoreWidget.of(context), accountId);
                   });
               },
               child: Text(zulipLocalizations.chooseAccountPageLogOutButton)),

test/widgets/actions_test.dart

@@ -109,7 +109,7 @@ void main() {
       final newConnection = separateConnection()
         ..prepare(delay: unregisterDelay, json: {'msg': '', 'result': 'success'});
 
-      final future = logOutAccount(context, eg.selfAccount.id);
+      final future = logOutAccount(testBinding.globalStore, eg.selfAccount.id);
       // Unregister-token request and account removal dispatched together
       checkSingleUnregisterRequest(newConnection);
       check(testBinding.globalStore.takeDoRemoveAccountCalls())
@@ -141,7 +141,7 @@ void main() {
       final newConnection = separateConnection()
         ..prepare(delay: unregisterDelay, exception: exception);
 
-      final future = logOutAccount(context, eg.selfAccount.id);
+      final future = logOutAccount(testBinding.globalStore, eg.selfAccount.id);
       // Unregister-token request and account removal dispatched together
       checkSingleUnregisterRequest(newConnection);
       check(testBinding.globalStore.takeDoRemoveAccountCalls())
@@ -210,8 +210,7 @@ void main() {
       final removedRoutes = <Route<dynamic>>[];
       testNavObserver.onRemoved = (route, prevRoute) => removedRoutes.add(route);
 
-      final context = tester.element(find.byType(MaterialApp));
-      final future = logOutAccount(context, account1.id);
+      final future = logOutAccount(testBinding.globalStore, account1.id);
       await tester.pump(TestGlobalStore.removeAccountDuration);
       await future;
       check(removedRoutes).single.identicalTo(account1Route);

test/widgets/home_test.dart

@@ -453,8 +453,7 @@ void main () {
     await tester.pump(); // wait for the loading page
     checkOnLoadingPage();
 
-    final element = tester.element(find.byType(MaterialApp));
-    final future = logOutAccount(element, eg.selfAccount.id);
+    final future = logOutAccount(testBinding.globalStore, eg.selfAccount.id);
     await tester.pump(TestGlobalStore.removeAccountDuration);
     await future;
     // No error expected from briefly not having
@@ -471,8 +470,7 @@ void main () {
     await tester.pump(); // wait for store
     checkOnHomePage(tester, expectedAccount: eg.selfAccount);
 
-    final element = tester.element(find.byType(HomePage));
-    final future = logOutAccount(element, eg.selfAccount.id);
+    final future = logOutAccount(testBinding.globalStore, eg.selfAccount.id);
     await tester.pump(TestGlobalStore.removeAccountDuration);
     await future;
     // No error expected from briefly not having