summaryrefslogtreecommitdiff
path: root/server/internal
diff options
context:
space:
mode:
authorPeter Stone <thepeterstone@gmail.com>2026-11-11 09:50:42 +0000
committerPeter Stone <thepeterstone@gmail.com>2026-11-11 09:50:42 +0000
commite77fb81d9ec3e4c1b1d6ea2d2103660d121d8ace (patch)
tree57bd0f059cd51099b396e1844931905c2a4abdc9 /server/internal
parent4559946420c4e7317ce68acc34f9ece28492163b (diff)
Two instrumented tests had gone stale, and nothing was running them
Both describe behaviour that was deliberately changed and neither was updated with it, because the instrumented suite is not in the commit gate: it needs a device, so it only runs when someone asks for it. 108 tests, two of them asserting the opposite of what the app now does. The library row asserted a Download button. The row's three became play next, queue and mark played a commit ago -- play is a long press on the line and a queued episode downloads itself -- so the test named a control that no longer exists. Mark played now has a test of its own; it was the one the change was for and the only one of the three with none. The car's resume test seeded an episode with no file, from when playback would fall back to the publisher's copy. That fallback was removed on purpose, so the car now starts a fetch and plays nothing this time round -- the test was asserting a resume against an empty playlist and failing on the position being zero, which is the right answer to the wrong question. Confirmed from the device's own log, "nothing to play for resume-ep from outside the app", rather than inferred. It seeds a downloaded episode now, so the defect it exists for -- the position being dropped at the session boundary -- is what it measures. Whole instrumented suite: 109 tests, no failures. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018AwnALZL6bD4VBtATDVv3B
Diffstat (limited to 'server/internal')
0 files changed, 0 insertions, 0 deletions