Merge pull request #72 from oslabs-beta/bryan/testing

dropdown unit testing

Author: starkspark

src/app/__tests__/MainContainer.test.js

@@ -13,7 +13,7 @@ configure({ adapter: new Adapter() });
 const state = {
   tabs: {
     87: {
-      snapshots: [1,2,3,4],
+      snapshots: [1, 2, 3, 4],
       sliderIndex: 0,
       viewIndex: -1,
     },

src/app/__tests__/actionContainer.test.js

@@ -9,31 +9,31 @@ configure({ adapter: new Adapter() });
 describe('unit testing for Dropdown.jsx', () => {
   let wrapper;
   const props = {
-    options: [
-      { value: 2000, label: '0.5x' },
-      { value: 1000, label: '1.0x' },
-      { value: 500, label: '2.0x' },
+    speeds: [
+      { value: 1234, label: '0.5x' },
+      { value: 312, label: '1.0x' },
+      { value: 23, label: '2.0x' },
     ],
-    handleChangeSpeed: jest.fn(),
-    selectedOption: { value: 1000, label: '1.0x' },
+    setSpeed: jest.fn(),
+    selectedOption: { value: 312, label: '1.0x' },
   };
   beforeEach(() => {
     wrapper = shallow(<Dropdown {...props} />);
   });
 
   describe('Component', () => {
     test('array of objects that have value and label should be options props', () => {
-      expect(wrapper.props().options).toEqual(props.options);
+      expect(wrapper.props().options).toEqual(props.speeds);
     });
     test('selectedOption should be value property', () => {
-      expect(wrapper.props().value).toEqual(props.selectedOption);
+      expect(wrapper.props().value).toEqual(props.selectedSpeed);
     });
   });
 
   describe('handlechangeSpeed', () => {
     test('should invoke handleChangeSpeed onChange', () => {
       wrapper.simulate('change', { value: 2000, label: '0.5x' });
-      expect(props.handleChangeSpeed).toHaveBeenCalled();
+      expect(props.setSpeed).toHaveBeenCalled();
     });
   });
 });