Cleaned up ReacType files and console.logs

>
>
Co-authored-by: slhoehn <[email protected]>

Author: elenaconn

__tests__/componentReducer.test.ts

@@ -26,26 +26,26 @@ describe('Testing componentReducer functionality', () => {
     });
   });
 
- // TEST 'ADD COMPONENT' with new root 
- describe('ADD COMPONENT reducer', () => {
-  it('should add new reuseable component to state as the new root', () => {
-    const action: Action = {
-      type: 'ADD COMPONENT',
-      payload: {
-        componentName: 'TestRootChange',
-        id: 3,
-        root: true,
-      },
-    };
-    state = reducer(state, action);
-    
-    // expect state.components array to have length 3
-    const length = state.components.length;
-    expect(length).toEqual(3);
-    // expect new root to match id of component id of TestRootChange
-    expect(state.rootComponents[state.rootComponents.length - 1]).toEqual(action.payload.id);
+  // TEST 'ADD COMPONENT' with new root 
+  describe('ADD COMPONENT reducer', () => {
+    it('should add new reuseable component to state as the new root', () => {
+      const action: Action = {
+        type: 'ADD COMPONENT',
+        payload: {
+          componentName: 'TestRootChange',
+          id: 3,
+          root: true,
+        },
+      };
+      state = reducer(state, action);
+      
+      // expect state.components array to have length 3
+      const length = state.components.length;
+      expect(length).toEqual(3);
+      // expect new root to match id of component id of TestRootChange
+      expect(state.rootComponents[state.rootComponents.length - 1]).toEqual(action.payload.id);
+    });
   });
-});
 
   // TEST 'ADD CHILD'
   describe('ADD CHILD reducer', () => {
@@ -204,6 +204,7 @@ describe('Testing componentReducer functionality', () => {
         }
       }
       state = reducer(state, actionHTML2);
+      // invoking snapShotFunc is necessary to push actions into the past array, referenced in the UNDO functionality to define children
       snapShotFuncCopy();
 
       const actionUndo: Action = {

app/src/Dashboard/FormsContainer.tsx

@@ -19,7 +19,6 @@ const FormsContainer = () => {
   if (error) return <p>Error :{error}</p>;
   // based on resolver(readAllTests) for this query, the data is stored in the data object with the key 'readAllTests'
   const myTests = data.readAllTests;
-  console.log('myTests', myTests);
   // generate an array of Form components based on data
   const forms = myTests.map((test, index) => <Form key={index} id={test.id} description={test.description} likes={test.likes}/>);
 

app/src/components/login/SignIn.tsx

@@ -75,7 +75,6 @@ const SignIn: React.FC<LoginInt & RouteComponentProps> = props => {
       window.api.setCookie();
       window.api.getCookie(cookie => {
         // if a cookie exists, set localstorage item with cookie data, clear interval, go back to '/' route to load app
-        console.log(cookie);
         if (cookie[0]) {
           window.localStorage.setItem('ssid', cookie[0].value);
           clearInterval(githubCookie);

app/src/components/main/Canvas.tsx

@@ -97,4 +97,3 @@ function Canvas() {
 }
 
 export default Canvas;
-

app/src/components/main/DirectChildHTML.tsx

@@ -48,7 +48,6 @@ function DirectChildHTML({
   function onClickHandler(event) {
     event.stopPropagation();
     changeFocus(state.canvasFocus.componentId, childId);
-    console.log('state in directChild', state)
   }
 
   // combine all styles so that higher priority style specifications overrule lower priority style specifications

app/src/reducers/componentReducer.ts

@@ -619,6 +619,7 @@ const reducer = (state: State, action: Action) => {
         HTMLTypes
       };
     }
+
     case 'UNDO': {
       const focusIndex = state.canvasFocus.componentId - 1;
       // if the past array is empty, return state
@@ -644,6 +645,7 @@ const reducer = (state: State, action: Action) => {
         ...state
       };
     }
+    
     case 'REDO': {
       const focusIndex = state.canvasFocus.componentId - 1;
       //if future array is empty, return state