@torrent-tv/proxy 2.89.7 → 2.89.8

package/test/viewer-states.test.js

@@ -1,189 +1,192 @@
-/**
- * @file The three states a viewer can be in, and what each is worth.
- *
- * Written against the failure of 2026-09-14: a page whose tab was hidden for
- * 145 seconds, which had measured nothing and therefore said nothing, was
- * carried 146 seconds into a film it had not begun — and the soundtrack's
- * encoder was placed there while the browser asked for segment #0 and was
- * refused for sixty seconds.
- */
-
-import assert from "node:assert/strict";
-import test from "node:test";
-
-import { Viewer } from "../services/viewer/Viewer.js";
-import { Viewers } from "../services/viewer/Viewers.js";
-
-const AT = 1_000_000;
-
-test("a viewer who has stated nothing does not move, however long they are silent", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.moveTo(0, AT);
-
-  // 146 seconds of silence — the field figure exactly.
-  assert.equal(viewer.positionSeconds(AT + 146_000), 0);
-});
-
-test("a viewer who has stated nothing is waiting, and their work is due now", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.moveTo(0, AT);
-
-  assert.equal(viewer.playing, false);
-  assert.equal(viewer.waiting, true);
-  assert.equal(viewer.wantsFilmNow(), true);
-});
-
-test("a playing viewer moves with the film, and no further than what they hold", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.report({ bufferedAheadSec: 30, positionSeconds: 100, playing: true, waiting: false }, AT);
-
-  // Ten seconds of silence, ten seconds of film.
-  assert.equal(viewer.positionSeconds(AT + 10_000), 110);
-  // Two minutes of silence buys only the thirty seconds they said they held.
-  assert.equal(viewer.positionSeconds(AT + 120_000), 130);
-});
-
-test("a viewer holding nothing cannot move at all, even reported as playing", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.report({ bufferedAheadSec: 0, positionSeconds: 0, playing: true, waiting: false }, AT);
-
-  assert.equal(viewer.positionSeconds(AT + 146_000), 0);
-});
-
-test("a viewer who stopped the picture is not waiting, and nothing of theirs falls due", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.report({ bufferedAheadSec: 42, positionSeconds: 100, playing: false, waiting: false }, AT);
-
-  assert.equal(viewer.wantsFilmNow(), false);
-  assert.equal(viewer.positionSeconds(AT + 60_000), 100);
-});
-
-test("a viewer starved of material is waiting, and is as urgent as one playing", () => {
-  const playing = new Viewer("playing", AT);
-  playing.report({ bufferedAheadSec: 30, positionSeconds: 100, playing: true, waiting: false }, AT);
-  const starved = new Viewer("starved", AT);
-  starved.report({ bufferedAheadSec: 0, positionSeconds: 100, playing: false, waiting: true }, AT);
-
-  // Both want film now, and that is the whole point: the starved one used to
-  // count as somebody who had chosen to stop.
-  assert.equal(starved.wantsFilmNow(), true);
-  assert.equal(playing.wantsFilmNow(), true);
-  // And the starved one cannot be carried forward from where they stand,
-  // because they hold nothing to play.
-  assert.equal(starved.positionSeconds(AT + 60_000), 100);
-});
-
-test("a report without a link measurement still states everything about the viewer", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.report({ bufferedAheadSec: 0, positionSeconds: 12, playing: false, waiting: true }, AT);
-
-  assert.equal(viewer.positionSeconds(AT), 12);
-  assert.equal(viewer.waiting, true);
-  // Nothing was measured, so nothing is claimed about the link.
-  assert.equal(viewer.netReport, null);
-});
-
-test("a page one release behind, which cannot say `waiting`, is read from what it does say", () => {
-  const starved = new Viewer("starved", AT);
-  starved.report({ linkMbps: 5, bufferedAheadSec: 0, positionSeconds: 0, playing: false }, AT);
-  assert.equal(starved.waiting, true, "stopped and holding nothing is starved");
-
-  const paused = new Viewer("paused", AT);
-  paused.report({ linkMbps: 5, bufferedAheadSec: 40, positionSeconds: 0, playing: false }, AT);
-  assert.equal(paused.waiting, false, "stopped while holding film is a viewer's own pause");
-});
-
-test("a seek does not carry the cushion of the place it left", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.report({ bufferedAheadSec: 120, positionSeconds: 100, playing: true, waiting: false }, AT);
-
-  viewer.moveTo(2000, AT + 1000);
-  // Nothing is held at the new place until they say so, so the position stands.
-  assert.equal(viewer.positionSeconds(AT + 61_000), 2000);
-});
-
-test("a page that is not on screen wants nothing now, whatever else it says", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.report(
-    { bufferedAheadSec: 0, positionSeconds: 100, playing: false, waiting: true, onScreen: false },
-    AT
-  );
-
-  assert.equal(viewer.wantsFilmNow(), false);
-});
-
-test("the intake hands the statement to the viewer on the output they are watching", async () => {
-  const { recordViewerReport } = await import("../services/viewer/report-intake.js");
-  const picture = { id: "picture" };
-  const rung = { id: "rung" };
-  const sessions = new Map([
-    ["picture", picture],
-    ["rung", rung]
-  ]);
-  const viewers = new Viewers();
-  viewers.of(picture, "one", AT).activeVariantId = "rung";
-
-  // The page always addresses the PICTURE — it is never told which rung it is
-  // on — and the statement belongs to the rung actually on screen.
-  const taken = recordViewerReport({
-    outputs: sessions,
-    viewers,
-    sessionId: "picture",
-    report: { linkMbps: 8, bufferedAheadSec: 10, positionSeconds: 5, playing: true, consumerId: "one" },
-    now: AT
-  });
-
-  assert.equal(taken, true);
-  assert.equal(viewers.forOutput(rung).get("one").positionSeconds(AT), 5);
-  assert.equal(viewers.forOutput(picture).has("one"), true);
-});
-
-test("a link reading does not expire — presence is what decides whose it is", () => {
-  const viewer = new Viewer("one", AT);
-  viewer.report({ linkMbps: 8, bufferedAheadSec: 10, positionSeconds: 5, playing: true }, AT);
-
-  // A link does not stop being what it was measured to be because nobody
-  // measured it for a while. The page says as much by keeping its own last
-  // figure rather than reporting nothing.
-  assert.equal(viewer.linkReading()?.linkMbps, 8);
-  assert.equal(viewer.linkReading()?.linkMbps, 8, "and an hour later it is still the last thing known");
-
-  // What ends it is the person leaving, which is a different fact with a
-  // different owner.
-  assert.equal(viewer.isPresent(), true);
-});
-
-test("each viewer's link is read for that viewer alone, never as a worst over several", async () => {
-  // Roadmap item 97, step 11: a thin link decides for the person on it. The
-  // worst reading across everybody watching used to decide for all of them.
-  const { linkReportOf, presentOn } = await import("../services/viewer/choices.js");
-  const output = { id: "out" };
-  const viewers = new Viewers();
-  const slow = viewers.of(output, "slow", AT);
-  const fast = viewers.of(output, "fast", AT);
-  slow.report({ linkMbps: 2, bufferedAheadSec: 3, playing: true }, AT);
-  fast.report({ linkMbps: 40, bufferedAheadSec: 90, playing: true }, AT);
-
-  assert.equal(linkReportOf(viewers, output, "slow")?.linkMbps, 2);
-  assert.equal(linkReportOf(viewers, output, "fast")?.linkMbps, 40, "the slow link says nothing about this one");
-  assert.equal(linkReportOf(viewers, output, "nobody"), null);
-
-  // The slow one leaves: they are no longer among the viewers asked about.
-  slow.gone = true;
-  assert.deepEqual(presentOn(viewers, output), ["fast"]);
-});
-
-test("a report into an output absent from the live registry is refused rather than invented", async () => {
-  const { recordViewerReport } = await import("../services/viewer/report-intake.js");
-  const sessions = new Map();
-  const viewers = { of() { throw new Error("must not be asked"); } };
-
-  assert.equal(
-    recordViewerReport({ outputs: sessions, viewers, sessionId: "dead", report: { bufferedAheadSec: 0 }, now: AT }),
-    false
-  );
-  assert.equal(
-    recordViewerReport({ outputs: sessions, viewers, sessionId: "never", report: { bufferedAheadSec: 0 }, now: AT }),
-    false
-  );
-});
+/**
+ * @file The three states a viewer can be in, and what each is worth.
+ *
+ * Written against the failure of 2026-09-14: a page whose tab was hidden for
+ * 145 seconds, which had measured nothing and therefore said nothing, was
+ * carried 146 seconds into a film it had not begun — and the soundtrack's
+ * encoder was placed there while the browser asked for segment #0 and was
+ * refused for sixty seconds.
+ */
+
+import assert from "node:assert/strict";
+import test from "node:test";
+
+import { Viewer } from "../services/viewer/Viewer.js";
+import { Viewers } from "../services/viewer/Viewers.js";
+
+const AT = 1_000_000;
+
+test("a viewer who has stated nothing does not move, however long they are silent", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.moveTo(0, AT);
+
+  // 146 seconds of silence — the field figure exactly.
+  assert.equal(viewer.positionSeconds(AT + 146_000), 0);
+});
+
+test("a viewer who has stated nothing is waiting, and their work is due now", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.moveTo(0, AT);
+
+  assert.equal(viewer.playing, false);
+  assert.equal(viewer.waiting, true);
+  assert.equal(viewer.wantsFilmNow(), true);
+});
+
+test("a playing viewer moves with the film, and no further than what they hold", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.report({ bufferedAheadSec: 30, positionSeconds: 100, playing: true, waiting: false }, AT);
+
+  // Ten seconds of silence, ten seconds of film.
+  assert.equal(viewer.positionSeconds(AT + 10_000), 110);
+  // Two minutes of silence buys only the thirty seconds they said they held.
+  assert.equal(viewer.positionSeconds(AT + 120_000), 130);
+});
+
+test("a viewer holding nothing cannot move at all, even reported as playing", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.report({ bufferedAheadSec: 0, positionSeconds: 0, playing: true, waiting: false }, AT);
+
+  assert.equal(viewer.positionSeconds(AT + 146_000), 0);
+});
+
+test("a viewer who stopped the picture is not waiting, and nothing of theirs falls due", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.report({ bufferedAheadSec: 42, positionSeconds: 100, playing: false, waiting: false }, AT);
+
+  assert.equal(viewer.wantsFilmNow(), false);
+  assert.equal(viewer.positionSeconds(AT + 60_000), 100);
+});
+
+test("a viewer starved of material is waiting, and is as urgent as one playing", () => {
+  const playing = new Viewer("playing", AT);
+  playing.report({ bufferedAheadSec: 30, positionSeconds: 100, playing: true, waiting: false }, AT);
+  const starved = new Viewer("starved", AT);
+  starved.report({ bufferedAheadSec: 0, positionSeconds: 100, playing: false, waiting: true }, AT);
+
+  // Both want film now, and that is the whole point: the starved one used to
+  // count as somebody who had chosen to stop.
+  assert.equal(starved.wantsFilmNow(), true);
+  assert.equal(playing.wantsFilmNow(), true);
+  // And the starved one cannot be carried forward from where they stand,
+  // because they hold nothing to play.
+  assert.equal(starved.positionSeconds(AT + 60_000), 100);
+});
+
+test("a report without a link measurement still states everything about the viewer", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.report({ bufferedAheadSec: 0, positionSeconds: 12, playing: false, waiting: true }, AT);
+
+  assert.equal(viewer.positionSeconds(AT), 12);
+  assert.equal(viewer.waiting, true);
+  // The report is kept from the first one the page sends (proxy 2.86.0), and
+  // since nothing was measured, nothing is claimed about the link.
+  assert.equal(viewer.netReport.linkMbps, null);
+  assert.equal(viewer.netReport.linkSampleMbps, null);
+  assert.equal(viewer.netReport.positionSeconds, 12);
+});
+
+test("a page one release behind, which cannot say `waiting`, is read from what it does say", () => {
+  const starved = new Viewer("starved", AT);
+  starved.report({ linkMbps: 5, bufferedAheadSec: 0, positionSeconds: 0, playing: false }, AT);
+  assert.equal(starved.waiting, true, "stopped and holding nothing is starved");
+
+  const paused = new Viewer("paused", AT);
+  paused.report({ linkMbps: 5, bufferedAheadSec: 40, positionSeconds: 0, playing: false }, AT);
+  assert.equal(paused.waiting, false, "stopped while holding film is a viewer's own pause");
+});
+
+test("a seek does not carry the cushion of the place it left", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.report({ bufferedAheadSec: 120, positionSeconds: 100, playing: true, waiting: false }, AT);
+
+  viewer.moveTo(2000, AT + 1000);
+  // Nothing is held at the new place until they say so, so the position stands.
+  assert.equal(viewer.positionSeconds(AT + 61_000), 2000);
+});
+
+test("a page that is not on screen wants nothing now, whatever else it says", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.report(
+    { bufferedAheadSec: 0, positionSeconds: 100, playing: false, waiting: true, onScreen: false },
+    AT
+  );
+
+  assert.equal(viewer.wantsFilmNow(), false);
+});
+
+test("the intake hands the statement to the viewer on the output they are watching", async () => {
+  const { recordViewerReport } = await import("../services/viewer/report-intake.js");
+  const picture = { id: "picture" };
+  const rung = { id: "rung" };
+  const sessions = new Map([
+    ["picture", picture],
+    ["rung", rung]
+  ]);
+  const viewers = new Viewers();
+  viewers.of(picture, "one", AT).activeVariantId = "rung";
+
+  // The page always addresses the PICTURE — it is never told which rung it is
+  // on — and the statement belongs to the rung actually on screen.
+  const taken = recordViewerReport({
+    outputs: sessions,
+    viewers,
+    sessionId: "picture",
+    report: { linkMbps: 8, bufferedAheadSec: 10, positionSeconds: 5, playing: true, consumerId: "one" },
+    now: AT
+  });
+
+  assert.equal(taken, true);
+  assert.equal(viewers.forOutput(rung).get("one").positionSeconds(AT), 5);
+  assert.equal(viewers.forOutput(picture).has("one"), true);
+});
+
+test("a link reading does not expire — presence is what decides whose it is", () => {
+  const viewer = new Viewer("one", AT);
+  viewer.report({ linkMbps: 8, bufferedAheadSec: 10, positionSeconds: 5, playing: true }, AT);
+
+  // A link does not stop being what it was measured to be because nobody
+  // measured it for a while. The page says as much by keeping its own last
+  // figure rather than reporting nothing.
+  assert.equal(viewer.linkReading()?.linkMbps, 8);
+  assert.equal(viewer.linkReading()?.linkMbps, 8, "and an hour later it is still the last thing known");
+
+  // What ends it is the person leaving, which is a different fact with a
+  // different owner.
+  assert.equal(viewer.isPresent(), true);
+});
+
+test("each viewer's link is read for that viewer alone, never as a worst over several", async () => {
+  // Roadmap item 97, step 11: a thin link decides for the person on it. The
+  // worst reading across everybody watching used to decide for all of them.
+  const { linkReportOf, presentOn } = await import("../services/viewer/choices.js");
+  const output = { id: "out" };
+  const viewers = new Viewers();
+  const slow = viewers.of(output, "slow", AT);
+  const fast = viewers.of(output, "fast", AT);
+  slow.report({ linkMbps: 2, bufferedAheadSec: 3, playing: true }, AT);
+  fast.report({ linkMbps: 40, bufferedAheadSec: 90, playing: true }, AT);
+
+  assert.equal(linkReportOf(viewers, output, "slow")?.linkMbps, 2);
+  assert.equal(linkReportOf(viewers, output, "fast")?.linkMbps, 40, "the slow link says nothing about this one");
+  assert.equal(linkReportOf(viewers, output, "nobody"), null);
+
+  // The slow one leaves: they are no longer among the viewers asked about.
+  slow.gone = true;
+  assert.deepEqual(presentOn(viewers, output), ["fast"]);
+});
+
+test("a report into an output absent from the live registry is refused rather than invented", async () => {
+  const { recordViewerReport } = await import("../services/viewer/report-intake.js");
+  const sessions = new Map();
+  const viewers = { of() { throw new Error("must not be asked"); } };
+
+  assert.equal(
+    recordViewerReport({ outputs: sessions, viewers, sessionId: "dead", report: { bufferedAheadSec: 0 }, now: AT }),
+    false
+  );
+  assert.equal(
+    recordViewerReport({ outputs: sessions, viewers, sessionId: "never", report: { bufferedAheadSec: 0 }, now: AT }),
+    false
+  );
+});

package/test/viewer-subtitle-subscription.test.js

@@ -1,96 +1,96 @@
-/**
- * @file Who wants subtitle cues, and why the answer is a person and not a channel.
- *
- * The subscription used to be a `Set<DataChannel>` inside the transport, keyed
- * by `sourceKey:fileIndex`. Two things followed from that, and both are field
- * facts rather than worries:
- *
- * 1. a seamless reconnect lost subtitles for the rest of the session, because
- *    the new channel is a different object and nothing re-subscribed it;
- * 2. the transport had to sniff `/api/subtitles` out of the request path and
- *    derive a torrent key to build that key at all — application routing and
- *    torrent identity, inside the layer that is only supposed to carry bytes.
- *
- * Held against the viewer instead, both go away by construction: whatever
- * channel the person is reachable on next, they are still subscribed, and the
- * transport never learns what a subtitle is.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-
-import { Viewers } from "../services/viewer/Viewers.js";
-
-/** An output is only ever held by id here, so the thinnest possible stand-in. */
-const anOutput = (id) => ({ id, viewers: new Set() });
-
-test("a viewer who switched subtitles on is listed for that file", () => {
-  const viewers = new Viewers();
-  viewers.of(anOutput("picture"), "alice");
-
-  assert.equal(viewers.wantsCues("alice", "src-1", 3), true);
-  assert.deepEqual(viewers.wantingCues("src-1", 3), ["alice"]);
-});
-
-test("nobody is listed for a file nobody asked about", () => {
-  const viewers = new Viewers();
-  viewers.of(anOutput("picture"), "alice");
-  viewers.wantsCues("alice", "src-1", 3);
-
-  assert.deepEqual(viewers.wantingCues("src-1", 4), []);
-  assert.deepEqual(viewers.wantingCues("src-2", 3), []);
-});
-
-test("two viewers of one file are both listed, and one file each is kept apart", () => {
-  const viewers = new Viewers();
-  const picture = anOutput("picture");
-  viewers.of(picture, "alice");
-  viewers.of(picture, "bob");
-
-  viewers.wantsCues("alice", "src-1", 0);
-  viewers.wantsCues("bob", "src-1", 0);
-  viewers.wantsCues("bob", "src-1", 1);
-
-  assert.deepEqual(viewers.wantingCues("src-1", 0).sort(), ["alice", "bob"]);
-  assert.deepEqual(viewers.wantingCues("src-1", 1), ["bob"]);
-});
-
-test("the subscription outlives the connection, which is the whole point", () => {
-  // Nothing here mentions a channel, and that is the assertion: the registry
-  // has no way to express "this channel", so a channel dying cannot take a
-  // subscription with it.
-  const viewers = new Viewers();
-  viewers.of(anOutput("picture"), "alice");
-  viewers.wantsCues("alice", "src-1", 3);
-
-  // A reconnect is, from here, nothing at all: the same person is still known.
-  viewers.seen("alice");
-
-  assert.deepEqual(viewers.wantingCues("src-1", 3), ["alice"]);
-});
-
-test("a viewer who has gone is no longer listed", () => {
-  const viewers = new Viewers();
-  const picture = anOutput("picture");
-  viewers.of(picture, "alice");
-  viewers.wantsCues("alice", "src-1", 3);
-
-  viewers.hasGone("alice");
-
-  assert.deepEqual(viewers.wantingCues("src-1", 3), []);
-});
-
-test("an unknown name subscribes to nothing", () => {
-  const viewers = new Viewers();
-  assert.equal(viewers.wantsCues("nobody", "src-1", 3), false);
-  assert.deepEqual(viewers.wantingCues("src-1", 3), []);
-});
-
-test("a request missing the file index registers nothing", () => {
-  const viewers = new Viewers();
-  viewers.of(anOutput("picture"), "alice");
-
-  assert.equal(viewers.wantsCues("alice", "src-1", Number.NaN), false);
-  assert.equal(viewers.wantsCues("alice", "", 3), false);
-  assert.deepEqual(viewers.wantingCues("src-1", 3), []);
-});
+/**
+ * @file Who wants subtitle cues, and why the answer is a person and not a channel.
+ *
+ * The subscription used to be a `Set<DataChannel>` inside the transport, keyed
+ * by `sourceKey:fileIndex`. Two things followed from that, and both are field
+ * facts rather than worries:
+ *
+ * 1. a seamless reconnect lost subtitles for the rest of the session, because
+ *    the new channel is a different object and nothing re-subscribed it;
+ * 2. the transport had to sniff `/api/subtitles` out of the request path and
+ *    derive a torrent key to build that key at all — application routing and
+ *    torrent identity, inside the layer that is only supposed to carry bytes.
+ *
+ * Held against the viewer instead, both go away by construction: whatever
+ * channel the person is reachable on next, they are still subscribed, and the
+ * transport never learns what a subtitle is.
+ */
+
+import test from "node:test";
+import assert from "node:assert/strict";
+
+import { Viewers } from "../services/viewer/Viewers.js";
+
+/** An output is only ever held by id here, so the thinnest possible stand-in. */
+const anOutput = (id) => ({ id, viewers: new Set() });
+
+test("a viewer who switched subtitles on is listed for that file", () => {
+  const viewers = new Viewers();
+  viewers.of(anOutput("picture"), "alice");
+
+  assert.equal(viewers.wantsCues("alice", "src-1", 3), true);
+  assert.deepEqual(viewers.wantingCues("src-1", 3), ["alice"]);
+});
+
+test("nobody is listed for a file nobody asked about", () => {
+  const viewers = new Viewers();
+  viewers.of(anOutput("picture"), "alice");
+  viewers.wantsCues("alice", "src-1", 3);
+
+  assert.deepEqual(viewers.wantingCues("src-1", 4), []);
+  assert.deepEqual(viewers.wantingCues("src-2", 3), []);
+});
+
+test("two viewers of one file are both listed, and one file each is kept apart", () => {
+  const viewers = new Viewers();
+  const picture = anOutput("picture");
+  viewers.of(picture, "alice");
+  viewers.of(picture, "bob");
+
+  viewers.wantsCues("alice", "src-1", 0);
+  viewers.wantsCues("bob", "src-1", 0);
+  viewers.wantsCues("bob", "src-1", 1);
+
+  assert.deepEqual(viewers.wantingCues("src-1", 0).sort(), ["alice", "bob"]);
+  assert.deepEqual(viewers.wantingCues("src-1", 1), ["bob"]);
+});
+
+test("the subscription outlives the connection, which is the whole point", () => {
+  // Nothing here mentions a channel, and that is the assertion: the registry
+  // has no way to express "this channel", so a channel dying cannot take a
+  // subscription with it.
+  const viewers = new Viewers();
+  viewers.of(anOutput("picture"), "alice");
+  viewers.wantsCues("alice", "src-1", 3);
+
+  // A reconnect is, from here, nothing at all: the same person is still known.
+  viewers.seen("alice");
+
+  assert.deepEqual(viewers.wantingCues("src-1", 3), ["alice"]);
+});
+
+test("a viewer who has gone is no longer listed", () => {
+  const viewers = new Viewers();
+  const picture = anOutput("picture");
+  viewers.of(picture, "alice");
+  viewers.wantsCues("alice", "src-1", 3);
+
+  viewers.hasGone("alice");
+
+  assert.deepEqual(viewers.wantingCues("src-1", 3), []);
+});
+
+test("an unknown name subscribes to nothing", () => {
+  const viewers = new Viewers();
+  assert.equal(viewers.wantsCues("nobody", "src-1", 3), false);
+  assert.deepEqual(viewers.wantingCues("src-1", 3), []);
+});
+
+test("a request missing the file index registers nothing", () => {
+  const viewers = new Viewers();
+  viewers.of(anOutput("picture"), "alice");
+
+  assert.equal(viewers.wantsCues("alice", "src-1", Number.NaN), false);
+  assert.equal(viewers.wantsCues("alice", "", 3), false);
+  assert.deepEqual(viewers.wantingCues("src-1", 3), []);
+});