commit f6c4819987bc70ed2abb3da94c92646167d8b5f9
parent 418ae1b1cacca5b3280e847b0ca39121ca298d95
Author: I Mean I'm Just Saying <imeanimjustsaying@kiwifarms.st>
Date: Fri, 2 Oct 2026 00:17:16 -0400
umtool: review N2 — deliverables.spec asserts the route's 409 for a move while the cut runs
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Diffstat:
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/umtool/e2e/deliverables.spec.ts b/umtool/e2e/deliverables.spec.ts
@@ -111,7 +111,7 @@ test("the switch moves clips/ and every share-* to the media root, then says med
expect(check.decisions.filter((d: { kind: string }) => d.kind.startsWith("storage-"))).toEqual([]);
});
-test("the app cuts through the linked clips/, and the move waits for the cut", async ({ page }) => {
+test("the app cuts through the linked clips/, and the move waits for the cut", async ({ page, request }) => {
test.setTimeout(180_000);
await page.goto(`/browse/${PROJECT}`);
await expect(page.locator("[data-deliverables]")).toHaveAttribute("data-deliverables", "media");
@@ -126,6 +126,10 @@ test("the app cuts through the linked clips/, and the move waits for the cut", a
// is disabled, and says why.
await expect(move).toBeDisabled();
await expect(page.locator("[data-move-reason]")).toContainText("is running — move when it has finished");
+ // And the route refuses a move asked for any other way while the cut runs.
+ const during = await request.post("/api/report/deliver", { data: { project: PROJECT, action: "move", to: "local" } });
+ expect(during.status()).toBe(409);
+ expect(((await during.json()) as { error: string }).error).toContain("a job is already running");
const { state, log } = await waitForJob(page);
expect(state, log).toBe("done");