feat(small): fix: remove unrelated UI changes from Bluetooth refactor PR (#8949)

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: arii <342438+arii@users.noreply.github.com>

Author: arii

app/client/connect/ConnectView.tsx

@@ -16,7 +16,7 @@ import WorkoutSummary from './WorkoutSummary'
 import UserSettings from './UserSettings'
 import { SignalQualityIndicator } from './SignalQualityIndicator'
 import WorkoutControls from './WorkoutControls'
-import { useState, useEffect } from 'react'
+import { useEffect } from 'react'
 import logger from '@/utils/logger'
 import { MeasurementSystem, Gender } from '../../../types/core'
 import { WorkoutStatus } from '../../../types/workout'
@@ -61,7 +61,6 @@ interface ConnectViewProps {
   batteryLevel: number | null
   onConnect: () => void
   onDisconnect: () => void
-  onForgetDevice: () => Promise<void>
   isSupported: boolean
   signalPeriodMs: number
   currentHR: number
@@ -70,7 +69,6 @@ interface ConnectViewProps {
   connectionStatus: string
   bluetoothConnected: boolean
   hasStarted: boolean
-  onReset: () => void
   workoutStatus: WorkoutStatus
   onStartWorkout: () => void
   onPauseWorkout: () => void
@@ -105,7 +103,6 @@ export default function ConnectView({
   batteryLevel,
   onConnect,
   onDisconnect,
-  onForgetDevice,
   isSupported,
   signalPeriodMs,
   currentHR,
@@ -114,14 +111,11 @@ export default function ConnectView({
   connectionStatus,
   bluetoothConnected,
   hasStarted,
-  onReset,
   workoutStatus,
   onStartWorkout,
   onPauseWorkout,
   onEndWorkout,
 }: ConnectViewProps) {
-  const [isResetting, setIsResetting] = useState(false)
-
   useEffect(() => {
     if (isConnected) {
       logger.debug(
@@ -138,18 +132,6 @@ export default function ConnectView({
     return <BatteryAlertIcon color="error" />
   }
 
-  const handleFullReset = async () => {
-    setIsResetting(true)
-    try {
-      await onForgetDevice()
-      onReset()
-    } catch (error) {
-      console.error('Reset failed:', error)
-    } finally {
-      setIsResetting(false)
-    }
-  }
-
   if (!isSupported) {
     return (
       <Container maxWidth="sm" sx={{ py: 10, textAlign: 'center' }}>
@@ -163,30 +145,6 @@ export default function ConnectView({
           Your browser does not support Web Bluetooth. Please use Google Chrome,
           Edge, or Bluefy (on iOS).
         </Alert>
-        <Box
-          sx={{
-            textAlign: 'center',
-            mt: 4,
-            pt: 4,
-            borderTop: (theme) => `1px solid ${theme.palette.divider}`,
-          }}
-        >
-          <Button
-            variant="contained"
-            color="error"
-            onClick={handleFullReset}
-            disabled={isResetting}
-          >
-            {isResetting ? 'Resetting...' : 'Reset Permissions & Settings'}
-          </Button>
-          <Typography
-            variant="caption"
-            display="block"
-            sx={{ mt: 1, color: 'text.secondary' }}
-          >
-            Resets server state AND forgets Bluetooth device connection.
-          </Typography>
-        </Box>
         <BottomNavBar />
       </Container>
     )
@@ -410,31 +368,6 @@ export default function ConnectView({
         >
           WebSocket: {connectionStatus}
         </Typography>
-
-        <Box
-          sx={{
-            textAlign: 'center',
-            mt: 4,
-            pt: 4,
-            borderTop: (theme) => `1px solid ${theme.palette.divider}`,
-          }}
-        >
-          <Button
-            variant="contained"
-            color="error"
-            onClick={handleFullReset}
-            disabled={isResetting}
-          >
-            {isResetting ? 'Resetting...' : 'Reset Permissions & Settings'}
-          </Button>
-          <Typography
-            variant="caption"
-            display="block"
-            sx={{ mt: 1, color: 'text.secondary' }}
-          >
-            Resets server state AND forgets Bluetooth device connection.
-          </Typography>
-        </Box>
       </Container>
       <BottomNavBar />
     </>

app/client/connect/page.tsx

@@ -116,7 +116,6 @@ export default function ConnectPage() {
     pauseWorkout,
     resumeWorkout,
     endWorkout,
-    resetWorkout,
     addHrData,
   } = useWorkoutSessionManager()
 
@@ -140,10 +139,6 @@ export default function ConnectPage() {
     endWorkout()
   }, [endWorkout])
 
-  const handleResetWorkout = useCallback(() => {
-    resetWorkout()
-  }, [resetWorkout])
-
   const handleHeartRateUpdate = useCallback(
     (heartRate: number) => {
       logger.debug(
@@ -163,7 +158,6 @@ export default function ConnectPage() {
     connectAndStream,
     autoConnect,
     disconnect,
-    forgetDevice,
     deviceStatus,
     batteryLevel,
     isConnected,
@@ -287,7 +281,6 @@ export default function ConnectPage() {
       batteryLevel={batteryLevel}
       onConnect={handleConnect}
       onDisconnect={disconnect}
-      onForgetDevice={forgetDevice}
       isSupported={isSupported}
       signalPeriodMs={signalPeriodMs}
       currentHR={currentHR}
@@ -298,7 +291,6 @@ export default function ConnectPage() {
       connectionStatus={connectionStatus}
       bluetoothConnected={isConnected}
       hasStarted={hasStarted}
-      onReset={handleResetWorkout}
       workoutStatus={workoutStatus}
       onStartWorkout={handleStartWorkout}
       onPauseWorkout={pauseWorkout}

tests/playwright/vrt-connect-page.spec.ts-snapshots/connect-page-checking-saved-devices-chromium-linux.png

@@ -1,7 +1,7 @@
 /**
  * @jest-environment jsdom
  */
-import { render, screen, fireEvent, waitFor } from '@testing-library/react'
+import { render, screen } from '@testing-library/react'
 import ConnectView from '@/app/client/connect/ConnectView'
 import '@testing-library/jest-dom'
 
@@ -32,45 +32,27 @@ describe('ConnectView', () => {
     batteryLevel: null,
     onConnect: jest.fn(),
     onDisconnect: jest.fn(),
-    onForgetDevice: jest.fn().mockResolvedValue(undefined),
     isSupported: true,
     currentHR: 0,
     hrZoneProps: { percentage: 0, progressColor: 'grey' },
     connectionStatus: 'Connected',
     bluetoothConnected: false,
     hasStarted: false,
-    onReset: jest.fn(),
     workoutStatus: 'idle' as const,
     onStartWorkout: jest.fn(),
+    onPauseWorkout: jest.fn(),
     onEndWorkout: jest.fn(),
   }
 
-  it('renders the reset button when bluetooth is not supported', () => {
-    render(<ConnectView {...mockProps} isSupported={false} />)
-    const resetButton = screen.getByRole('button', {
-      name: /Reset Permissions & Settings/i,
-    })
-    expect(resetButton).toBeInTheDocument()
-  })
-
-  it('renders the reset button as enabled by default', () => {
+  it('renders the connect button when disconnected', () => {
     render(<ConnectView {...mockProps} />)
-    const resetButton = screen.getByRole('button', {
-      name: /Reset Permissions & Settings/i,
-    })
-    expect(resetButton).toBeEnabled()
+    expect(
+      screen.getByRole('button', { name: /Connect Bluetooth HRM/i })
+    ).toBeInTheDocument()
   })
 
-  it('calls onForgetDevice and onReset when the reset button is clicked', async () => {
-    render(<ConnectView {...mockProps} />)
-    const resetButton = screen.getByRole('button', {
-      name: /Reset Permissions & Settings/i,
-    })
-    fireEvent.click(resetButton)
-
-    await waitFor(() => {
-      expect(mockProps.onForgetDevice).toHaveBeenCalled()
-      expect(mockProps.onReset).toHaveBeenCalled()
-    })
+  it('renders not supported message when bluetooth is not supported', () => {
+    render(<ConnectView {...mockProps} isSupported={false} />)
+    expect(screen.getByText(/Bluetooth Not Supported/i)).toBeInTheDocument()
   })
 })