chore: update test

Author: leangseu-edx

src/components/FileUpload/ActionCell.jsx

@@ -1,8 +1,9 @@
 import React, { useCallback } from 'react';
-import { IconButton, Icon } from '@edx/paragon';
+import PropTypes from 'prop-types';
 
-import { useIntl } from '@edx/frontend-platform/i18n';
+import { IconButton, Icon } from '@edx/paragon';
 import { Delete, Preview } from '@edx/paragon/icons';
+import { useIntl } from '@edx/frontend-platform/i18n';
 
 import messages from './messages';
 
@@ -15,7 +16,7 @@ const ActionCell = ({
   const deleteFile = useCallback(async () => {
     console.log('deleteFile', row.index);
     await onDeletedFile(row.index);
-  }, []);
+  }, [onDeletedFile, row.index]);
   return (
     <>
       <IconButton
@@ -35,4 +36,16 @@ const ActionCell = ({
   );
 };
 
+ActionCell.defaultProps = {
+  onDeletedFile: () => {},
+};
+
+ActionCell.propTypes = {
+  onDeletedFile: PropTypes.func,
+  disabled: PropTypes.bool.isRequired,
+  row: PropTypes.shape({
+    index: PropTypes.number.isRequired,
+  }).isRequired,
+};
+
 export default ActionCell;

src/components/FileUpload/ActionCell.test.jsx

@@ -2,8 +2,15 @@ import { shallow } from '@edx/react-unit-test-utils';
 import ActionCell from './ActionCell';
 
 describe('<ActionCell />', () => {
+  const props = {
+    onDeletedFile: jest.fn(),
+    disabled: false,
+    row: {
+      index: 0,
+    },
+  };
   it('renders', () => {
-    const wrapper = shallow(<ActionCell />);
+    const wrapper = shallow(<ActionCell {...props} />);
     expect(wrapper.snapshot).toMatchSnapshot();
   });
 });

src/components/FileUpload/__snapshots__/ActionCell.test.jsx.snap

@@ -4,12 +4,14 @@ exports[`<ActionCell /> renders 1`] = `
 <Fragment>
   <IconButton
     alt="Delete"
+    disabled={false}
     iconAs="Icon"
     onClick={[Function]}
     src={[Function]}
   />
   <IconButton
     alt="Preview"
+    disabled={false}
     iconAs="Icon"
     src={[Function]}
   />

src/components/FileUpload/index.jsx

@@ -14,7 +14,9 @@ import messages from './messages';
 
 import './styles.scss';
 
-const FileUpload = ({ isReadOnly, uploadedFiles, onFileUploaded, onDeletedFile }) => {
+const FileUpload = ({
+  isReadOnly, uploadedFiles, onFileUploaded, onDeletedFile,
+}) => {
   const { formatMessage } = useIntl();
 
   const {
@@ -55,6 +57,7 @@ const FileUpload = ({ isReadOnly, uploadedFiles, onFileUploaded, onDeletedFile }
               {
                 Header: formatMessage(messages.fileActionsTitle),
                 accessor: 'actions',
+                // eslint-disable-next-line react/no-unstable-nested-components
                 Cell: (props) => <ActionCell {...props} onDeletedFile={onDeletedFile} disabled={isReadOnly} />,
               },
             ]}
@@ -82,6 +85,7 @@ FileUpload.defaultProps = {
   isReadOnly: false,
   uploadedFiles: [],
   onFileUploaded: () => { },
+  onDeletedFile: () => { },
 };
 FileUpload.propTypes = {
   isReadOnly: PropTypes.bool,
@@ -93,6 +97,7 @@ FileUpload.propTypes = {
     }),
   ),
   onFileUploaded: PropTypes.func,
+  onDeletedFile: PropTypes.func,
 };
 
 export default FileUpload;

src/components/TextResponse/index.jsx

@@ -6,7 +6,9 @@ import RichTextEditor from 'components/TextResponse/RichTextEditor';
 
 import './index.scss';
 
-const TextResponse = ({ submissionConfig, value, onChange, isReadOnly }) => {
+const TextResponse = ({
+  submissionConfig, value, onChange, isReadOnly,
+}) => {
   const { textResponseConfig } = submissionConfig;
   const { optional, enabled } = textResponseConfig;
   const props = {
@@ -25,6 +27,11 @@ const TextResponse = ({ submissionConfig, value, onChange, isReadOnly }) => {
   );
 };
 
+TextResponse.defaultProps = {
+  onChange: () => {},
+  isReadOnly: false,
+};
+
 TextResponse.propTypes = {
   submissionConfig: PropTypes.shape({
     textResponseConfig: PropTypes.shape({
@@ -35,6 +42,7 @@ TextResponse.propTypes = {
   }).isRequired,
   value: PropTypes.string.isRequired,
   onChange: PropTypes.func,
+  isReadOnly: PropTypes.bool,
 };
 
 export default TextResponse;

src/views/SelfAssessmentView/AssessmentContent.jsx

@@ -1,8 +1,6 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 
-import { Icon } from '@edx/paragon';
-import { CheckCircle } from '@edx/paragon/icons';
 import { useIntl } from '@edx/frontend-platform/i18n';
 
 import Prompt from 'components/Prompt';

src/views/SelfAssessmentView/AssessmentContent.test.jsx

@@ -24,7 +24,7 @@ describe('<AssessmentContent />', () => {
       submissionConfig: {
         maxFileSize: 100,
       },
-    }
+    },
   };
 
   describe('render', () => {

src/views/SelfAssessmentView/AssessmentContentLayout.jsx

@@ -21,7 +21,7 @@ const AssessmentContentLayout = ({
             oraConfigData={oraConfigData}
           />
         </Col>
-        <Rubric isGrading={true} />
+        <Rubric isGrading />
       </Row>
     </div>
   </div>

src/views/SelfAssessmentView/AssessmentContentLayout.test.jsx

@@ -6,7 +6,8 @@ jest.mock('./AssessmentContent', () => 'AssessmentContent');
 
 describe('<AssessmentContentLayout />', () => {
   const props = {
-    submission: 'submission'
+    submission: 'submission',
+    oraConfigData: 'oraConfigData',
   };
 
   it('render', () => {

src/views/SelfAssessmentView/__snapshots__/AssessmentContentLayout.test.jsx.snap

@@ -14,6 +14,7 @@ exports[`<AssessmentContentLayout /> render 1`] = `
         className="p-0"
       >
         <AssessmentContent
+          oraConfigData="oraConfigData"
           submission="submission"
         />
       </Col>