mirror of
https://github.com/lingble/chatwoot.git
synced 2025-11-05 13:37:55 +00:00
# Pull Request Template
## Description
**Issue**
This PR fixes template variables in messages (e.g., {{customer.name}})
that were being incorrectly converted to clickable links by the
`MessageFormatter's linkify` functionality. This caused formatting
issues and broken links.
**Solution**
Added a `linkify` parameter to `MessageFormatter` to optionally disable
link conversion
## Type of change
- [x] Bug fix (non-breaking change which fixes an issue)
## How Has This Been Tested?
**Screenshots**
**Before**
<img width="1012" alt="image"
src="https://github.com/user-attachments/assets/70abb238-b4d9-439d-9e51-c7513cf482fb"
/>
**After**
<img width="1012" alt="image"
src="https://github.com/user-attachments/assets/387acb74-674e-4b26-85cc-2d7190d256b1"
/>
## Checklist:
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my code
- [x] I have commented on my code, particularly in hard-to-understand
areas
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [x] I have added tests that prove my fix is effective or that my
feature works
- [x] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream
modules
92 lines
3.0 KiB
JavaScript
92 lines
3.0 KiB
JavaScript
import { useMessageFormatter } from '../useMessageFormatter';
|
|
|
|
describe('useMessageFormatter', () => {
|
|
let messageFormatter;
|
|
|
|
beforeEach(() => {
|
|
messageFormatter = useMessageFormatter();
|
|
});
|
|
|
|
describe('formatMessage', () => {
|
|
it('should format a regular message correctly', () => {
|
|
const message = 'This is a [test](https://example.com) message';
|
|
const result = messageFormatter.formatMessage(message, false, false);
|
|
expect(result).toContain('<a href="https://example.com"');
|
|
expect(result).toContain('class="link"');
|
|
});
|
|
|
|
it('should format a tweet correctly', () => {
|
|
const message = '@user #hashtag';
|
|
const result = messageFormatter.formatMessage(message, true, false);
|
|
expect(result).toContain('<a href="http://twitter.com/user"');
|
|
expect(result).toContain('<a href="https://twitter.com/hashtag/hashtag"');
|
|
});
|
|
|
|
it('should not format mentions and hashtags for private notes', () => {
|
|
const message = '@user #hashtag';
|
|
const result = messageFormatter.formatMessage(message, false, true);
|
|
expect(result).not.toContain('<a href="http://twitter.com/user"');
|
|
expect(result).not.toContain(
|
|
'<a href="https://twitter.com/hashtag/hashtag"'
|
|
);
|
|
});
|
|
|
|
it('should disable link formatting when linkify is false', () => {
|
|
const message = 'Check https://example.com and {{user.id}}';
|
|
const result = messageFormatter.formatMessage(
|
|
message,
|
|
false,
|
|
false,
|
|
false
|
|
);
|
|
expect(result).not.toContain('<a href="https://example.com"');
|
|
expect(result).toContain('{{user.id}}');
|
|
});
|
|
});
|
|
|
|
describe('truncateMessage', () => {
|
|
it('should not truncate short messages', () => {
|
|
const message = 'Short message';
|
|
const result = messageFormatter.truncateMessage(message);
|
|
expect(result).toBe(message);
|
|
});
|
|
|
|
it('should truncate long messages', () => {
|
|
const message = 'A'.repeat(150);
|
|
const result = messageFormatter.truncateMessage(message);
|
|
expect(result.length).toBe(100);
|
|
expect(result.endsWith('...')).toBe(true);
|
|
});
|
|
});
|
|
|
|
describe('highlightContent', () => {
|
|
it('should highlight search term in content', () => {
|
|
const content = 'This is a test message';
|
|
const searchTerm = 'test';
|
|
const highlightClass = 'highlight';
|
|
const result = messageFormatter.highlightContent(
|
|
content,
|
|
searchTerm,
|
|
highlightClass
|
|
);
|
|
expect(result.trim()).toBe(
|
|
'This is a <span class="highlight">test</span> message'
|
|
);
|
|
});
|
|
|
|
it('should handle special characters in search term', () => {
|
|
const content = 'This (message) contains [special] characters';
|
|
const searchTerm = '(message)';
|
|
const highlightClass = 'highlight';
|
|
const result = messageFormatter.highlightContent(
|
|
content,
|
|
searchTerm,
|
|
highlightClass
|
|
);
|
|
expect(result.trim()).toBe(
|
|
'This <span class="highlight">(message)</span> contains [special] characters'
|
|
);
|
|
});
|
|
});
|
|
});
|