commit 9c96c35f1aae9009bd43de239d23d8be77e92416
parent 870d26ff9fa9941e203b2a51b7ebea58d5e05798
Author: I Mean I'm Just Saying <imeanimjustsaying@kiwifarms.st>
Date: Thu, 1 Oct 2026 23:49:51 -0400
umtool: deliverables.spec — a batch's count includes the ids its LIST.md says were already shared
share-second's LIST.md names d01 and d02 and says d03 was "already shared";
sharedIdsIn reads both (by design), so the panel lists it as (3).
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Diffstat:
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/umtool/e2e/deliverables.spec.ts b/umtool/e2e/deliverables.spec.ts
@@ -168,9 +168,11 @@ test("a new batch: refused by an app with no media root, made as a link by a CLI
expect(list).not.toContain("d03_");
expect(list).toContain("1 already shared (d03)");
- // And the panel lists it, through its link.
+ // And the panel lists it, through its link. Three ids, not two: a batch's
+ // LIST.md is read for the ids it says were "already shared" as well as the
+ // files it names (deliver.mjs sharedIdsIn), so d03 counts as shipped here too.
await page.reload();
- await expect(page.locator('[data-batch="share-second"]')).toContainText("(2)");
+ await expect(page.locator('[data-batch="share-second"]')).toContainText("share-second (3)");
await expect(page.locator('[data-deliverable="share-second"]')).toHaveAttribute("data-deliverable-state", "link");
});