From 32f8e43c90e24b43201886faab70f596cc82f6be Mon Sep 17 00:00:00 2001
From: ogerly <fridolin@tutanota.com>
Date: Sun, 29 May 2022 12:08:28 +0200
Subject: [PATCH] remove files and code from coinanimation settings

---
 .../UserSettings/UserCoinAnimation.spec.js    | 127 ------------------
 .../UserSettings/UserCoinAnimation.vue        |  65 ---------
 frontend/src/graphql/mutations.js             |   2 -
 frontend/src/graphql/queries.js               |   2 -
 frontend/src/locales/de.json                  |   5 -
 frontend/src/locales/en.json                  |   5 -
 frontend/src/pages/Profile.spec.js            |   4 -
 frontend/src/pages/Profile.vue                |   4 -
 frontend/src/store/store.js                   |   6 -
 frontend/src/store/store.test.js              |  40 ++----
 10 files changed, 10 insertions(+), 250 deletions(-)
 delete mode 100644 frontend/src/components/UserSettings/UserCoinAnimation.spec.js
 delete mode 100644 frontend/src/components/UserSettings/UserCoinAnimation.vue

diff --git a/frontend/src/components/UserSettings/UserCoinAnimation.spec.js b/frontend/src/components/UserSettings/UserCoinAnimation.spec.js
deleted file mode 100644
index aabf927fb..000000000
--- a/frontend/src/components/UserSettings/UserCoinAnimation.spec.js
+++ /dev/null
@@ -1,127 +0,0 @@
-import { mount } from '@vue/test-utils'
-import UserCoinAnimation from './UserCoinAnimation'
-import { updateUserInfos } from '@/graphql/mutations'
-
-import { toastErrorSpy, toastSuccessSpy } from '@test/testSetup'
-
-const localVue = global.localVue
-
-const mockAPIcall = jest.fn()
-
-const storeCommitMock = jest.fn()
-
-describe('UserCard_CoinAnimation', () => {
-  let wrapper
-
-  const mocks = {
-    $t: jest.fn((t) => t),
-    $store: {
-      state: {
-        language: 'de',
-        coinanimation: true,
-      },
-      commit: storeCommitMock,
-    },
-    $apollo: {
-      mutate: mockAPIcall,
-    },
-  }
-
-  const Wrapper = () => {
-    return mount(UserCoinAnimation, { localVue, mocks })
-  }
-
-  describe('mount', () => {
-    beforeEach(() => {
-      jest.clearAllMocks()
-      wrapper = Wrapper()
-    })
-
-    it('renders the component', () => {
-      expect(wrapper.find('div#formusercoinanimation').exists()).toBeTruthy()
-    })
-
-    it('has an edit BFormCheckbox switch', () => {
-      expect(wrapper.find('.Test-BFormCheckbox').exists()).toBeTruthy()
-    })
-
-    describe('enable with success', () => {
-      beforeEach(async () => {
-        await wrapper.setData({ CoinAnimationStatus: false })
-        mockAPIcall.mockResolvedValue({
-          data: {
-            updateUserInfos: {
-              validValues: 1,
-            },
-          },
-        })
-        await wrapper.find('input').setChecked()
-      })
-
-      it('calls the updateUserInfos mutation', () => {
-        expect(mockAPIcall).toBeCalledWith({
-          mutation: updateUserInfos,
-          variables: {
-            coinanimation: true,
-          },
-        })
-      })
-
-      it('updates the store', () => {
-        expect(storeCommitMock).toBeCalledWith('coinanimation', true)
-      })
-
-      it('toasts a success message', () => {
-        expect(toastSuccessSpy).toBeCalledWith('settings.coinanimation.True')
-      })
-    })
-
-    describe('disable with success', () => {
-      beforeEach(async () => {
-        await wrapper.setData({ CoinAnimationStatus: true })
-        mockAPIcall.mockResolvedValue({
-          data: {
-            updateUserInfos: {
-              validValues: 1,
-            },
-          },
-        })
-        await wrapper.find('input').setChecked(false)
-      })
-
-      it('calls the subscribe mutation', () => {
-        expect(mockAPIcall).toBeCalledWith({
-          mutation: updateUserInfos,
-          variables: {
-            coinanimation: false,
-          },
-        })
-      })
-
-      it('updates the store', () => {
-        expect(storeCommitMock).toBeCalledWith('coinanimation', false)
-      })
-
-      it('toasts a success message', () => {
-        expect(toastSuccessSpy).toBeCalledWith('settings.coinanimation.False')
-      })
-    })
-
-    describe('disable with server error', () => {
-      beforeEach(() => {
-        mockAPIcall.mockRejectedValue({
-          message: 'Ouch',
-        })
-        wrapper.find('input').trigger('change')
-      })
-
-      it('resets the CoinAnimationStatus', () => {
-        expect(wrapper.vm.CoinAnimationStatus).toBeTruthy()
-      })
-
-      it('toasts an error message', () => {
-        expect(toastErrorSpy).toBeCalledWith('Ouch')
-      })
-    })
-  })
-})
diff --git a/frontend/src/components/UserSettings/UserCoinAnimation.vue b/frontend/src/components/UserSettings/UserCoinAnimation.vue
deleted file mode 100644
index 040825105..000000000
--- a/frontend/src/components/UserSettings/UserCoinAnimation.vue
+++ /dev/null
@@ -1,65 +0,0 @@
-<template>
-  <b-card
-    id="formusercoinanimation"
-    class="bg-transparent gradido-custom-background gradido-no-border-radius"
-  >
-    <div>
-      <b-row class="mb-3">
-        <b-col class="mb-2 col-12">
-          <small>
-            <b>{{ $t('settings.coinanimation.coinanimation') }}</b>
-          </small>
-        </b-col>
-        <b-col class="col-12">
-          <b-form-checkbox
-            class="Test-BFormCheckbox"
-            v-model="CoinAnimationStatus"
-            name="check-button"
-            switch
-            @change="onSubmit"
-          >
-            {{
-              CoinAnimationStatus
-                ? $t('settings.coinanimation.True')
-                : $t('settings.coinanimation.False')
-            }}
-          </b-form-checkbox>
-        </b-col>
-      </b-row>
-    </div>
-  </b-card>
-</template>
-<script>
-import { updateUserInfos } from '@/graphql/mutations'
-export default {
-  name: 'UserCoinAnimation',
-  data() {
-    return {
-      CoinAnimationStatus: this.$store.state.coinanimation,
-    }
-  },
-  methods: {
-    async onSubmit() {
-      this.$apollo
-        .mutate({
-          mutation: updateUserInfos,
-          variables: {
-            coinanimation: this.CoinAnimationStatus,
-          },
-        })
-        .then(() => {
-          this.$store.commit('coinanimation', this.CoinAnimationStatus)
-          this.toastSuccess(
-            this.CoinAnimationStatus
-              ? this.$t('settings.coinanimation.True')
-              : this.$t('settings.coinanimation.False'),
-          )
-        })
-        .catch((error) => {
-          this.CoinAnimationStatus = this.$store.state.coinanimation
-          this.toastError(error.message)
-        })
-    },
-  },
-}
-</script>
diff --git a/frontend/src/graphql/mutations.js b/frontend/src/graphql/mutations.js
index 672af5f04..9b035cba6 100644
--- a/frontend/src/graphql/mutations.js
+++ b/frontend/src/graphql/mutations.js
@@ -31,7 +31,6 @@ export const updateUserInfos = gql`
     $password: String
     $passwordNew: String
     $locale: String
-    $coinanimation: Boolean
   ) {
     updateUserInfos(
       firstName: $firstName
@@ -39,7 +38,6 @@ export const updateUserInfos = gql`
       password: $password
       passwordNew: $passwordNew
       language: $locale
-      coinanimation: $coinanimation
     )
   }
 `
diff --git a/frontend/src/graphql/queries.js b/frontend/src/graphql/queries.js
index 2bd905e5e..601880a51 100644
--- a/frontend/src/graphql/queries.js
+++ b/frontend/src/graphql/queries.js
@@ -7,7 +7,6 @@ export const login = gql`
       firstName
       lastName
       language
-      coinanimation
       klickTipp {
         newsletterState
       }
@@ -25,7 +24,6 @@ export const verifyLogin = gql`
       firstName
       lastName
       language
-      coinanimation
       klickTipp {
         newsletterState
       }
diff --git a/frontend/src/locales/de.json b/frontend/src/locales/de.json
index 3b4912a2f..a3e412a0e 100644
--- a/frontend/src/locales/de.json
+++ b/frontend/src/locales/de.json
@@ -186,11 +186,6 @@
   "send_gdd": "GDD versenden",
   "send_per_link": "GDD versenden per Link",
   "settings": {
-    "coinanimation": {
-      "coinanimation": "Münzanimation",
-      "False": "Münzanimation ausgeschaltet",
-      "True": "Münzanimation eingeschaltet"
-    },
     "language": {
       "changeLanguage": "Sprache ändern",
       "de": "Deutsch",
diff --git a/frontend/src/locales/en.json b/frontend/src/locales/en.json
index 63f2510a3..4a7245147 100644
--- a/frontend/src/locales/en.json
+++ b/frontend/src/locales/en.json
@@ -186,11 +186,6 @@
   "send_gdd": "GDD send",
   "send_per_link": "GDD send via link",
   "settings": {
-    "coinanimation": {
-      "coinanimation": "Coin animation",
-      "False": "Coin animation disabled",
-      "True": "Coin animation enabled"
-    },
     "language": {
       "changeLanguage": "Change language",
       "de": "Deutsch",
diff --git a/frontend/src/pages/Profile.spec.js b/frontend/src/pages/Profile.spec.js
index fb4ee2d42..6a6efcfe3 100644
--- a/frontend/src/pages/Profile.spec.js
+++ b/frontend/src/pages/Profile.spec.js
@@ -38,9 +38,5 @@ describe('Profile', () => {
     it('has a user change newsletter form', () => {
       expect(wrapper.findComponent({ name: 'UserNewsletter' }).exists()).toBeTruthy()
     })
-
-    it('has a user change coin animation form', () => {
-      expect(wrapper.findComponent({ name: 'UserCoinAnimation' }).exists()).toBeTruthy()
-    })
   })
 })
diff --git a/frontend/src/pages/Profile.vue b/frontend/src/pages/Profile.vue
index 01308b29a..703fea307 100644
--- a/frontend/src/pages/Profile.vue
+++ b/frontend/src/pages/Profile.vue
@@ -8,8 +8,6 @@
     <user-language />
     <hr />
     <user-newsletter />
-    <hr />
-    <user-coin-animation />
   </div>
 </template>
 <script>
@@ -18,7 +16,6 @@ import UserData from '@/components/UserSettings/UserData.vue'
 import UserPassword from '@/components/UserSettings/UserPassword.vue'
 import UserLanguage from '@/components/UserSettings/UserLanguage.vue'
 import UserNewsletter from '@/components/UserSettings/UserNewsletter.vue'
-import UserCoinAnimation from '@/components/UserSettings/UserCoinAnimation.vue'
 
 export default {
   name: 'Profile',
@@ -28,7 +25,6 @@ export default {
     UserPassword,
     UserLanguage,
     UserNewsletter,
-    UserCoinAnimation,
   },
   props: {
     balance: { type: Number, default: 0 },
diff --git a/frontend/src/store/store.js b/frontend/src/store/store.js
index 526a2c84f..129337e7f 100644
--- a/frontend/src/store/store.js
+++ b/frontend/src/store/store.js
@@ -38,9 +38,6 @@ export const mutations = {
   isAdmin: (state, isAdmin) => {
     state.isAdmin = !!isAdmin
   },
-  coinanimation: (state, coinanimation) => {
-    state.coinanimation = coinanimation
-  },
   hasElopage: (state, hasElopage) => {
     state.hasElopage = hasElopage
   },
@@ -53,7 +50,6 @@ export const actions = {
     // commit('username', data.username)
     commit('firstName', data.firstName)
     commit('lastName', data.lastName)
-    commit('coinanimation', data.coinanimation)
     commit('newsletterState', data.klickTipp.newsletterState)
     commit('hasElopage', data.hasElopage)
     commit('publisherId', data.publisherId)
@@ -65,7 +61,6 @@ export const actions = {
     // commit('username', '')
     commit('firstName', '')
     commit('lastName', '')
-    commit('coinanimation', true)
     commit('newsletterState', null)
     commit('hasElopage', false)
     commit('publisherId', null)
@@ -91,7 +86,6 @@ try {
       // username: '',
       token: null,
       isAdmin: false,
-      coinanimation: true,
       newsletterState: null,
       hasElopage: false,
       publisherId: null,
diff --git a/frontend/src/store/store.test.js b/frontend/src/store/store.test.js
index 9eadedd6b..e8d680f94 100644
--- a/frontend/src/store/store.test.js
+++ b/frontend/src/store/store.test.js
@@ -20,7 +20,6 @@ const {
   token,
   firstName,
   lastName,
-  coinanimation,
   newsletterState,
   publisherId,
   isAdmin,
@@ -78,14 +77,6 @@ describe('Vuex store', () => {
       })
     })
 
-    describe('coinanimation', () => {
-      it('sets the state of coinanimation', () => {
-        const state = { coinanimation: true }
-        coinanimation(state, false)
-        expect(state.coinanimation).toEqual(false)
-      })
-    })
-
     describe('newsletterState', () => {
       it('sets the state of newsletterState', () => {
         const state = { newsletterState: null }
@@ -134,7 +125,6 @@ describe('Vuex store', () => {
         language: 'de',
         firstName: 'Peter',
         lastName: 'Lustig',
-        coinanimation: false,
         klickTipp: {
           newsletterState: true,
         },
@@ -145,7 +135,7 @@ describe('Vuex store', () => {
 
       it('calls nine commits', () => {
         login({ commit, state }, commitedData)
-        expect(commit).toHaveBeenCalledTimes(9)
+        expect(commit).toHaveBeenCalledTimes(8)
       })
 
       it('commits email', () => {
@@ -168,29 +158,24 @@ describe('Vuex store', () => {
         expect(commit).toHaveBeenNthCalledWith(4, 'lastName', 'Lustig')
       })
 
-      it('commits coinanimation', () => {
-        login({ commit, state }, commitedData)
-        expect(commit).toHaveBeenNthCalledWith(5, 'coinanimation', false)
-      })
-
       it('commits newsletterState', () => {
         login({ commit, state }, commitedData)
-        expect(commit).toHaveBeenNthCalledWith(6, 'newsletterState', true)
+        expect(commit).toHaveBeenNthCalledWith(5, 'newsletterState', true)
       })
 
       it('commits hasElopage', () => {
         login({ commit, state }, commitedData)
-        expect(commit).toHaveBeenNthCalledWith(7, 'hasElopage', false)
+        expect(commit).toHaveBeenNthCalledWith(6, 'hasElopage', false)
       })
 
       it('commits publisherId', () => {
         login({ commit, state }, commitedData)
-        expect(commit).toHaveBeenNthCalledWith(8, 'publisherId', 1234)
+        expect(commit).toHaveBeenNthCalledWith(7, 'publisherId', 1234)
       })
 
       it('commits isAdmin', () => {
         login({ commit, state }, commitedData)
-        expect(commit).toHaveBeenNthCalledWith(9, 'isAdmin', true)
+        expect(commit).toHaveBeenNthCalledWith(8, 'isAdmin', true)
       })
     })
 
@@ -200,7 +185,7 @@ describe('Vuex store', () => {
 
       it('calls nine commits', () => {
         logout({ commit, state })
-        expect(commit).toHaveBeenCalledTimes(9)
+        expect(commit).toHaveBeenCalledTimes(8)
       })
 
       it('commits token', () => {
@@ -223,29 +208,24 @@ describe('Vuex store', () => {
         expect(commit).toHaveBeenNthCalledWith(4, 'lastName', '')
       })
 
-      it('commits coinanimation', () => {
-        logout({ commit, state })
-        expect(commit).toHaveBeenNthCalledWith(5, 'coinanimation', true)
-      })
-
       it('commits newsletterState', () => {
         logout({ commit, state })
-        expect(commit).toHaveBeenNthCalledWith(6, 'newsletterState', null)
+        expect(commit).toHaveBeenNthCalledWith(5, 'newsletterState', null)
       })
 
       it('commits hasElopage', () => {
         logout({ commit, state })
-        expect(commit).toHaveBeenNthCalledWith(7, 'hasElopage', false)
+        expect(commit).toHaveBeenNthCalledWith(6, 'hasElopage', false)
       })
 
       it('commits publisherId', () => {
         logout({ commit, state })
-        expect(commit).toHaveBeenNthCalledWith(8, 'publisherId', null)
+        expect(commit).toHaveBeenNthCalledWith(7, 'publisherId', null)
       })
 
       it('commits isAdmin', () => {
         logout({ commit, state })
-        expect(commit).toHaveBeenNthCalledWith(9, 'isAdmin', false)
+        expect(commit).toHaveBeenNthCalledWith(8, 'isAdmin', false)
       })
 
       // how to get this working?