refactor some of the shared hooks

Author: tariknz

src/frontend/components/Standings/hooks/useDriverPositions.tsx

@@ -3,9 +3,9 @@ import {
   useTelemetryValue,
   useTelemetry,
   useSessionDrivers,
-  useSessionType,
   useDriverCarIdx,
   useSessionQualifyingResults,
+  useCurrentSessionType,
 } from '@irdashies/context';
 import { Standings } from '../createStandings';
 
@@ -73,8 +73,7 @@ export const useDriverStandings = () => {
   const radioTransmitCarIdx = useTelemetryValue('RadioTransmitCarIdx');
   const carStates = useCarState();
   const playerCarIdx = useDriverCarIdx();
-  const sessionNum = useTelemetryValue('SessionNum');
-  const sessionType = useSessionType(sessionNum);
+  const sessionType = useCurrentSessionType();
   const qualifyingPositions = useSessionQualifyingResults();
 
   const driverStandings: Standings[] = useMemo(() => {

src/frontend/context/index.ts

@@ -4,5 +4,4 @@ export * from './SessionStore/SessionProvider';
 export * from './SessionStore/SessionStore';
 export * from './TelemetryStore/TelemetryProvider';
 export * from './TelemetryStore/TelemetryStore';
-export * from './CarSpeedStore/useCarIdxSpeed';
-export * from './LapTimesStore/useCarIdxAverageLapTime';
+export * from './shared';

src/frontend/context/shared/index.ts

@@ -0,0 +1,3 @@
+export * from './useCarIdxSpeed';
+export * from './useCarIdxAverageLapTime';
+export * from './useCurrentSessionType';

src/frontend/context/LapTimesStore/useCarIdxAverageLapTime.spec.tsx renamed to src/frontend/context/shared/useCarIdxAverageLapTime.spec.tsx

@@ -2,7 +2,7 @@ import { describe, it, expect, beforeEach, vi } from 'vitest';
 import { renderHook } from '@testing-library/react';
 import { useCarIdxAverageLapTime } from './useCarIdxAverageLapTime';
 import { useTelemetryStore } from '../TelemetryStore/TelemetryStore';
-import { useLapTimesStore, useLapTimes } from './LapTimesStore';
+import { useLapTimesStore, useLapTimes } from '../LapTimesStore/LapTimesStore';
 import { useCarIdxClassEstLapTime } from '../SessionStore/SessionStore';
 import type { Telemetry } from '@irdashies/types';
 
@@ -11,7 +11,7 @@ vi.mock('../TelemetryStore/TelemetryStore', () => ({
   useTelemetryStore: vi.fn(),
 }));
 
-vi.mock('./LapTimesStore', () => ({
+vi.mock('../LapTimesStore/LapTimesStore', () => ({
   useLapTimesStore: vi.fn(),
   useLapTimes: vi.fn(),
 }));

src/frontend/context/LapTimesStore/useCarIdxAverageLapTime.ts renamed to src/frontend/context/shared/useCarIdxAverageLapTime.ts

@@ -1,6 +1,6 @@
 import { useEffect } from 'react';
 import { useTelemetryStore } from '../TelemetryStore/TelemetryStore';
-import { useLapTimes, useLapTimesStore } from './LapTimesStore';
+import { useLapTimes, useLapTimesStore } from '../LapTimesStore/LapTimesStore';
 import { useCarIdxClassEstLapTime } from '../SessionStore/SessionStore';
 
 /**

src/frontend/context/CarSpeedStore/useCarIdxSpeed.spec.tsx renamed to src/frontend/context/shared/useCarIdxSpeed.spec.tsx

@@ -2,7 +2,7 @@ import { describe, it, expect, beforeEach, vi } from 'vitest';
 import { renderHook } from '@testing-library/react';
 import { useCarIdxSpeed } from './useCarIdxSpeed';
 import { useTelemetryStore } from '../TelemetryStore/TelemetryStore';
-import { useCarSpeedsStore, useCarSpeeds } from './CarSpeedsStore';
+import { useCarSpeedsStore, useCarSpeeds } from '../CarSpeedStore/CarSpeedsStore';
 import { useSessionStore } from '../SessionStore/SessionStore';
 import type { Telemetry } from '@irdashies/types';
 
@@ -11,7 +11,7 @@ vi.mock('../TelemetryStore/TelemetryStore', () => ({
   useTelemetryStore: vi.fn(),
 }));
 
-vi.mock('./CarSpeedsStore', () => ({
+vi.mock('../CarSpeedStore/CarSpeedsStore', () => ({
   useCarSpeedsStore: vi.fn(),
   useCarSpeeds: vi.fn(),
 }));

src/frontend/context/CarSpeedStore/useCarIdxSpeed.ts renamed to src/frontend/context/shared/useCarIdxSpeed.ts

@@ -1,7 +1,7 @@
 import { useEffect } from 'react';
 import { useTelemetryStore } from '../TelemetryStore/TelemetryStore';
 import { useSessionStore } from '../SessionStore/SessionStore';
-import { useCarSpeeds, useCarSpeedsStore } from './CarSpeedsStore';
+import { useCarSpeeds, useCarSpeedsStore } from '../CarSpeedStore/CarSpeedsStore';
 
 /**
  * First time this hook is called, it will update the car speeds using both telemetry and session (track length) state.

src/frontend/context/shared/useCurrentSessionType.spec.tsx

@@ -0,0 +1,42 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { renderHook } from '@testing-library/react';
+import { useCurrentSessionType } from './useCurrentSessionType';
+import { useSessionType, useTelemetryValue } from '@irdashies/context';
+
+// Mock the context hooks
+vi.mock('@irdashies/context', () => ({
+  useSessionType: vi.fn(),
+  useTelemetryValue: vi.fn(),
+}));
+
+describe('useCurrentSession', () => {
+  beforeEach(() => {
+    vi.resetAllMocks();
+  });
+
+  it('should return the session type when session number is available', () => {
+    // Mock the telemetry value for session number
+    vi.mocked(useTelemetryValue).mockReturnValue(1);
+    // Mock the session type
+    vi.mocked(useSessionType).mockReturnValue('Race');
+
+    const { result } = renderHook(() => useCurrentSessionType());
+
+    expect(result.current).toBe('Race');
+    expect(useTelemetryValue).toHaveBeenCalledWith('SessionNum');
+    expect(useSessionType).toHaveBeenCalledWith(1);
+  });
+
+  it('should return undefined when session number is not available', () => {
+    // Mock the telemetry value as undefined
+    vi.mocked(useTelemetryValue).mockReturnValue(undefined);
+    // Mock the session type
+    vi.mocked(useSessionType).mockReturnValue(undefined);
+
+    const { result } = renderHook(() => useCurrentSessionType());
+
+    expect(result.current).toBeUndefined();
+    expect(useTelemetryValue).toHaveBeenCalledWith('SessionNum');
+    expect(useSessionType).toHaveBeenCalledWith(undefined);
+  });
+});

src/frontend/context/shared/useCurrentSessionType.tsx

@@ -0,0 +1,13 @@
+import { useSessionType, useTelemetryValue } from '@irdashies/context';
+
+type SessionType = 'Race' | 'Lone Qualify' | 'Open Qualify' | 'Practice' | 'Offline Testing';
+
+/**
+ * @returns The current session type. Undefined if sessionNum is unknown.
+ */
+export const useCurrentSessionType = (): SessionType | undefined => {
+  const sessionNum = useTelemetryValue('SessionNum');
+  const sessionType = useSessionType(sessionNum);
+
+  return sessionType as SessionType | undefined;
+};