refactor(events): remove unsupported mark validation from bulkToggleEventMarks and clean up related tests

Author: Dobrunia

src/resolvers/event.js

@@ -170,10 +170,6 @@ module.exports = {
      * @return {Promise<{ updatedCount: number, updatedEventIds: string[], failedEventIds: string[] }>}
      */
     async bulkToggleEventMarks(_obj, { projectId, eventIds, mark }, context) {
-      if (mark !== 'resolved' && mark !== 'ignored' && mark !== 'starred') {
-        throw new UserInputError('bulkToggleEventMarks supports only resolved, ignored and starred marks');
-      }
-
       const { validEventIds, invalidEventIds } = parseBulkEventIds(eventIds);
 
       if (validEventIds.length === 0) {

test/resolvers/event-bulk-toggle-marks.test.ts

@@ -1,7 +1,5 @@
 import '../../src/env-test';
 
-import { UserInputError } from 'apollo-server-express';
-
 jest.mock('../../src/resolvers/helpers/eventsFactory', () => ({
   __esModule: true,
   default: jest.fn(),
@@ -31,26 +29,6 @@ describe('Mutation.bulkToggleEventMarks', () => {
     });
   });
 
-  it('should throw when mark is not supported', async () => {
-    await expect(
-      eventResolvers.Mutation.bulkToggleEventMarks(
-        {},
-        { projectId: 'p1', eventIds: [ '507f1f77bcf86cd799439012' ], mark: 'some-unknown-mark' },
-        ctx
-      )
-    ).rejects.toThrow(UserInputError);
-
-    await expect(
-      eventResolvers.Mutation.bulkToggleEventMarks(
-        {},
-        { projectId: 'p1', eventIds: [ '507f1f77bcf86cd799439012' ], mark: 'some-unknown-mark' },
-        ctx
-      )
-    ).rejects.toThrow('bulkToggleEventMarks supports only resolved, ignored and starred marks');
-
-    expect(bulkToggleEventMark).not.toHaveBeenCalled();
-  });
-
   it('should throw when eventIds is empty', async () => {
     await expect(
       eventResolvers.Mutation.bulkToggleEventMarks(