Clean settings model tests

benoitletondor · Oct 14, 2018 · 689ac3c · 689ac3c
1 parent bb9375f
commit 689ac3c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/lib/page/settings/state.dart b/lib/page/settings/state.dart
@@ -7,9 +7,7 @@ class SettingsState extends Union2Impl<SettingsStateLoading, SettingsStateLoad>
   static final Doublet<SettingsStateLoading, SettingsStateLoad> factory
     = const Doublet<SettingsStateLoading, SettingsStateLoad>();
 
-  final Union2<SettingsStateLoading, SettingsStateLoad> _union;
-
-  SettingsState._(this._union) : super(_union);
+  SettingsState._(Union2<SettingsStateLoading, SettingsStateLoad> union) : super(union);
 
   factory SettingsState.loading() => SettingsState._(factory.first(SettingsStateLoading()));
   factory SettingsState.load(String email, bool hapticFeedbackEnabled, bool analyticsEnabled) => SettingsState._(factory.second(SettingsStateLoad(email, hapticFeedbackEnabled, analyticsEnabled)));

diff --git a/test/page/settings/model_test.dart b/test/page/settings/model_test.dart
@@ -55,9 +55,11 @@ void main() {
 
     items.analyticsSettingStream.add(false);
     expect(await items.stateStream.next, SettingsState.load(email, true, false));
+    verify(items.authService.setAnalyticsEnabled(false));
 
     items.analyticsSettingStream.add(true);
     expect(await items.stateStream.next, SettingsState.load(email, true, true));
+    verify(items.authService.setAnalyticsEnabled(true));
   });
 
   test('test haptic feedback enabled change is sent to the UI', () async {
@@ -74,9 +76,11 @@ void main() {
 
     items.hapticFeedbackSettingStream.add(false);
     expect(await items.stateStream.next, SettingsState.load(email, false, true));
+    verify(items.authService.setHapticFeedbackEnabled(false));
 
     items.hapticFeedbackSettingStream.add(true);
     expect(await items.stateStream.next, SettingsState.load(email, true, true));
+    verify(items.authService.setHapticFeedbackEnabled(true));
   });
 
   test('clicking on ToS button push the ToS page', () async {