Add unit tests

Author: AhyoungRyu

package.json

@@ -102,6 +102,7 @@
     "@svgr/rollup": "^8.1.0",
     "@testing-library/jest-dom": "^5.16.5",
     "@testing-library/react": "^13.4.0",
+    "@testing-library/react-hooks": "^8.0.1",
     "@testing-library/user-event": "^14.4.3",
     "@types/node": "^22.7.2",
     "@typescript-eslint/eslint-plugin": "^6.17.0",

src/modules/GroupChannel/context/__tests__/useMessageActions.spec.ts

@@ -0,0 +1,154 @@
+import { renderHook } from '@testing-library/react-hooks';
+import { useMessageActions } from '../hooks/useMessageActions';
+import { UserMessageCreateParams, FileMessageCreateParams } from '@sendbird/chat/message';
+
+const mockEventHandlers = {
+  message: {
+    onSendMessageFailed: jest.fn(),
+    onUpdateMessageFailed: jest.fn(),
+    onFileUploadFailed: jest.fn(),
+  },
+};
+
+jest.mock('../../../../hooks/useSendbirdStateContext', () => ({
+  __esModule: true,
+  default: () => ({
+    eventHandlers: mockEventHandlers,
+  }),
+}));
+
+describe('useMessageActions', () => {
+  const mockParams = {
+    sendUserMessage: jest.fn(),
+    sendFileMessage: jest.fn(),
+    sendMultipleFilesMessage: jest.fn(),
+    updateUserMessage: jest.fn(),
+    scrollToBottom: jest.fn(),
+    replyType: 'NONE',
+  };
+
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  describe('processParams', () => {
+    it('should handle successful user message', async () => {
+      const { result } = renderHook(() => useMessageActions(mockParams));
+      const params: UserMessageCreateParams = { message: 'test' };
+
+      await result.current.sendUserMessage(params);
+
+      expect(mockParams.sendUserMessage).toHaveBeenCalledWith(
+        expect.objectContaining({ message: 'test' }),
+        expect.any(Function),
+      );
+    });
+
+    it('should handle void return from onBeforeSendFileMessage', async () => {
+      const onBeforeSendFileMessage = jest.fn();
+      const { result } = renderHook(() => useMessageActions({
+        ...mockParams,
+        onBeforeSendFileMessage,
+      }),
+      );
+
+      const fileParams: FileMessageCreateParams = {
+        file: new File([], 'test.txt'),
+      };
+
+      await result.current.sendFileMessage(fileParams);
+
+      expect(onBeforeSendFileMessage).toHaveBeenCalled();
+      expect(mockParams.sendFileMessage).toHaveBeenCalledWith(
+        expect.objectContaining(fileParams),
+        expect.any(Function),
+      );
+    });
+
+    it('should handle file upload error', async () => {
+      // Arrange
+      const error = new Error('Upload failed');
+      const onBeforeSendFileMessage = jest.fn().mockRejectedValue(error);
+      const fileParams: FileMessageCreateParams = {
+        file: new File([], 'test.txt'),
+        fileName: 'test.txt',
+      };
+
+      const { result } = renderHook(() => useMessageActions({
+        ...mockParams,
+        onBeforeSendFileMessage,
+      }),
+      );
+
+      await expect(async () => {
+        await result.current.sendFileMessage(fileParams);
+      }).rejects.toThrow('Upload failed');
+
+      // Wait for next tick to ensure all promises are resolved
+      await new Promise(process.nextTick);
+
+      expect(onBeforeSendFileMessage).toHaveBeenCalled();
+      expect(mockEventHandlers.message.onFileUploadFailed).toHaveBeenCalledWith(error);
+      expect(mockEventHandlers.message.onSendMessageFailed).toHaveBeenCalledWith(
+        expect.objectContaining({
+          file: fileParams.file,
+          fileName: fileParams.fileName,
+        }),
+        error,
+      );
+    });
+
+    it('should handle message update error', async () => {
+      // Arrange
+      const error = new Error('Update failed');
+      const onBeforeUpdateUserMessage = jest.fn().mockRejectedValue(error);
+      const messageParams = {
+        messageId: 1,
+        message: 'update message',
+      };
+
+      const { result } = renderHook(() => useMessageActions({
+        ...mockParams,
+        onBeforeUpdateUserMessage,
+      }),
+      );
+
+      await expect(async () => {
+        await result.current.updateUserMessage(messageParams.messageId, {
+          message: messageParams.message,
+        });
+      }).rejects.toThrow('Update failed');
+
+      // Wait for next tick to ensure all promises are resolved
+      await new Promise(process.nextTick);
+
+      expect(onBeforeUpdateUserMessage).toHaveBeenCalled();
+      expect(mockEventHandlers.message.onUpdateMessageFailed).toHaveBeenCalledWith(
+        expect.objectContaining({
+          message: messageParams.message,
+        }),
+        error,
+      );
+    });
+
+    it('should preserve modified params from onBefore handlers', async () => {
+      const onBeforeSendUserMessage = jest.fn().mockImplementation((params) => ({
+        ...params,
+        message: 'modified',
+      }));
+
+      const { result } = renderHook(() => useMessageActions({
+        ...mockParams,
+        onBeforeSendUserMessage,
+      }),
+      );
+
+      await result.current.sendUserMessage({ message: 'original' });
+
+      expect(mockParams.sendUserMessage).toHaveBeenCalledWith(
+        expect.objectContaining({ message: 'modified' }),
+        expect.any(Function),
+      );
+    });
+  });
+});

yarn.lock

@@ -2778,6 +2778,7 @@ __metadata:
     "@svgr/rollup": ^8.1.0
     "@testing-library/jest-dom": ^5.16.5
     "@testing-library/react": ^13.4.0
+    "@testing-library/react-hooks": ^8.0.1
     "@testing-library/user-event": ^14.4.3
     "@types/node": ^22.7.2
     "@typescript-eslint/eslint-plugin": ^6.17.0
@@ -3468,6 +3469,28 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@testing-library/react-hooks@npm:^8.0.1":
+  version: 8.0.1
+  resolution: "@testing-library/react-hooks@npm:8.0.1"
+  dependencies:
+    "@babel/runtime": ^7.12.5
+    react-error-boundary: ^3.1.0
+  peerDependencies:
+    "@types/react": ^16.9.0 || ^17.0.0
+    react: ^16.9.0 || ^17.0.0
+    react-dom: ^16.9.0 || ^17.0.0
+    react-test-renderer: ^16.9.0 || ^17.0.0
+  peerDependenciesMeta:
+    "@types/react":
+      optional: true
+    react-dom:
+      optional: true
+    react-test-renderer:
+      optional: true
+  checksum: 7fe44352e920deb5cb1876f80d64e48615232072c9d5382f1e0284b3aab46bb1c659a040b774c45cdf084a5257b8fe463f7e08695ad8480d8a15635d4d3d1f6d
+  languageName: node
+  linkType: hard
+
 "@testing-library/react@npm:^13.4.0":
   version: 13.4.0
   resolution: "@testing-library/react@npm:13.4.0"
@@ -12532,6 +12555,17 @@ __metadata:
   languageName: node
   linkType: hard
 
+"react-error-boundary@npm:^3.1.0":
+  version: 3.1.4
+  resolution: "react-error-boundary@npm:3.1.4"
+  dependencies:
+    "@babel/runtime": ^7.12.5
+  peerDependencies:
+    react: ">=16.13.1"
+  checksum: f36270a5d775a25c8920f854c0d91649ceea417b15b5bc51e270a959b0476647bb79abb4da3be7dd9a4597b029214e8fe43ea914a7f16fa7543c91f784977f1b
+  languageName: node
+  linkType: hard
+
 "react-is@npm:18.1.0":
   version: 18.1.0
   resolution: "react-is@npm:18.1.0"