diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-11-11 09:50:42 +0000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-11-11 09:50:42 +0000 |
| commit | e77fb81d9ec3e4c1b1d6ea2d2103660d121d8ace (patch) | |
| tree | 57bd0f059cd51099b396e1844931905c2a4abdc9 /gradle | |
| parent | 4559946420c4e7317ce68acc34f9ece28492163b (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 'gradle')
0 files changed, 0 insertions, 0 deletions
