vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-06 07:10:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: drop duplicate markdown cases

Browse Source
This commit is contained in:
Peter Steinberger
2026-04-17 18:28:40 +01:00
parent 81d6cf9c82
commit f665c767e6
1 changed files with 17 additions and 148 deletions
Download Patch File Download Diff File Expand all files Collapse all files

165
ui/src/ui/markdown.test.ts
View File

@@ -3,11 +3,6 @@ import { md, toSanitizedMarkdownHtml } from "./markdown.ts";
describe("toSanitizedMarkdownHtml", () => {
// ── Original tests from before markdown-it migration ──
it("renders basic markdown", () => {
const html = toSanitizedMarkdownHtml("Hello **world**");
expect(html).toContain("<strong>world</strong>");
});
it("strips scripts and unsafe links", () => {
const html = toSanitizedMarkdownHtml(
[
@@ -23,146 +18,6 @@ describe("toSanitizedMarkdownHtml", () => {
expect(html).toContain("https://example.com");
});
it("renders fenced code blocks", () => {
const html = toSanitizedMarkdownHtml(["```ts", "console.log(1)", "```"].join("\n"));
expect(html).toContain("<pre>");
expect(html).toContain("<code");
expect(html).toContain("console.log(1)");
});
it("flattens remote markdown images into alt text", () => {
const html = toSanitizedMarkdownHtml("![Alt text](https://example.com/image.png)");
expect(html).not.toContain("<img");
expect(html).toContain("Alt text");
});
it("preserves base64 data URI images (#15437)", () => {
const html = toSanitizedMarkdownHtml("![Chart](data:image/png;base64,iVBORw0KGgo=)");
expect(html).toContain("<img");
expect(html).toContain('class="markdown-inline-image"');
expect(html).toContain("data:image/png;base64,");
});
it("flattens non-data markdown image urls", () => {
const html = toSanitizedMarkdownHtml("![X](javascript:alert(1))");
expect(html).not.toContain("<img");
expect(html).not.toContain("javascript:");
expect(html).toContain("X");
});
it("uses a plain fallback label for unlabeled markdown images", () => {
const html = toSanitizedMarkdownHtml("![](https://example.com/image.png)");
expect(html).not.toContain("<img");
expect(html).toContain("image");
});
it("renders GFM markdown tables (#20410)", () => {
const md = [
"| Feature | Status |",
"|---------|--------|",
"| Tables | ✅ |",
"| Borders | ✅ |",
].join("\n");
const html = toSanitizedMarkdownHtml(md);
expect(html).toContain("<table");
expect(html).toContain("<thead");
expect(html).toContain("<th>");
expect(html).toContain("Feature");
expect(html).toContain("Tables");
expect(html).not.toContain("|---------|");
});
it("renders GFM tables surrounded by text (#20410)", () => {
const md = [
"Text before.",
"",
"| Col1 | Col2 |",
"|------|------|",
"| A | B |",
"",
"Text after.",
].join("\n");
const html = toSanitizedMarkdownHtml(md);
expect(html).toContain("<table");
expect(html).toContain("Col1");
expect(html).toContain("Col2");
// Pipes from table delimiters must not appear as raw text
expect(html).not.toContain("|------|");
});
it("does not throw on deeply nested emphasis markers (#36213)", () => {
// Pathological patterns that can trigger catastrophic backtracking / recursion
const nested = "*".repeat(500) + "text" + "*".repeat(500);
expect(() => toSanitizedMarkdownHtml(nested)).not.toThrow();
const html = toSanitizedMarkdownHtml(nested);
expect(html).toContain("text");
});
it("does not throw on deeply nested brackets (#36213)", () => {
const nested = "[".repeat(200) + "link" + "]".repeat(200) + "(" + "x".repeat(200) + ")";
expect(() => toSanitizedMarkdownHtml(nested)).not.toThrow();
const html = toSanitizedMarkdownHtml(nested);
expect(html).toContain("link");
});
it("keeps oversized plain-text replies readable instead of forcing code-block chrome", () => {
const input =
Array.from(
{ length: 320 },
(_, i) => `Paragraph ${i + 1}: ${"Long plain-text reply. ".repeat(8)}`,
).join("\n\n") + "\n";
const html = toSanitizedMarkdownHtml(input);
expect(html).not.toContain('<pre class="code-block">');
expect(html).toContain('class="markdown-plain-text-fallback"');
expect(html).toContain("Paragraph 1:");
expect(html).toContain("Paragraph 320:");
});
it("preserves indentation in oversized plain-text replies", () => {
const input = `${"Header line\n".repeat(5000)}\n indented log line\n deeper indent`;
const html = toSanitizedMarkdownHtml(input);
expect(html).toContain('class="markdown-plain-text-fallback"');
expect(html).toContain(" indented log line");
expect(html).toContain(" deeper indent");
});
it("exercises the cached oversized fallback branch", () => {
const input =
Array.from(
{ length: 240 },
(_, i) => `Paragraph ${i + 1}: ${"Cacheable long reply. ".repeat(8)}`,
).join("\n\n") + "\n";
expect(input.length).toBeGreaterThan(40_000);
expect(input.length).toBeLessThan(50_000);
const first = toSanitizedMarkdownHtml(input);
const second = toSanitizedMarkdownHtml(input);
expect(first).toContain('class="markdown-plain-text-fallback"');
expect(second).toBe(first);
});
it("falls back to escaped plain text if md.render throws (#36213)", () => {
const renderSpy = vi.spyOn(md, "render").mockImplementation(() => {
throw new Error("forced render failure");
});
const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
const input = `Fallback **probe** ${Date.now()}`;
try {
const html = toSanitizedMarkdownHtml(input);
expect(html).toContain('<pre class="code-block">');
expect(html).toContain("Fallback **probe**");
expect(warnSpy).toHaveBeenCalledOnce();
} finally {
renderSpy.mockRestore();
warnSpy.mockRestore();
}
});
// ── Additional tests for markdown-it migration ──
describe("www autolinks", () => {
it("links www.example.com", () => {
@@ -474,11 +329,22 @@ describe("toSanitizedMarkdownHtml", () => {
expect(html).toContain("<s>deleted</s>");
});
it("renders tables", () => {
const md = "| A | B |\n|---|---|\n| 1 | 2 |";
it("renders tables surrounded by text", () => {
const md = [
"Text before.",
"",
"| A | B |",
"|---|---|",
"| 1 | 2 |",
"",
"Text after.",
].join("\n");
const html = toSanitizedMarkdownHtml(md);
expect(html).toContain("<table");
expect(html).toContain("<th>");
expect(html).toContain("Text before.");
expect(html).toContain("Text after.");
expect(html).not.toContain("|---|");
});
it("renders basic markdown", () => {
@@ -617,9 +483,12 @@ describe("toSanitizedMarkdownHtml", () => {
});
it("caches oversized fallback results", () => {
const input = Array.from({ length: 240 }, (_, i) => `P${i}`).join("\n\n") + "x".repeat(35000);
const input =
Array.from({ length: 240 }, (_, i) => `P${i}`).join("\n\n") + "x".repeat(45_000);
const first = toSanitizedMarkdownHtml(input);
const second = toSanitizedMarkdownHtml(input);
expect(input.length).toBeGreaterThan(40_000);
expect(first).toContain('class="markdown-plain-text-fallback"');
expect(second).toBe(first);
});