Fix some frontend tests

Coursemology · Jan 15, 2018 · b7709df · b7709df
1 parent e6e4488
commit b7709df
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 5 deletions.
diff --git a/...s/course/assessment/containers/MaterialUploader/__test__/__snapshots__/index.test.js.snap b/...s/course/assessment/containers/MaterialUploader/__test__/__snapshots__/index.test.js.snap
@@ -96,13 +96,15 @@ exports[`<MaterialList /> renders the component with materials 1`] = `
     <Material
       deleting={false}
       id={1}
+      key="1"
       name="Material 1"
       onMaterialDelete={[Function]}
       updatedAt="2017-01-01T01:00:00.0000000Z"
     />
     <Material
       deleting={false}
       id={2}
+      key="2"
       name="Material 2"
       onMaterialDelete={[Function]}
       updatedAt="2017-01-01T02:00:00.0000000Z"

diff --git a/client/app/bundles/course/assessment/question/scribing/__test__/index.test.js b/client/app/bundles/course/assessment/question/scribing/__test__/index.test.js
@@ -20,9 +20,11 @@ const scribingId = '3';
 
 const mockFields = {
   question_scribing: {
-    title: 'Scribing Exercise',
+    description: '',
     maximum_grade: 10,
     skill_ids: [],
+    staff_only_comments: '',
+    title: 'Scribing Exercise',
   },
 };
 

diff --git a/client/app/bundles/course/survey/containers/ResponseForm/__test__/index.test.jsx b/client/app/bundles/course/survey/containers/ResponseForm/__test__/index.test.jsx
@@ -107,10 +107,10 @@ describe('<ResponseForm />', () => {
       textResponseAnswer = responseAnswers.at(0);
       multipleChoiceAnswer = responseAnswers.at(1);
       multipleResponseAnswer = responseAnswers.at(2);
-    }
+    };
     updateResponse();
 
-    const lastMRQOptionCheckbox = multipleResponseAnswer.find('OptionsListItem').last().find('Checkbox');
+    let lastMRQOptionCheckbox = multipleResponseAnswer.find('OptionsListItem').last().find('Checkbox');
     lastMRQOptionCheckbox.props().onCheck(null, true);
 
     const submitButton = responseForm.find('button').last();
@@ -155,6 +155,7 @@ describe('<ResponseForm />', () => {
     textResponse.simulate('change', { target: { value: newAnswer } });
     const firstMCQOptionRadio = multipleChoiceAnswer.find('OptionsListItem').first().find('RadioButton');
     firstMCQOptionRadio.props().onCheck(null, firstMCQOptionRadio.props().value);
+    lastMRQOptionCheckbox = multipleResponseAnswer.find('OptionsListItem').last().find('Checkbox');
     lastMRQOptionCheckbox.props().onCheck(null, false);
 
     submitButton.simulate('click');

diff --git a/...nt/app/bundles/course/survey/pages/SurveyShow/Section/__test__/NewQuestionButton.test.jsx b/...nt/app/bundles/course/survey/pages/SurveyShow/Section/__test__/NewQuestionButton.test.jsx
@@ -1,4 +1,5 @@
 import React from 'react';
+import ReactDOM from 'react-dom';
 import ReactTestUtils from 'react-dom/test-utils';
 import { mount } from 'enzyme';
 import CourseAPI from 'api/course';

diff --git a/client/app/lib/components/redux-form/__test__/SingleFileInput.test.js b/client/app/lib/components/redux-form/__test__/SingleFileInput.test.js
@@ -3,7 +3,6 @@ import PropTypes from 'prop-types';
 import { mount } from 'enzyme';
 import SingleFileInput from '../SingleFileInput';
 
-
 describe('<SingleFileInput />', () => {
   it('renders with url and name when provided in badge/avatar style', () => {
     const singleFileInput = mount(
@@ -30,7 +29,6 @@ describe('<SingleFileInput />', () => {
     );
 
     const avatar = singleFileInput.find('Avatar').first();
-
     expect(singleFileInput.find('.file-name').text().includes('bar')).toBeTruthy();
     expect(avatar.prop('src')).toEqual('foo');
     expect(avatar.prop('icon')).toBeUndefined();