fix(worker): multi case method for fcm (#6405)

novuhq · Aug 28, 2024 · cb3b817 · cb3b817
1 parent 02aa790
commit cb3b817
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/packages/providers/src/lib/push/fcm/fcm.provider.spec.ts b/packages/providers/src/lib/push/fcm/fcm.provider.spec.ts
@@ -4,7 +4,7 @@ import app from 'firebase-admin/app';
 
 import { FcmPushProvider } from './fcm.provider';
 
-const sendMulticast = vi.fn().mockResolvedValue({ successCount: 1 });
+const sendEachForMulticast = vi.fn().mockResolvedValue({ successCount: 1 });
 const mockApp = {
   appCheck: vi.fn() as any,
   auth: vi.fn() as any,
@@ -31,7 +31,7 @@ vi.mock(import('firebase-admin/messaging'), async (importOriginal) => {
         sendEach: vi.fn(),
         sendEachForMulticast: vi.fn(),
         sendAll: vi.fn(),
-        sendMulticast,
+        sendEachForMulticast,
         sendToDevice: vi.fn(),
         sendToDeviceGroup: vi.fn(),
         sendToTopic: vi.fn(),
@@ -87,7 +87,7 @@ describe.skip('FcmPushProvider', () => {
     spy = vi
       // eslint-disable-next-line @typescript-eslint/ban-ts-comment
       // @ts-expect-error
-      .spyOn(provider.messaging, 'sendMulticast')
+      .spyOn(provider.messaging, 'sendEachForMulticast')
       .mockImplementation(async () => {
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         return {} as any;

diff --git a/packages/providers/src/lib/push/fcm/fcm.provider.ts b/packages/providers/src/lib/push/fcm/fcm.provider.ts
@@ -68,7 +68,7 @@ export class FcmPushProvider extends BaseProvider implements IPushProvider {
     let res;
 
     if (type === 'data') {
-      res = await this.messaging.sendMulticast(
+      res = await this.messaging.sendEachForMulticast(
         this.transform<MulticastMessage>(bridgeProviderData, {
           tokens: options.target,
           data: {
@@ -84,7 +84,7 @@ export class FcmPushProvider extends BaseProvider implements IPushProvider {
         }).body
       );
     } else {
-      res = await this.messaging.sendMulticast(
+      res = await this.messaging.sendEachForMulticast(
         this.transform<MulticastMessage>(bridgeProviderData, {
           tokens: options.target,
           notification: {