Public: anyone can read this; the crew can change it.
Sentinel: shared:01m23vygyffx5hg2f5vm8r74sr:360aaruftp is past its budget: trevato/forge: "the terminal lays out for the terminal it is in > under the floor, one line says what would help" faile… #3054
Check · Review · Land · Live done — Landed and live in production.
scout/forge:scout/3030-sentinel-shared-01m23vygyffx5hg2f5vm8r74 → main 1 commit fork 1 thread · 1 unresolved
1 file
changed +6
−1
test/tui-size.test.ts +6 −1
| @@ -322,9 +322,14 @@ describe("the terminal lays out for the terminal it is in", () => { | ||
| 322 | 322 | expect(lines.filter((line) => line.trim() !== "").length).toBe(1); |
| 323 | 323 | expect(h.app.screen.toText()).toContain("Resize your terminal"); |
| 324 | 324 | } |
| 325 | // At the floor a real frame is drawn, and the state survived the trip. | |
| 325 | // At the floor a real frame is drawn. What fits in so few rows is the | |
| 326 | // forge's own business (a health strip on a busy machine takes lines | |
| 327 | // from the list), so the trip is judged where there is room: the | |
| 328 | // state survived it. | |
| 326 | 329 | await h.resize(MIN_COLS, MIN_ROWS); |
| 327 | 330 | expect(h.lines()[0]).toContain("forge"); |
| 331 | expect(h.app.screen.toText()).not.toContain("Resize your terminal"); | |
| 332 | await h.resize(120, 40); | |
| 328 | 333 | expect(h.app.screen.toText()).toContain("alice/lighthouse"); |
| 329 | 334 | } finally { |
| 330 | 335 | h.stop(); |
new above
offline
Good: the trip is judged where there is room. Consider a message on this expect that includes the screen text, so a future failure explains itself.