Skip to content

Commit 6b78211

Browse files
committed
test(dsh-plugin): drop redundant thumbnail read-back case
Coverage already lives in the attachment-store and ring tests; this case only re-hit readThumbnail and was flaky on CI.
1 parent eebf334 commit 6b78211

1 file changed

Lines changed: 0 additions & 16 deletions

File tree

‎packages/dsh-plugin-browserskill/tests/observation.test.ts‎

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -525,22 +525,6 @@ describe("availability and dead sessions", () => {
525525
});
526526
});
527527

528-
describe("thumbnail read-back", () => {
529-
it("serves captured frames through readThumbnail and 404s unknown ids", async () => {
530-
const scheduler = fakeScheduler();
531-
const runner = fakeRunner();
532-
const { service } = setup({ runner, scheduler });
533-
service.addSession("s1");
534-
scheduler.runNext();
535-
await waitFor(() => service.getState()[0]?.thumbnailAttachmentId !== undefined);
536-
const frame = await service.readThumbnail("obs-s1-1");
537-
expect(frame?.mediaType).toBe("image/png");
538-
expect([...(frame?.data ?? [])]).toEqual([0x89, 0x50, 0x4e, 0x47]);
539-
expect(await service.readThumbnail("nope")).toBeUndefined();
540-
service.dispose();
541-
});
542-
});
543-
544528
describe("actionForLabel", () => {
545529
it("maps tool labels onto observation verbs", () => {
546530
expect(actionForLabel("navigate")).toBe("navigating");

0 commit comments

Comments
 (0)