From 818239e439d66bf7a2c7684fc6b2280f0eaea57e Mon Sep 17 00:00:00 2001
From: Chris Bobbe <cbobbe@zulip.com>
Date: Fri, 14 Jun 2024 14:45:53 -0700
Subject: [PATCH] inbox test: Set up whole ZulipApp, not just MaterialApp

This should help the tests be more representative generally; but in
particular, ZulipApp is going to start providing the stream color
swatches, instead of code in api/model, for #393. So, we have the
test setup use ZulipApp so that we don't get a crash in InboxPage.

Related: #393
---
 test/widgets/inbox_test.dart | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/test/widgets/inbox_test.dart b/test/widgets/inbox_test.dart
index b68221945e..44f4b2e221 100644
--- a/test/widgets/inbox_test.dart
+++ b/test/widgets/inbox_test.dart
@@ -4,9 +4,9 @@ import 'package:flutter_test/flutter_test.dart';
 import 'package:zulip/api/model/events.dart';
 import 'package:zulip/api/model/model.dart';
 import 'package:zulip/model/store.dart';
+import 'package:zulip/widgets/app.dart';
 import 'package:zulip/widgets/icons.dart';
 import 'package:zulip/widgets/inbox.dart';
-import 'package:zulip/widgets/store.dart';
 
 import '../example_data.dart' as eg;
 import '../flutter_checks.dart';
@@ -71,12 +71,10 @@ void main() {
     }
 
     await tester.pumpWidget(
-      GlobalStoreWidget(
-        child: MaterialApp(
-          navigatorObservers: [if (navigatorObserver != null) navigatorObserver],
-          home: PerAccountStoreWidget(
-            accountId: eg.selfAccount.id,
-            child: const InboxPage()))));
+      ZulipApp(navigatorObservers: [if (navigatorObserver != null) navigatorObserver]));
+    await tester.pump();
+    final navigator = await ZulipApp.navigator;
+    navigator.push(InboxPage.buildRoute(accountId: eg.selfAccount.id));
 
     // global store and per-account store get loaded
     await tester.pumpAndSettle();