Fix compile errors

Also added a TODO for a testing util.

Author: RichDom2185

src/commons/sagas/SideContentSaga.ts

@@ -2,7 +2,6 @@ import { Action } from 'redux';
 import type { SagaIterator } from 'redux-saga';
 import { put, take } from 'redux-saga/effects';
 import { notifyStoriesEvaluated } from 'src/features/stories/StoriesActions';
-import { NOTIFY_STORIES_EVALUATED } from 'src/features/stories/StoriesTypes';
 
 import * as actions from '../sideContent/SideContentActions';
 import {
@@ -52,7 +51,7 @@ export default function* SideContentSaga(): SagaIterator {
   );
 
   yield takeEvery(
-    NOTIFY_STORIES_EVALUATED,
+    notifyStoriesEvaluated.type,
     function* (action: ReturnType<typeof notifyStoriesEvaluated>) {
       yield put(actions.spawnSideContent(`stories.${action.payload.env}`, action.payload));
     }

src/commons/sagas/WorkspaceSaga/helpers/evalCode.ts

@@ -11,7 +11,6 @@ import * as Sourceror from 'sourceror';
 import { makeCCompilerConfig, specialCReturnObject } from 'src/commons/utils/CToWasmHelper';
 import { javaRun } from 'src/commons/utils/JavaHelper';
 import { notifyStoriesEvaluated } from 'src/features/stories/StoriesActions';
-import { EVAL_STORY } from 'src/features/stories/StoriesTypes';
 
 import { EventType } from '../../../../features/achievement/AchievementTypes';
 import { isSchemeLanguage, OverallState } from '../../../application/ApplicationTypes';
@@ -48,7 +47,7 @@ export function* evalCode(
 ): SagaIterator {
   context.runtime.debuggerOn =
     (actionType === EVAL_EDITOR || actionType === DEBUG_RESUME) && context.chapter > 2;
-  const isStoriesBlock = actionType === EVAL_STORY || workspaceLocation === 'stories';
+  const isStoriesBlock = actionType === actions.evalStory.type || workspaceLocation === 'stories';
 
   // Logic for execution of substitution model visualizer
   const correctWorkspace = workspaceLocation === 'playground' || workspaceLocation === 'sicp';

src/commons/utils/TestUtils.ts

@@ -47,3 +47,19 @@ export function deepFilter<T>(
   helper(nestedObject);
   return matches;
 }
+
+// TODO: Fix type inference for this function,
+// then use it in tests. We no longer need to
+// check the action type as everything gets migrated
+// to RTK and explicit action types are no longer needed.
+// /**
+//  * The `expectActionPayload` function is used to test the payload
+//  * of an action.
+//  * @param action The action to test
+//  */
+// export function expectActionPayload<Action extends ActionCreatorWithPreparedPayload<any, any>>(
+//   action: ReturnType<Action>
+// ) {
+//   const payload: ReturnType<Action>['payload'] = action.payload;
+//   return expect(payload);
+// }