fix: lint issues

Author: Anas12091101

src/editors/containers/ProblemEditor/components/EditProblemView/SettingsWidget/index.jsx

@@ -195,6 +195,8 @@ SettingsWidget.propTypes = {
   blockTitle: PropTypes.string.isRequired,
   correctAnswerCount: PropTypes.number.isRequired,
   problemType: PropTypes.string.isRequired,
+  // eslint-disable-next-line react/forbid-prop-types
+  editorRef: PropTypes.object.isRequired,
   setBlockTitle: PropTypes.func.isRequired,
   updateAnswer: PropTypes.func.isRequired,
   updateField: PropTypes.func.isRequired,

src/editors/containers/ProblemEditor/components/EditProblemView/SettingsWidget/index.test.tsx

@@ -6,7 +6,6 @@ import { editorRender, type PartialEditorState } from '@src/editors/editorTestRe
 import { mockWaffleFlags } from '@src/data/apiHooks.mock';
 import * as hooks from './hooks';
 import { SettingsWidgetInternal as SettingsWidget } from '.';
-import { initialState } from '@src/course-outline/__mocks__/courseOutlineIndex';
 
 jest.mock('./settingsComponents/GeneralFeedback', () => 'GeneralFeedback');
 jest.mock('./settingsComponents/GroupFeedback', () => 'GroupFeedback');
@@ -18,8 +17,6 @@ jest.mock('./settingsComponents/ShowAnswerCard', () => 'ShowAnswerCard');
 jest.mock('./settingsComponents/SwitchEditorCard', () => 'SwitchEditorCard');
 jest.mock('./settingsComponents/TimerCard', () => 'TimerCard');
 jest.mock('./settingsComponents/TypeCard', () => 'TypeCard');
-// NOTE: Do NOT mock useSelector here. We rely on the real hook so that
-// editorRender's provided initialState flows into the component under test.
 mockWaffleFlags();
 
 describe('SettingsWidget', () => {
@@ -30,7 +27,7 @@ describe('SettingsWidget', () => {
 
   const props = {
     problemType: ProblemTypeKeys.TEXTINPUT,
-    editorRef: { current: null},
+    editorRef: { current: null },
     settings: {},
     defaultSettings: {
       maxAttempts: 2,

src/editors/containers/ProblemEditor/components/EditProblemView/SettingsWidget/settingsComponents/SwitchEditorCard.jsx

@@ -56,6 +56,8 @@ const SwitchEditorCard = ({
 SwitchEditorCard.propTypes = {
   problemType: PropTypes.string.isRequired,
   editorType: PropTypes.string.isRequired,
+  // eslint-disable-next-line react/forbid-prop-types
+  editorRef: PropTypes.object.isRequired,
 };
 
 export default SwitchEditorCard;

src/editors/containers/ProblemEditor/components/EditProblemView/SettingsWidget/settingsComponents/SwitchEditorCard.test.tsx

@@ -59,5 +59,4 @@ describe('SwitchEditorCard - markdown', () => {
     const reduxWrapper = (container.firstChild as HTMLElement | null);
     expect(reduxWrapper?.innerHTML).toBe('');
   });
-
 });

src/editors/data/redux/thunkActions/problem.test.ts

@@ -90,7 +90,7 @@ describe('problem thunkActions', () => {
       expect(dispatch).toHaveBeenCalledWith(
         actions.problem.updateField({
           problemType: ProblemTypeKeys.ADVANCED,
-          rawOLX: `<problem>\n<p>MockMarkdownContent</p>\n</problem>`,
+          rawOLX: '<problem>\n<p>MockMarkdownContent</p>\n</problem>',
           isMarkdownEditorEnabled: false,
         }),
       );

src/editors/data/redux/thunkActions/problem.ts

@@ -1,4 +1,5 @@
 import { get, isEmpty } from 'lodash';
+import { camelizeKeys, convertMarkdownToXml } from '@src/editors/utils';
 import { actions as problemActions } from '../problem';
 import { actions as requestActions } from '../requests';
 import { selectors as appSelectors } from '../app';
@@ -8,16 +9,13 @@ import { OLXParser } from '../../../containers/ProblemEditor/data/OLXParser';
 import { parseSettings } from '../../../containers/ProblemEditor/data/SettingsParser';
 import { ProblemTypeKeys } from '../../constants/problem';
 import ReactStateOLXParser from '../../../containers/ProblemEditor/data/ReactStateOLXParser';
-import { camelizeKeys } from '../../../utils';
 import { fetchEditorContent } from '../../../containers/ProblemEditor/components/EditProblemView/hooks';
 import { RequestKeys } from '../../constants/requests';
-import { convertMarkdownToXml } from '../../../utils';
 
 // Similar to `import { actions, selectors } from '..';` but avoid circular imports:
 const actions = { problem: problemActions, requests: requestActions };
 const selectors = { app: appSelectors };
 
-
 export const switchToAdvancedEditor = (editorRef) => (dispatch, getState) => {
   const { problem } = getState();
   const editorObject = fetchEditorContent({ format: '' });
@@ -28,8 +26,7 @@ export const switchToAdvancedEditor = (editorRef) => (dispatch, getState) => {
     if (editorRef?.current?.state?.doc) {
       const markdownContent = editorRef.current.state.doc.toString();
       rawOLX = convertMarkdownToXml(markdownContent);
-    }
-    else{
+    } else {
       // Fallback to previously saved olx
       rawOLX = problem.rawOLX;
     }

src/editors/sharedComponents/CodeEditor/index.test.tsx

@@ -141,7 +141,7 @@ describe('CodeEditor', () => {
         expect(hooks.createCodeMirrorDomNode).toHaveBeenCalled();
         fireEvent.click(screen.getByRole('button', { name: 'Unescape HTML Literals' }));
         expect(mockEscapeHTMLSpecialChars).toHaveBeenCalled();
-        // Prevent React.useRef mock leakage into subsequent tests 
+        // Prevent React.useRef mock leakage into subsequent tests
         useRefSpy.mockRestore();
       });
     });