From 9d522353cc36c52b15d57c71c83bbe9c459625a3 Mon Sep 17 00:00:00 2001
From: roschaefer <git@roschaefer.de>
Date: Thu, 24 Oct 2019 11:07:13 +0200
Subject: [PATCH 1/3] fix: import missing files in maintenance image

@ogerly the maintenance page needs just two components of the webapp
folder. For that reason we import only these. If you add imports to
`LocaleSwitch` component, you need to copy those files in the
Dockerfile.
---
 webapp/Dockerfile.maintenance | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/webapp/Dockerfile.maintenance b/webapp/Dockerfile.maintenance
index b32bca8e1..0e4a618ed 100644
--- a/webapp/Dockerfile.maintenance
+++ b/webapp/Dockerfile.maintenance
@@ -1,5 +1,5 @@
 FROM node:12.13.0-alpine as build
-LABEL Description="Web Frontend of the Social Network Human-Connection.org" Vendor="Human-Connection gGmbH" Version="0.0.1" Maintainer="Human-Connection gGmbH (developer@human-connection.org)"
+LABEL Description="Maintenance page of the Social Network Human-Connection.org" Vendor="Human-Connection gGmbH" Version="0.0.1" Maintainer="Human-Connection gGmbH (developer@human-connection.org)"
 
 EXPOSE 3000
 CMD ["yarn", "run", "start"]
@@ -19,6 +19,7 @@ RUN yarn install --production=false --frozen-lockfile --non-interactive
 
 COPY assets assets
 COPY components/LocaleSwitch/ components/LocaleSwitch
+COPY graphql graphql
 COPY components/Dropdown.vue components/Dropdown.vue
 COPY layouts/blank.vue layouts/blank.vue
 COPY locales locales

From 821ea6d6711744face1366227070f8786dec52cb Mon Sep 17 00:00:00 2001
From: mattwr18 <mattwr18@gmail.com>
Date: Thu, 24 Oct 2019 12:45:55 +0200
Subject: [PATCH 2/3] Update LocaleSwitch with guard clause

- return if there is no currentUser
- do not copy graphql directory as there is no need to send messages to
backend
---
 webapp/Dockerfile.maintenance                   |  1 -
 webapp/components/LocaleSwitch/LocaleSwitch.vue | 12 ++++++++++--
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/webapp/Dockerfile.maintenance b/webapp/Dockerfile.maintenance
index 0e4a618ed..7daf4f822 100644
--- a/webapp/Dockerfile.maintenance
+++ b/webapp/Dockerfile.maintenance
@@ -19,7 +19,6 @@ RUN yarn install --production=false --frozen-lockfile --non-interactive
 
 COPY assets assets
 COPY components/LocaleSwitch/ components/LocaleSwitch
-COPY graphql graphql
 COPY components/Dropdown.vue components/Dropdown.vue
 COPY layouts/blank.vue layouts/blank.vue
 COPY locales locales
diff --git a/webapp/components/LocaleSwitch/LocaleSwitch.vue b/webapp/components/LocaleSwitch/LocaleSwitch.vue
index 307425aca..38334621e 100644
--- a/webapp/components/LocaleSwitch/LocaleSwitch.vue
+++ b/webapp/components/LocaleSwitch/LocaleSwitch.vue
@@ -33,12 +33,12 @@
 </template>
 
 <script>
+import gql from 'graphql-tag'
 import Dropdown from '~/components/Dropdown'
 import find from 'lodash/find'
 import orderBy from 'lodash/orderBy'
 import locales from '~/locales'
 import { mapGetters, mapMutations } from 'vuex'
-import { localeMutation } from '~/graphql/User.js'
 
 export default {
   components: {
@@ -84,9 +84,17 @@ export default {
       setCurrentUser: 'auth/SET_USER',
     }),
     async updateUserLocale() {
+      if (!this.currentUser || !this.currentUser.id) return null
       try {
         await this.$apollo.mutate({
-          mutation: localeMutation(),
+          mutation: gql`
+            mutation($id: ID!, $locale: String) {
+              UpdateUser(id: $id, locale: $locale) {
+                id
+                locale
+              }
+            }
+          `,
           variables: {
             id: this.currentUser.id,
             locale: this.$i18n.locale(),

From 5c12914e02bdc835b88c1887b553f7c23834b296 Mon Sep 17 00:00:00 2001
From: mattwr18 <mattwr18@gmail.com>
Date: Thu, 24 Oct 2019 13:52:21 +0200
Subject: [PATCH 3/3] Add test cases for LocaleSwitch

- if there is a currentUser then update their locale in the database,
otherwise, do not.
---
 .../LocaleSwitch/LocaleSwitch.spec.js         | 70 ++++++++++++++++---
 1 file changed, 59 insertions(+), 11 deletions(-)

diff --git a/webapp/components/LocaleSwitch/LocaleSwitch.spec.js b/webapp/components/LocaleSwitch/LocaleSwitch.spec.js
index eba7bd068..3e9e17443 100644
--- a/webapp/components/LocaleSwitch/LocaleSwitch.spec.js
+++ b/webapp/components/LocaleSwitch/LocaleSwitch.spec.js
@@ -2,26 +2,43 @@ import { mount, createLocalVue } from '@vue/test-utils'
 import Styleguide from '@human-connection/styleguide'
 import VTooltip from 'v-tooltip'
 import LocaleSwitch from './LocaleSwitch.vue'
+import Vuex from 'vuex'
 
 const localVue = createLocalVue()
 
 localVue.use(Styleguide)
 localVue.use(VTooltip)
+localVue.use(Vuex)
 
 describe('LocaleSwitch.vue', () => {
-  let wrapper
-  let mocks
-  let computed
-  let deutschLanguageItem
+  let wrapper, mocks, computed, deutschLanguageItem, getters
 
   beforeEach(() => {
     mocks = {
       $i18n: {
-        locale: () => 'de',
-        set: jest.fn(),
+        locale: () => 'en',
+        set: jest.fn(locale => locale),
       },
       $t: jest.fn(),
+      $toast: {
+        success: jest.fn(a => a),
+        error: jest.fn(a => a),
+      },
       setPlaceholderText: jest.fn(),
+      $apollo: {
+        mutate: jest
+          .fn()
+          .mockResolvedValueOnce({
+            data: {
+              UpdateUser: {
+                locale: 'de',
+              },
+            },
+          })
+          .mockRejectedValueOnce({
+            message: 'Please log in!',
+          }),
+      },
     }
     computed = {
       current: () => {
@@ -40,12 +57,21 @@ describe('LocaleSwitch.vue', () => {
         ]
       },
     }
+    getters = {
+      'auth/user': () => {
+        return { id: 'u35' }
+      },
+    }
   })
 
-  describe('mount', () => {
-    const Wrapper = () => {
-      return mount(LocaleSwitch, { mocks, localVue, computed })
-    }
+  const Wrapper = () => {
+    const store = new Vuex.Store({
+      getters,
+    })
+    return mount(LocaleSwitch, { mocks, localVue, computed, store })
+  }
+
+  describe('with current user', () => {
     beforeEach(() => {
       wrapper = Wrapper()
       wrapper.find('.locale-menu').trigger('click')
@@ -53,8 +79,30 @@ describe('LocaleSwitch.vue', () => {
       deutschLanguageItem.trigger('click')
     })
 
-    it("changes a user's locale", () => {
+    it("sets a user's locale", () => {
       expect(mocks.$i18n.set).toHaveBeenCalledTimes(1)
     })
+
+    it("updates the user's locale in the database", () => {
+      expect(mocks.$apollo.mutate).toHaveBeenCalledTimes(1)
+    })
+  })
+
+  describe('no current user', () => {
+    beforeEach(() => {
+      getters = {
+        'auth/user': () => {
+          return null
+        },
+      }
+      wrapper = Wrapper()
+      wrapper.find('.locale-menu').trigger('click')
+      deutschLanguageItem = wrapper.findAll('li').at(1)
+      deutschLanguageItem.trigger('click')
+    })
+
+    it('does not send a UpdateUser mutation', () => {
+      expect(mocks.$apollo.mutate).not.toHaveBeenCalled()
+    })
   })
 })