added Stream.empty() tests

README.md

@@ -1,14 +1,10 @@
-<p align="center">
+<img src="resources/social_preview.png" width="100%" />
-  <img src="assets/launcher_icon_circle.png" width="100" height="100">
-</p>
-<p align="center", style="font-size:60px;">
-  <b>Material Lightmeter</b>
-</p>
 
 # Table of contents
 
 - [Table of contents](#table-of-contents)
 - [Backstory](#backstory)
+- [Screenshots](#screenshots)
 - [Build](#build)
 - [Contribution](#contribution)
 - [iOS Limitations](#ios-limitations)
@@ -21,7 +17,7 @@ But as the existing repo contained some sensitive data, that I've pushed due to
 
 Without further delay behold my new Lightmeter app inspired by Material You (a.k.a. M3)
 
-# Features
+# Screenshots
 
 <p float="center">
   <img src="https://lh3.googleusercontent.com/8Sd-pmNcQ0xAr5opuTeJKWr2OXeQvCoFSdVDSoKQSHHKeNmqF71hqeAdm3yjunY12zY" width="18.8%" />

lib/data/light_sensor_service.dart

@@ -17,5 +17,10 @@ class LightSensorService {
     }
   }
 
-  Stream<int> luxStream() => LightSensor.lightSensorStream;
+  Stream<int> luxStream() {
+    if (!localPlatform.isAndroid) {
+      return const Stream<int>.empty();
+    }
+    return LightSensor.lightSensorStream;
+  }
 }

lib/data/models/volume_action.dart

@@ -1,3 +1,3 @@
-enum VolumeAction { shutter, zoom, none }
+enum VolumeAction { shutter, none }
 
 enum VolumeKey { up, down }

lib/l10n/intl_en.arb

@@ -56,9 +56,7 @@
     "general": "General",
     "keepScreenOn": "Keep screen on",
     "haptics": "Haptics",
-    "volumeKeysAction": "Volume keys action",
+    "volumeKeysAction": "Shutter by volume keys",
-    "shutter": "Shutter",
-    "zoom": "Zoom",
     "language": "Language",
     "chooseLanguage": "Choose language",
     "theme": "Theme",

lib/l10n/intl_fr.arb

@@ -56,9 +56,7 @@
     "general": "Général",
     "keepScreenOn": "Garder l'écran allumé",
     "haptics": "Haptiques",
-    "volumeKeysAction": "Action des touches de volume",
+    "volumeKeysAction": "Obturateur par boutons de volume",
-    "shutter": "Obturateur",
-    "zoom": "Zoom",
     "language": "Langue",
     "chooseLanguage": "Choisissez la langue",
     "theme": "Thème",

lib/l10n/intl_ru.arb

@@ -56,9 +56,7 @@
     "general": "Общие",
     "keepScreenOn": "Запрет блокировки",
     "haptics": "Вибрация",
-    "volumeKeysAction": "Кнопки регулировки громкости",
+    "volumeKeysAction": "Затвор по кнопкам громкости",
-    "shutter": "Затвор",
-    "zoom": "Зум",
     "language": "Язык",
     "chooseLanguage": "Выберите язык",
     "theme": "Тема",

lib/screens/metering/components/camera_container/bloc_container_camera.dart

@@ -8,7 +8,6 @@ import 'package:camera/camera.dart';
 import 'package:exif/exif.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:lightmeter/data/models/volume_action.dart';
 import 'package:lightmeter/interactors/metering_interactor.dart';
 import 'package:lightmeter/screens/metering/communication/bloc_communication_metering.dart';
 import 'package:lightmeter/screens/metering/communication/event_communication_metering.dart'
@@ -19,12 +18,10 @@ import 'package:lightmeter/screens/metering/components/camera_container/event_co
 import 'package:lightmeter/screens/metering/components/camera_container/models/camera_error_type.dart';
 import 'package:lightmeter/screens/metering/components/camera_container/state_container_camera.dart';
 import 'package:lightmeter/screens/metering/components/shared/ev_source_base/bloc_base_ev_source.dart';
-import 'package:lightmeter/screens/metering/components/shared/volume_keys_notifier/notifier_volume_keys.dart';
 import 'package:lightmeter/utils/log_2.dart';
 
 class CameraContainerBloc extends EvSourceBlocBase<CameraContainerEvent, CameraContainerState> {
   final MeteringInteractor _meteringInteractor;
-  final VolumeKeysNotifier _volumeKeysNotifier;
   late final _WidgetsBindingObserver _observer;
 
   CameraController? _cameraController;
@@ -44,13 +41,11 @@ class CameraContainerBloc extends EvSourceBlocBase<CameraContainerEvent, CameraC
 
   CameraContainerBloc(
     this._meteringInteractor,
-    this._volumeKeysNotifier,
     MeteringCommunicationBloc communicationBloc,
   ) : super(
           communicationBloc,
           const CameraInitState(),
         ) {
-    _volumeKeysNotifier.addListener(onVolumeKey);
     _observer = _WidgetsBindingObserver(_appLifecycleStateObserver);
     WidgetsBinding.instance.addObserver(_observer);
 
@@ -66,7 +61,6 @@ class CameraContainerBloc extends EvSourceBlocBase<CameraContainerEvent, CameraC
   @override
   Future<void> close() async {
     WidgetsBinding.instance.removeObserver(_observer);
-    _volumeKeysNotifier.removeListener(onVolumeKey);
     unawaited(_cameraController?.dispose().then((_) => _cameraController = null));
     communicationBloc.add(communication_event.MeteringEndedEvent(_ev100));
     return super.close();
@@ -165,7 +159,8 @@ class CameraContainerBloc extends EvSourceBlocBase<CameraContainerEvent, CameraC
   }
 
   Future<void> _onDeinitialize(DeinitializeEvent _, Emitter emit) async {
-    emit(const CameraLoadingState());
+    emit(const CameraInitState());
+    communicationBloc.add(communication_event.MeteringEndedEvent(_ev100));
     await _cameraController?.dispose().then((_) => _cameraController = null);
   }
 
@@ -245,18 +240,6 @@ class CameraContainerBloc extends EvSourceBlocBase<CameraContainerEvent, CameraC
       }
     }
   }
-
-  @visibleForTesting
-  void onVolumeKey() {
-    if (_meteringInteractor.volumeAction == VolumeAction.zoom) {
-      switch (_volumeKeysNotifier.value) {
-        case VolumeKey.up:
-          add(ZoomChangedEvent(_currentZoom + 0.5));
-        case VolumeKey.down:
-          add(ZoomChangedEvent(_currentZoom - 0.5));
-      }
-    }
-  }
 }
 
 /// This is needed only because we cannot use `with` with mixins

lib/screens/metering/components/camera_container/provider_container_camera.dart

@@ -7,7 +7,6 @@ import 'package:lightmeter/screens/metering/communication/bloc_communication_met
 import 'package:lightmeter/screens/metering/components/camera_container/bloc_container_camera.dart';
 import 'package:lightmeter/screens/metering/components/camera_container/event_container_camera.dart';
 import 'package:lightmeter/screens/metering/components/camera_container/widget_container_camera.dart';
-import 'package:lightmeter/screens/metering/components/shared/volume_keys_notifier/notifier_volume_keys.dart';
 import 'package:lightmeter/utils/inherited_generics.dart';
 import 'package:m3_lightmeter_resources/m3_lightmeter_resources.dart';
 
@@ -41,7 +40,6 @@ class CameraContainerProvider extends StatelessWidget {
       lazy: false,
       create: (context) => CameraContainerBloc(
         context.get<MeteringInteractor>(),
-        context.get<VolumeKeysNotifier>(),
         context.read<MeteringCommunicationBloc>(),
       )..add(const RequestPermissionEvent()),
       child: CameraContainer(

lib/screens/metering/components/light_sensor_container/bloc_container_light_sensor.dart

@@ -25,6 +25,7 @@ class LightSensorContainerBloc
           communicationBloc,
           const LightSensorContainerState(null),
         ) {
+    on<StartLuxMeteringEvent>(_onStartLuxMeteringEvent);
     on<LuxMeteringEvent>(_onLuxMeteringEvent);
     on<CancelLuxMeteringEvent>(_onCancelLuxMeteringEvent);
   }
@@ -34,22 +35,27 @@ class LightSensorContainerBloc
     switch (communicationState) {
       case communication_states.MeasureState():
         if (_luxSubscriptions == null) {
-          _startMetering();
+          add(const StartLuxMeteringEvent());
         } else {
-          _cancelMetering();
+          add(const CancelLuxMeteringEvent());
         }
       case communication_states.SettingsOpenedState():
-        _cancelMetering();
+        add(const CancelLuxMeteringEvent());
       default:
     }
   }
 
   @override
   Future<void> close() async {
-    _cancelMetering();
+    communicationBloc.add(communication_event.MeteringEndedEvent(state.ev100));
+    _luxSubscriptions?.cancel().then((_) => _luxSubscriptions = null);
     return super.close();
   }
 
+  void _onStartLuxMeteringEvent(StartLuxMeteringEvent event, _) {
+    _luxSubscriptions = _meteringInteractor.luxStream().listen((lux) => add(LuxMeteringEvent(lux)));
+  }
+
   void _onLuxMeteringEvent(LuxMeteringEvent event, Emitter<LightSensorContainerState> emit) {
     final ev100 = log2(event.lux.toDouble() / 2.5) + _meteringInteractor.lightSensorEvCalibration;
     emit(LightSensorContainerState(ev100));
@@ -57,14 +63,6 @@ class LightSensorContainerBloc
   }
 
   void _onCancelLuxMeteringEvent(CancelLuxMeteringEvent event, _) {
-    _cancelMetering();
-  }
-
-  void _startMetering() {
-    _luxSubscriptions = _meteringInteractor.luxStream().listen((lux) => add(LuxMeteringEvent(lux)));
-  }
-
-  void _cancelMetering() {
     communicationBloc.add(communication_event.MeteringEndedEvent(state.ev100));
     _luxSubscriptions?.cancel().then((_) => _luxSubscriptions = null);
   }

lib/screens/metering/components/light_sensor_container/event_container_light_sensor.dart

@@ -2,6 +2,10 @@ abstract class LightSensorContainerEvent {
   const LightSensorContainerEvent();
 }
 
+class StartLuxMeteringEvent extends LightSensorContainerEvent {
+  const StartLuxMeteringEvent();
+}
+
 class LuxMeteringEvent extends LightSensorContainerEvent {
   final int lux;
 

lib/screens/settings/components/general/components/caffeine/widget_list_tile_caffeine.dart

@@ -1,6 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:lightmeter/generated/l10n.dart';
+import 'package:lightmeter/res/dimens.dart';
 
 import 'package:lightmeter/screens/settings/components/general/components/caffeine/bloc_list_tile_caffeine.dart';
 
@@ -15,6 +16,7 @@ class CaffeineListTile extends StatelessWidget {
         title: Text(S.of(context).keepScreenOn),
         value: state,
         onChanged: context.read<CaffeineListTileBloc>().onCaffeineChanged,
+        contentPadding: const EdgeInsets.symmetric(horizontal: Dimens.paddingM),
       ),
     );
   }

lib/screens/settings/components/general/components/haptics/widget_list_tile_haptics.dart

@@ -1,6 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:lightmeter/generated/l10n.dart';
+import 'package:lightmeter/res/dimens.dart';
 
 import 'package:lightmeter/screens/settings/components/general/components/haptics/bloc_list_tile_haptics.dart';
 
@@ -15,6 +16,7 @@ class HapticsListTile extends StatelessWidget {
         title: Text(S.of(context).haptics),
         value: state,
         onChanged: context.read<HapticsListTileBloc>().onHapticsChanged,
+        contentPadding: const EdgeInsets.symmetric(horizontal: Dimens.paddingM),
       ),
     );
   }

lib/screens/settings/components/general/components/volume_actions/bloc_list_tile_volume_actions.dart

@@ -2,15 +2,15 @@ import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:lightmeter/data/models/volume_action.dart';
 import 'package:lightmeter/interactors/settings_interactor.dart';
 
-class VolumeActionsListTileBloc extends Cubit<VolumeAction> {
+class VolumeActionsListTileBloc extends Cubit<bool> {
   final SettingsInteractor _settingsInteractor;
 
   VolumeActionsListTileBloc(
     this._settingsInteractor,
-  ) : super(_settingsInteractor.volumeAction);
+  ) : super(_settingsInteractor.volumeAction == VolumeAction.shutter);
 
-  void onVolumeActionChanged(VolumeAction value) {
+  void onVolumeActionChanged(bool value) {
-    _settingsInteractor.setVolumeAction(value);
+    _settingsInteractor.setVolumeAction(value ? VolumeAction.shutter : VolumeAction.none);
 
     // while in settings we allow system to handle volume
     // so that volume keys action works only when necessary - on the metering screen

lib/screens/settings/components/general/components/volume_actions/widget_list_tile_volume_actions.dart

@@ -1,48 +1,22 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:lightmeter/data/models/volume_action.dart';
 import 'package:lightmeter/generated/l10n.dart';
+import 'package:lightmeter/res/dimens.dart';
 import 'package:lightmeter/screens/settings/components/general/components/volume_actions/bloc_list_tile_volume_actions.dart';
-import 'package:lightmeter/screens/settings/components/shared/dialog_picker.dart/widget_dialog_picker.dart';
 
 class VolumeActionsListTile extends StatelessWidget {
   const VolumeActionsListTile({super.key});
 
   @override
   Widget build(BuildContext context) {
-    return BlocBuilder<VolumeActionsListTileBloc, VolumeAction>(
+    return BlocBuilder<VolumeActionsListTileBloc, bool>(
-      builder: (context, state) => ListTile(
+      builder: (context, state) => SwitchListTile(
-        leading: const Icon(Icons.volume_up),
+        secondary: const Icon(Icons.volume_up),
         title: Text(S.of(context).volumeKeysAction),
-        trailing: Text(actionToString(context, state)),
+        value: state,
-        onTap: () {
+        onChanged: context.read<VolumeActionsListTileBloc>().onVolumeActionChanged,
-          showDialog<VolumeAction>(
+        contentPadding: const EdgeInsets.symmetric(horizontal: Dimens.paddingM),
-            context: context,
-            builder: (_) => DialogPicker<VolumeAction>(
-              icon: Icons.volume_up,
-              title: S.of(context).volumeKeysAction,
-              selectedValue: state,
-              values: VolumeAction.values,
-              titleAdapter: (context, value) => actionToString(context, value),
-            ),
-          ).then((value) {
-            if (value != null) {
-              context.read<VolumeActionsListTileBloc>().onVolumeActionChanged(value);
-            }
-          });
-        },
       ),
     );
   }
-
-  String actionToString(BuildContext context, VolumeAction themeType) {
-    switch (themeType) {
-      case VolumeAction.shutter:
-        return S.of(context).shutter;
-      case VolumeAction.zoom:
-        return S.of(context).zoom;
-      case VolumeAction.none:
-        return S.of(context).none;
-    }
-  }
 }

lib/screens/settings/components/theme/components/dynamic_color/widget_list_tile_dynamic_color.dart

@@ -2,6 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:lightmeter/data/models/dynamic_colors_state.dart';
 import 'package:lightmeter/generated/l10n.dart';
 import 'package:lightmeter/providers/theme_provider.dart';
+import 'package:lightmeter/res/dimens.dart';
 import 'package:lightmeter/utils/inherited_generics.dart';
 
 class DynamicColorListTile extends StatelessWidget {
@@ -14,6 +15,7 @@ class DynamicColorListTile extends StatelessWidget {
       title: Text(S.of(context).dynamicColor),
       value: context.listen<DynamicColorState>() == DynamicColorState.enabled,
       onChanged: ThemeProvider.of(context).enableDynamicColor,
+      contentPadding: const EdgeInsets.symmetric(horizontal: Dimens.paddingM),
     );
   }
 }

test/data/light_sensor_service_test.dart

@@ -83,4 +83,16 @@ void main() {
       });
     },
   );
+
+  group('luxStream', () {
+    // test('Android', () async {
+    //   when(() => localPlatform.isAndroid).thenReturn(true);
+    //   expect(service.luxStream(), const Stream.empty());
+    // });
+
+    test('iOS', () async {
+      when(() => localPlatform.isAndroid).thenReturn(false);
+      expect(service.luxStream(), const Stream<int>.empty());
+    });
+  });
 }

test/data/volume_events_service_test.dart

@@ -58,4 +58,16 @@ void main() {
       expectLater(service.setVolumeHandling(false), completion(false));
     });
   });
+
+  group('volumeButtonsEventStream', () {
+    // test('Android', () async {
+    //   when(() => localPlatform.isAndroid).thenReturn(true);
+    //   expect(service.volumeButtonsEventStream(), const Stream.empty());
+    // });
+
+    test('iOS', () async {
+      when(() => localPlatform.isAndroid).thenReturn(false);
+      expect(service.volumeButtonsEventStream(), const Stream<int>.empty());
+    });
+  });
 }

test/screens/metering/bloc_metering_test.dart

@@ -641,19 +641,6 @@ void main() {
         expect: () => [isA<LoadingState>()],
       );
 
-      blocTest<MeteringBloc, MeteringState>(
-        'onVolumeKey & VolumeAction.zoom',
-        build: () => bloc,
-        act: (bloc) async {
-          bloc.onVolumeKey();
-        },
-        setUp: () {
-          when(() => meteringInteractor.volumeAction).thenReturn(VolumeAction.zoom);
-        },
-        verify: (_) {},
-        expect: () => [],
-      );
-
       blocTest<MeteringBloc, MeteringState>(
         'onVolumeKey & VolumeAction.none',
         build: () => bloc,
@@ -668,4 +655,25 @@ void main() {
       );
     },
   );
+
+  group(
+    '`SettingOpenedEvent`/`SettingsClosedEvent`',
+    () {
+      blocTest<MeteringBloc, MeteringState>(
+        'Settings opened & closed',
+        build: () => bloc,
+        act: (bloc) async {
+          bloc.add(const SettingsOpenedEvent());
+          bloc.add(const SettingsClosedEvent());
+        },
+        verify: (_) {
+          verify(() => communicationBloc.add(const communication_events.SettingsOpenedEvent()))
+              .called(1);
+          verify(() => communicationBloc.add(const communication_events.SettingsClosedEvent()))
+              .called(1);
+        },
+        expect: () => [],
+      );
+    },
+  );
 }

test/screens/metering/communication/bloc_communication_metering_test.dart

@@ -98,4 +98,30 @@ void main() {
       );
     },
   );
+
+  group(
+    '`SettingsOpenedEvent`/`SettingsClosedEvent`',
+    () {
+      blocTest<MeteringCommunicationBloc, MeteringCommunicationState>(
+        'Multiple consequtive settings events',
+        build: () => bloc,
+        act: (bloc) async {
+          bloc.add(const SettingsOpenedEvent());
+          bloc.add(const SettingsOpenedEvent());
+          bloc.add(const SettingsOpenedEvent());
+          bloc.add(const SettingsClosedEvent());
+          bloc.add(const SettingsClosedEvent());
+          bloc.add(const SettingsClosedEvent());
+          bloc.add(const SettingsOpenedEvent());
+          bloc.add(const SettingsClosedEvent());
+        },
+        expect: () => [
+          isA<SettingsOpenedState>(),
+          isA<SettingsClosedState>(),
+          isA<SettingsOpenedState>(),
+          isA<SettingsClosedState>(),
+        ],
+      );
+    },
+  );
 }

test/screens/metering/components/camera/bloc_container_camera_test.dart

@@ -2,7 +2,6 @@ import 'package:bloc_test/bloc_test.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter_test/flutter_test.dart';
-import 'package:lightmeter/data/models/volume_action.dart';
 import 'package:lightmeter/interactors/metering_interactor.dart';
 import 'package:lightmeter/screens/metering/communication/bloc_communication_metering.dart';
 import 'package:lightmeter/screens/metering/communication/event_communication_metering.dart'
@@ -13,13 +12,10 @@ import 'package:lightmeter/screens/metering/components/camera_container/bloc_con
 import 'package:lightmeter/screens/metering/components/camera_container/event_container_camera.dart';
 import 'package:lightmeter/screens/metering/components/camera_container/models/camera_error_type.dart';
 import 'package:lightmeter/screens/metering/components/camera_container/state_container_camera.dart';
-import 'package:lightmeter/screens/metering/components/shared/volume_keys_notifier/notifier_volume_keys.dart';
 import 'package:mocktail/mocktail.dart';
 
 class _MockMeteringInteractor extends Mock implements MeteringInteractor {}
 
-class _MockVolumeKeysNotifier extends Mock implements VolumeKeysNotifier {}
-
 class _MockMeteringCommunicationBloc extends MockBloc<
     communication_events.MeteringCommunicationEvent,
     communication_states.MeteringCommunicationState> implements MeteringCommunicationBloc {}
@@ -28,7 +24,6 @@ void main() {
   TestWidgetsFlutterBinding.ensureInitialized();
 
   late _MockMeteringInteractor meteringInteractor;
-  late _MockVolumeKeysNotifier volumeKeysNotifier;
   late _MockMeteringCommunicationBloc communicationBloc;
   late CameraContainerBloc bloc;
 
@@ -117,7 +112,6 @@ void main() {
 
   setUpAll(() {
     meteringInteractor = _MockMeteringInteractor();
-    volumeKeysNotifier = _MockVolumeKeysNotifier();
     communicationBloc = _MockMeteringCommunicationBloc();
 
     when(() => meteringInteractor.cameraEvCalibration).thenReturn(0.0);
@@ -127,7 +121,6 @@ void main() {
   setUp(() {
     bloc = CameraContainerBloc(
       meteringInteractor,
-      volumeKeysNotifier,
       communicationBloc,
     );
     TestDefaultBinaryMessengerBinding.instance.defaultBinaryMessenger
@@ -317,6 +310,30 @@ void main() {
         },
         expect: () => [
           ...initializedStateSequence,
+          const CameraInitState(),
+          ...initializedStateSequence,
+        ],
+      );
+
+      blocTest<CameraContainerBloc, CameraContainerState>(
+        'onCommunicationState',
+        setUp: () {
+          when(() => meteringInteractor.checkCameraPermission()).thenAnswer((_) async => true);
+        },
+        build: () => bloc,
+        act: (bloc) async {
+          bloc.add(const InitializeEvent());
+          await Future.delayed(Duration.zero);
+          bloc.onCommunicationState(const communication_states.SettingsOpenedState());
+          await Future.delayed(Duration.zero);
+          bloc.onCommunicationState(const communication_states.SettingsClosedState());
+        },
+        verify: (_) {
+          verify(() => meteringInteractor.checkCameraPermission()).called(2);
+        },
+        expect: () => [
+          ...initializedStateSequence,
+          const CameraInitState(),
           ...initializedStateSequence,
         ],
       );
@@ -483,123 +500,6 @@ void main() {
       );
     },
   );
-
-  group(
-    '`Volume keys shutter action`',
-    () {
-      blocTest<CameraContainerBloc, CameraContainerState>(
-        'Add/remove listener',
-        build: () => bloc,
-        verify: (_) {
-          verify(() => volumeKeysNotifier.addListener(bloc.onVolumeKey)).called(1);
-          verify(() => volumeKeysNotifier.removeListener(bloc.onVolumeKey)).called(1);
-        },
-        expect: () => [],
-      );
-
-      blocTest<CameraContainerBloc, CameraContainerState>(
-        'onVolumeKey & VolumeAction.shutter',
-        build: () => bloc,
-        act: (bloc) async {
-          bloc.onVolumeKey();
-        },
-        setUp: () {
-          when(() => meteringInteractor.volumeAction).thenReturn(VolumeAction.shutter);
-        },
-        verify: (_) {},
-        expect: () => [],
-      );
-
-      blocTest<CameraContainerBloc, CameraContainerState>(
-        'onVolumeKey.up & VolumeAction.zoom',
-        build: () => bloc,
-        act: (bloc) async {
-          bloc.add(const InitializeEvent());
-          await Future.delayed(Duration.zero);
-          bloc.onVolumeKey();
-          await Future.delayed(Duration.zero);
-          bloc.onVolumeKey();
-          await Future.delayed(Duration.zero);
-          bloc.onVolumeKey();
-        },
-        setUp: () {
-          when(() => meteringInteractor.checkCameraPermission()).thenAnswer((_) async => true);
-          when(() => meteringInteractor.volumeAction).thenReturn(VolumeAction.zoom);
-          when(() => volumeKeysNotifier.value).thenReturn(VolumeKey.up);
-        },
-        verify: (_) {},
-        expect: () => [
-          ...initializedStateSequence,
-          isA<CameraActiveState>()
-              .having((state) => state.zoomRange, 'zoomRange', const RangeValues(1.0, 7.0))
-              .having((state) => state.currentZoom, 'currentZoom', 1.5)
-              .having(
-                (state) => state.exposureOffsetRange,
-                'exposureOffsetRange',
-                const RangeValues(-4.0, 4.0),
-              )
-              .having((state) => state.exposureOffsetStep, 'exposureOffsetStep', 0.1666666)
-              .having((state) => state.currentExposureOffset, 'currentExposureOffset', 0.0),
-          isA<CameraActiveState>()
-              .having((state) => state.zoomRange, 'zoomRange', const RangeValues(1.0, 7.0))
-              .having((state) => state.currentZoom, 'currentZoom', 2.0)
-              .having(
-                (state) => state.exposureOffsetRange,
-                'exposureOffsetRange',
-                const RangeValues(-4.0, 4.0),
-              )
-              .having((state) => state.exposureOffsetStep, 'exposureOffsetStep', 0.1666666)
-              .having((state) => state.currentExposureOffset, 'currentExposureOffset', 0.0),
-          isA<CameraActiveState>()
-              .having((state) => state.zoomRange, 'zoomRange', const RangeValues(1.0, 7.0))
-              .having((state) => state.currentZoom, 'currentZoom', 2.5)
-              .having(
-                (state) => state.exposureOffsetRange,
-                'exposureOffsetRange',
-                const RangeValues(-4.0, 4.0),
-              )
-              .having((state) => state.exposureOffsetStep, 'exposureOffsetStep', 0.1666666)
-              .having((state) => state.currentExposureOffset, 'currentExposureOffset', 0.0),
-        ],
-      );
-
-      blocTest<CameraContainerBloc, CameraContainerState>(
-        'onVolumeKey.down & VolumeAction.zoom',
-        build: () => bloc,
-        act: (bloc) async {
-          bloc.add(const InitializeEvent());
-          await Future.delayed(Duration.zero);
-          bloc.onVolumeKey();
-          await Future.delayed(Duration.zero);
-          bloc.onVolumeKey();
-          await Future.delayed(Duration.zero);
-          bloc.onVolumeKey();
-        },
-        setUp: () {
-          when(() => meteringInteractor.checkCameraPermission()).thenAnswer((_) async => true);
-          when(() => meteringInteractor.volumeAction).thenReturn(VolumeAction.zoom);
-          when(() => volumeKeysNotifier.value).thenReturn(VolumeKey.down);
-        },
-        verify: (_) {},
-        expect: () => [
-          ...initializedStateSequence,
-        ],
-      );
-
-      blocTest<CameraContainerBloc, CameraContainerState>(
-        'onVolumeKey & VolumeAction.none',
-        build: () => bloc,
-        act: (bloc) async {
-          bloc.onVolumeKey();
-        },
-        setUp: () {
-          when(() => meteringInteractor.volumeAction).thenReturn(VolumeAction.none);
-        },
-        verify: (_) {},
-        expect: () => [],
-      );
-    },
-  );
 }
 
 extension _MethodChannelMock on MethodChannel {

test/screens/metering/components/light_sensor/bloc_container_light_sensor_test.dart

@@ -78,4 +78,67 @@ void main() {
       );
     },
   );
+
+  group(
+    '`communication_states.SettingsOpenedState()`',
+    () {
+      const List<int> luxIterable = [1, 2, 2, 2, 3];
+      final List<double> resultList = luxIterable.map((lux) => log2(lux / 2.5)).toList();
+      blocTest<LightSensorContainerBloc, LightSensorContainerState>(
+        'Metering is already canceled',
+        build: () => bloc,
+        setUp: () {
+          when(() => meteringInteractor.luxStream())
+              .thenAnswer((_) => Stream.fromIterable(luxIterable));
+          when(() => meteringInteractor.lightSensorEvCalibration).thenReturn(0.0);
+        },
+        act: (bloc) async {
+          bloc.onCommunicationState(const communication_states.SettingsOpenedState());
+        },
+        verify: (_) {
+          verifyNever(() => meteringInteractor.luxStream().listen((_) {}));
+          verifyNever(() => meteringInteractor.lightSensorEvCalibration);
+          verify(() {
+            communicationBloc.add(const communication_events.MeteringEndedEvent(null));
+          }).called(2); // +1 from dispose
+        },
+        expect: () => [],
+      );
+
+      blocTest<LightSensorContainerBloc, LightSensorContainerState>(
+        'Metering is in progress',
+        build: () => bloc,
+        setUp: () {
+          when(() => meteringInteractor.luxStream())
+              .thenAnswer((_) => Stream.fromIterable(luxIterable));
+          when(() => meteringInteractor.lightSensorEvCalibration).thenReturn(0.0);
+        },
+        act: (bloc) async {
+          bloc.onCommunicationState(const communication_states.MeasureState());
+          await Future.delayed(Duration.zero);
+          bloc.onCommunicationState(const communication_states.SettingsOpenedState());
+          bloc.onCommunicationState(const communication_states.SettingsClosedState());
+        },
+        verify: (_) {
+          verify(() => meteringInteractor.luxStream().listen((_) {})).called(1);
+          verify(() => meteringInteractor.lightSensorEvCalibration).called(5);
+          verify(() {
+            communicationBloc.add(communication_events.MeteringInProgressEvent(resultList.first));
+          }).called(1);
+          verify(() {
+            communicationBloc.add(communication_events.MeteringInProgressEvent(resultList[1]));
+          }).called(3);
+          verify(() {
+            communicationBloc.add(communication_events.MeteringInProgressEvent(resultList.last));
+          }).called(1);
+          verify(() {
+            communicationBloc.add(communication_events.MeteringEndedEvent(resultList.last));
+          }).called(3); // +1 from settings closed, +1 from dispose
+        },
+        expect: () => resultList.map(
+          (e) => isA<LightSensorContainerState>().having((state) => state.ev100, 'ev100', e),
+        ),
+      );
+    },
+  );
 }