updatePost with event type

backend/src/schema/resolvers/helpers/events.js

@@ -1,6 +1,7 @@
 import { UserInputError } from 'apollo-server'
 
 export const validateEventParams = (params) => {
+  if (params.postType && params.postType === 'Event') {
     const { eventInput } = params
     validateEventDate(eventInput.eventStart)
     params.eventStart = eventInput.eventStart
@@ -10,6 +11,18 @@ export const validateEventParams = (params) => {
     params.eventVenue = eventInput.eventVenue
     params.eventLocation = eventInput.eventLocation
   }
+  delete params.eventInput
+  let locationName
+  if (params.eventLocation) {
+    params.eventLocationName = params.eventLocation
+    locationName = params.eventLocation
+  } else {
+    params.eventLocationName = null
+    locationName = null
+  }
+  delete params.eventLocation
+  return locationName
+}
 
 const validateEventDate = (dateString) => {
   const date = new Date(dateString)

backend/src/schema/resolvers/posts.js

@@ -84,20 +84,7 @@ export default {
       const { categoryIds, groupId } = params
       const { image: imageInput } = params
 
-      if (params.postType && params.postType === 'Event') {
+      const locationName = validateEventParams(params)
-        validateEventParams(params)
-      }
-      delete params.eventInput
-
-      let locationName
-      if (params.eventLocation) {
-        params.eventLocationName = params.eventLocation
-        locationName = params.eventLocation
-      } else {
-        params.eventLocationName = null
-        locationName = null
-      }
-      delete params.eventLocation
 
       delete params.categoryIds
       delete params.image
@@ -176,6 +163,9 @@ export default {
     UpdatePost: async (_parent, params, context, _resolveInfo) => {
       const { categoryIds } = params
       const { image: imageInput } = params
+
+      const locationName = validateEventParams(params)
+
       delete params.categoryIds
       delete params.image
       const session = context.driver.session()
@@ -227,6 +217,9 @@ export default {
           return post
         })
         const post = await writeTxResultPromise
+        if (locationName) {
+          await createOrUpdateLocations('Post', post.id, locationName, session)
+        }
         return post
       } finally {
         session.close()

backend/src/schema/resolvers/posts.spec.js

@@ -496,6 +496,7 @@ describe('UpdatePost', () => {
       $image: ImageInput
       $categoryIds: [ID]
       $postType: PostType
+      $eventInput: _EventInput
     ) {
       UpdatePost(
         id: $id
@@ -504,6 +505,7 @@ describe('UpdatePost', () => {
         image: $image
         categoryIds: $categoryIds
         postType: $postType
+        eventInput: $eventInput
       ) {
         id
         title
@@ -518,6 +520,13 @@ describe('UpdatePost', () => {
           id
         }
         postType
+        eventStart
+        eventLocationName
+        eventVenue
+        eventLocation {
+          lng
+          lat
+        }
       }
     }
   `
@@ -639,15 +648,115 @@ describe('UpdatePost', () => {
       })
     })
 
-    describe('post type', () => {
+    describe('change post type to event', () => {
-      it('changes the post type', async () => {
+      describe('with missing event start date', () => {
+        it('throws an error', async () => {
           await expect(
-          mutate({ mutation: updatePostMutation, variables: { ...variables, postType: 'Event' } }),
+            mutate({
+              mutation: updatePostMutation,
+              variables: { ...variables, postType: 'Event' },
+            }),
+          ).resolves.toMatchObject({
+            errors: [
+              {
+                message: "Cannot read properties of undefined (reading 'eventStart')",
+              },
+            ],
+          })
+        })
+      })
+
+      describe('with invalid event start date', () => {
+        it('throws an error', async () => {
+          await expect(
+            mutate({
+              mutation: updatePostMutation,
+              variables: {
+                ...variables,
+                postType: 'Event',
+                eventInput: {
+                  eventStart: 'no-date',
+                },
+              },
+            }),
+          ).resolves.toMatchObject({
+            errors: [
+              {
+                message: 'Event start date must be a valid date!',
+              },
+            ],
+          })
+        })
+      })
+
+      describe('with event start date in the past', () => {
+        it('throws an error', async () => {
+          const now = new Date()
+          await expect(
+            mutate({
+              mutation: updatePostMutation,
+              variables: {
+                ...variables,
+                postType: 'Event',
+                eventInput: {
+                  eventStart: new Date(now.getFullYear(), now.getMonth() - 1).toISOString(),
+                },
+              },
+            }),
+          ).resolves.toMatchObject({
+            errors: [
+              {
+                message: 'Event start date must be in the future!',
+              },
+            ],
+          })
+        })
+      })
+
+      describe('event location is given but event venue is missing', () => {
+        it('throws an error', async () => {
+          const now = new Date()
+          await expect(
+            mutate({
+              mutation: updatePostMutation,
+              variables: {
+                ...variables,
+                postType: 'Event',
+                eventInput: {
+                  eventStart: new Date(now.getFullYear(), now.getMonth() + 1).toISOString(),
+                  eventLocation: 'Berlin',
+                },
+              },
+            }),
+          ).resolves.toMatchObject({
+            errors: [
+              {
+                message: 'Event venue must be present if event location is given!',
+              },
+            ],
+          })
+        })
+      })
+
+      describe('valid event input without location', () => {
+        it('has label "Event" set', async () => {
+          const now = new Date()
+          await expect(
+            mutate({
+              mutation: updatePostMutation,
+              variables: {
+                ...variables,
+                postType: 'Event',
+                eventInput: {
+                  eventStart: new Date(now.getFullYear(), now.getMonth() + 1).toISOString(),
+                },
+              },
+            }),
           ).resolves.toMatchObject({
             data: {
               UpdatePost: {
-              id: newlyCreatedPost.id,
                 postType: ['Event'],
+                eventStart: new Date(now.getFullYear(), now.getMonth() + 1).toISOString(),
               },
             },
             errors: undefined,
@@ -655,6 +764,41 @@ describe('UpdatePost', () => {
         })
       })
 
+      describe('valid event input with location', () => {
+        it('has label "Event" set', async () => {
+          const now = new Date()
+          await expect(
+            mutate({
+              mutation: updatePostMutation,
+              variables: {
+                ...variables,
+                postType: 'Event',
+                eventInput: {
+                  eventStart: new Date(now.getFullYear(), now.getMonth() + 1).toISOString(),
+                  eventLocation: 'Leipzig',
+                  eventVenue: 'Connewitzer Kreuz',
+                },
+              },
+            }),
+          ).resolves.toMatchObject({
+            data: {
+              UpdatePost: {
+                postType: ['Event'],
+                eventStart: new Date(now.getFullYear(), now.getMonth() + 1).toISOString(),
+                eventLocationName: 'Leipzig',
+                eventVenue: 'Connewitzer Kreuz',
+                eventLocation: {
+                  lng: 12.374733,
+                  lat: 51.340632,
+                },
+              },
+            },
+            errors: undefined,
+          })
+        })
+      })
+    })
+
     describe.skip('params.image', () => {
       describe('is object', () => {
         beforeEach(() => {