From 099579a32b4cc2454deb044436a11aa362b120ee Mon Sep 17 00:00:00 2001 From: Roo Code Date: Thu, 15 Jan 2026 20:34:31 +0000 Subject: [PATCH] fix: add soft assertion for list format validation in markdown-lists test --- apps/vscode-e2e/src/suite/markdown-lists.test.ts | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/apps/vscode-e2e/src/suite/markdown-lists.test.ts b/apps/vscode-e2e/src/suite/markdown-lists.test.ts index c3e2a770e3..066960b6b5 100644 --- a/apps/vscode-e2e/src/suite/markdown-lists.test.ts +++ b/apps/vscode-e2e/src/suite/markdown-lists.test.ts @@ -35,16 +35,25 @@ suite("Markdown List Rendering", function () { assert.ok(allText.includes("Banana"), "Response should mention Banana") assert.ok(allText.includes("Orange"), "Response should mention Orange") - // OPTIONAL: Check for list formatting (log but don't fail) + // SOFT ASSERTION: Validate at least one common list format is present + // Flexible about which format (dash, asterisk, or bullet) but requires some list formatting const hasListFormat = allText.includes("- ") || allText.includes("* ") || allText.includes("• ") || allText.match(/^\s*[-*•]/m) + // Log which format was detected for debugging if (hasListFormat) { console.log("✓ AI used list formatting") } else { console.log("⚠ AI did not use traditional list formatting") console.log("Response format:", allText.substring(0, 200)) } + + // Assert that at least one list format is present (soft assertion - flexible about which format) + assert.ok( + hasListFormat, + "Response should contain at least one list format (dash, asterisk, or bullet). " + + "The test is flexible about which format is used, but requires some list formatting.", + ) }) test("Should render ordered lists with numbers in chat", async () => {