fix(ci): check decoded reference paths and restore formatting (#1648)

* style(browser): format cached SVG ancestry check

* fix(tests): assert decoded reference-source error paths
This commit is contained in:
morluto
2026-10-11 06:29:26 +08:00
committed by GitHub
parent 0f605bca09
commit 54e2890b30
2 changed files with 18 additions and 8 deletions
+1 -2
View File
@@ -412,8 +412,7 @@ const elementInSvgContext = (
}
const parent = Math.trunc(parents[current] ?? -1);
if (parent < 0 || parent === current) break;
if (indexedString(strings, nodeNames[parent]) === "foreignObject")
break;
if (indexedString(strings, nodeNames[parent]) === "foreignObject") break;
current = parent;
}
// Cache the entire examined chain, including forward references and cycles,
@@ -118,10 +118,17 @@ describe("compiled reference-source import preflight failures", () => {
} else {
expect(result.json).toMatchObject(invalidRootOutput);
}
expect(JSON.stringify(result.json)).toContain(
"Check that the path exists, is readable, and points to a directory.",
const messagePath = scenario.fullOutput ? "data.message" : "message";
expect(result.json).toHaveProperty(
messagePath,
expect.stringContaining(
"Check that the path exists, is readable, and points to a directory.",
),
);
expect(result.json).toHaveProperty(
messagePath,
expect.stringContaining(root),
);
expect(JSON.stringify(result.json)).toContain(root);
expect.soft(result.exitCode).toBe(1);
if (scenario.logging) {
const records: unknown[] = result.stderr
@@ -169,9 +176,13 @@ describe("compiled reference-source import preflight failures", () => {
category: "execution_failure",
message: expect.stringContaining("EACCES: permission denied"),
});
expect(JSON.stringify(result.json)).toContain(root);
expect(JSON.stringify(result.json)).toContain(
"Check directory permissions and try again.",
expect(result.json).toHaveProperty(
"message",
expect.stringContaining(root),
);
expect(result.json).toHaveProperty(
"message",
expect.stringContaining("Check directory permissions and try again."),
);
expect(result.exitCode).toBe(1);
} finally {