Refactor after Roberts suggestions, number I

Start writing tests

backend/src/middleware/handleHtmlContent/handleContentData.js

@@ -10,12 +10,15 @@ const notifyMentions = async (label, id, idsOfMentionedUsers, context) => {
     MATCH (source)
     WHERE source.id = $id AND $label IN LABELS(source)
     MATCH (source)<-[:WROTE]-(author: User)
-    MATCH (u: User)
+    MATCH (user: User)
-    WHERE u.id in $idsOfMentionedUsers
+    WHERE user.id in $idsOfMentionedUsers
-    AND NOT (u)<-[:BLOCKED]-(author)
+    AND NOT (user)<-[:BLOCKED]-(author)
-    CREATE (n: Notification {id: apoc.create.uuid(), read: false, createdAt: $createdAt })
+    CREATE (notification: Notification {id: apoc.create.uuid(), read: false, createdAt: $createdAt })
-    MERGE (source)-[:NOTIFIED]->(n)-[:NOTIFIED]->(u)
+    MERGE (source)-[:NOTIFIED]->(notification)-[:NOTIFIED]->(user)
     `
+  // "author" of comment, blocked Peter: Jenny
+  // "user" mentioned on post by Jenny: Peter
+  // owner of post: Bob
   await session.run(cypher, {
     idsOfMentionedUsers,
     label,

backend/src/middleware/handleHtmlContent/handleContentData.spec.js

@@ -1,7 +1,14 @@
-import { gql } from '../../jest/helpers'
+import {
+  gql
+} from '../../jest/helpers'
 import Factory from '../../seed/factories'
-import { createTestClient } from 'apollo-server-testing'
+import {
-import { neode, getDriver } from '../../bootstrap/neo4j'
+  createTestClient
+} from 'apollo-server-testing'
+import {
+  neode,
+  getDriver
+} from '../../bootstrap/neo4j'
 import createServer from '../../server'
 
 const factory = Factory()
@@ -63,12 +70,12 @@ describe('notifications', () => {
     })
 
     describe('given another user', () => {
-      let author
+      let postAuthor
       beforeEach(async () => {
-        author = await instance.create('User', {
+        postAuthor = await instance.create('User', {
-          email: 'author@example.org',
+          email: 'post-author@example.org',
           password: '1234',
-          id: 'author',
+          id: 'postAuthor',
         })
       })
 
@@ -87,10 +94,14 @@ describe('notifications', () => {
               }
             }
           `
-          authenticatedUser = await author.toJson()
+          authenticatedUser = await postAuthor.toJson()
           await mutate({
             mutation: createPostMutation,
-            variables: { id: 'p47', title, content },
+            variables: {
+              id: 'p47',
+              title,
+              content
+            },
           })
           authenticatedUser = await user.toJson()
         }
@@ -101,12 +112,26 @@ describe('notifications', () => {
             'Hey <a class="mention" data-mention-id="you" href="/profile/you/al-capone" target="_blank">@al-capone</a> how do you do?'
           const expected = expect.objectContaining({
             data: {
-              currentUser: { notifications: [{ read: false, post: { content: expectedContent } }] },
+              currentUser: {
+                notifications: [{
+                  read: false,
+                  post: {
+                    content: expectedContent
+                  }
+                }]
+              },
             },
           })
-          const { query } = createTestClient(server)
+          const {
+            query
+          } = createTestClient(server)
           await expect(
-            query({ query: notificationQuery, variables: { read: false } }),
+            query({
+              query: notificationQuery,
+              variables: {
+                read: false
+              }
+            }),
           ).resolves.toEqual(expected)
         })
 
@@ -134,7 +159,7 @@ describe('notifications', () => {
                 }
               }
             `
-            authenticatedUser = await author.toJson()
+            authenticatedUser = await postAuthor.toJson()
             await mutate({
               mutation: updatePostMutation,
               variables: {
@@ -154,32 +179,115 @@ describe('notifications', () => {
             const expected = expect.objectContaining({
               data: {
                 currentUser: {
-                  notifications: [
+                  notifications: [{
-                    { read: false, post: { content: expectedContent } },
+                      read: false,
-                    { read: false, post: { content: expectedContent } },
+                      post: {
+                        content: expectedContent
+                      }
+                    },
+                    {
+                      read: false,
+                      post: {
+                        content: expectedContent
+                      }
+                    },
                   ],
                 },
               },
             })
             await expect(
-              query({ query: notificationQuery, variables: { read: false } }),
+              query({
+                query: notificationQuery,
+                variables: {
+                  read: false
+                }
+              }),
             ).resolves.toEqual(expected)
           })
         })
 
         describe('but the author of the post blocked me', () => {
           beforeEach(async () => {
-            await author.relateTo(user, 'blocked')
+            await postAuthor.relateTo(user, 'blocked')
           })
 
           it('sends no notification', async () => {
             await createPostAction()
             const expected = expect.objectContaining({
-              data: { currentUser: { notifications: [] } },
+              data: {
+                currentUser: {
+                  notifications: []
+                }
+              },
             })
-            const { query } = createTestClient(server)
+            const {
+              query
+            } = createTestClient(server)
             await expect(
-              query({ query: notificationQuery, variables: { read: false } }),
+              query({
+                query: notificationQuery,
+                variables: {
+                  read: false
+                }
+              }),
+            ).resolves.toEqual(expected)
+          })
+        })
+
+        describe('but the author of the post blocked a user I mention in a comment', () => {
+          const createCommentOnPostAction = async () => {
+            await createPostAction()
+            const createCommentMutation = gql `
+              mutation($id: ID, $title: String!, $content: String!) {
+                CreateComment(id: $id, postId: $postId, content: $commentContent) {
+                  id
+                  content
+                }
+              }
+            `
+            // authenticatedUser = await postAuthor.toJson()
+            // authenticatedUser = await user.toJson()
+            await mutate({
+              mutation: createCommentMutation,
+              variables: {
+                id: 'c47',
+                postId: 'p47',
+                commentContent: 'One mention of me with <a data-mention-id="you" class="mention" href="/profile/you" target="_blank">.'
+              },
+            })
+          }
+          let mentioner
+
+          beforeEach(async () => {
+            await postAuthor.relateTo(user, 'blocked')
+            mentioner = await instance.create('User', {
+              id: 'mentioner',
+              name: 'Mr Mentioner',
+              slug: 'mr-mentioner',
+              email: 'mentioner@example.org',
+              password: '1234',
+            })
+          })
+
+          it('sends no notification', async () => {
+            await createPostAction()
+            const expected = expect.objectContaining({
+              data: {
+                currentUser: {
+                  notifications: []
+                }
+              },
+            })
+            const {
+              query
+            } = createTestClient(server)
+            await expect(
+              query({
+                query: notificationQuery,
+                variables: {
+                  read: false
+                }
+              }),
             ).resolves.toEqual(expected)
           })
         })
@@ -234,20 +342,26 @@ describe('Hashtags', () => {
       })
 
       it('both Hashtags are created with the "id" set to their "name"', async () => {
-        const expected = [
+        const expected = [{
-          { id: 'Democracy', name: 'Democracy' },
+            id: 'Democracy',
-          { id: 'Liberty', name: 'Liberty' },
+            name: 'Democracy'
+          },
+          {
+            id: 'Liberty',
+            name: 'Liberty'
+          },
         ]
         await expect(
-          query({ query: postWithHastagsQuery, variables: postWithHastagsVariables }),
+          query({
+            query: postWithHastagsQuery,
+            variables: postWithHastagsVariables
+          }),
         ).resolves.toEqual(
           expect.objectContaining({
             data: {
-              Post: [
+              Post: [{
-                {
                 tags: expect.arrayContaining(expected),
-                },
+              }, ],
-              ],
             },
           }),
         )
@@ -277,16 +391,26 @@ describe('Hashtags', () => {
             },
           })
 
-          const expected = [
+          const expected = [{
-            { id: 'Elections', name: 'Elections' },
+              id: 'Elections',
-            { id: 'Liberty', name: 'Liberty' },
+              name: 'Elections'
+            },
+            {
+              id: 'Liberty',
+              name: 'Liberty'
+            },
           ]
           await expect(
-            query({ query: postWithHastagsQuery, variables: postWithHastagsVariables }),
+            query({
+              query: postWithHastagsQuery,
+              variables: postWithHastagsVariables
+            }),
           ).resolves.toEqual(
             expect.objectContaining({
               data: {
-                Post: [{ tags: expect.arrayContaining(expected) }],
+                Post: [{
+                  tags: expect.arrayContaining(expected)
+                }],
               },
             }),
           )

webapp/components/notifications/Notification/Notification.vue

@@ -3,19 +3,21 @@
     <no-ssr>
       <ds-space margin-bottom="x-small">
         <hc-user
-          :user="post.author || comment.author"
+          v-if="resourceType == 'Post'"
-          :date-time="post.createdAt || comment.createdAt"
+          :user="post.author"
+          :date-time="post.createdAt"
           :trunc="35"
         />
+        <hc-user v-else :user="comment.author" :date-time="comment.createdAt" :trunc="35" />
       </ds-space>
       <ds-text color="soft">
-        {{ $t('notifications.menu.mentioned', { resource: post.id ? 'post' : 'comment' }) }}
+        {{ $t('notifications.menu.mentioned', { resource: resourceType }) }}
       </ds-text>
     </no-ssr>
     <ds-space margin-bottom="x-small" />
     <nuxt-link
       class="notification-mention-post"
-      :to="{ name: 'post-id-slug', params: postParams, ...hashParam }"
+      :to="{ name: 'post-id-slug', params, ...hashParam }"
       @click.native="$emit('read')"
     >
       <ds-space margin-bottom="x-small">
@@ -26,9 +28,14 @@
           class="notifications-card"
         >
           <ds-space margin-bottom="x-small" />
-          <!-- eslint-disable vue/no-v-html -->
+          <div v-if="resourceType == 'Post'">{{ post.contentExcerpt | removeHtml }}</div>
-          <div v-html="excerpt" />
+          <div v-else>
-          <!-- eslint-enable vue/no-v-html -->
+            <b>
+              Comment:
+              <nbsp />
+            </b>
+            {{ comment.contentExcerpt | removeHtml }}
+          </div>
         </ds-card>
       </ds-space>
     </nuxt-link>
@@ -50,11 +57,8 @@ export default {
     },
   },
   computed: {
-    excerpt() {
+    resourceType() {
-      const excerpt = this.post.id ? this.post.contentExcerpt : this.comment.contentExcerpt
+      return this.post.id ? 'Post' : 'Comment'
-      return (
-        (!this.post.id ? '<b>Comment: </b>' : '') + excerpt.replace(/<(?:.|\n)*?>/gm, '').trim()
-      )
     },
     post() {
       return this.notification.post || {}
@@ -62,7 +66,7 @@ export default {
     comment() {
       return this.notification.comment || {}
     },
-    postParams() {
+    params() {
       return {
         id: this.post.id || this.comment.post.id,
         slug: this.post.slug || this.comment.post.slug,

webapp/components/notifications/NotificationMenu/NotificationMenu.spec.js

@@ -1,5 +1,5 @@
 import { config, shallowMount, createLocalVue } from '@vue/test-utils'
-import NotificationMenu from '.'
+import NotificationMenu from './NotificationMenu'
 
 import Styleguide from '@human-connection/styleguide'
 import Filters from '~/plugins/vue-filters'

webapp/components/notifications/NotificationMenu/NotificationMenu.vue

@@ -17,80 +17,9 @@
 </template>
 
 <script>
-import NotificationList from '../NotificationList/NotificationList'
 import Dropdown from '~/components/Dropdown'
-import gql from 'graphql-tag'
+import { currentUserNotificationsQuery, updateNotificationMutation } from '~/graphql/User'
-
+import NotificationList from '../NotificationList/NotificationList'
-const MARK_AS_READ = gql`
-  mutation($id: ID!, $read: Boolean!) {
-    UpdateNotification(id: $id, read: $read) {
-      id
-      read
-    }
-  }
-`
-
-const NOTIFICATIONS = gql`
-  {
-    currentUser {
-      id
-      notifications(read: false, orderBy: createdAt_desc) {
-        id
-        read
-        createdAt
-        post {
-          id
-          createdAt
-          disabled
-          deleted
-          title
-          contentExcerpt
-          slug
-          author {
-            id
-            slug
-            name
-            disabled
-            deleted
-            avatar
-          }
-        }
-        comment {
-          id
-          createdAt
-          disabled
-          deleted
-          contentExcerpt
-          author {
-            id
-            slug
-            name
-            disabled
-            deleted
-            avatar
-          }
-          post {
-            id
-            createdAt
-            disabled
-            deleted
-            title
-            contentExcerpt
-            slug
-            author {
-              id
-              slug
-              name
-              disabled
-              deleted
-              avatar
-            }
-          }
-        }
-      }
-    }
-  }
-`
 
 export default {
   name: 'NotificationMenu',
@@ -111,7 +40,7 @@ export default {
       const variables = { id: notificationId, read: true }
       try {
         await this.$apollo.mutate({
-          mutation: MARK_AS_READ,
+          mutation: updateNotificationMutation(),
           variables,
         })
       } catch (err) {
@@ -121,7 +50,7 @@ export default {
   },
   apollo: {
     notifications: {
-      query: NOTIFICATIONS,
+      query: currentUserNotificationsQuery(),
       update: data => {
         const {
           currentUser: { notifications },

webapp/graphql/User.js

@@ -75,3 +75,78 @@ export default i18n => {
     }
   `
 }
+
+export const currentUserNotificationsQuery = () => {
+  return gql`
+    {
+      currentUser {
+        id
+        notifications(read: false, orderBy: createdAt_desc) {
+          id
+          read
+          createdAt
+          post {
+            id
+            createdAt
+            disabled
+            deleted
+            title
+            contentExcerpt
+            slug
+            author {
+              id
+              slug
+              name
+              disabled
+              deleted
+              avatar
+            }
+          }
+          comment {
+            id
+            createdAt
+            disabled
+            deleted
+            contentExcerpt
+            author {
+              id
+              slug
+              name
+              disabled
+              deleted
+              avatar
+            }
+            post {
+              id
+              createdAt
+              disabled
+              deleted
+              title
+              contentExcerpt
+              slug
+              author {
+                id
+                slug
+                name
+                disabled
+                deleted
+                avatar
+              }
+            }
+          }
+        }
+      }
+    }
+  `
+}
+
+export const updateNotificationMutation = () => {
+  return gql`
+    mutation($id: ID!, $read: Boolean!) {
+      UpdateNotification(id: $id, read: $read) {
+        id
+        read
+      }
+    }
+  `
+}

webapp/layouts/default.vue

@@ -88,9 +88,11 @@
                           {{ $t('login.hello') }}
                           <b>{{ userName }}</b>
                           <template v-if="user.role !== 'user'">
-                            <ds-text color="softer" size="small" style="margin-bottom: 0">
+                            <ds-text
-                              {{ user.role | camelCase }}
+                              color="softer"
-                            </ds-text>
+                              size="small"
+                              style="margin-bottom: 0"
+                            >{{ user.role | camelCase }}</ds-text>
                           </template>
                           <hr />
                           <ds-menu :routes="routes" :matcher="matcher">
@@ -129,14 +131,10 @@
     <div id="footer" class="ds-footer">
       <a href="https://human-connection.org" target="_blank" v-html="$t('site.made')"></a>
       &nbsp;-&nbsp;
-      <nuxt-link to="/imprint">{{ $t('site.imprint') }}</nuxt-link>
+      <nuxt-link to="/imprint">{{ $t('site.imprint') }}</nuxt-link>&nbsp;‑&nbsp;
-      &nbsp;‑&nbsp;
+      <nuxt-link to="/terms-and-conditions">{{ $t('site.termsAndConditions') }}</nuxt-link>&nbsp;‑&nbsp;
-      <nuxt-link to="/terms-and-conditions">{{ $t('site.termsAndConditions') }}</nuxt-link>
+      <nuxt-link to="/code-of-conduct">{{ $t('site.code-of-conduct') }}</nuxt-link>&nbsp;‑&nbsp;
-      &nbsp;‑&nbsp;
+      <nuxt-link to="/data-privacy">{{ $t('site.data-privacy') }}</nuxt-link>&nbsp;‑&nbsp;
-      <nuxt-link to="/code-of-conduct">{{ $t('site.code-of-conduct') }}</nuxt-link>
-      &nbsp;‑&nbsp;
-      <nuxt-link to="/data-privacy">{{ $t('site.data-privacy') }}</nuxt-link>
-      &nbsp;‑&nbsp;
       <nuxt-link to="/changelog">{{ $t('site.changelog') }}</nuxt-link>
     </div>
     <div id="overlay" />
@@ -151,7 +149,7 @@ import { mapGetters, mapActions, mapMutations } from 'vuex'
 import LocaleSwitch from '~/components/LocaleSwitch/LocaleSwitch'
 import SearchInput from '~/components/SearchInput.vue'
 import Modal from '~/components/Modal'
-import NotificationMenu from '~/components/notifications/NotificationMenu'
+import NotificationMenu from '~/components/notifications/NotificationMenu/NotificationMenu'
 import Dropdown from '~/components/Dropdown'
 import HcAvatar from '~/components/Avatar/Avatar.vue'
 import seo from '~/mixins/seo'

webapp/pages/post/_id/_slug/index.vue

@@ -110,12 +110,10 @@ export default {
     }
   },
   data() {
-    // Wolle: const { commentId = null } = this.$route.query
     return {
       post: null,
       ready: false,
       title: 'loading',
-      // Wolle: commentId,
     }
   },
   watch: {

webapp/pages/profile/_id/_slug.vue

@@ -248,7 +248,7 @@ import ContentMenu from '~/components/ContentMenu'
 import HcUpload from '~/components/Upload'
 import HcAvatar from '~/components/Avatar/Avatar.vue'
 import PostQuery from '~/graphql/UserProfile/Post.js'
-import UserQuery from '~/graphql/UserProfile/User.js'
+import UserQuery from '~/graphql/User.js'
 import { Block, Unblock } from '~/graphql/settings/BlockedUsers.js'
 
 const tabToFilterMapping = ({ tab, id }) => {

webapp/plugins/vue-filters.js

@@ -83,6 +83,15 @@ export default ({ app = {} }) => {
 
       return excerpt
     },
+    removeHtml: content => {
+      if (!content) return ''
+      // replace linebreaks with spaces first
+      let contentExcerpt = content.replace(/<br>/gim, ' ').trim()
+      // remove the rest of the HTML
+      contentExcerpt = contentExcerpt.replace(/<(?:.|\n)*?>/gm, '').trim()
+
+      return contentExcerpt
+    },
     proxyApiUrl: url => {
       if (!url) return url
       return url.startsWith('/') ? url.replace('/', '/api/') : url

webapp/store/auth.js

@@ -69,7 +69,8 @@ export const actions = {
     const {
       data: { currentUser },
     } = await client.query({
-      query: gql(`{
+      query: gql`
+        {
           currentUser {
             id
             name
@@ -103,7 +104,8 @@ export const actions = {
               }
             }
           }
-      }`),
+        }
+      `,
     })
     if (!currentUser) return dispatch('logout')
     commit('SET_USER', currentUser)
@@ -122,7 +124,10 @@ export const actions = {
               login(email: $email, password: $password)
             }
           `),
-        variables: { email, password },
+        variables: {
+          email,
+          password,
+        },
       })
       await this.app.$apolloHelpers.onLogin(login)
       commit('SET_TOKEN', login)

webapp/store/notifications.js

@@ -19,7 +19,10 @@ export const mutations = {
     const toBeUpdated = notifications.find(n => {
       return n.id === notification.id
     })
-    state.notifications = { ...toBeUpdated, ...notification }
+    state.notifications = {
+      ...toBeUpdated,
+      ...notification,
+    }
   },
 }
 export const getters = {
@@ -38,7 +41,8 @@ export const actions = {
       const {
         data: { currentUser },
       } = await client.query({
-        query: gql(`{
+        query: gql`
+          {
             currentUser {
               id
               notifications(orderBy: createdAt_desc) {
@@ -59,7 +63,8 @@ export const actions = {
                 }
               }
             }
-        }`),
+          }
+        `,
       })
       notifications = currentUser.notifications
       commit('SET_NOTIFICATIONS', notifications)
@@ -71,18 +76,24 @@ export const actions = {
 
   async markAsRead({ commit, rootGetters }, notificationId) {
     const client = this.app.apolloProvider.defaultClient
-    const mutation = gql(`
+    const mutation = gql`
       mutation($id: ID!, $read: Boolean!) {
         UpdateNotification(id: $id, read: $read) {
           id
           read
         }
       }
-    `)
+    `
-    const variables = { id: notificationId, read: true }
+    const variables = {
+      id: notificationId,
+      read: true,
+    }
     const {
       data: { UpdateNotification },
-    } = await client.mutate({ mutation, variables })
+    } = await client.mutate({
+      mutation,
+      variables,
+    })
     commit('UPDATE_NOTIFICATIONS', UpdateNotification)
   },
 }