Update sinon spy tests

Use the BDD syntax from sinon-chai.

Author: eatyourgreens

app/classifier/tasks/components/TaskInputField/TaskInputField.spec.jsx

@@ -12,8 +12,8 @@ import sinon from 'sinon';
 import { TaskInputField, StyledTaskInputField } from './TaskInputField';
 import { mockReduxStore, radioTypeAnnotation } from '../../testHelpers';
 
-describe('TaskInputField', function() {
-  describe('render', function() {
+describe('TaskInputField', function () {
+  describe('render', function () {
     let wrapper;
     before(function () {
       wrapper = mount(<TaskInputField annotation={radioTypeAnnotation} index={0} type="radio" />, mockReduxStore);
@@ -39,24 +39,24 @@ describe('TaskInputField', function() {
       wrapper.setProps({ className: 'active' });
       expect(wrapper.props().className).to.include('active');
     });
-  })
+  });
 
 
-  describe('onChange method', function() {
+  describe('onChange method', function () {
     let onChangePropsSpy;
     let wrapper;
-    before(function() {
+    before(function () {
       onChangePropsSpy = sinon.spy();
       wrapper = mount(<TaskInputField annotation={radioTypeAnnotation} onChange={onChangePropsSpy} index={0} type="radio" />, mockReduxStore);
     });
 
-    afterEach(function() {
+    afterEach(function () {
       onChangePropsSpy.resetHistory();
     });
 
-    it('should call onChange when the on change event is fired', function() {
+    it('should call onChange when the on change event is fired', function () {
       wrapper.find('input').simulate('change');
-      expect(onChangePropsSpy.calledOnce).to.be.true;
+      expect(onChangePropsSpy).to.have.been.calledOnce;
     });
   });
 });

app/classifier/tasks/multiple/index.spec.js

@@ -71,14 +71,14 @@ describe('MultipleChoiceTask', function () {
     it('should call props.onChange when an answer changes', function () {
       const firstAnswer = wrapper.find(GenericTask).prop('answers')[0];
       firstAnswer.props.onChange({ target: { checked: true } });
-      expect(onChangeSpy.calledOnce).to.be.true;
+      expect(onChangeSpy).to.have.been.calledOnce;
     });
 
     it('should call handleChange with the answer array index', function() {
       wrapper.setProps({ annotation });
       const firstAnswer = wrapper.find(GenericTask).prop('answers')[0];
       firstAnswer.props.onChange({ target: { checked: true } });
-      expect(handleChangeSpy.calledWith(0)).to.be.true;
+      expect(handleChangeSpy).to.have.been.calledWith(0);
     });
   });
 

app/classifier/tasks/single/index.spec.js

@@ -70,13 +70,13 @@ describe('SingleChoiceTask', function () {
     it('should call handleChange with the answer array index', function () {
       const firstAnswer = wrapper.find(GenericTask).prop('answers')[0];
       firstAnswer.props.onChange({ target: { checked: true } });
-      expect(handleChangeSpy.calledWith(0)).to.be.true;
+      expect(handleChangeSpy).to.have.been.calledWith(0);
     });
 
     it('should call props.onChange when an answer changes and the target is checked', function () {
       const firstAnswer = wrapper.find(GenericTask).prop('answers')[0];
       firstAnswer.props.onChange({ target: { checked: true } });
-      expect(onChangeSpy.calledOnce).to.be.true;
+      expect(onChangeSpy).to.have.been.calledOnce;
     });
   });