nav: Go straight to an account's home page on launch, when available

Author: gnprice

lib/widgets/app.dart

@@ -114,6 +114,9 @@ class ZulipApp extends StatelessWidget {
 
     return GlobalStoreWidget(
       child: Builder(builder: (context) {
+        final globalStore = GlobalStoreWidget.of(context);
+        // TODO(#524) choose initial account as last one used
+        final initialAccountId = globalStore.accounts.firstOrNull?.id;
         return MaterialApp(
           title: 'Zulip',
           localizationsDelegates: ZulipLocalizations.localizationsDelegates,
@@ -143,6 +146,9 @@ class ZulipApp extends StatelessWidget {
           onGenerateInitialRoutes: (_) {
             return [
               MaterialWidgetRoute(page: const ChooseAccountPage()),
+              if (initialAccountId != null) ...[
+                HomePage.buildRoute(accountId: initialAccountId),
+              ],
             ];
           });
         }));

test/notifications_test.dart

@@ -191,15 +191,21 @@ void main() {
   group('NotificationDisplayManager open', () {
     late List<Route<dynamic>> pushedRoutes;
 
-    void takeStartingRoutes() {
+    void takeStartingRoutes({bool withAccount = true}) {
       final expected = [
         (Subject it) => it.isA<WidgetRoute>().page.isA<ChooseAccountPage>(),
+        if (withAccount) ...[
+          (Subject it) => it.isA<MaterialAccountWidgetRoute>()
+            ..accountId.equals(eg.selfAccount.id)
+            ..page.isA<HomePage>(),
+        ],
       ];
       check(pushedRoutes.take(expected.length)).deepEquals(expected);
       pushedRoutes.removeRange(0, expected.length);
     }
 
-    Future<void> prepare(WidgetTester tester, {bool early = false}) async {
+    Future<void> prepare(WidgetTester tester,
+        {bool early = false, bool withAccount = true}) async {
       await init();
       pushedRoutes = [];
       final testNavObserver = TestNavigatorObserver()
@@ -210,7 +216,7 @@ void main() {
         return;
       }
       await tester.pump();
-      takeStartingRoutes();
+      takeStartingRoutes(withAccount: withAccount);
       check(pushedRoutes).isEmpty();
     }
 
@@ -250,7 +256,7 @@ void main() {
     });
 
     testWidgets('no accounts', (tester) async {
-      await prepare(tester);
+      await prepare(tester, withAccount: false);
       await openNotification(eg.selfAccount, eg.streamMessage());
       check(pushedRoutes).isEmpty();
     });

test/widgets/app_test.dart

@@ -4,6 +4,7 @@ import 'package:flutter_test/flutter_test.dart';
 import 'package:zulip/widgets/app.dart';
 import 'package:zulip/widgets/page.dart';
 
+import '../example_data.dart' as eg;
 import '../model/binding.dart';
 import '../test_navigation.dart';
 import 'page_checks.dart';
@@ -23,11 +24,27 @@ void main() {
       return pushedRoutes;
     }
 
-    testWidgets('go to choose-account page', (tester) async {
+    testWidgets('when no accounts, go to choose account', (tester) async {
       addTearDown(testBinding.reset);
       check(await initialRoutes(tester)).deepEquals([
         (Subject it) => it.isA<WidgetRoute>().page.isA<ChooseAccountPage>(),
       ]);
     });
+
+    testWidgets('when have accounts, go to home page for first account', (tester) async {
+      addTearDown(testBinding.reset);
+
+      // We'll need per-account data for the account that a page will be opened
+      // for, but not for the other account.
+      await testBinding.globalStore.add(eg.selfAccount, eg.initialSnapshot());
+      await testBinding.globalStore.insertAccount(eg.otherAccount.toCompanion(false));
+
+      check(await initialRoutes(tester)).deepEquals([
+        (Subject it) => it.isA<WidgetRoute>().page.isA<ChooseAccountPage>(),
+        (Subject it) => it.isA<MaterialAccountWidgetRoute>()
+          ..accountId.equals(eg.selfAccount.id)
+          ..page.isA<HomePage>(),
+      ]);
+    });
   });
 }