@torrent-tv/proxy 2.89.17 → 2.89.18

package/test/piece-from-whole-file.test.js

@@ -1,129 +0,0 @@
-/**
- * @file A piece read out of the files it was assembled into.
- *
- * Without this a whole file is a second copy of bytes the piece store also
- * holds, and neither copy can be dropped. With it the spilled copy is a
- * duplicate and can go — one episode on the field host of 2026-09-11 was
- * 1417 MB of segments beside 1424 MB of spilled pieces — and a torrent can be
- * destroyed and added again without fetching a byte.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import fs from "node:fs/promises";
-import os from "node:os";
-import path from "node:path";
-import { pieceFromWholeFiles, pieceIsInWholeFiles } from "../services/storage/files/piece-from-whole-file.js";
-
-const PIECE = 16;
-
-/**
- * Two files laid end to end, as a torrent lays them out, with a piece straddling
- * the boundary between them.
- *
- * @returns {Promise<{ root: string, files: object[], length: number, bytes: Buffer }>}
- */
-async function twoFiles() {
-  const root = await fs.mkdtemp(path.join(os.tmpdir(), "piece-of-whole-"));
-  const first = Buffer.alloc(20, 1);
-  const second = Buffer.alloc(24, 2);
-  await fs.writeFile(path.join(root, "0"), first);
-  await fs.writeFile(path.join(root, "1"), second);
-  return {
-    root,
-    files: [
-      { offset: 0, length: first.length },
-      { offset: first.length, length: second.length }
-    ],
-    length: first.length + second.length,
-    bytes: Buffer.concat([first, second])
-  };
-}
-
-/**
- * @param {string} root
- * @param {number[]} whole - Which file indexes this proxy holds whole.
- * @returns {(fileIndex: number) => { path: string, length: number } | null}
- */
-const holds = (root, whole) => (fileIndex) =>
-  whole.includes(fileIndex) ? { path: path.join(root, String(fileIndex)), length: 0 } : null;
-
-test("a piece inside one file comes back byte for byte", async () => {
-  const { root, files, length, bytes } = await twoFiles();
-  try {
-    const piece = await pieceFromWholeFiles({
-      index: 0,
-      pieceLength: PIECE,
-      length,
-      files,
-      wholeFileAt: holds(root, [0, 1])
-    });
-    assert.deepEqual(piece, bytes.subarray(0, PIECE));
-  } finally {
-    await fs.rm(root, { recursive: true, force: true, maxRetries: 10, retryDelay: 20 });
-  }
-});
-
-test("a piece straddling two files is stitched from both", async () => {
-  const { root, files, length, bytes } = await twoFiles();
-  try {
-    // Piece 1 covers bytes 16..31: four from the first file, twelve from the
-    // second.
-    const piece = await pieceFromWholeFiles({
-      index: 1,
-      pieceLength: PIECE,
-      length,
-      files,
-      wholeFileAt: holds(root, [0, 1])
-    });
-    assert.deepEqual(piece, bytes.subarray(PIECE, PIECE * 2));
-  } finally {
-    await fs.rm(root, { recursive: true, force: true, maxRetries: 10, retryDelay: 20 });
-  }
-});
-
-test("the last piece is read to the end of the torrent and no further", async () => {
-  const { root, files, length, bytes } = await twoFiles();
-  try {
-    // 44 bytes in pieces of 16: the last piece is 12 long.
-    const piece = await pieceFromWholeFiles({
-      index: 2,
-      pieceLength: PIECE,
-      length,
-      files,
-      wholeFileAt: holds(root, [0, 1])
-    });
-    assert.equal(piece.length, 12);
-    assert.deepEqual(piece, bytes.subarray(32));
-  } finally {
-    await fs.rm(root, { recursive: true, force: true, maxRetries: 10, retryDelay: 20 });
-  }
-});
-
-test("a piece any part of which is not held whole is refused, not half read", async () => {
-  const { root, files, length } = await twoFiles();
-  try {
-    // Only the first file is here; piece 1 straddles both. Half a piece is
-    // worse than none: the layer above would hash it and mark it bad.
-    const piece = await pieceFromWholeFiles({
-      index: 1,
-      pieceLength: PIECE,
-      length,
-      files,
-      wholeFileAt: holds(root, [0])
-    });
-    assert.equal(piece, null);
-    assert.equal(
-      pieceIsInWholeFiles({ index: 1, pieceLength: PIECE, length, files, wholeFileAt: holds(root, [0]) }),
-      false
-    );
-    // And one wholly inside the file that IS held is both readable and known to
-    // be a duplicate of what is on the spill.
-    assert.equal(
-      pieceIsInWholeFiles({ index: 0, pieceLength: PIECE, length, files, wholeFileAt: holds(root, [0]) }),
-      true
-    );
-  } finally {
-    await fs.rm(root, { recursive: true, force: true, maxRetries: 10, retryDelay: 20 });
-  }
-});

package/test/piece-lru.test.js

@@ -1,319 +0,0 @@
-/**
- * @file Recency and pinning for resident pieces.
- *
- * The pinning cases matter more than the ordering ones: proxy 2.9.71 removed a
- * torrent's data while a reader was mid-read, and every later read hung. Here
- * that is meant to be impossible by construction, so it is worth stating in
- * tests rather than trusting to eviction order.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { PieceLru } from "../services/storage/piece-store/piece-lru.js";
-
-test("evicts the least recently used piece", () => {
-  const lru = new PieceLru(3);
-  lru.touch(1);
-  lru.touch(2);
-  lru.touch(3);
-
-  assert.equal(lru.evictionCandidate(), 1);
-});
-
-test("using a piece again moves it out of the firing line", () => {
-  const lru = new PieceLru(3);
-  lru.touch(1);
-  lru.touch(2);
-  lru.touch(3);
-  lru.touch(1);
-
-  assert.equal(lru.evictionCandidate(), 2);
-});
-
-test("a pinned piece is never offered for eviction", () => {
-  const lru = new PieceLru(3);
-  lru.touch(1);
-  lru.touch(2);
-  lru.touch(3);
-  lru.pin(1);
-
-  assert.equal(lru.evictionCandidate(), 2, "the pinned piece was offered up");
-});
-
-test("pins nest, so one reader leaving does not expose the piece", () => {
-  const lru = new PieceLru(2);
-  lru.touch(1);
-  lru.touch(2);
-
-  // Two sessions reading the same piece — the union-window case.
-  lru.pin(1);
-  lru.pin(1);
-  lru.unpin(1);
-
-  assert.ok(lru.isPinned(1), "the piece stopped being held while a reader still had it");
-  assert.equal(lru.evictionCandidate(), 2);
-
-  lru.unpin(1);
-  assert.equal(lru.isPinned(1), false);
-});
-
-test("reports no candidate rather than evicting a piece in use", () => {
-  const lru = new PieceLru(2);
-  lru.touch(1);
-  lru.touch(2);
-  lru.pin(1);
-  lru.pin(2);
-
-  assert.equal(
-    lru.evictionCandidate(),
-    null,
-    "with every piece held, the caller must wait — not have memory taken from under it"
-  );
-});
-
-test("unpinning something that was never pinned is harmless", () => {
-  const lru = new PieceLru(2);
-  lru.touch(1);
-  lru.unpin(1);
-  lru.unpin(1);
-
-  assert.equal(lru.isPinned(1), false);
-  assert.equal(lru.evictionCandidate(), 1);
-});
-
-test("removal takes a piece out of the ordering", () => {
-  const lru = new PieceLru(3);
-  lru.touch(1);
-  lru.touch(2);
-  lru.remove(1);
-
-  assert.equal(lru.has(1), false);
-  assert.equal(lru.evictionCandidate(), 2);
-  assert.equal(lru.size, 1);
-});
-
-test("fullness follows capacity", () => {
-  const lru = new PieceLru(2);
-  assert.equal(lru.isFull(), false);
-  lru.touch(1);
-  lru.touch(2);
-  assert.equal(lru.isFull(), true);
-  lru.remove(1);
-  assert.equal(lru.isFull(), false);
-});
-
-test("a nonsensical capacity is refused at construction", () => {
-  assert.throws(() => new PieceLru(0), /positive integer/);
-  assert.throws(() => new PieceLru(-1), /positive integer/);
-  assert.throws(() => new PieceLru(1.5), /positive integer/);
-});
-
-test("a declared window is the last thing evicted, not the first", () => {
-  const lru = new PieceLru(4);
-  // Oldest first: 10 was touched longest ago, so plain recency would take it.
-  for (const index of [10, 11, 12, 13]) {
-    lru.touch(index);
-  }
-  lru.protect("reader-a", 10, 11);
-
-  assert.equal(
-    lru.evictionCandidate(),
-    12,
-    "recency alone would have taken 10; the reader said it is about to read it"
-  );
-});
-
-test("protection yields when everything resident is protected", () => {
-  // The case the store cannot survive if protection were absolute: at its
-  // smallest budget only two pieces are resident, and both belong to a window.
-  const lru = new PieceLru(2);
-  lru.touch(4);
-  lru.touch(5);
-  lru.protect("reader-a", 4, 5);
-
-  assert.equal(lru.evictionCandidate(), 4, "a full store must still be able to admit a piece");
-});
-
-test("a pin is never overruled, protected or not", () => {
-  const lru = new PieceLru(2);
-  lru.touch(7);
-  lru.touch(8);
-  lru.pin(7);
-  lru.protect("reader-a", 7, 8);
-
-  assert.equal(lru.evictionCandidate(), 8, "the piece being read now must stay");
-  lru.pin(8);
-  assert.equal(lru.evictionCandidate(), null, "nothing may be taken from under a reader");
-});
-
-test("windows of several readers add up, and leaving takes only your own", () => {
-  const lru = new PieceLru(6);
-  for (const index of [20, 21, 22, 23, 24, 25]) {
-    lru.touch(index);
-  }
-  lru.protect("reader-a", 20, 21);
-  lru.protect("reader-b", 22, 23);
-  assert.equal(lru.evictionCandidate(), 24);
-
-  lru.unprotect("reader-b");
-  assert.equal(lru.evictionCandidate(), 22, "b's window is free once b has gone");
-  assert.equal(lru.protectedCount, 1);
-});
-
-test("a reader moving its window replaces it instead of accumulating", () => {
-  const lru = new PieceLru(4);
-  for (const index of [30, 31, 32, 33]) {
-    lru.touch(index);
-  }
-  lru.protect("reader-a", 30, 31);
-  lru.protect("reader-a", 32, 33);
-
-  assert.equal(lru.evictionCandidate(), 30, "the pieces already read are free again");
-  assert.equal(lru.protectedCount, 1);
-});
-
-test("the capacity follows the store's live allowance", () => {
-  const lru = new PieceLru(4);
-  for (const index of [40, 41]) {
-    lru.touch(index);
-  }
-  assert.equal(lru.isFull(), false, "two of four is not full");
-
-  // The store's allowance moves with the machine's free memory, and the LRU is
-  // told. Before this it kept the capacity it was built with for ever, so
-  // `isFull` answered against a number that had stopped being the limit.
-  lru.setCapacity(2);
-  assert.equal(lru.capacity, 2);
-  assert.equal(lru.isFull(), true, "two of two is full");
-
-  lru.setCapacity(0);
-  assert.equal(lru.capacity, 2, "a capacity below one is refused, not obeyed");
-});
-
-test("the demand is the union of the readers' windows, not their sum", () => {
-  const lru = new PieceLru(88);
-  // Two readers of one file — picture and sound — overlap by construction.
-  // Summing them would say the store is short when it is not.
-  lru.protect("video", 100, 149);
-  lru.protect("audio", 130, 179);
-
-  const demand = lru.demand();
-  assert.equal(demand.readers, 2);
-  // NAMED, because the count read as five encoders on a session that had two:
-  // four of the five were zones of the priority map, and only the names could
-  // say so.
-  assert.deepEqual(demand.names, ["audio", "video"]);
-  assert.equal(demand.unionPieces, 80, "100..179 is eighty pieces, not a hundred");
-  assert.equal(demand.widestPieces, 50);
-  assert.equal(demand.capacity, 88);
-
-  lru.protect("second-viewer", 900, 979);
-  assert.equal(lru.demand().unionPieces, 160, "windows that do not touch add up");
-
-  lru.unprotect("second-viewer");
-  lru.unprotect("audio");
-  lru.unprotect("video");
-  assert.deepEqual(
-    lru.demand(),
-    { readers: 0, names: [], unionPieces: 0, widestPieces: 0, capacity: 88 },
-    "no reader asking for anything is not the same as asking for one piece"
-  );
-});
-
-test("an eviction says whether it had to take a piece a reader declared", () => {
-  const lru = new PieceLru(3);
-  lru.touch(10);
-  lru.touch(11);
-  lru.touch(12);
-  lru.protect("video", 11, 12);
-
-  const spare = lru.evictionChoice();
-  assert.equal(spare.index, 10, "the piece outside every window goes first");
-  assert.equal(spare.protectionYielded, false);
-  assert.equal(spare.distance, 1, "one piece away from the window at 11");
-
-  // Nothing spare left: both survivors are inside the declared window.
-  lru.remove(10);
-  const forced = lru.evictionChoice();
-  assert.equal(forced.index, 11);
-  assert.equal(forced.protectionYielded, true, "the store is holding less than it is asked to");
-  assert.equal(forced.distance, 0, "inside a window");
-
-  assert.equal(lru.evictionCandidate(), 11, "the older answer is the same choice");
-});
-
-test("with nothing declared there is no distance to report", () => {
-  const lru = new PieceLru(2);
-  lru.touch(7);
-  const choice = lru.evictionChoice();
-  assert.equal(choice.index, 7);
-  assert.equal(choice.distance, -1, "-1 is absence, and 0 would read as inside a window");
-
-  lru.pin(7);
-  assert.deepEqual(
-    lru.evictionChoice(),
-    { index: null, protectionYielded: false, distance: -1 },
-    "a pinned piece is never a candidate, and says so without a distance"
-  );
-});
-
-test("the least wanted piece goes, however recently it was touched", () => {
-  // Field 2026-09-05: fifteen reads declared fifteen windows on a store holding
-  // sixteen pieces, and with everything resident declared, eviction fell back
-  // to recency — 392 of 780 evictions took a piece a reader had said it wanted.
-  // Recency cannot separate them; the priority map's number can.
-  const lru = new PieceLru(3);
-  for (const index of [10, 20, 30]) {
-    lru.touch(index);
-  }
-  // 10 is the stalest, and the map wants it most: it is where a viewer is.
-  lru.protect("map:blocked", 10, 10, 0);
-  lru.protect("map:ahead", 20, 20, 2);
-  lru.protect("map:tail", 30, 30, 3);
-
-  const choice = lru.evictionChoice();
-  assert.equal(choice.index, 30, "eviction took the piece the map wants most");
-  assert.equal(choice.protectionYielded, true, "every resident piece was wanted, and it said so");
-});
-
-test("recency decides only between pieces the map wants equally", () => {
-  const lru = new PieceLru(3);
-  for (const index of [10, 20, 30]) {
-    lru.touch(index);
-  }
-  lru.protect("map:blocked", 10, 10, 0);
-  lru.protect("map:ahead", 20, 30, 2);
-
-  // 20 and 30 sit in one zone, so the stalest of the two goes.
-  assert.equal(lru.evictionCandidate(), 20);
-  lru.touch(20);
-  assert.equal(lru.evictionCandidate(), 30);
-});
-
-test("a piece no zone covers goes before any piece a zone covers", () => {
-  const lru = new PieceLru(4);
-  for (const index of [10, 20, 30, 40]) {
-    lru.touch(index);
-  }
-  // Only the last-touched piece is outside every zone — and it still goes
-  // first, because nobody has said they will read it.
-  lru.protect("map:tail", 10, 30, 4);
-
-  const choice = lru.evictionChoice();
-  assert.equal(choice.index, 40);
-  assert.equal(choice.protectionYielded, false, "nothing the map wanted was taken");
-});
-
-test("a claimant that states no number cannot displace one that did", () => {
-  const lru = new PieceLru(2);
-  lru.touch(10);
-  lru.touch(20);
-  lru.protect("map:tail", 10, 10, 4);
-  lru.protect("nameless", 20, 20);
-
-  assert.equal(
-    lru.evictionCandidate(),
-    20,
-    "a range with no stated number was treated as more wanted than the map's own"
-  );
-});

package/test/piece-reader.test.js

@@ -1,162 +0,0 @@
-/**
- * @file Turning a byte range into positions in shared memory.
- *
- * This arithmetic fails silently when it is wrong: the response comes back the
- * right length and full of the wrong bytes. Piece numbers are torrent-wide
- * while a read is expressed in file coordinates, and the first and last pieces
- * of a range are almost always partial — so every case here is a boundary.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { EventEmitter } from "node:events";
-import { readFragments } from "../services/torrent/worker/piece-reader.js";
-import { SharedPieceStore } from "../services/storage/piece-store/shared-piece-store.js";
-import os from "node:os";
-import path from "node:path";
-import fs from "node:fs/promises";
-
-const PIECE = 1024;
-
-/**
- * A torrent whose pieces are all present, backed by a real store so that
- * `locate`/`reside`/`pin` behave as they do in production.
- *
- * @param {{ fileOffset: number, fileLength: number, totalLength: number }} shape
- */
-async function fakeTorrent({ fileOffset, fileLength, totalLength }) {
-  const directory = await fs.mkdtemp(path.join(os.tmpdir(), "piece-reader-test-"));
-  const store = new SharedPieceStore(PIECE, {
-    length: totalLength,
-    memoryBytes: 64 * PIECE,
-    path: directory,
-    name: "test"
-  });
-
-  const pieceCount = Math.ceil(totalLength / PIECE);
-  for (let index = 0; index < pieceCount; index += 1) {
-    const length = index === pieceCount - 1 ? totalLength - index * PIECE : PIECE;
-    const piece = Buffer.alloc(length);
-    // Each byte encodes its own absolute position, so a misplaced offset is
-    // visible in the value itself rather than only in the length.
-    for (let at = 0; at < length; at += 1) {
-      piece[at] = (index * PIECE + at) % 251;
-    }
-    await new Promise((resolve, reject) => {
-      store.put(index, piece, (error) => (error ? reject(error) : resolve()));
-    });
-  }
-
-  const torrent = Object.assign(new EventEmitter(), {
-    pieceLength: PIECE,
-    store,
-    bitfield: { get: () => true },
-    files: [{ offset: fileOffset, length: fileLength, name: "file.bin" }],
-    select() {},
-    critical() {}
-  });
-
-  return { torrent, store, directory };
-}
-
-/** Collect a range through the reader, as the worker does. */
-async function readRange(torrent, start, end) {
-  const collected = [];
-  const positions = [];
-  for await (const fragment of readFragments({
-    torrent,
-    fileIndex: 0,
-    start,
-    end,
-    cancellation: { isCancelled: () => false }
-  })) {
-    const source = fragment.buffer ? Buffer.from(fragment.buffer, fragment.offset, fragment.length) : Buffer.alloc(0);
-    collected.push(Buffer.from(source));
-    positions.push({ piece: fragment.pieceIndex, length: fragment.length });
-    fragment.release();
-  }
-  return { bytes: Buffer.concat(collected), positions };
-}
-
-/** What the bytes at an absolute torrent offset should be. */
-function expectedBytes(absoluteStart, length) {
-  const expected = Buffer.alloc(length);
-  for (let at = 0; at < length; at += 1) {
-    expected[at] = (absoluteStart + at) % 251;
-  }
-  return expected;
-}
-
-test("a range inside one piece is read from that piece only", async () => {
-  const { torrent, store, directory } = await fakeTorrent({
-    fileOffset: 0,
-    fileLength: 4 * PIECE,
-    totalLength: 4 * PIECE
-  });
-  try {
-    const { bytes, positions } = await readRange(torrent, 100, 199);
-    assert.equal(positions.length, 1);
-    assert.deepEqual(bytes, expectedBytes(100, 100));
-  } finally {
-    store.destroy(() => undefined);
-    await fs.rm(directory, { recursive: true, force: true, maxRetries: 10, retryDelay: 20 });
-  }
-});
-
-test("a range spanning pieces reassembles in order", async () => {
-  const { torrent, store, directory } = await fakeTorrent({
-    fileOffset: 0,
-    fileLength: 4 * PIECE,
-    totalLength: 4 * PIECE
-  });
-  try {
-    const start = PIECE - 10;
-    const end = 2 * PIECE + 9;
-    const { bytes, positions } = await readRange(torrent, start, end);
-    assert.deepEqual(positions.map((entry) => entry.piece), [0, 1, 2]);
-    assert.deepEqual(bytes, expectedBytes(start, end - start + 1));
-  } finally {
-    store.destroy(() => undefined);
-    await fs.rm(directory, { recursive: true, force: true, maxRetries: 10, retryDelay: 20 });
-  }
-});
-
-test("a file that does not start at a piece boundary is still read correctly", async () => {
-  // The usual case in a multi-file torrent, and the one where using file
-  // offsets as if they were torrent offsets returns the wrong bytes at the
-  // right length.
-  const fileOffset = PIECE + 300;
-  const { torrent, store, directory } = await fakeTorrent({
-    fileOffset,
-    fileLength: 2 * PIECE,
-    totalLength: 5 * PIECE
-  });
-  try {
-    const { bytes } = await readRange(torrent, 0, 1499);
-    assert.deepEqual(bytes, expectedBytes(fileOffset, 1500));
-  } finally {
-    store.destroy(() => undefined);
-    await fs.rm(directory, { recursive: true, force: true, maxRetries: 10, retryDelay: 20 });
-  }
-});
-
-test("every fragment releases its pin, so nothing stays held", async () => {
-  const { torrent, store, directory } = await fakeTorrent({
-    fileOffset: 0,
-    fileLength: 4 * PIECE,
-    totalLength: 4 * PIECE
-  });
-  try {
-    await readRange(torrent, 0, 4 * PIECE - 1);
-    // With every piece unpinned the store can still make room; if a pin leaked
-    // it would eventually refuse.
-    const before = store.stats().blockedByPins;
-    for (let index = 0; index < 200; index += 1) {
-      await store.reside(index % 4);
-    }
-    assert.equal(store.stats().blockedByPins, before, "a pin was left behind");
-  } finally {
-    store.destroy(() => undefined);
-    await fs.rm(directory, { recursive: true, force: true, maxRetries: 10, retryDelay: 20 });
-  }
-});

package/test/piece-selection-offset.test.js

@@ -1,85 +0,0 @@
-/**
- * @file What WebTorrent's own selection bookkeeping does when a seek demotes
- * the pieces behind the playhead.
- *
- * This is the library's behaviour, not ours, and `prioritizeByteRange` depends
- * on it: a selection carries an `offset` — how many pieces from its start are
- * already downloaded — and the picker scans from `from + offset`
- * (`torrent.js`, `for (piece = next.from + next.offset; piece <= next.to; …)`).
- * `deselect` subtracts an interval and copies that offset into what survives,
- * so the remaining selection can end up scanning past its own end and yield
- * nothing at all.
- *
- * Measured consequence before the fix: a seek to 89.1% of a 4.7 GB film left
- * the seek target wanted by nobody, a later range-less read re-selected the
- * whole file, and the swarm walked it from the first missing piece — 2.47 GB
- * over 93 s before the segment could be served.
- *
- * If a WebTorrent upgrade changes any of this, these tests fail rather than the
- * behaviour silently regressing.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { Selections } from "webtorrent/lib/selections.js";
-
-// The numbers are the measured session: 588 pieces of 8 MiB, sequential
-// download had reached 38.4% (piece 226), the viewer seeked to 89.1% (piece
-// 523).
-const LAST_PIECE = 587;
-const DOWNLOADED_TO = 226;
-const PLAYHEAD = 523;
-
-/** Where the picker would start scanning this selection. */
-const scanStart = (selection) => selection.from + selection.offset;
-
-test("deselecting the gap behind the playhead leaves a selection that yields nothing", () => {
-  const selections = new Selections();
-  selections.insert({ from: 0, to: LAST_PIECE, offset: 0, priority: 1 });
-  // What `_gcSelections` does as pieces arrive.
-  selections.get(0).offset = DOWNLOADED_TO;
-
-  selections.remove({ from: 0, to: PLAYHEAD - 1, isStreamSelection: false });
-
-  assert.equal(selections.length, 1);
-  const survivor = selections.get(0);
-  assert.equal(survivor.from, PLAYHEAD, "the surviving selection starts at the playhead");
-  assert.equal(survivor.offset, DOWNLOADED_TO, "and it kept the offset of the range it came from");
-  assert.ok(
-    scanStart(survivor) > survivor.to,
-    `scan would start at piece ${scanStart(survivor)} of ${survivor.to} — nothing is downloadable`
-  );
-});
-
-test("re-selecting the same range restores a scan that starts at the playhead", () => {
-  const selections = new Selections();
-  selections.insert({ from: 0, to: LAST_PIECE, offset: 0, priority: 1 });
-  selections.get(0).offset = DOWNLOADED_TO;
-
-  // Exactly what prioritizeByteRange does on a forward seek.
-  selections.remove({ from: 0, to: PLAYHEAD - 1, isStreamSelection: false });
-  selections.insert({ from: PLAYHEAD, to: LAST_PIECE, offset: 0, priority: 1 });
-
-  assert.equal(selections.length, 1, "the dead selection was replaced, not added to");
-  const selection = selections.get(0);
-  assert.equal(scanStart(selection), PLAYHEAD, "the picker now starts at the seek target");
-  assert.equal(selection.to, LAST_PIECE);
-});
-
-test("selecting the whole file again undoes the demotion", () => {
-  const selections = new Selections();
-  selections.insert({ from: 0, to: LAST_PIECE, offset: 0, priority: 1 });
-  selections.get(0).offset = DOWNLOADED_TO;
-  selections.remove({ from: 0, to: PLAYHEAD - 1, isStreamSelection: false });
-  selections.insert({ from: PLAYHEAD, to: LAST_PIECE, offset: 0, priority: 1 });
-
-  // A range-less read reporting position 0 used to land here.
-  selections.insert({ from: 0, to: LAST_PIECE, offset: 0, priority: 1 });
-
-  assert.equal(selections.length, 1);
-  assert.equal(
-    scanStart(selections.get(0)),
-    0,
-    "the whole file is selected again, so the picker falls back to the first missing piece"
-  );
-});