Add custom resolver for createSocialMedia

backend/src/middleware/permissionsMiddleware.js

@@ -57,7 +57,7 @@ const permissions = shield({
     UpdateBadge: isAdmin,
     DeleteBadge: isAdmin,
     AddUserBadges: isAdmin,
-    addSocialMedia: isAuthenticated,
+    CreateSocialMedia: isAuthenticated,
     // AddBadgeRewarded: isAdmin,
     // RemoveBadgeRewarded: isAdmin,
     reward: isAdmin,

backend/src/resolvers/socialMedia.js

@@ -0,0 +1,22 @@
+import { neo4jgraphql } from 'neo4j-graphql-js'
+
+export default {
+  Mutation: {
+    createSocialMedia: async (object, params, context, resolveInfo) => {
+      const result = await neo4jgraphql(object, params, context, resolveInfo, true)
+
+      const session = context.driver.session()
+      await session.run(
+        'MATCH (owner:User {id: $userId}), (socialMedia:SocialMedia {id: $socialMediaId}) ' +
+        'MERGE (socialMedia)<-[:OWNED]-(owner) ' +
+        'RETURN owner', {
+          userId: context.user.id,
+          socialMediaId: result.id
+        }
+      )
+      session.close()
+
+      return result
+    }
+  }
+}

backend/src/resolvers/user_management.js

@@ -100,27 +100,6 @@ export default {
 
         return encode(currentUser)
       }
-    },
-    addSocialMedia: async (_, { url }, { driver, user }) => {
-      const session = driver.session()
-
-      const { id } = user
-      const result = await session.run(
-        `MATCH (user:User {id: $userId})
-        SET user.socialMedia = user.socialMedia + $url
-        RETURN user {.socialMedia}
-        `,
-        {
-          userId: id,
-          url
-        }
-      )
-      session.close()
-      const [currentUser] = result.records.map(record => {
-        return record.get('user')
-      })
-
-      return currentUser.socialMedia
     }
   }
-}
+}

backend/src/resolvers/user_management.spec.js

@@ -310,12 +310,14 @@ describe('change password', () => {
   })
 })
 
-describe('addSocialMedia', () => {
+describe('CreateSocialMedia', () => {
   let client
   let headers
   const mutation = `
-    mutation($url: String!) {
+    mutation($input: SocialMediaInput) {
-      addSocialMedia(url: $url)
+      CreateSocialMedia(input: $input) {
+        url
+      }
     }
   `
 
@@ -326,12 +328,12 @@ describe('addSocialMedia', () => {
     })
 
     it('rejects empty string', async () => {
-      const variables = { url: '' }
+      const variables = { input: { url: '' } }
       await expect(client.request(mutation, variables)).rejects.toThrow('Input is not a URL')
     })
 
     it('validates URLs', async () => {
-      const variables = { url: 'not-a-url' }
+      const variables = { input: { url: 'not-a-url' } }
       await expect(client.request(mutation, variables)).rejects.toThrow('Input is not a URL')
     })
   })

backend/src/schema.graphql

@@ -26,7 +26,7 @@ type Mutation {
   disable(id: ID!): ID
   enable(id: ID!): ID
   reward(fromBadgeId: ID!, toUserId: ID!): ID
-  addSocialMedia(url: urlInput): [String]!
+  createSocialMedia(input: SocialMediaInput): SocialMedia
   unreward(fromBadgeId: ID!, toUserId: ID!): ID
   "Shout the given Type and ID"
   shout(id: ID!, type: ShoutTypeEnum): Boolean! @cypher(statement: """
@@ -121,7 +121,7 @@ type User {
   location: Location @cypher(statement: "MATCH (this)-[:IS_IN]->(l:Location) RETURN l")
   locationName: String
   about: String
-  socialMedia: [String]
+  socialMedia: [SocialMedia]! @relation(name: "OWNED", direction: "OUT")
 
   createdAt: String
   updatedAt: String
@@ -313,9 +313,14 @@ type SharedInboxEndpoint {
 
 type SocialMedia {
   id: ID!
-  uri: String
+  url: String
+  ownedBy: [User]! @relation(name: "OWNED", direction: "IN")
 }
 
-input urlInput {
+input SocialMediaInput {
   url: String! @constraint(format: "uri")
-}
+}
+
+directive @constraint(
+  format: String
+) on INPUT_FIELD_DEFINITION

backend/src/seed/factories/users.js

@@ -1,7 +1,7 @@
 import faker from 'faker'
 import uuid from 'uuid/v4'
 
-export default function create (params) {
+export default function create(params) {
   const {
     id = uuid(),
     name = faker.name.findName(),
@@ -27,8 +27,7 @@ export default function create (params) {
         about: "${about}",
         role: ${role},
         disabled: ${disabled},
-        deleted: ${deleted},
+        deleted: ${deleted}
-        socialMedia: []
       ) {
         id
         name
@@ -38,7 +37,6 @@ export default function create (params) {
         role
         deleted
         disabled
-        socialMedia
       }
     }
   `