smashimo/oh-my-pi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: bump version to 17.1.2

Browse Source 
- fixed eval preview windowing fixture to use valid identifiers so the display formatter's operator spacing does not rewrite the asserted lines
This commit is contained in:
can1357
2026-07-24 17:23:19 +02:00
parent 7ebf141743
commit a38cd95d7d
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/coding-agent/test/streaming-preview-height.test.ts
+7 -5
View File
@@ -457,7 +457,9 @@ describe("streaming tool call preview height (bounded across renderers)", () =>
const window = previewWindowRows();
const total = window + 5;
const hidden = total - window;
const longLines = Array.from({ length: total }, (_, i) => `line-${i}`);
// Underscore identifiers: the display formatter would space `line-1` as a
// subtraction, and this test asserts windowing, not operator layout.
const longLines = Array.from({ length: total }, (_, i) => `line_${i}`);
const { lines, text } = renderPending("eval", {
language: "js",
title: "big",
@@ -466,10 +468,10 @@ describe("streaming tool call preview height (bounded across renderers)", () =>
expect(lines.length, "eval code preview should stay bounded").toBeLessThan(window + 10);
const renderedLines = getRenderedLines(lines);
expect(renderedLines).toContain(`const line-${total - 1} = 1;`);
expect(renderedLines).toContain(`const line-${hidden} = 1;`);
expect(renderedLines).not.toContain("const line-0 = 1;");
expect(renderedLines).not.toContain(`const line-${hidden - 1} = 1;`);
expect(renderedLines).toContain(`const line_${total - 1} = 1;`);
expect(renderedLines).toContain(`const line_${hidden} = 1;`);
expect(renderedLines).not.toContain("const line_0 = 1;");
expect(renderedLines).not.toContain(`const line_${hidden - 1} = 1;`);
expect(text).toContain(`… ${hidden} earlier lines`);
}, 30_000);
});