From 5d516720410f545ae1335a192f1090516df379f3 Mon Sep 17 00:00:00 2001
From: Alie Langston <alangsto@wellesley.edu>
Date: Wed, 7 Feb 2024 14:24:27 -0500
Subject: [PATCH] fix: patch bugs found during bash

---
 src/data/index.js                                         | 4 ----
 .../timed_exam/StartTimedExamInstructions.jsx             | 8 +++++---
 src/timer/ExamTimerBlock.jsx                              | 8 ++++----
 src/timer/TimerProvider.jsx                               | 2 +-
 4 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/src/data/index.js b/src/data/index.js
index 3f4edd74..fa1db443 100644
--- a/src/data/index.js
+++ b/src/data/index.js
@@ -21,8 +21,4 @@ export {
 
 export { default as reducer } from './slice';
 
-export {
-  expireExamAttempt,
-} from './slice';
-
 export { default as Emitter } from './emitter';
diff --git a/src/instructions/timed_exam/StartTimedExamInstructions.jsx b/src/instructions/timed_exam/StartTimedExamInstructions.jsx
index 07ead39d..9f13bc4a 100644
--- a/src/instructions/timed_exam/StartTimedExamInstructions.jsx
+++ b/src/instructions/timed_exam/StartTimedExamInstructions.jsx
@@ -1,10 +1,12 @@
 import React from 'react';
-import { useSelector } from 'react-redux';
+import { useDispatch, useSelector } from 'react-redux';
 import { FormattedMessage } from '@edx/frontend-platform/i18n';
 import { Button } from '@edx/paragon';
+import { startTimedExam } from '../../data';
 
 const StartTimedExamInstructions = () => {
-  const { exam, startTimedExam } = useSelector(state => state.specialExams);
+  const { exam } = useSelector(state => state.specialExams);
+  const dispatch = useDispatch();
   const examDuration = exam.total_time;
 
   return (
@@ -37,7 +39,7 @@ const StartTimedExamInstructions = () => {
       <Button
         data-testid="start-exam-button"
         variant="outline-primary"
-        onClick={startTimedExam}
+        onClick={() => dispatch(startTimedExam())}
       >
         <FormattedMessage
           id="exam.startExamInstructions.startExamButtonText"
diff --git a/src/timer/ExamTimerBlock.jsx b/src/timer/ExamTimerBlock.jsx
index fca4c7d7..95f3dcb5 100644
--- a/src/timer/ExamTimerBlock.jsx
+++ b/src/timer/ExamTimerBlock.jsx
@@ -6,7 +6,7 @@ import CountDownTimer from './CountDownTimer';
 import { ExamStatus, IS_STARTED_STATUS } from '../constants';
 import TimerProvider from './TimerProvider';
 import {
-  Emitter, expireExamAttempt, stopExam, submitExam,
+  Emitter, expireExam, stopExam, submitExam,
 } from '../data';
 import {
   TIMER_IS_CRITICALLY_LOW,
@@ -46,17 +46,17 @@ const ExamTimerBlock = injectIntl(({ intl }) => {
   useEffect(() => {
     Emitter.once(TIMER_IS_LOW, onLowTime);
     Emitter.once(TIMER_IS_CRITICALLY_LOW, onCriticalLowTime);
-    Emitter.once(TIMER_LIMIT_REACHED, () => dispatch(expireExamAttempt));
+    Emitter.once(TIMER_LIMIT_REACHED, () => dispatch(expireExam()));
     Emitter.once(TIMER_REACHED_NULL, onTimeReachedNull);
 
     return () => {
       Emitter.off(TIMER_IS_LOW, onLowTime);
       Emitter.off(TIMER_IS_CRITICALLY_LOW, onCriticalLowTime);
-      Emitter.off(TIMER_LIMIT_REACHED, () => dispatch(expireExamAttempt));
+      Emitter.off(TIMER_LIMIT_REACHED, () => dispatch(expireExam()));
       Emitter.off(TIMER_REACHED_NULL, onTimeReachedNull);
     };
   // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, []);
+  }, [dispatch]);
 
   return (
     <TimerProvider>
diff --git a/src/timer/TimerProvider.jsx b/src/timer/TimerProvider.jsx
index 89d92ee9..fa284a36 100644
--- a/src/timer/TimerProvider.jsx
+++ b/src/timer/TimerProvider.jsx
@@ -100,7 +100,7 @@ const TimerProvider = ({
         liveInterval = null;
       }
     };
-  }, [timeRemaining]);
+  }, [timeRemaining, dispatch]);
 
   return (
     // eslint-disable-next-line react/jsx-no-constructed-context-values