196e51e91b
The quick-reply box built its body with appendPlainTextSignature, which runs the identity's HTML signature through htmlToPlainText, and sent a text-only message (htmlBody was undefined). A formatted signature (e.g. <strong>…) was therefore flattened to plain text in the sent mail, even though it previewed correctly in the identity editor. The full composer already builds an HTML signature block; quick reply did not. Add an appendHtmlSignature helper (mirrors the composer's send-time block) and, when the sending identity has an HTML signature, send a matching HTML body from handleQuickReply so the markup is preserved. Text-only identities keep the plain-text-only behavior. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
61 lines
2.4 KiB
TypeScript
61 lines
2.4 KiB
TypeScript
import { describe, expect, it } from 'vitest';
|
|
|
|
import {
|
|
appendHtmlSignature,
|
|
appendPlainTextSignature,
|
|
getPlainTextSignature,
|
|
hasMeaningfulHtmlBody,
|
|
} from '../signature-utils';
|
|
|
|
describe('signature-utils', () => {
|
|
describe('getPlainTextSignature', () => {
|
|
it('prefers text signatures when present', () => {
|
|
expect(getPlainTextSignature({ textSignature: 'Regards,\nAlice', htmlSignature: '<p>Ignored</p>' })).toBe('Regards,\nAlice');
|
|
});
|
|
|
|
it('converts html-only signatures into plain text', () => {
|
|
expect(getPlainTextSignature({ htmlSignature: '<p>Alice Example<br><a href="mailto:alice@example.com">alice@example.com</a></p>' })).toBe('Alice Example\nalice@example.com');
|
|
});
|
|
});
|
|
|
|
describe('appendPlainTextSignature', () => {
|
|
it('appends a converted html signature to the text body', () => {
|
|
expect(appendPlainTextSignature('Hello there', { htmlSignature: '<p>Alice<br>Engineering</p>' })).toBe('Hello there\n\n-- \nAlice\nEngineering');
|
|
});
|
|
|
|
it('leaves the body untouched when no signature exists', () => {
|
|
expect(appendPlainTextSignature('Hello there', {})).toBe('Hello there');
|
|
});
|
|
});
|
|
|
|
describe('appendHtmlSignature', () => {
|
|
it('appends a sanitized html signature, preserving formatting', () => {
|
|
expect(appendHtmlSignature('<div>Hello</div>', { htmlSignature: '<strong>Alice</strong>' }))
|
|
.toBe('<div>Hello</div><br><br>-- <br><strong>Alice</strong>');
|
|
});
|
|
|
|
it('escapes and appends a text signature when no html signature exists', () => {
|
|
expect(appendHtmlSignature('<div>Hello</div>', { textSignature: 'Alice\nEng' }))
|
|
.toBe('<div>Hello</div><br><br>-- <br>Alice<br>Eng');
|
|
});
|
|
|
|
it('omits the separator marker when disabled', () => {
|
|
expect(appendHtmlSignature('<div>Hi</div>', { htmlSignature: '<strong>A</strong>' }, { separator: false }))
|
|
.toBe('<div>Hi</div><br><br><strong>A</strong>');
|
|
});
|
|
|
|
it('leaves the body untouched when no signature exists', () => {
|
|
expect(appendHtmlSignature('<div>Hi</div>', {})).toBe('<div>Hi</div>');
|
|
});
|
|
});
|
|
|
|
describe('hasMeaningfulHtmlBody', () => {
|
|
it('prefers html bodies that preserve signature formatting', () => {
|
|
expect(hasMeaningfulHtmlBody('<div>Hello</div><br><p>Alice</p>')).toBe(true);
|
|
});
|
|
|
|
it('ignores minimal wrapper html with a single block', () => {
|
|
expect(hasMeaningfulHtmlBody('<div>Hello world</div>')).toBe(false);
|
|
});
|
|
});
|
|
}); |