From be690c81943073b5ce8e172c4e49d266d528036a Mon Sep 17 00:00:00 2001 From: just-stuff-tm Date: Sun, 15 Mar 2026 16:48:40 -0400 Subject: [PATCH] fix: provide AppSettingsService in tcp_flow_test TcpScreen.initState reads AppSettingsService from context to pre-fill host/port fields, but the test helper only provided MeshCoreConnector. Switch to MultiProvider so AppSettingsService is also in the widget tree. --- test/screens/tcp_flow_test.dart | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/test/screens/tcp_flow_test.dart b/test/screens/tcp_flow_test.dart index 725388a..1d8174c 100644 --- a/test/screens/tcp_flow_test.dart +++ b/test/screens/tcp_flow_test.dart @@ -6,6 +6,7 @@ import 'package:meshcore_open/connector/meshcore_connector.dart'; import 'package:meshcore_open/l10n/app_localizations.dart'; import 'package:meshcore_open/screens/scanner_screen.dart'; import 'package:meshcore_open/screens/tcp_screen.dart'; +import 'package:meshcore_open/services/app_settings_service.dart'; class _FakeMeshCoreConnector extends MeshCoreConnector { _FakeMeshCoreConnector(); @@ -44,8 +45,13 @@ Widget _buildTestApp({ required Widget child, Locale? locale, }) { - return ChangeNotifierProvider.value( - value: connector, + return MultiProvider( + providers: [ + ChangeNotifierProvider.value(value: connector), + ChangeNotifierProvider( + create: (_) => AppSettingsService(), + ), + ], child: MaterialApp( locale: locale, localizationsDelegates: AppLocalizations.localizationsDelegates,