refactor(event): Update after simplification of event tables

__tests__/__utils__/assertions.ts

@@ -236,11 +236,16 @@ export async function assertErrorEvent(args?: {
 
     if (args?.errorContext !== undefined) {
       for (const contextName in args.errorContext) {
-        const contextValue = event.contexts?.error?.[contextName]
+        const contextValueString = event.contexts?.error?.[contextName]
+        const contextValue = destringifyProperties(contextValueString)
         const targetValue = args.errorContext[contextName]
 
-        if (!R.equals(destringifyProperties(contextValue), targetValue))
-          return false
+        if (typeof targetValue === 'object' && targetValue !== null) {
+          if (!R.equals(R.pick(R.keys(targetValue), contextValue), targetValue))
+            return false
+        } else {
+          if (contextValue !== targetValue) return false
+        }
       }
     }
 

__tests__/schema/event-types.ts

@@ -81,7 +81,7 @@ test('adds a notification', async () => {
 
   expect(
     await databaseForTests.fetchAll(
-      'select * from notification_event where event_log_id = ?',
+      'select * from notification_event where event_id = ?',
       [lastEvent.id],
     ),
   ).not.toHaveLength(0)
@@ -100,19 +100,6 @@ test('fails if object does not exist', async () => {
   ).rejects.toThrow()
 })
 
-test('fails if name from parameters is invalid', async () => {
-  const initialEventsNumber = await getEventsNumber()
-
-  await global.databaseForTests.mutate(
-    'delete from event_parameter_name where name = "discussion"',
-  )
-
-  await expect(createEvent(basePayload, getContext())).rejects.toThrow()
-
-  const finalEventsNumber = await getEventsNumber()
-  expect(finalEventsNumber).toEqual(initialEventsNumber)
-})
-
 test('fails if uuid number in parameters does not exist', async () => {
   await expect(
     createEvent({ ...basePayload, threadId: 40000 }, getContext()),
@@ -309,7 +296,7 @@ function getApiResult(event: Record<string, unknown>): Record<string, unknown> {
 async function getEventsNumber() {
   return (
     await global.databaseForTests.fetchOne<{ n: number }>(
-      'SELECT count(*) AS n FROM event_log',
+      'SELECT count(*) AS n FROM event',
     )
   ).n
 }

__tests__/schema/events.ts

@@ -97,7 +97,7 @@ describe('query endpoint "events"', () => {
   test('with filter "objectId"', async () => {
     await query.withVariables({ first: 1, objectId: 1855 }).shouldReturnData({
       events: {
-        nodes: [{ __typename: 'CreateEntityNotificationEvent', id: 1198 }],
+        nodes: [{ __typename: 'CheckoutRevisionNotificationEvent', id: 77030 }],
       },
     })
   })

__tests__/schema/user/delete-bots.ts

@@ -2,14 +2,14 @@ import { createHash } from 'crypto'
 import gql from 'graphql-tag'
 import { HttpResponse, ResponseResolver, http } from 'msw'
 
-import { article, user, user2 } from '../../../__fixtures__'
+import { article, user } from '../../../__fixtures__'
 import {
-  given,
   Client,
   Query,
-  returnsJson,
   assertErrorEvent,
   assertNoErrorEvents,
+  given,
+  returnsJson,
   userQuery,
 } from '../../__utils__'
 
@@ -137,11 +137,13 @@ describe('community chat', () => {
       returnsJson({ json: { success: false, errorType: 'unknown' } }),
     )
 
-    await mutation.withInput({ botIds: [user2.id] }).execute()
+    await mutation
+      .withInput({ botIds: input.botIds.slice(1) })
+      .shouldReturnData({ user: { deleteBots: { success: true } } })
 
     await assertErrorEvent({
       message: 'Cannot delete a user from community.serlo.org',
-      errorContext: { user: user2 },
+      errorContext: { user: { id: input.botIds[1] } },
     })
   })
 })

docker-compose.yml

@@ -4,7 +4,7 @@ services:
     ports:
       - '6379:6379'
   mysql:
-    image: eu.gcr.io/serlo-shared/serlo-mysql-database:prerelease-simplify-taxonomy-tables
+    image: eu.gcr.io/serlo-shared/serlo-mysql-database:prerelease-entityfield-coursepage-event-taxonomy
     platform: linux/x86_64
     pull_policy: always
     ports: