Merge pull request #147 from open-source-labs/bryan/testRefactorTwo

Bryan/test refactor two

Author: MrBeeAreWhy

src/store/actions.js

@@ -100,7 +100,7 @@ const actions = {
 
   // copy the active component
   [types.copyActiveComponent]: ({ commit }, payload) => {
-    commit(types.COPY_ACTIVE_COMPONENT, payload);
+    commit(types.COPY_ACTIVE_COMPONENT);
   },
   // paste the active component copy
   [types.pasteActiveComponent]: ({ commit, state }) => {

src/store/mutations.js

@@ -248,7 +248,7 @@ const mutations = {
     state.userActions.splice(index, 1);
   },
   // copy and paste functionality
-  [types.COPY_ACTIVE_COMPONENT]: (state, payload) => {
+  [types.COPY_ACTIVE_COMPONENT]: (state) => {
     // copy activeComponentObj and place in the state but without children
     const copy = { ...state.activeComponentObj };
     // copy.componentName;

test/jest/__tests__/copyPaste.spec.js

@@ -0,0 +1,63 @@
+import { mount, shallowMount } from "@vue/test-utils";
+import { createApp } from "vue";
+import * as All from "quasar";
+const { Quasar, date } = All;
+import actions from "../../../src/store/actions";
+import mutations from "../../../src/store/mutations";
+import * as types from "../../../src/store/types";
+import Vuex from "vuex";
+
+describe("Tests for copy and pasting", () => {
+  const App = {}
+  const app = createApp(App);
+  app.use(Quasar, Vuex);
+  let state;
+  beforeEach(() => {
+    state = {
+      componentMap: {
+        testComp: {
+          componentName: "testComp",
+          children: [],
+          htmlList: [],
+          componentActions: [],
+          state: ["state1", "state2"],
+          actions: ["action1", "action2"]
+        }
+      },
+      activeComponent: "test",
+      userState: ['state1', 'state2'],
+      userActions: ['action1', 'action2'],
+      userProps: ['prop1', 'prop2'],
+      routes: {
+        HomeView: [{componentName: 'testComp', parent: []}],
+      },
+      activeRoute: 'HomeView',
+      pasteTimer: 0,
+    };
+  });
+
+  test('"[types.copyActiveComponent]" action to commit COPY_ACTIVE_COMPONENT', () => {
+    const commit = jest.fn();
+    actions[types.copyActiveComponent]({ commit });
+    expect(commit).toHaveBeenCalledWith(types.COPY_ACTIVE_COMPONENT);
+  });
+
+  test('"[types.COPY_ACTIVE_COMPONENT]" to update state successfully', ()=>{
+    mutations[types.SET_ACTIVE_COMPONENT](state, 'testComp');
+    mutations[types.COPY_ACTIVE_COMPONENT](state);
+    expect(state.copiedComponent.componentName).toBe('testComp');
+  })
+
+  test('"[types.PASTE_ACTIVE_COMPONENT]" to update state successfully', () => {
+    mutations[types.SET_ACTIVE_COMPONENT](state, 'testComp');
+    mutations[types.COPY_ACTIVE_COMPONENT](state);
+    mutations[types.PASTE_ACTIVE_COMPONENT](state);
+    expect(Object.keys(state.componentMap).length).toBe(2);
+  });
+
+  test('"[types.PASTE_ACTIVE_COMPONENT]" to throttle via state paste timer', () => {
+    mutations[types.UPDATE_PASTE_TIMER](state);
+    expect(state.pasteTimer).not.toBe(0);
+  });
+
+});

test/jest/__tests__/routeTests.spec.js

@@ -93,16 +93,14 @@ describe("Test Suite for route actions and related mutations", () => {
     test('"[types.ADD_COMPONENT_TO_COMPONENT_CHILDREN]" mutation to add created route as one of Apps children', () => {
       let component = 'App';
       let value = state.componentMap[state.activeRoute].componentName;
-      // console.log('pre: ', state.componentMap[component].children);
       expect(state.componentMap[component].children.length).toBe(1);
       mutations[types.ADD_COMPONENT_TO_COMPONENT_CHILDREN](state, {component, value});
-      // console.log('post: ', state.componentMap[component].children);
       expect(state.componentMap[component].children.length).toBe(2);
       expect(state.componentMap[component].children[state.componentMap[component].children.length-1]).toBe(value);
 
     });
 
-    test('"[types.DELETE_ROUTE]" mutation to ---', () => {
+    test('"[types.DELETE_ROUTE]" mutation to update state by removing a route', () => {
       const payload = 'testRoute';
       let flag = false;
       // pre mutation tests