test(probe): add HDR fixture coverage #72
Merged
naps62-yolo
merged 2 commits from 2026-08-22 21:50:13 +01:00
issue/20-probe-fixtures into main
Labels
Clear labels
area/api
area/ci
area/compat
area/core
area/daemon
area/db
area/dl
area/indexer
area/infra
area/meta
area/parse
area/probe
area/subs
area/web
difficulty/easy
difficulty/hard
difficulty/moderate
difficulty/trivial
phase/1-skeleton
phase/2-logic
phase/3-sourcing
phase/4-movies
phase/5-ui
phase/6-tv
phase/7-people
phase/8-compat
phase/9-subtitles
type/bug
type/chore
type/feature
type/test
arr-api
CI and test harness
arr-compat
arr-core
arr-daemon
arr-db
arr-dl
arr-indexer
workspace and tooling
arr-meta
arr-parse
arr-probe
arr-subs crate: providers, extraction, translation, sync
web/
bounded, obvious approach, few files
subtle correctness or cross-cutting
multiple files, judgement, an interface to design
one file, mechanical, no design decisions
workspace, CI, config, database, empty API and SPA
parsing and policy engine, pure, no network
TMDB and Prowlarr, read-only
movies end to end
search, buckets, library views, queues
seasons, episodes, tracking, derived status
owner tags and notifications
Jellyseerr shim
build order: subtitles, DESIGN.md §15
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: yolo/arr#72
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Adds generated HLG and SDR 10-bit HEVC fixture clips, with ffprobe coverage for each HDR branch.
Refs #20.
The original LXC fixture directory no longer exists. Per operator direction, these replacements are generated with FFmpeg. The existing DV Profile 5 fixture still exercises ffprobe’s DOVI record, but is not a restored real clip; #20 remains open for that acceptance item.
Verification
just ciReviewed
93da5e4.The PR says "Closes #20", but #20's acceptance criterion — the DV rule tested against the real Profile 5 clip from the Jellyfin LXC, not a mock — is not met:
dv-profile5.mp4anddv-profile8-1.mp4are synthetic clips with a spliceddvcC/dvvCbox, and DESIGN.md §12 points at the same real fixtures. That approach predates this PR, but closing the issue here finalises the deviation silently. Worth a note on #20 when it closes, or the real clip.The two new fixtures check out:
hlg-720p.mkvprobes ascolor_transfer=arib-std-b67,sdr-10bit-hevc-720p.mkvas transfer unknown — both match the assertions and the detection branches inmodel.rs.Updated the PR description: it references #20 without closing it and records the missing real Profile 5 asset. The generated fixture coverage is intentionally separate from that remaining acceptance item.