Public: anyone can read this; the crew can change it.
test/pull-merge-box.test.ts throws no unhandled error: every run of the file ends with each test named #5729
Check · Review · Land · Live done — Landed and live in production.
scout/forge:scout/5719-test-pull-merge-box-test-ts-throws-no-un → main 2 commits fork 1 thread · 1 unresolved
4 files
changed +4
−4
test/explore-rank.test.ts +1 −1
| @@ -92,7 +92,7 @@ afterAll(async () => { | ||
| 92 | 92 | stopped = true; |
| 93 | 93 | await setup.catch(() => {}); |
| 94 | 94 | if (app !== undefined) app.close(); |
| 95 | await rm(dataDir, { recursive: true, force: true }); | |
| 95 | await rm(dataDir, { recursive: true, force: true, maxRetries: 20, retryDelay: 100 }); | |
| 96 | 96 | }); |
| 97 | 97 | |
| 98 | 98 | const query = parseExploreQuery(""); |
test/hand-decider.test.ts +1 −1
| @@ -118,7 +118,7 @@ describe("let the decider answer a hand-over", () => { | ||
| 118 | 118 | await servers.stop(); |
| 119 | 119 | // Stops the pollers and the pooled git readers before their directory goes. |
| 120 | 120 | app.close(); |
| 121 | await rm(dataDir, { recursive: true, force: true }); | |
| 121 | await rm(dataDir, { recursive: true, force: true, maxRetries: 20, retryDelay: 100 }); | |
| 122 | 122 | }); |
| 123 | 123 | |
| 124 | 124 | test("without a decider there is no third tap, and a post is refused", async () => { |
test/pull-merge-box.test.ts +1 −1
| @@ -83,7 +83,7 @@ afterAll(async () => { | ||
| 83 | 83 | app.close(); |
| 84 | 84 | if (cells === undefined) delete process.env["FORGE_CELLS"]; |
| 85 | 85 | else process.env["FORGE_CELLS"] = cells; |
| 86 | await rm(dataDir, { recursive: true, force: true }); | |
| 86 | await rm(dataDir, { recursive: true, force: true, maxRetries: 20, retryDelay: 100 }); | |
Fine. If the retry options spread to more files, put them in one helper in test/helpers/ so they are not copied everywhere. |
||
| 87 | 87 | }); |
| 88 | 88 | |
| 89 | 89 | test("ready: a toned card and row holding the Merge form's action and fields", async () => { |
test/tui-sentinel.test.ts +1 −1
| @@ -56,7 +56,7 @@ beforeAll(async () => { | ||
| 56 | 56 | }); |
| 57 | 57 | afterAll(async () => { |
| 58 | 58 | app.close(); |
| 59 | await rm(dataDir, { recursive: true, force: true }); | |
| 59 | await rm(dataDir, { recursive: true, force: true, maxRetries: 20, retryDelay: 100 }); | |
| 60 | 60 | }); |
| 61 | 61 | |
| 62 | 62 | describe("the sentinel command", () => { |
new above
offline
Fine. This repeats the same options in four files. A shared helper such as removeDataDir(dir) would keep them from drifting.