refactor(events): rename bulkVisitEvent to bulkVisitEvents for consistency across the codebase

Author: Dobrunia

src/models/eventsFactory.js

@@ -889,7 +889,7 @@ class EventsFactory extends Factory {
    * @param {string|ObjectId} userId - id of the user who is visiting events
    * @returns {Promise<{ updatedCount: number, updatedEventIds: string[], failedEventIds: string[] }>}
    */
-  async bulkVisitEvent(eventIds, userId) {
+  async bulkVisitEvents(eventIds, userId) {
     const {
       collection,
       found,

src/resolvers/event.js

@@ -162,7 +162,7 @@ module.exports = {
       }
 
       const factory = getEventsFactory(context, projectId);
-      const result = await factory.bulkVisitEvent(validEventIds, user.id);
+      const result = await factory.bulkVisitEvents(validEventIds, user.id);
 
       return {
         ...result,

test/models/eventsFactory-bulk-visit.test.ts

@@ -35,7 +35,7 @@ jest.mock('../../src/mongo', () => ({
 // eslint-disable-next-line @typescript-eslint/no-var-requires, @typescript-eslint/no-explicit-any -- CJS class
 const EventsFactory = require('../../src/models/eventsFactory') as any;
 
-describe('EventsFactory.bulkVisitEvent', () => {
+describe('EventsFactory.bulkVisitEvents', () => {
   const projectId = '507f1f77bcf86cd799439011';
 
   beforeEach(() => {
@@ -57,7 +57,7 @@ describe('EventsFactory.bulkVisitEvent', () => {
     });
     collectionMock.updateMany.mockResolvedValue({ modifiedCount: 1 });
 
-    const result = await factory.bulkVisitEvent([ a.toString(), b.toString() ], userId.toString());
+    const result = await factory.bulkVisitEvents([ a.toString(), b.toString() ], userId.toString());
 
     expect(result.updatedCount).toBe(1);
     expect(result.updatedEventIds).toEqual([ b.toString() ]);
@@ -72,7 +72,7 @@ describe('EventsFactory.bulkVisitEvent', () => {
       toArray: () => Promise.resolve([]),
     });
 
-    const result = await factory.bulkVisitEvent([ missing.toString() ], new ObjectId().toString());
+    const result = await factory.bulkVisitEvents([ missing.toString() ], new ObjectId().toString());
 
     expect(result.updatedCount).toBe(0);
     expect(result.updatedEventIds).toEqual([]);

test/resolvers/event-bulk-visit.test.ts

@@ -17,7 +17,7 @@ const eventResolvers = require('../../src/resolvers/event') as {
   };
 };
 
-const bulkVisitEvent = jest.fn();
+const bulkVisitEvents = jest.fn();
 
 describe('Mutation.bulkVisitEvents', () => {
   const ctx = {
@@ -26,11 +26,11 @@ describe('Mutation.bulkVisitEvents', () => {
 
   beforeEach(() => {
     jest.clearAllMocks();
-    (getEventsFactory as unknown as jest.Mock).mockReturnValue({ bulkVisitEvent });
+    (getEventsFactory as unknown as jest.Mock).mockReturnValue({ bulkVisitEvents });
   });
 
   it('should call factory with valid ids only and merge invalid ids', async () => {
-    bulkVisitEvent.mockResolvedValue({
+    bulkVisitEvents.mockResolvedValue({
       updatedCount: 1,
       updatedEventIds: [ '507f1f77bcf86cd799439012' ],
       failedEventIds: [ '507f1f77bcf86cd799439099' ],
@@ -42,7 +42,7 @@ describe('Mutation.bulkVisitEvents', () => {
       ctx
     );
 
-    expect(bulkVisitEvent).toHaveBeenCalledWith(
+    expect(bulkVisitEvents).toHaveBeenCalledWith(
       [ '507f1f77bcf86cd799439012' ],
       '507f1f77bcf86cd799439011'
     );
@@ -60,7 +60,7 @@ describe('Mutation.bulkVisitEvents', () => {
       ctx
     );
 
-    expect(bulkVisitEvent).not.toHaveBeenCalled();
+    expect(bulkVisitEvents).not.toHaveBeenCalled();
     expect(result).toEqual({
       updatedCount: 0,
       updatedEventIds: [],