Split language config into separate store module with it's own spec.

Author: glenpike

src/store/index.js

@@ -1,14 +1,14 @@
 import Vue from 'vue';
 import Vuex from 'vuex';
 import order from './modules/order';
+import language from './modules/language';
 
 Vue.use(Vuex);
 
 export default new Vuex.Store({
   state: {
     loggedInUser: null,
     dojo: null,
-    chosenLanguageConfig: null,
   },
   mutations: {
     setLoggedInUser(state, user) {
@@ -17,9 +17,6 @@ export default new Vuex.Store({
     setDojo(state, dojo) {
       Vue.set(state, 'dojo', dojo);
     },
-    setChosenLanguageConfig(state, chosenLanguageConfig) {
-      Vue.set(state, 'chosenLanguageConfig', chosenLanguageConfig);
-    },
   },
   actions: {
     async getLoggedInUser({ commit }) {
@@ -32,9 +29,6 @@ export default new Vuex.Store({
       const dojo = (await Vue.http.get(`${Vue.config.apiServer}/api/2.0/dojos/${dojoId}`)).body;
       commit('setDojo', dojo);
     },
-    updateChosenLanguageConfig({ commit }, chosenLanguageConfig) {
-      commit('setChosenLanguageConfig', chosenLanguageConfig);
-    },
   },
   getters: {
     loggedInUser: (state) => {
@@ -52,13 +46,13 @@ export default new Vuex.Store({
       state.loggedInUser.login !== null &&
       state.loggedInUser.login.id !== undefined,
     dojo: state => state.dojo,
-    chosenLanguageConfig: state => state.chosenLanguageConfig,
     // User properties
     hasRequests: (state, getters) => getters.isLoggedIn &&
       getters.loggedInUser.joinRequests &&
       getters.loggedInUser.joinRequests.length > 0,
   },
   modules: {
     order,
+    language,
   },
 });

src/store/modules/language.js

@@ -0,0 +1,29 @@
+import Vue from 'vue';
+
+const state = {
+  chosenLanguageConfig: null,
+};
+
+export const mutations = {
+  setChosenLanguageConfig(state, chosenLanguageConfig) {
+    Vue.set(state, 'chosenLanguageConfig', chosenLanguageConfig);
+  },
+};
+
+export const actions = {
+  updateChosenLanguageConfig({ commit }, chosenLanguageConfig) {
+    commit('setChosenLanguageConfig', chosenLanguageConfig);
+  },
+}
+
+export const getters = {
+  chosenLanguageConfig: state => state.chosenLanguageConfig,
+}
+
+export default {
+  namespaced: false,
+  state,
+  mutations,
+  actions,
+  getters,
+};

test/unit/specs/store/modules/language.spec.js

@@ -0,0 +1,36 @@
+import { mutations, actions, getters } from '@/store/modules/language';
+import { expect } from 'chai';
+
+describe('Language Store', () => {
+  describe('mutations', () => {
+    const { setChosenLanguageConfig } = mutations
+    it('setChosenLanguageConfig', () => {
+      const state = { chosenLanguageConfig: null };
+      const newLanguageConfig = { dir: 'test' };
+
+      setChosenLanguageConfig(state, newLanguageConfig);
+
+      expect(state.chosenLanguageConfig.dir).to.eq('test');
+    });
+  });
+
+  describe('actions', () => {
+    it('updateChosenLanguageConfig', () => {
+      const commit = sinon.spy()
+      const state = {};
+
+      const chosenLanguageConfig = { dir: 'down' };
+
+      actions.updateChosenLanguageConfig({ commit, state }, chosenLanguageConfig);
+
+      expect(commit.args).to.deep.equal([['setChosenLanguageConfig', chosenLanguageConfig]]);
+    });
+  });
+
+  describe('getters', () => {
+    it('chosenLanguageConfig', () => {
+      const state = { chosenLanguageConfig: { dir: 'left' } };
+      expect(getters.chosenLanguageConfig(state).dir).to.eq('left');
+    });
+  });
+});

test/unit/specs/store/store.spec.js

@@ -1,15 +1,16 @@
 import store from '@/store/';
 import { expect } from 'chai';
 
-describe.only('Store', () => {
+describe('Store', () => {
   describe('mutations', () => {
-    it('setChosenLanguageConfig', () => {
-      const newLanguageConfig = { dir: 'test' };
+    // it('setChosenLanguageConfig', () => {
+    //   console.log('store ', store);
+    //   const newLanguageConfig = { dir: 'test' };
 
-      store.commit('setChosenLanguageConfig', newLanguageConfig);
+    //   store.commit('setChosenLanguageConfig', newLanguageConfig);
 
-      expect(store.state.chosenLanguageConfig.dir).to.eq('test');
-    });
+    //   expect(store.state.chosenLanguageConfig.dir).to.eq('test');
+    // });
 
     it('setLoggedInUser', () => {
       const user = { user: 'bob' };
@@ -26,5 +27,13 @@ describe.only('Store', () => {
 
       expect(store.state.dojo.name).to.eq('SuperDojo');
     });
-  })
+  });
+
+  // describe('actions', () => {
+  //   it('updateChosenLanguageConfig', () => {
+  //     console.log('actions: ', store._actions);
+  //     const commit = sinon.spy()
+  //     const state = {};
+  //   });
+  // });
 });

test/unit/specs/users/service.spec.js

@@ -2,16 +2,10 @@ import Vue from 'vue';
 import UserService from 'inject-loader!@/users/service';
 
 describe('UserService', () => {
-  let storeMock;
   let UserServiceWithMocks;
 
   beforeEach(() => {
-    storeMock = {
-      dispatch: sinon.stub(),
-    };
-    UserServiceWithMocks = UserService({
-      '@/store': storeMock,
-    }).default;
+    UserServiceWithMocks = UserService().default;
   });
 
   afterEach(() => {