From 583345fdfee63e6b99b0a7c9eeb6c8385c34f25b Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 07:09:31 +0000 Subject: [PATCH] test: collapse redundant markdown conversion micro-tests --- src/line/markdown-to-line.test.ts | 41 ++++++++++--------------------- 1 file changed, 13 insertions(+), 28 deletions(-) diff --git a/src/line/markdown-to-line.test.ts b/src/line/markdown-to-line.test.ts index d5e3ca75d..7deff0b6e 100644 --- a/src/line/markdown-to-line.test.ts +++ b/src/line/markdown-to-line.test.ts @@ -261,9 +261,6 @@ describe("convertCodeBlockToFlexBubble", () => { const bubble = convertCodeBlockToFlexBubble(block); - expect(bubble.type).toBe("bubble"); - expect(bubble.body).toBeDefined(); - const body = bubble.body as { contents: Array<{ text: string }> }; expect(body.contents[0].text).toBe("Code (typescript)"); }); @@ -303,7 +300,6 @@ Done.`; const result = processLineMessage(text); expect(result.flexMessages).toHaveLength(1); - expect(result.flexMessages[0].type).toBe("flex"); expect(result.text).toContain("Here's the data:"); expect(result.text).toContain("Done."); expect(result.text).not.toContain("|"); @@ -368,32 +364,21 @@ print("done") }); describe("hasMarkdownToConvert", () => { - it("detects tables", () => { - const text = `| A | B | + it("detects supported markdown patterns", () => { + const cases = [ + `| A | B | |---|---| -| 1 | 2 |`; - expect(hasMarkdownToConvert(text)).toBe(true); - }); +| 1 | 2 |`, + "```js\ncode\n```", + "**bold**", + "~~deleted~~", + "# Title", + "> quote", + ]; - it("detects code blocks", () => { - const text = "```js\ncode\n```"; - expect(hasMarkdownToConvert(text)).toBe(true); - }); - - it("detects bold", () => { - expect(hasMarkdownToConvert("**bold**")).toBe(true); - }); - - it("detects strikethrough", () => { - expect(hasMarkdownToConvert("~~deleted~~")).toBe(true); - }); - - it("detects headers", () => { - expect(hasMarkdownToConvert("# Title")).toBe(true); - }); - - it("detects blockquotes", () => { - expect(hasMarkdownToConvert("> quote")).toBe(true); + for (const text of cases) { + expect(hasMarkdownToConvert(text)).toBe(true); + } }); it("returns false for plain text", () => {