add unit tests

Author: nazabucciarelli

apps/meteor/client/components/message/content/attachments/QuoteAttachment.spec.tsx

@@ -0,0 +1,44 @@
+import { render, screen } from '@testing-library/react';
+
+import { QuoteAttachment } from './QuoteAttachment';
+
+jest.mock('../../hooks/useMaxMessageParseSize', () => ({
+	useMaxMessageParseSize: () => 100,
+}));
+
+jest.mock('@rocket.chat/ui-contexts', () => ({
+	useUserPreference: () => true,
+}));
+
+jest.mock('../../../../hooks/useTimeAgo', () => ({
+	useTimeAgo: () => (date: Date) => date.toISOString(),
+}));
+
+jest.mock('../../MessageContentBody', () => ({
+	__esModule: true,
+	default: () => <div data-testid='message-content-body' />,
+}));
+
+const baseAttachment = {
+	author_name: 'User',
+	author_icon: '',
+	ts: new Date(),
+	text: 'short text',
+	md: [{ type: 'PARAGRAPH', value: [{ type: 'PLAIN_TEXT', value: 'short text' }] }],
+};
+
+describe('QuoteAttachment', () => {
+	it('renders MessageContentBody when text length is within maxMessageParseSize', () => {
+		render(<QuoteAttachment attachment={baseAttachment as any} />);
+		expect(screen.getByTestId('message-content-body')).toBeInTheDocument();
+	});
+
+	it('renders plain text when text exceeds maxMessageParseSize', () => {
+		const longText = 'a'.repeat(101);
+		const attachment = { ...baseAttachment, text: longText };
+
+		render(<QuoteAttachment attachment={attachment as any} />);
+
+		expect(screen.queryByTestId('message-content-body')).not.toBeInTheDocument();
+	});
+});

apps/meteor/client/components/message/hooks/useNormalizedMessage.spec.ts

@@ -0,0 +1,68 @@
+import { renderHook } from '@testing-library/react';
+
+import { useNormalizedMessage } from './useNormalizedMessage';
+
+const mockParseMessageTextToAstMarkdown = jest.fn((msg: any, ..._args: any[]) => msg);
+
+jest.mock('../list/MessageListContext', () => ({
+	useMessageListKatex: () => null,
+	useMessageListAutoTranslate: () => ({
+		showAutoTranslate: () => false,
+		autoTranslateLanguage: '',
+	}),
+	useMessageListShowColors: () => false,
+}));
+
+jest.mock('../../../lib/parseMessageTextToAstMarkdown', () => ({
+	parseMessageTextToAstMarkdown: (msg: any, ...args: any[]) => mockParseMessageTextToAstMarkdown(msg, ...args),
+}));
+
+jest.mock('../../../views/room/MessageList/hooks/useAutoLinkDomains', () => ({ useAutoLinkDomains: () => [] }));
+
+const baseMessage = {
+	_id: 'msg1',
+	rid: 'room1',
+	u: { _id: 'u1', username: 'user', name: 'User' },
+	ts: new Date(),
+	_updatedAt: new Date(),
+};
+
+describe('useNormalizedMessage', () => {
+	beforeEach(() => {
+		mockParseMessageTextToAstMarkdown.mockClear();
+	});
+
+	it('should skip parsing and returns PARAGRAPH node when msg exceeds maxMessageParseSize', () => {
+		const longMsg = 'a'.repeat(101);
+		const message = { ...baseMessage, msg: longMsg };
+
+		const { result } = renderHook(() => useNormalizedMessage(message as any, 100));
+
+		expect(mockParseMessageTextToAstMarkdown).not.toHaveBeenCalled();
+		expect(result.current.md).toEqual([
+			{
+				type: 'PARAGRAPH',
+				value: [{ type: 'PLAIN_TEXT', value: longMsg }],
+			},
+		]);
+	});
+
+	it('should call parseMessageTextToAstMarkdown when msg is within maxMessageParseSize', () => {
+		const message = { ...baseMessage, msg: 'Hello world' };
+
+		renderHook(() => useNormalizedMessage(message as any, 100));
+
+		expect(mockParseMessageTextToAstMarkdown).toHaveBeenCalledWith(message, expect.anything(), expect.anything());
+	});
+
+	it('should preserve attachments when bypassing parsing due to size', () => {
+		const longMsg = 'a'.repeat(101);
+		const attachments = [{ type: 'quote', text: 'quoted' }];
+		const message = { ...baseMessage, msg: longMsg, attachments };
+
+		const { result } = renderHook(() => useNormalizedMessage(message as any, 100));
+
+		expect(mockParseMessageTextToAstMarkdown).not.toHaveBeenCalled();
+		expect(result.current.attachments).toBe(attachments);
+	});
+});

apps/meteor/client/views/room/MessageList/hooks/useMessageBody.spec.ts

@@ -0,0 +1,64 @@
+import { renderHook } from '@testing-library/react';
+
+import { useMessageBody } from './useMessageBody';
+
+const mockParseMessageTextToAstMarkdown = jest.fn();
+
+jest.mock('./useAutoLinkDomains', () => ({
+	useAutoLinkDomains: () => [],
+}));
+
+jest.mock('../../../../components/message/list/MessageListContext', () => ({
+	useMessageListAutoTranslate: () => ({
+		showAutoTranslate: () => false,
+		autoTranslateLanguage: '',
+	}),
+}));
+
+jest.mock('../../../../lib/parseMessageTextToAstMarkdown', () => ({
+	parseMessageTextToAstMarkdown: (msg: any, ...args: any[]) => mockParseMessageTextToAstMarkdown(msg, ...args),
+}));
+
+const baseMessage = {
+	_id: 'msg1',
+	rid: 'room1',
+	u: { _id: 'u1', username: 'user', name: 'User' },
+	ts: new Date(),
+	_updatedAt: new Date(),
+};
+
+describe('useMessageBody', () => {
+	beforeEach(() => {
+		mockParseMessageTextToAstMarkdown.mockClear();
+	});
+
+	it('should return raw msg and skips parsing when msg exceeds maxMessageParseSize', () => {
+		const longMsg = 'a'.repeat(101);
+		const message = { ...baseMessage, msg: longMsg, md: [{ type: 'PARAGRAPH', value: [] }] };
+
+		const { result } = renderHook(() => useMessageBody(message as any, 100));
+
+		expect(result.current).toBe(longMsg);
+		expect(mockParseMessageTextToAstMarkdown).not.toHaveBeenCalled();
+	});
+
+	it('should call parser when message has md and is within maxMessageParseSize', () => {
+		const md = [{ type: 'PARAGRAPH', value: [] }];
+		const message = { ...baseMessage, msg: 'Hello world', md };
+		mockParseMessageTextToAstMarkdown.mockReturnValue({ ...message, md });
+
+		const { result } = renderHook(() => useMessageBody(message as any, 100));
+
+		expect(mockParseMessageTextToAstMarkdown).toHaveBeenCalledWith(message, expect.anything(), expect.anything());
+		expect(result.current).toBe(md);
+	});
+
+	it('should return raw msg without parsing when message has no md', () => {
+		const message = { ...baseMessage, msg: 'Hello world' };
+
+		const { result } = renderHook(() => useMessageBody(message as any, 100));
+
+		expect(mockParseMessageTextToAstMarkdown).not.toHaveBeenCalled();
+		expect(result.current).toBe('Hello world');
+	});
+});