DevOverlayにバックグラウンド位置情報取得のON/OFF状態表示を追加 (#5402)

Author: TinyKitten

src/components/DevOverlay.test.tsx

@@ -64,15 +64,17 @@ const mockUseNextStation = useNextStation as jest.MockedFunction<
 describe('DevOverlay', () => {
   beforeEach(() => {
     // Default mock implementations for useAtomValue
-    // locationAtom, accuracyHistoryAtomの順で呼ばれる (useTelemetryEnabledは別途モック済み)
+    // locationAtom, accuracyHistoryAtom, backgroundLocationTrackingAtomの順で呼ばれる
+    // (useTelemetryEnabledは別途モック済み)
     mockUseAtomValue
       .mockReturnValueOnce({
         coords: {
           speed: 10,
           accuracy: 15,
         },
       }) // locationAtom
-      .mockReturnValue([10, 15, 20]); // accuracyHistoryAtom
+      .mockReturnValueOnce([10, 15, 20]) // accuracyHistoryAtom
+      .mockReturnValue(false); // backgroundLocationTrackingAtom
 
     mockUseDistanceToNextStation.mockReturnValue('500');
     mockUseNextStation.mockReturnValue({
@@ -106,6 +108,24 @@ describe('DevOverlay', () => {
       const { getByText } = render(<DevOverlay />);
       expect(getByText('Telemetry: ON')).toBeTruthy();
     });
+
+    it('バックグラウンド位置情報のOFF状態を表示する', () => {
+      const { getByText } = render(<DevOverlay />);
+      expect(getByText('BG Loc: OFF')).toBeTruthy();
+    });
+
+    it('バックグラウンド位置情報のON状態を表示する', () => {
+      mockUseAtomValue.mockReset();
+      mockUseAtomValue
+        .mockReturnValueOnce({
+          coords: { speed: 10, accuracy: 15 },
+        }) // locationAtom
+        .mockReturnValueOnce([10, 15, 20]) // accuracyHistoryAtom
+        .mockReturnValue(true); // backgroundLocationTrackingAtom
+
+      const { getByText } = render(<DevOverlay />);
+      expect(getByText('BG Loc: ON')).toBeTruthy();
+    });
   });
 
   describe('位置情報の表示', () => {
@@ -138,7 +158,7 @@ describe('DevOverlay', () => {
       mockUseAtomValue
         .mockReturnValueOnce(null) // locationAtom
         .mockReturnValueOnce([]) // accuracyHistoryAtom
-        .mockReturnValue({ telemetryEnabled: true });
+        .mockReturnValue(false); // backgroundLocationTrackingAtom
 
       expect(() => {
         render(<DevOverlay />);
@@ -152,7 +172,7 @@ describe('DevOverlay', () => {
           coords: { speed: null, accuracy: 15 },
         }) // locationAtom
         .mockReturnValueOnce([]) // accuracyHistoryAtom
-        .mockReturnValue({ telemetryEnabled: true });
+        .mockReturnValue(false); // backgroundLocationTrackingAtom
 
       const { getByText } = render(<DevOverlay />);
       expect(getByText('Speed: 0km/h')).toBeTruthy();
@@ -165,7 +185,7 @@ describe('DevOverlay', () => {
           coords: { speed: -5, accuracy: 15 },
         }) // locationAtom
         .mockReturnValueOnce([]) // accuracyHistoryAtom
-        .mockReturnValue({ telemetryEnabled: true });
+        .mockReturnValue(false); // backgroundLocationTrackingAtom
 
       const { getByText } = render(<DevOverlay />);
       expect(getByText('Speed: 0km/h')).toBeTruthy();
@@ -178,7 +198,7 @@ describe('DevOverlay', () => {
           coords: { speed: 10, accuracy: null },
         }) // locationAtom
         .mockReturnValueOnce([]) // accuracyHistoryAtom
-        .mockReturnValue({ telemetryEnabled: true });
+        .mockReturnValue(false); // backgroundLocationTrackingAtom
 
       const { getByText } = render(<DevOverlay />);
       expect(getByText('Accuracy: m')).toBeTruthy();
@@ -191,7 +211,7 @@ describe('DevOverlay', () => {
           coords: { speed: 10, accuracy: 15 },
         }) // locationAtom
         .mockReturnValueOnce([]) // accuracyHistoryAtom
-        .mockReturnValue({ telemetryEnabled: true });
+        .mockReturnValue(false); // backgroundLocationTrackingAtom
 
       expect(() => {
         render(<DevOverlay />);
@@ -242,7 +262,7 @@ describe('DevOverlay', () => {
           coords: { speed: 0, accuracy: 15 },
         }) // locationAtom
         .mockReturnValueOnce([]) // accuracyHistoryAtom
-        .mockReturnValue({ telemetryEnabled: true });
+        .mockReturnValue(false); // backgroundLocationTrackingAtom
 
       const { getByText } = render(<DevOverlay />);
       expect(getByText('Speed: 0km/h')).toBeTruthy();
@@ -255,7 +275,7 @@ describe('DevOverlay', () => {
           coords: { speed: 13.89, accuracy: 15 }, // 約50 km/h
         }) // locationAtom
         .mockReturnValueOnce([]) // accuracyHistoryAtom
-        .mockReturnValue({ telemetryEnabled: true });
+        .mockReturnValue(false); // backgroundLocationTrackingAtom
 
       const { getByText } = render(<DevOverlay />);
       expect(getByText('Speed: 50km/h')).toBeTruthy();

src/components/DevOverlay.tsx

@@ -4,7 +4,11 @@ import React, { useMemo } from 'react';
 import { StyleSheet, Text, useWindowDimensions, View } from 'react-native';
 import { useDistanceToNextStation, useNextStation } from '~/hooks';
 import { useTelemetryEnabled } from '~/hooks/useTelemetryEnabled';
-import { accuracyHistoryAtom, locationAtom } from '~/store/atoms/location';
+import {
+  accuracyHistoryAtom,
+  backgroundLocationTrackingAtom,
+  locationAtom,
+} from '~/store/atoms/location';
 import { generateAccuracyChart } from '~/utils/accuracyChart';
 import Typography from './Typography';
 
@@ -35,6 +39,9 @@ const DevOverlay: React.FC = () => {
   const distanceToNextStation = useDistanceToNextStation();
   const nextStation = useNextStation(false);
   const isTelemetryEnabled = useTelemetryEnabled();
+  const isBackgroundLocationTracking = useAtomValue(
+    backgroundLocationTrackingAtom
+  );
 
   const coordsSpeed = ((speed ?? 0) < 0 ? 0 : speed) ?? 0;
 
@@ -88,6 +95,9 @@ const DevOverlay: React.FC = () => {
       <Typography style={styles.text}>
         Telemetry: {isTelemetryEnabled ? 'ON' : 'OFF'}
       </Typography>
+      <Typography style={styles.text}>
+        BG Loc: {isBackgroundLocationTracking ? 'ON' : 'OFF'}
+      </Typography>
     </View>
   );
 };

src/hooks/useStartBackgroundLocationUpdates.test.tsx

@@ -10,7 +10,11 @@ import { useStartBackgroundLocationUpdates } from './useStartBackgroundLocationU
 
 jest.mock('expo-location');
 jest.mock('./useLocationPermissionsGranted');
+jest.mock('~/store', () => ({
+  store: { set: jest.fn() },
+}));
 jest.mock('~/store/atoms/location', () => ({
+  backgroundLocationTrackingAtom: {},
   setLocation: jest.fn(),
 }));
 jest.mock('~/store/atoms/navigation', () => ({

src/hooks/useStartBackgroundLocationUpdates.ts

@@ -1,7 +1,11 @@
 import * as Location from 'expo-location';
 import { useAtomValue } from 'jotai';
 import { useEffect } from 'react';
-import { setLocation } from '~/store/atoms/location';
+import { store } from '~/store';
+import {
+  backgroundLocationTrackingAtom,
+  setLocation,
+} from '~/store/atoms/location';
 import navigationState from '~/store/atoms/navigation';
 import {
   LOCATION_START_MAX_RETRIES,
@@ -60,6 +64,8 @@ export const useStartBackgroundLocationUpdates = () => {
                 stopError
               );
             }
+          } else {
+            store.set(backgroundLocationTrackingAtom, true);
           }
           return;
         } catch (error) {
@@ -82,6 +88,7 @@ export const useStartBackgroundLocationUpdates = () => {
 
     return () => {
       cancelled = true;
+      store.set(backgroundLocationTrackingAtom, false);
       Location.stopLocationUpdatesAsync(LOCATION_TASK_NAME).catch((error) => {
         console.warn(
           'バックグラウンド位置情報の更新停止に失敗しました:',

src/store/atoms/location.ts

@@ -12,6 +12,7 @@ const SMOOTHING_ALPHA = 0.6;
 
 export const locationAtom = atom<Location.LocationObject | null>(null);
 export const accuracyHistoryAtom = atom<number[]>([]);
+export const backgroundLocationTrackingAtom = atom(false);
 
 export const setLocation = (location: Location.LocationObject) => {
   const prev = store.get(locationAtom);