Preserve video thumbnail height and verify social target outputs
This commit is contained in:
1 parent
df382ed550
commit
0fa3c01a16
3 files changed
+41
-2
No files matched your search
@@ -114,6 +114,45 @@ test("marketing image dimensions reject invalid or injected values", async () =>
|
||||
}
|
||||
});
|
||||
|
||||
test("video exports preserve thumbnail proportions, safe destination and accessible text", async () => {
|
||||
const draft = document("content.video", { title: "Watch & learn", href: "https://example.com/watch?a=1&b=2", thumbnail: "https://example.com/poster.png", alt: 'Demo & "preview"', width: "360", height: "240" });
|
||||
const before = JSON.stringify(draft);
|
||||
const result = await renderEmail(draft, { amp: true, mjml: true });
|
||||
expect(result.ok).toBe(true);
|
||||
if (!result.ok) return;
|
||||
for (const output of [result.value.parts.html.content, result.value.parts.amp?.content, result.value.exports?.mjml]) {
|
||||
expect(output).toContain('href="https://example.com/watch?a=1&b=2"');
|
||||
expect(output).toContain('src="https://example.com/poster.png"');
|
||||
expect(output).toContain('alt="Demo & "preview""');
|
||||
}
|
||||
expect(result.value.parts.html.content).toContain('width="360" height="240"');
|
||||
expect(result.value.parts.amp?.content).toContain('width="360" height="240" layout="responsive"');
|
||||
expect(result.value.exports?.mjml).toContain('width="360px" height="240px"');
|
||||
expect(result.value.parts.text.content).toContain("Watch & learn: https://example.com/watch?a=1&b=2");
|
||||
expect(result.value.parts.amp?.validation.valid).toBe(true);
|
||||
expect(await validateMjml(result.value.exports!.mjml!)).toEqual([]);
|
||||
expect(JSON.stringify(draft)).toBe(before);
|
||||
});
|
||||
|
||||
test("social exports include only configured networks and retain safe links in plain text", async () => {
|
||||
const props = { label: "Follow & connect", facebook: "https://example.com/facebook?a=1&b=2", linkedin: "https://example.com/linkedin", instagram: "", x: "" };
|
||||
const result = await renderEmail(document("content.social", props), { amp: true, mjml: true });
|
||||
expect(result.ok).toBe(true);
|
||||
if (!result.ok) return;
|
||||
for (const output of [result.value.parts.html.content, result.value.parts.amp?.content, result.value.exports?.mjml]) {
|
||||
expect(output).toContain("Follow & connect");
|
||||
expect(output).toContain('href="https://example.com/facebook?a=1&b=2"');
|
||||
expect(output).toContain('href="https://example.com/linkedin"');
|
||||
expect(output).not.toContain(">Instagram</a>");
|
||||
expect(output).not.toContain(">X</a>");
|
||||
}
|
||||
expect(result.value.parts.text.content).toContain("Facebook: https://example.com/facebook?a=1&b=2");
|
||||
expect(result.value.parts.text.content).toContain("LinkedIn: https://example.com/linkedin");
|
||||
expect(result.value.parts.amp?.validation.valid).toBe(true);
|
||||
expect(await validateMjml(result.value.exports!.mjml!)).toEqual([]);
|
||||
expect((await renderEmail(document("content.social", { ...props, facebook: "javascript:alert(1)" }))).ok).toBe(false);
|
||||
});
|
||||
|
||||
test("basic images, video thumbnails and spacers validate dimensions before rendering", async () => {
|
||||
const cases: Array<{ type: string; props: Record<string, string> }> = [
|
||||
{ type: "content.image", props: { src: "https://example.com/image.png", alt: "Image" } },
|
||||
|
||||
Reference in new issue
Block a user