fix: Renamed a couple of things and removed unneeded check

src/data/slice.js

@@ -1,5 +1,5 @@
 import { createSlice } from '@reduxjs/toolkit';
-import { aggregateActiveAttemptData } from '../helpers';
+import { appendTimerEnd } from '../helpers';
 
 /* eslint-disable no-param-reassign */
 export const examSlice = createSlice({
@@ -77,10 +77,10 @@ export const examSlice = createSlice({
     },
     setExamState: (state, { payload }) => {
       state.exam = payload.exam;
-      state.activeAttempt = aggregateActiveAttemptData(payload.activeAttempt);
+      state.activeAttempt = appendTimerEnd(payload.activeAttempt);
     },
     setActiveAttempt: (state, { payload }) => {
-      state.activeAttempt = aggregateActiveAttemptData(payload.activeAttempt);
+      state.activeAttempt = appendTimerEnd(payload.activeAttempt);
       state.apiErrorMsg = '';
     },
     setProctoringSettings: (state, { payload }) => {

src/helpers.js

@@ -40,7 +40,7 @@ export const generateHumanizedTime = (timeRemainingSeconds) => {
 // The only information we get on the remaining time on the active exam attempt
 // from the endpoint is the remaining seconds. We need to have a fixed time reference
 // on the time limit to be able to calculate the remaining time accurately.
-export const aggregateActiveAttemptData = (activeAttempt) => {
+export const appendTimerEnd = (activeAttempt) => {
   if (!activeAttempt?.time_remaining_seconds) {
     return activeAttempt;
   }

src/timer/TimerProvider.jsx

@@ -50,10 +50,9 @@ const TimerProvider = ({
 
   const pollExam = useCallback(() => {
     // poll url may be null if this is an LTI exam
-    if (attempt?.exam_started_poll_url) {
-      dispatch(pollAttempt(attempt.exam_started_poll_url));
-    }
+    dispatch(pollAttempt(attempt.exam_started_poll_url));
   }, [attempt.exam_started_poll_url, dispatch]);
+
   const processTimeLeft = useCallback(() => (secondsLeft) => {
     const criticalLowTime = timeLimitMins * 60 * TIME_LIMIT_CRITICAL_PCT;
     const lowTime = timeLimitMins * 60 * TIME_LIMIT_LOW_PCT;
@@ -107,8 +106,8 @@ const TimerProvider = ({
         dispatch(pingAttempt(pingInterval, workerUrl));
       }
 
-      const keepRunning = processTimeLeft(secondsLeft);
-      if (!keepRunning) {
+      const keepTimerRunning = processTimeLeft(secondsLeft);
+      if (!keepTimerRunning) {
         clearInterval(timerHandler);
       }
     }, 1000);