+
+`;
+
+exports[`
+
+`;
diff --git a/src/components/Assessment/EditableAssessment/__snapshots__/index.test.jsx.snap b/src/components/Assessment/EditableAssessment/__snapshots__/index.test.jsx.snap
new file mode 100644
index 00000000..20848fff
--- /dev/null
+++ b/src/components/Assessment/EditableAssessment/__snapshots__/index.test.jsx.snap
@@ -0,0 +1,125 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`+ Rubric +
++
+
+ Rubric +
++
+
Children
+ +
+ Children
+
+
+ Children
+
+
+
+
+
+
+
+
+
+
+
+
+ + Criterion Name +
++ Criterion Description +
++ Selected Option + : + 5 + + Points +
+
+
+
+
+
+
+
+
++ Comment Header comment +
+
+
+ Read less
+
+
+ + Comment Body +
+
+
+
+
+
+
+ + Criterion Name +
++ Criterion Description +
++ Selected Option + : + 5 + + Points +
+
+
+
+
+ + Criterion Name +
++ Selected Option + : + 5 + + Points +
+
+
+
+
+
+
+
+
++ Comment Header comment +
+
+
+ Read less
+
+
+ + Comment Body +
+
+
+
+
+
+
+ + Criterion Name +
++ Criterion Description +
+
+
+
+
+
+
+
+
++ Comment Header comment +
+
+
+ Read less
+
+
+ + Comment Body +
+
+
+
+
+
+
+
+
+
+
+`;
+
+exports[`+ + 1 + : +
++
+ + 2 + : +
++
+
+ Option 1
+
+
+`;
+
+exports[`
+
+
+
+
+
+
+ Feedback
+
+
+
+ Option 1
+
+
+`;
diff --git a/src/components/CriterionContainer/__snapshots__/RadioCriterion.test.jsx.snap b/src/components/CriterionContainer/__snapshots__/RadioCriterion.test.jsx.snap
new file mode 100644
index 00000000..93dc382c
--- /dev/null
+++ b/src/components/CriterionContainer/__snapshots__/RadioCriterion.test.jsx.snap
@@ -0,0 +1,136 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`
+
+
+
+
+
+
+
+
+
+ Option 1
+
+
+
+
+
+
+
+
+
+
+ Option 2
+
+
+
+
+
+
+
+
+
+ description
+
+ +
+
+ Option 1
+
+
+ description1 +
+ + description1 +
+
+ Option 2
+
+
+ description2 +
+ + description2 +
+
+
+ input
+
+
+ feedback
+
+
+ description
+
+ +
+
+ Option 1
+
+
+ description1 +
+ + description1 +
+
+ Option 2
+
+
+ description2 +
+ + description2 +
input
,
+ feedback: feedback
,
+ criterion: {
+ name: 'criterionName',
+ description: 'description',
+ options: [
+ {
+ name: 'option1',
+ label: 'Option 1',
+ description: 'description1',
+ },
+ {
+ name: 'option2',
+ label: 'Option 2',
+ description: 'description2',
+ },
+ ],
+ },
+ };
+
+ it('renders default', () => {
+ const wrapper = shallow(
+
+`;
+
+exports[`
+
+
+
+
+
+ Page
+
+
+ of
+ 10
+
+
+
+
+`;
+
+exports[`PDF Renderer Component snapshots on last page, next is disabled 1`] = `
+
+
+
+
+
+
+
+
+ Page
+
+
+ of
+ 10
+
+
+
+
+`;
diff --git a/src/components/FilePreview/components/FileRenderer/BaseRenderers/__snapshots__/TXTRenderer.test.jsx.snap b/src/components/FilePreview/components/FileRenderer/BaseRenderers/__snapshots__/TXTRenderer.test.jsx.snap
new file mode 100644
index 00000000..58c2c859
--- /dev/null
+++ b/src/components/FilePreview/components/FileRenderer/BaseRenderers/__snapshots__/TXTRenderer.test.jsx.snap
@@ -0,0 +1,9 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`
+
+ + content ++`; diff --git a/src/components/FilePreview/components/FileRenderer/BaseRenderers/pdfHooks.test.js b/src/components/FilePreview/components/FileRenderer/BaseRenderers/pdfHooks.test.js new file mode 100644 index 00000000..4d6e9c11 --- /dev/null +++ b/src/components/FilePreview/components/FileRenderer/BaseRenderers/pdfHooks.test.js @@ -0,0 +1,92 @@ +import React from 'react'; +import { mockUseKeyedState } from '@edx/react-unit-test-utils'; +import { when } from 'jest-when'; + +import { + usePDFRendererData, safeSetPageNumber, stateKeys, initialState, +} from './pdfHooks'; + +const state = mockUseKeyedState(stateKeys); + +describe('PDF Renderer hooks', () => { + describe('safeSetPageNumber', () => { + it('returns value handler that sets page number if valid', () => { + const rawSetPageNumber = jest.fn(); + const numPages = 10; + const goToPage = safeSetPageNumber({ numPages, rawSetPageNumber }); + // should not call rawSetPageNumber when page number smaller than 1 or greater than numPages + goToPage(0); + expect(rawSetPageNumber).not.toHaveBeenCalled(); + goToPage(numPages + 1); + expect(rawSetPageNumber).not.toHaveBeenCalled(); + // should call rawSetPageNumber when page number is valid + goToPage(numPages); + expect(rawSetPageNumber).toHaveBeenCalledWith(numPages); + }); + }); + + describe('usePDFRendererData', () => { + const onSuccess = jest.fn(); + const onError = jest.fn(); + + beforeEach(() => { + jest.clearAllMocks(); + state.mock(); + }); + afterEach(() => { + state.resetVals(); + }); + + it('start with initial state', () => { + usePDFRendererData({}); + state.expectInitializedWith(stateKeys.pageNumber, initialState.pageNumber); + state.expectInitializedWith(stateKeys.numPages, initialState.numPages); + state.expectInitializedWith(stateKeys.relativeHeight, initialState.relativeHeight); + }); + + it('calls onSuccess and sets numPages based on args', () => { + const out = usePDFRendererData({ onSuccess }); + out.onDocumentLoadSuccess({ numPages: 5 }); + expect(onSuccess).toHaveBeenCalled(); + state.expectSetStateCalledWith(stateKeys.numPages, 5); + }); + + it('sets relative height based on page size', () => { + when(React.useRef) + .calledWith() + .mockReturnValueOnce({ + current: { getBoundingClientRect: () => ({ width: 20 }) }, + }); + const out = usePDFRendererData({}); + + const page = { view: [0, 0, 20, 30] }; + out.onLoadPageSuccess(page); + expect(state.setState.relativeHeight).toHaveBeenCalledWith(30); + }); + + it('calls onErro if error happened', () => { + const out = usePDFRendererData({ onError }); + out.onDocumentLoadError('notFound'); + expect(onError).toHaveBeenCalledWith('notFound'); + }); + + it('has good page logic', () => { + // start with 3 pages + // this seems to be the only way to mock initial value + initialState.pageNumber = 2; + initialState.numPages = 3; + const out = usePDFRendererData({ onSuccess }); + // go to next page + out.onNextPageButtonClick(); + state.expectSetStateCalledWith(stateKeys.pageNumber, 3); + + // go to prev page + out.onPrevPageButtonClick(); + state.expectSetStateCalledWith(stateKeys.pageNumber, 1); + + // reset initial state + initialState.pageNumber = 1; + initialState.numPages = 1; + }); + }); +}); diff --git a/src/components/FilePreview/components/FileRenderer/BaseRenderers/textHooks.test.js b/src/components/FilePreview/components/FileRenderer/BaseRenderers/textHooks.test.js new file mode 100644 index 00000000..1a5c6179 --- /dev/null +++ b/src/components/FilePreview/components/FileRenderer/BaseRenderers/textHooks.test.js @@ -0,0 +1,60 @@ +import React from 'react'; +import { mockUseKeyedState } from '@edx/react-unit-test-utils'; + +import axios from 'axios'; +import { useTextRendererData, fetchFile, stateKeys } from './textHooks'; + +jest.mock('axios'); + +const state = mockUseKeyedState(stateKeys); + +describe('textHooks', () => { + const url = 'http://example.com'; + const setContent = jest.fn(); + const onError = jest.fn(); + const onSuccess = jest.fn(); + + describe('fetchFile', () => { + it('should call onSuccess and setContent when the request is successful', async () => { + const response = { data: 'file content' }; + axios.get.mockResolvedValue(response); + await fetchFile({ setContent, url, onSuccess }); + expect(onSuccess).toHaveBeenCalled(); + expect(setContent).toHaveBeenCalledWith(response.data); + }); + + it('should call onError when the request fails', async () => { + const response = { response: { status: 404 } }; + axios.get.mockRejectedValue(response); + await fetchFile({ url, onError }); + expect(onError).toHaveBeenCalledWith(response.response.status); + }); + }); + + describe('useTextRendererData', () => { + beforeEach(() => { + jest.clearAllMocks(); + state.mock(); + }); + afterEach(() => { state.resetVals(); }); + + it('start with empty content', () => { + useTextRendererData({}); + state.expectInitializedWith(stateKeys.content, ''); + }); + + it('update content after useEffect get call', async () => { + axios.get.mockResolvedValue({ data: 'file content' }); + useTextRendererData({ url, onError, onSuccess }); + // wouldn't call before useEffect + expect(axios.get).not.toHaveBeenCalled(); + const [[cb]] = React.useEffect.mock.calls; + cb(); + // because fetchFile was written with async/await, we need to wait for the next tick + await new Promise(process.nextTick); + expect(axios.get).toHaveBeenCalled(); + expect(onSuccess).toHaveBeenCalled(); + state.expectSetStateCalledWith(stateKeys.content, 'file content'); + }); + }); +}); diff --git a/src/components/FilePreview/components/FileRenderer/FileCard/__snapshots__/index.test.jsx.snap b/src/components/FilePreview/components/FileRenderer/FileCard/__snapshots__/index.test.jsx.snap new file mode 100644 index 00000000..69b7500f --- /dev/null +++ b/src/components/FilePreview/components/FileRenderer/FileCard/__snapshots__/index.test.jsx.snap @@ -0,0 +1,28 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[`
+
+
+ children
+
+ children
,
+ defaultOpen: true,
+ };
+
+ it('renders default', () => {
+ const wrapper = shallow(
+
+
+ name +
+ + name +
+
+
+ description +
+ + description +
+
+
+ 123 KB +
++ 123 KB +
+
+
+ name +
+ + name +
+
+
+ description +
+ + description +
+
+
+ Unknown +
++ Unknown +
+
+
+
+ Description for:
+
+
+ file name
+
+
+
+
+
+
+`;
+
+exports[`+ File upload +
+ + Uploaded files + +
+
+`;
+
+exports[`+ File upload +
+ + Uploaded files + +
+
+`;
+
+exports[`+ File upload +
+
+
+`;
+
+exports[`+ File upload +
+ + Uploaded files + +
+
+ Uploaded files
+
+
+`;
+
+exports[`createFileActionCell should return a function that is an action cell 1`] = `
+
+ Children
+
+ Children
+
+
+`;
diff --git a/src/components/Instructions/index.jsx b/src/components/Instructions/index.jsx
index f8617909..d6f5bb33 100644
--- a/src/components/Instructions/index.jsx
+++ b/src/components/Instructions/index.jsx
@@ -2,12 +2,6 @@ import React from 'react';
import { useIntl } from '@edx/frontend-platform/i18n';
-import { stepNames, stepStates } from 'constants/index';
-
-import { useGlobalState } from 'hooks/app';
-import { useViewStep } from 'hooks/routing';
-import { isXblockStep } from 'utils';
-
import useInstructionsMessage from './useInstructionsMessage';
import messages from './messages';
@@ -15,10 +9,7 @@ import messages from './messages';
const Instructions = () => {
const { formatMessage } = useIntl();
const message = useInstructionsMessage();
- const viewStep = useViewStep();
- const { activeStepName, stepState } = useGlobalState();
- const stepName = isXblockStep(viewStep) ? activeStepName : viewStep;
- if (stepState !== stepStates.inProgress || stepName === stepNames.staff) {
+ if (!message) {
return null;
}
return (
diff --git a/src/components/Instructions/index.test.jsx b/src/components/Instructions/index.test.jsx
new file mode 100644
index 00000000..7fd3064f
--- /dev/null
+++ b/src/components/Instructions/index.test.jsx
@@ -0,0 +1,23 @@
+import { shallow } from '@edx/react-unit-test-utils';
+
+import useInstructionsMessage from './useInstructionsMessage';
+
+import Instructions from './index';
+
+jest.mock('./useInstructionsMessage', () => jest.fn());
+
+describe('+ + Instructions + : + + arbitrarilyInstructionsMessage +
+
+
+`;
+
+exports[`
+
+`;
+
+exports[`children
+
{label}
{subLabel && (
-
diff --git a/src/components/ProgressBar/ProgressStep.test.jsx b/src/components/ProgressBar/ProgressStep.test.jsx
new file mode 100644
index 00000000..387a9d31
--- /dev/null
+++ b/src/components/ProgressBar/ProgressStep.test.jsx
@@ -0,0 +1,85 @@
+import { shallow } from '@edx/react-unit-test-utils';
+
+import { CheckCircle, Error } from '@edx/paragon/icons';
+
+import { stepNames } from 'constants/index';
+import { useProgressStepData } from './hooks';
+
+import ProgressStep, { stepIcons } from './ProgressStep';
+
+jest.mock('./hooks', () => ({
+ useProgressStepData: jest.fn(),
+}));
+
+describe('{subLabel}
+{subLabel}
)}
+ Test Step
+
++ Past due! +
+
+ Test Step
+
++ 1 / 2 +
+
+ Test Step
+
+
+ Test Step
+
+
+ Test Step
+
+
+ Test Step
+
+
+ Test Step
+
+
+ Test Step
+
+
+
+
+ +
+
+
+ +
+
+
+`;
+
+exports[`
+
+
+`;
+
+exports[`
+
+
+`;
diff --git a/src/components/TextResponse/index.test.jsx b/src/components/TextResponse/index.test.jsx
new file mode 100644
index 00000000..ba5c96e4
--- /dev/null
+++ b/src/components/TextResponse/index.test.jsx
@@ -0,0 +1,45 @@
+import React from 'react';
+import { shallow } from '@edx/react-unit-test-utils';
+
+import { useSubmissionConfig } from 'hooks/app';
+
+import TextResponse from './index';
+
+jest.mock('hooks/app', () => ({
+ useSubmissionConfig: jest.fn(),
+}));
+
+describe('
+
+`;
+
+exports[`
+
+ + title +
+
+
+
+
+
+
+ children
+
+
+
+`;
diff --git a/src/setupTest.js b/src/setupTest.js
index 72202834..ed81d118 100644
--- a/src/setupTest.js
+++ b/src/setupTest.js
@@ -12,7 +12,8 @@ jest.mock('react', () => ({
jest.mock('@edx/frontend-platform/i18n', () => {
const i18n = jest.requireActual('@edx/frontend-platform/i18n');
const { formatMessage } = jest.requireActual('@edx/react-unit-test-utils');
- const formatDate = jest.fn(date => new Date(date).toLocaleDateString()).mockName('useIntl.formatDate');
+ // this provide consistent for the test on different platform/timezone
+ const formatDate = jest.fn(date => new Date(date).toISOString()).mockName('useIntl.formatDate');
return {
...i18n,
useIntl: jest.fn(() => ({
@@ -73,6 +74,9 @@ jest.mock('@edx/paragon', () => jest.requireActual('@edx/react-unit-test-utils')
Hyperlink: 'Hyperlink',
Icon: 'Icon',
IconButton: 'IconButton',
+ Layout: {
+ Element: 'Layout.Element',
+ },
ModalDialog: {
Body: 'ModalDialog.Body',
Footer: 'ModalDialog.Footer',
@@ -81,6 +85,16 @@ jest.mock('@edx/paragon', () => jest.requireActual('@edx/react-unit-test-utils')
CloseButton: 'ModalDialog.CloseButton',
},
MultiSelectDropdownFilter: 'MultiSelectDropdownFilter',
+ Nav: {
+ Item: 'Nav.Item',
+ Link: 'Nav.Link',
+ },
+ Navbar: {
+ Brand: 'Navbar.Brand',
+ Collapse: 'Navbar.Collapse',
+ Nav: 'Navbar.Nav',
+ Toggle: 'Navbar.Toggle',
+ },
OverlayTrigger: 'OverlayTrigger',
PageBanner: 'PageBanner',
Popover: {
@@ -95,11 +109,19 @@ jest.mock('@edx/paragon', () => jest.requireActual('@edx/react-unit-test-utils')
Spinner: 'Spinner',
}));
jest.mock('@edx/paragon/icons', () => ({
+ CheckCircle: jest.fn().mockName('icons.CheckCircle'),
+ Edit: jest.fn().mockName('icons.Edit'),
+ Error: jest.fn().mockName('icons.Error'),
+ Highlight: jest.fn().mockName('icons.Highlight'),
Rule: jest.fn().mockName('icons.Rule'),
}));
jest.mock('@zip.js/zip.js', () => ({}));
+jest.mock('uuid', () => ({
+ v4: () => 'some_uuid',
+}));
+
Object.defineProperty(window, 'matchMedia', {
writable: true,
value: jest.fn().mockImplementation(query => ({
diff --git a/src/views/AssessmentView/BaseAssessmentView/__snapshots__/index.test.jsx.snap b/src/views/AssessmentView/BaseAssessmentView/__snapshots__/index.test.jsx.snap
new file mode 100644
index 00000000..89814d45
--- /dev/null
+++ b/src/views/AssessmentView/BaseAssessmentView/__snapshots__/index.test.jsx.snap
@@ -0,0 +1,45 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`
+
+ + title +
+
+
+
+
+
+
+ children
+
+
+
+`;
diff --git a/src/views/AssessmentView/BaseAssessmentView/index.test.jsx b/src/views/AssessmentView/BaseAssessmentView/index.test.jsx
new file mode 100644
index 00000000..8ef1e929
--- /dev/null
+++ b/src/views/AssessmentView/BaseAssessmentView/index.test.jsx
@@ -0,0 +1,24 @@
+import { shallow } from '@edx/react-unit-test-utils';
+
+import { useViewStep } from 'hooks/routing';
+
+import { stepNames } from 'constants/index';
+
+import BaseAssessmentView from './index';
+
+jest.mock('hooks/routing', () => ({
+ useViewStep: jest.fn(),
+}));
+jest.mock('components/Assessment', () => 'Assessment');
+jest.mock('components/Instructions', () => 'Instructions');
+jest.mock('components/ModalActions', () => 'ModalActions');
+jest.mock('components/StatusAlert', () => 'StatusAlert');
+jest.mock('components/StepProgressIndicator', () => 'StepProgressIndicator');
+
+describe('
+
+
+
+
+
+
+
+ + Self grading +
+
+
+
+
+
+
+
+
+ + Your response +
+
+
+
+
+ + Your response +
+- {effectiveAssessmentType === 'peer' + {effectiveAssessmentType === stepNames.peer ? formatMessage(messages.peerAsFinalGradeInfo) : formatMessage(messages.finalGradeInfo, { step: effectiveAssessmentType })}
diff --git a/src/views/GradeView/FinalGrade.test.jsx b/src/views/GradeView/FinalGrade.test.jsx new file mode 100644 index 00000000..aac5526d --- /dev/null +++ b/src/views/GradeView/FinalGrade.test.jsx @@ -0,0 +1,59 @@ +import { shallow } from '@edx/react-unit-test-utils'; + +import { + useAssessmentData, + useStepInfo, +} from 'hooks/app'; +import { stepNames } from 'constants/index'; + +import FinalGrade from './FinalGrade'; + +jest.mock('hooks/app', () => ({ + useAssessmentData: jest.fn(), + useStepInfo: jest.fn(), +})); +jest.mock('components/InfoPopover', () => 'InfoPopover'); +jest.mock('components/Assessment/ReadonlyAssessment', () => 'ReadOnlyAssessment'); + +describe('
+
+ About your grade:
+
+
+`;
+
+exports[`
+ Your grade is based on your Peer assessment score for this problem. Other assessments don't count towards your final score.
+
+ Only the required number of peer grades will counted against your final grade. The others are shown, but are not included in your grade calculation
+
+
+
+
+ + Your response +
+
+
+ + Your response +
+
+
+ About your grade:
+
+
+`;
diff --git a/src/views/GradeView/__snapshots__/FinalGrade.test.jsx.snap b/src/views/GradeView/__snapshots__/FinalGrade.test.jsx.snap
new file mode 100644
index 00000000..c7017d75
--- /dev/null
+++ b/src/views/GradeView/__snapshots__/FinalGrade.test.jsx.snap
@@ -0,0 +1,70 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`+ Your grade is based on your Self assessment score for this problem. Other assessments don't count towards your final score. +
+
+
+
+
+ Your final grade: {earned}/{possible}
+
+
+`;
+
+exports[`
+ Your final grade: {earned}/{possible}
+
+
+ Only the required number of peer grades will counted against your final grade. The others are shown, but are not included in your grade calculation
+
+
+
+ + Only the required number of peer grades will counted against your final grade. The others are shown, but are not included in your grade calculation +
+
+
+
+
+
+ Your final grade: {earned}/{possible}
+
+
+ Unweighted grades
+
+
+`;
diff --git a/src/views/GradeView/__snapshots__/index.test.jsx.snap b/src/views/GradeView/__snapshots__/index.test.jsx.snap
new file mode 100644
index 00000000..2762a2bb
--- /dev/null
+++ b/src/views/GradeView/__snapshots__/index.test.jsx.snap
@@ -0,0 +1,133 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`
+ Your final grade: {earned}/{possible}
+
+
+ Your grade is based on your self score for this problem. Other assessments don't count towards your final score.
+
+
+
+ + Your grade is based on your self score for this problem. Other assessments don't count towards your final score. +
+
+
+
+
+ Unweighted grades
+
+
+ These grades are given to your response. However, these are not used to compute your final grade.
+
+
+
+ + These grades are given to your response. However, these are not used to compute your final grade. +
+
+
+`;
diff --git a/src/views/GradeView/index.test.jsx b/src/views/GradeView/index.test.jsx
new file mode 100644
index 00000000..9cf2486d
--- /dev/null
+++ b/src/views/GradeView/index.test.jsx
@@ -0,0 +1,18 @@
+import { shallow } from '@edx/react-unit-test-utils';
+
+import GradeView from './index';
+
+jest.mock('components/ModalActions', () => 'ModalActions');
+jest.mock('./FinalGrade', () => 'FinalGrade');
+jest.mock('./Content', () => 'Content');
+
+describe('
+
+
+
+
+
+
+
+
+
+
+
+
-
- ));
+ return (
+ <>
+ {prompts.map((prompt, index) => (
+ // eslint-disable-next-line react/no-array-index-key
+
+
+ ))}
+
+ );
};
SubmissionPrompts.defaultProps = {
textResponses: null,
@@ -45,6 +47,7 @@ SubmissionPrompts.defaultProps = {
SubmissionPrompts.propTypes = {
textResponses: PropTypes.arrayOf(PropTypes.string),
onUpdateTextResponse: PropTypes.func.isRequired,
+ isReadOnly: PropTypes.bool.isRequired,
};
export default SubmissionPrompts;
diff --git a/src/views/SubmissionView/SubmissionPrompts.test.jsx b/src/views/SubmissionView/SubmissionPrompts.test.jsx
new file mode 100644
index 00000000..a53c81ab
--- /dev/null
+++ b/src/views/SubmissionView/SubmissionPrompts.test.jsx
@@ -0,0 +1,88 @@
+import { shallow } from '@edx/react-unit-test-utils';
+
+import {
+ usePrompts,
+ useSubmissionConfig,
+} from 'hooks/app';
+
+import SubmissionPrompts from './SubmissionPrompts';
+
+jest.mock('hooks/app', () => ({
+ usePrompts: jest.fn(),
+ useSubmissionConfig: jest.fn(),
+}));
+jest.mock('components/Prompt', () => 'Prompt');
+jest.mock('components/TextResponse', () => 'TextResponse');
+jest.mock('./TextResponseEditor', () => 'TextResponseEditor');
+
+describe('
+
+
+`;
diff --git a/src/views/SubmissionView/__snapshots__/SubmissionPrompts.test.jsx.snap b/src/views/SubmissionView/__snapshots__/SubmissionPrompts.test.jsx.snap
new file mode 100644
index 00000000..248add58
--- /dev/null
+++ b/src/views/SubmissionView/__snapshots__/SubmissionPrompts.test.jsx.snap
@@ -0,0 +1,72 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`
+
+
+
+
+
+
+
+
+
+
+
+
+
+`;
+
+exports[`
+
+
+
+
+
+
+
+ + Your response +
+
+
+`;
diff --git a/src/views/SubmissionView/hooks/useUploadedFilesData.js b/src/views/SubmissionView/hooks/useUploadedFilesData.js
index 743d733d..632ad0fa 100644
--- a/src/views/SubmissionView/hooks/useUploadedFilesData.js
+++ b/src/views/SubmissionView/hooks/useUploadedFilesData.js
@@ -14,11 +14,11 @@ const useUploadedFilesData = () => {
const deleteFileMutation = useDeleteFile();
const uploadFilesMutation = useUploadFiles();
- const response = useResponseData();
+ const response = useResponseData() || {};
const [value, setValue] = useKeyedState(
stateKeys.uploadedFiles,
- response ? response.uploadedFiles : [],
+ response?.uploadedFiles ? response.uploadedFiles : [],
);
React.useEffect(() => {
diff --git a/src/views/SubmissionView/hooks/useUploadedFilesData.test.js b/src/views/SubmissionView/hooks/useUploadedFilesData.test.js
new file mode 100644
index 00000000..c49b61f7
--- /dev/null
+++ b/src/views/SubmissionView/hooks/useUploadedFilesData.test.js
@@ -0,0 +1,46 @@
+import { mockUseKeyedState } from '@edx/react-unit-test-utils';
+
+import {
+ useResponseData,
+ useUploadFiles,
+ useDeleteFile,
+} from 'hooks/app';
+
+import useUploadedFilesData, { stateKeys } from './useUploadedFilesData';
+
+jest.mock('hooks/app', () => ({
+ useResponseData: jest.fn(),
+ useUploadFiles: jest.fn(),
+ useDeleteFile: jest.fn(),
+}));
+
+const state = mockUseKeyedState(stateKeys);
+
+describe('useUploadedFilesData', () => {
+ const mockUploadFilesMutation = jest.fn();
+ const mockDeleteFileMutation = jest.fn();
+ useUploadFiles.mockReturnValue({ mutateAsync: mockUploadFilesMutation });
+ useDeleteFile.mockReturnValue({ mutateAsync: mockDeleteFileMutation });
+
+ beforeEach(() => {
+ jest.clearAllMocks();
+ state.mock();
+ });
+ afterEach(() => { state.resetVals(); });
+
+ it('initializes uploadedFiles state to empty array if response is null', () => {
+ useResponseData.mockReturnValue();
+ useUploadedFilesData();
+ state.expectInitializedWith(stateKeys.uploadedFiles, []);
+ });
+ it('initializes uploadedFiles state to response.uploadedFiles', () => {
+ useResponseData.mockReturnValue({ uploadedFiles: ['file1', 'file2'] });
+ useUploadedFilesData();
+ state.expectInitializedWith(stateKeys.uploadedFiles, ['file1', 'file2']);
+ });
+ it('return correct mutation function', () => {
+ const { onFileUploaded, onDeletedFile } = useUploadedFilesData();
+ expect(onFileUploaded).toBe(mockUploadFilesMutation);
+ expect(onDeletedFile).toBe(mockDeleteFileMutation);
+ });
+});
diff --git a/src/views/SubmissionView/index.jsx b/src/views/SubmissionView/index.jsx
index 5e9dc7d6..e2b8e20c 100644
--- a/src/views/SubmissionView/index.jsx
+++ b/src/views/SubmissionView/index.jsx
@@ -17,7 +17,7 @@ import './index.scss';
import messages from './messages';
-export const SubmissionView = () => {
+const SubmissionView = () => {
const {
actionOptions,
showRubric,
diff --git a/src/views/SubmissionView/index.test.jsx b/src/views/SubmissionView/index.test.jsx
new file mode 100644
index 00000000..427fcf18
--- /dev/null
+++ b/src/views/SubmissionView/index.test.jsx
@@ -0,0 +1,47 @@
+import { shallow } from '@edx/react-unit-test-utils';
+
+import useSubmissionViewData from './hooks';
+
+import SubmissionView from './index';
+
+jest.mock('components/Rubric', () => 'Rubric');
+jest.mock('components/ModalActions', () => 'ModalActions');
+jest.mock('components/FileUpload', () => 'FileUpload');
+jest.mock('components/Instructions', () => 'Instructions');
+jest.mock('components/StatusAlert', () => 'StatusAlert');
+jest.mock('./SubmissionPrompts', () => 'SubmissionPrompts');
+jest.mock('./hooks', () => jest.fn());
+
+describe('
+
+
+
+
+
+
+
+ + Your response +
+
+
+
+
+`;
diff --git a/src/views/XBlockStudioView/components/StudioSchedule/__snapshots__/index.test.jsx.snap b/src/views/XBlockStudioView/components/StudioSchedule/__snapshots__/index.test.jsx.snap
new file mode 100644
index 00000000..d2a96b5f
--- /dev/null
+++ b/src/views/XBlockStudioView/components/StudioSchedule/__snapshots__/index.test.jsx.snap
@@ -0,0 +1,85 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`
+
+ test
+
+ start:
+
+
+
+ test
+
+ due:
+
+
+
+
+
+ Response
+
+ start:
+
+
+
+ Response
+
+ due:
+
+
+
+
+
+ Response
+
+ start:
+
+
+
+ Response
+
+ due:
+
+
+
+
+`;
+
+exports[`
+ Rubric
+
+ }
+>
+
+`;
+
+exports[`
+ Rubric
+
+ }
+/>
+`;
diff --git a/src/views/XBlockStudioView/components/__snapshots__/StudioViewSteps.test.jsx.snap b/src/views/XBlockStudioView/components/__snapshots__/StudioViewSteps.test.jsx.snap
new file mode 100644
index 00000000..3cb84d19
--- /dev/null
+++ b/src/views/XBlockStudioView/components/__snapshots__/StudioViewSteps.test.jsx.snap
@@ -0,0 +1,60 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`
+ Assessment steps
+
+ }
+>
+
+`;
+
+exports[`
+ Assessment steps
+
+ }
+>
+
+
+`;
diff --git a/src/views/XBlockStudioView/components/__snapshots__/StudioViewTitle.test.jsx.snap b/src/views/XBlockStudioView/components/__snapshots__/StudioViewTitle.test.jsx.snap
new file mode 100644
index 00000000..a4cf8482
--- /dev/null
+++ b/src/views/XBlockStudioView/components/__snapshots__/StudioViewTitle.test.jsx.snap
@@ -0,0 +1,33 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`
{formatMessage(messages.criteriaNameLabel)}
{criterion.name}
diff --git a/src/views/XBlockStudioView/components/StudioViewRubric.test.jsx b/src/views/XBlockStudioView/components/StudioViewRubric.test.jsx
new file mode 100644
index 00000000..e816da96
--- /dev/null
+++ b/src/views/XBlockStudioView/components/StudioViewRubric.test.jsx
@@ -0,0 +1,68 @@
+import { shallow } from '@edx/react-unit-test-utils';
+
+import { useRubricConfig } from 'hooks/app';
+
+import StudioViewRubric from './StudioViewRubric';
+
+jest.mock('hooks/app', () => ({
+ useRubricConfig: jest.fn(),
+}));
+jest.mock('./XBlockStudioViewProvider', () => ({
+ useXBlockStudioViewContext: () => ({
+ rubricIsOpen: true,
+ toggleRubric: jest.fn().mockName('toggleRubric'),
+ }),
+}));
+
+describe('
+
+ File uploads:
+
+
+
+ File upload limit:
+
+
+ 10
+
+
+
+ Text response:
+
+
+
+ Response editor:
+
+
+ WYSIWYG editor
+
+
+
+ Allow LaTeX responses:
+
+
+ True
+
+
+
+ Top responses:
+
+
+ 10
+
+
+
+ Teams enabled:
+
+
+ True
+
+
+
+ Show rubric during response:
+
+
+ False
+
+
+
+ Text response:
+
+
+
+ Response editor:
+
+
+ Text editor
+
+
+
+ Allow LaTeX responses:
+
+
+ False
+
+
+
+ Teams enabled:
+
+
+ False
+
+
+
+ Show rubric during response:
+
+
+ False
+
+
+
+
+
+
+
{formatMessage(messages.topResponsesLabel)} {leaderboardConfig.numberOfEntries}
diff --git a/src/views/XBlockStudioView/components/StudioViewSettings/index.test.jsx b/src/views/XBlockStudioView/components/StudioViewSettings/index.test.jsx new file mode 100644 index 00000000..100c7d0e --- /dev/null +++ b/src/views/XBlockStudioView/components/StudioViewSettings/index.test.jsx @@ -0,0 +1,72 @@ +import { shallow } from '@edx/react-unit-test-utils'; + +import { useORAConfigData } from 'hooks/app'; + +import StudioViewSettings from './index'; + +jest.mock('hooks/app', () => ({ + useORAConfigData: jest.fn(), +})); +jest.mock('../XBlockStudioViewProvider', () => ({ + useXBlockStudioViewContext: () => ({ + settingIsOpen: true, + toggleStudioViewSetting: jest.fn().mockName('toggleStudioViewSetting'), + }), +})); +jest.mock('./RequiredConfig', () => 'RequiredConfig'); +jest.mock('./FileUploadConfig', () => 'FileUploadConfig'); + +describe('
+
+ + + Criteria name: + + criterion1 +
++ + Criteria description: + + description1 +
++ + Criteria options: + +
+-
+
-
+
+ label1
+ :
+ 1
+
+ points
+
+
+ description1 +
+
+ -
+
+ label2
+ :
+ 2
+
+ points
+
+
+ description2 +
+
+
+
++ + Criteria name: + + criterion2 +
++ + Criteria description: + + description2 +
++ + Criteria options: + +
+-
+
-
+
+ label2
+ :
+ 2
+
+ points
+
+
+ description2 +
+
+
+
++ + Step + + 1 + : + + + Self assessment + +
++ + Step + + 2 + : + + + Peer assessment + +
+
+
+`;
+
+exports[`+ Test Title +
+ +
+
+`;
diff --git a/src/views/XBlockStudioView/index.jsx b/src/views/XBlockStudioView/index.jsx
index 778999ad..5099a134 100644
--- a/src/views/XBlockStudioView/index.jsx
+++ b/src/views/XBlockStudioView/index.jsx
@@ -10,7 +10,7 @@ import StudioViewPrompt from './components/StudioViewPrompt';
import './index.scss';
-export const XBlockStudioView = () => {
+const XBlockStudioView = () => {
useEffect(() => {
if (window.parent.length > 0) {
new ResizeObserver(() => {
diff --git a/src/views/XBlockStudioView/index.test.jsx b/src/views/XBlockStudioView/index.test.jsx
new file mode 100644
index 00000000..5dd19667
--- /dev/null
+++ b/src/views/XBlockStudioView/index.test.jsx
@@ -0,0 +1,26 @@
+import { shallow } from '@edx/react-unit-test-utils';
+
+import XBlockStudioView from './index';
+
+jest.mock('./components/XBlockStudioViewProvider', () => 'XBlockStudioViewProvider');
+jest.mock('./components/StudioSchedule', () => 'StudioSchedule');
+jest.mock('./components/StudioViewSteps', () => 'StudioViewSteps');
+jest.mock('./components/StudioViewSettings', () => 'StudioViewSettings');
+jest.mock('./components/StudioViewRubric', () => 'StudioViewRubric');
+jest.mock('./components/StudioViewTitle', () => 'StudioViewTitle');
+jest.mock('./components/StudioViewPrompt', () => 'StudioViewPrompt');
+
+describe('+ Test Title +
+ +
+
+
+`;
+
+exports[`
+
+
+`;
+
+exports[`
+
+
+`;
+
+exports[`
+
+
+`;
diff --git a/src/views/XBlockView/Actions/index.jsx b/src/views/XBlockView/Actions/index.jsx
index 48afa1cc..c80dea5d 100644
--- a/src/views/XBlockView/Actions/index.jsx
+++ b/src/views/XBlockView/Actions/index.jsx
@@ -30,6 +30,10 @@ const SubmissionActions = () => {
const loadNextAction = useLoadNextAction();
const stepConfigInfo = useAssessmentStepConfig().settings[activeStepName];
+ const onClick = React.useCallback(() => {
+ openModal({ view: activeStepName, title: activeStepName });
+ }, [activeStepName, openModal]);
+
const action = (() => {
if (
[stepNames.studentTraining, stepNames.peer].includes(activeStepName)
@@ -37,12 +41,17 @@ const SubmissionActions = () => {
&& stepInfo.numberOfAssessmentsCompleted > 0
&& !stepInfo.isWaitingForSubmissions
) {
- const onClick = () => openModal({ view: activeStepName, title: activeStepName });
const isOptional = activeStepName === stepNames.peer
- && stepInfo.numberOfAssessmentsCompleted >= stepConfigInfo.minNumberToGrade;
+ && stepInfo.numberOfAssessmentsCompleted
+ >= stepConfigInfo.minNumberToGrade;
return (
-