Missing files..

.github/workflows/test.yml

@@ -386,7 +386,7 @@ jobs:
           report_name: Coverage Backend
           type: lcov
           result_path: ./coverage/lcov.info
-          min_coverage: 4
+          min_coverage: 3
           token: ${{ github.token }}
 
   ##############################################################################

frontend/.env.dist

@@ -1,3 +1,4 @@
 ALLOW_REGISTER=true
 GRAPHQL_URI=http://localhost:4000/graphql
+DEFAULT_PUBLISHER_ID=2896
 //BUILD_COMMIT=0000000

frontend/src/components/SidebarPlugin/SideBar.spec.js

@@ -109,7 +109,7 @@ describe('SideBar', () => {
           })
         })
 
-        describe('member's area with publisher ID', () => {
+        describe("member's area with publisher ID", () => {
           beforeEach(() => {
             mocks.$store.state.hasElopage = true
           })

frontend/src/components/SidebarPlugin/SideBar.vue

@@ -68,6 +68,7 @@
 <script>
 import NavbarToggleButton from '@/components/NavbarToggleButton'
 import VueQrcode from 'vue-qrcode'
+import { CONFIG } from '../../config'
 
 export default {
   name: 'sidebar',
@@ -112,9 +113,14 @@ export default {
       this.$emit('logout')
     },
     getElopageLink() {
-      return this.$store.state.hasElopage
+      const pId = this.$store.state.publisherId
+        ? this.$store.state.publisherId
+        : CONFIG.DEFAULT_PUBLISHER_ID
+      return encodeURI(
+        this.$store.state.hasElopage
           ? `https://elopage.com/s/gradido/sign_in?locale=${this.$i18n.locale}`
-        : encodeURL(`https://elopage.com/s/gradido/basic-de/payment?locale=${this.$i18n.locale}&prid=111&pid=${this.$store.state.publisherId}&firstName=${this.$store.state.firstName}&lastName=${this.$store.state.lastName}&email=${this.$store.state.email})`
+          : `https://elopage.com/s/gradido/basic-de/payment?locale=${this.$i18n.locale}&prid=111&pid=${pId}&firstName=${this.$store.state.firstName}&lastName=${this.$store.state.lastName}&email=${this.$store.state.email}`,
+      )
     },
   },
 }

frontend/src/config/index.js

@@ -15,6 +15,7 @@ const environment = {
   NODE_ENV: process.env.NODE_ENV,
   DEBUG: process.env.NODE_ENV !== 'production' || false,
   PRODUCTION: process.env.NODE_ENV === 'production' || false,
+  DEFAULT_PUBLISHER_ID: process.env.DEFAULT_PUBLISHER_ID || 2896,
 }
 
 const server = {

frontend/src/graphql/mutations.js

@@ -51,6 +51,7 @@ export const registerUser = gql`
     $email: String!
     $password: String!
     $language: String!
+    $publisherId: Int!
   ) {
     createUser(
       email: $email
@@ -58,6 +59,7 @@ export const registerUser = gql`
       lastName: $lastName
       password: $password
       language: $language
+      publisherId: $publisherId
     )
   }
 `

frontend/src/graphql/queries.js

@@ -1,8 +1,8 @@
 import gql from 'graphql-tag'
 
 export const login = gql`
-  query($email: String!, $password: String!) {
+  query($email: String!, $password: String!, $publisherId: Int) {
-    login(email: $email, password: $password) {
+    login(email: $email, password: $password, publisherId: $publisherId) {
       email
       username
       firstName
@@ -14,6 +14,7 @@ export const login = gql`
         newsletterState
       }
       hasElopage
+      publisherId
     }
   }
 `

frontend/src/store/store.js

@@ -30,7 +30,9 @@ export const mutations = {
     state.newsletterState = newsletterState
   },
   publisherId: (state, publisherId) => {
-    state.publisherId = publisherId
+    let pubId = parseInt(publisherId)
+    if (isNaN(pubId)) pubId = null
+    state.publisherId = pubId
   },
   community: (state, community) => {
     state.community = community
@@ -54,6 +56,7 @@ export const actions = {
     commit('coinanimation', data.coinanimation)
     commit('newsletterState', data.klickTipp.newsletterState)
     commit('hasElopage', data.hasElopage)
+    commit('publisherId', data.publisherId)
   },
   logout: ({ commit, state }) => {
     commit('token', null)
@@ -65,6 +68,7 @@ export const actions = {
     commit('coinanimation', true)
     commit('newsletterState', null)
     commit('hasElopage', false)
+    commit('publisherId', null)
     localStorage.clear()
   },
 }

frontend/src/store/store.test.js

@@ -96,6 +96,12 @@ describe('Vuex store', () => {
         publisherId(state, 42)
         expect(state.publisherId).toEqual(42)
       })
+
+      it('sets publisherId to null with NaN', () => {
+        const state = {}
+        publisherId(state, 'abc')
+        expect(state.publisherId).toEqual(null)
+      })
     })
 
     describe('community', () => {
@@ -141,11 +147,12 @@ describe('Vuex store', () => {
           newsletterState: true,
         },
         hasElopage: false,
+        publisherId: 1234,
       }
 
-      it('calls nine commits', () => {
+      it('calls ten commits', () => {
         login({ commit, state }, commitedData)
-        expect(commit).toHaveBeenCalledTimes(9)
+        expect(commit).toHaveBeenCalledTimes(10)
       })
 
       it('commits email', () => {
@@ -192,15 +199,20 @@ describe('Vuex store', () => {
         login({ commit, state }, commitedData)
         expect(commit).toHaveBeenNthCalledWith(9, 'hasElopage', false)
       })
+
+      it('commits publisherId', () => {
+        login({ commit, state }, commitedData)
+        expect(commit).toHaveBeenNthCalledWith(10, 'publisherId', 1234)
+      })
     })
 
     describe('logout', () => {
       const commit = jest.fn()
       const state = {}
 
-      it('calls nine commits', () => {
+      it('calls ten commits', () => {
         logout({ commit, state })
-        expect(commit).toHaveBeenCalledTimes(9)
+        expect(commit).toHaveBeenCalledTimes(10)
       })
 
       it('commits token', () => {
@@ -248,6 +260,11 @@ describe('Vuex store', () => {
         expect(commit).toHaveBeenNthCalledWith(9, 'hasElopage', false)
       })
 
+      it('commits publisherId', () => {
+        logout({ commit, state })
+        expect(commit).toHaveBeenNthCalledWith(10, 'publisherId', null)
+      })
+
       // how to get this working?
       it.skip('calls localStorage.clear()', () => {
         const clearStorageMock = jest.fn()

frontend/src/views/Pages/Login.spec.js

@@ -44,6 +44,7 @@ describe('Login', () => {
           registerUrl: 'http://localhost/vue/register',
           description: 'Die lokale Entwicklungsumgebung von Gradido.',
         },
+        publisherId: 12345,
       },
     },
     $loading: {
@@ -199,6 +200,7 @@ describe('Login', () => {
               variables: {
                 email: 'user@example.org',
                 password: '1234',
+                publisherId: 12345,
               },
             }),
           )

frontend/src/views/Pages/Login.vue

@@ -96,6 +96,7 @@ export default {
           variables: {
             email: this.form.email,
             password: this.form.password,
+            publisherId: parseInt(this.$store.state.publisherId),
           },
           fetchPolicy: 'network-only',
         })

frontend/src/views/Pages/Register.spec.js

@@ -32,6 +32,7 @@ describe('Register', () => {
           registerUrl: 'http://localhost/vue/register',
           description: 'Die lokale Entwicklungsumgebung von Gradido.',
         },
+        publisherId: 12345,
       },
     },
   }
@@ -216,6 +217,7 @@ describe('Register', () => {
                 lastName: 'Mustermann',
                 password: 'Aa123456_',
                 language: 'en',
+                publisherId: 12345,
               },
             }),
           )

frontend/src/views/Pages/Register.vue

@@ -199,6 +199,7 @@ export default {
             lastName: this.form.lastname,
             password: this.form.password.password,
             language: this.language,
+            publisherId: parseInt(this.$store.state.publisherId),
           },
         })
         .then(() => {