finished component reducer test cases

Author: fredosauce

__mocks__/electron.ts

@@ -11,7 +11,7 @@ describe('Testing componentReducer functionality', function () {
   // TEST 'ADD COMPONENT'
   describe('ADD COMPONENT reducer', () => {
     it('should add new reuseable component to state', () => {
-      const action = {
+      const action: Action = {
         type: 'ADD COMPONENT',
         payload: {
           componentName: "TestRegular",
@@ -30,7 +30,7 @@ describe('Testing componentReducer functionality', function () {
   // TEST 'ADD CHILD'
   describe('ADD CHILD reducer', () => {
     it('should add child component to top-level component', () => {
-      const action = {
+      const action: Action = {
         type: 'ADD CHILD',
         payload: {
           type: 'Component',
@@ -40,40 +40,74 @@ describe('Testing componentReducer functionality', function () {
       }
       // switch focus to very first root component
       state.canvasFocus = { componentId: 1, childId: null };
-      console.log(state);
       state = reducer(state, action);
       const newParent = state.components[0];
       // expect new parent's children array to have length 1
       expect(newParent.children.length).toEqual(1);
       // expect new child to have type 'Component' 
-      console.log('new child ', newParent.children[0]);
       expect(newParent.children[0].type).toEqual('Component');
       const addedChild = state.components.find(comp => comp.id === newParent.children[0].typeId);
       // expect new child typeId to correspond to component with name 'TestRegular'
       expect(addedChild.name).toEqual('TestRegular');
     })
   })
-  
+
+  // TEST 'CHANGE POSITION'
+  describe('CHANGE POSITION reducer ', () => {
+    it('should move position of an instance', () => {
+      const actionHtml: Action = {
+        type: 'ADD CHILD',
+        payload: {
+          type: 'HTML Element',
+          typeId: 9,
+          childId: null
+        }
+      }
+      state = reducer(state, actionHtml);
+      const actionChangePos: Action = {
+        type: 'CHANGE POSITION',
+        payload: {
+          currentChildId: 1,
+          newParentChildId: null 
+        }
+      }
+      state = reducer(state, actionChangePos);
+      const changeParent = state.components.find(comp => comp.id === state.canvasFocus.componentId);
+      const changeParentChildLength = changeParent.children.length;
+      // expect last child of parent to be moved Component element
+      expect(changeParent.children[changeParentChildLength-1].type).toEqual('Component');
+      // expect last child of parent to have current child ID of payload
+      expect(changeParent.children[changeParentChildLength-1].childId).toEqual(1);
+    })
+  })
+
   // TEST 'DELETE CHILD'
   describe('DELETE CHILD reducer', () => {
     it('should delete child of focused top-level component', () => {
-      console.log('state before delete child ', state);
+      // canvas still focused on childId: 2, which is an HTML element
+      const action: Action = {
+        type: 'DELETE CHILD'
+      }
+      state = reducer(state, action);
+      // expect only one remaining child
+      const delParent = state.components.find(comp => comp.id === state.canvasFocus.componentId);
+      // expect remaining child to have type 'Component'
+      expect(delParent.children.length).toEqual(1);
+      expect(delParent.children[delParent.children.length -1].type).toEqual('Component');
     })
   })
 
   // TEST 'CHANGE FOCUS'
   describe('CHANGE FOCUS reducer', () => {
     it('should change focus to specified component', () => {
-      const action = {
+      const action: Action = {
         type: 'CHANGE FOCUS',
         payload: {
           componentId: 2,
           childId: null
         }
       }
-      console.log('before change focus ', state.canvasFocus);
       state = reducer(state, action);
-      console.log('after change focus ', state.canvasFocus);
       expect(state.canvasFocus.componentId).toEqual(2);
       expect(state.canvasFocus.childId).toEqual(null);
     })
@@ -82,7 +116,7 @@ describe('Testing componentReducer functionality', function () {
   // TEST 'UPDATE CSS'
   describe('UPDATE CSS reducer', () => {
     it('should add style to focused component', () => {
-      const action = {
+      const action: Action = {
         type: 'UPDATE CSS',
         payload: {
           style: {
@@ -100,7 +134,7 @@ describe('Testing componentReducer functionality', function () {
   // TEST 'UPDATE PROJECT NAME'
   describe('UPDATE PROJECT NAME reducer', () => {
     it('should update project with specified name', () => {
-      const action = {
+      const action: Action = {
         type: 'UPDATE PROJECT NAME',
         payload: 'TESTNAME'
       }
@@ -113,7 +147,7 @@ describe('Testing componentReducer functionality', function () {
   // TEST 'CHANGE PROJECT TYPE'
   describe('CHANGE PROJECT TYPE reducer', () => {
     it('should change project type to specified type', () => {
-      const action = {
+      const action: Action = {
         type: 'CHANGE PROJECT TYPE',
         payload: {
           projectType: 'Classic React'
@@ -127,7 +161,7 @@ describe('Testing componentReducer functionality', function () {
   // TEST 'RESET STATE'
   describe('RESET STATE reducer', () => {
     it('should reset project to initial state', () => {
-      const action = {
+      const action: Action = {
         type: 'RESET STATE'
       }
       state = reducer(state, action);