[FFmpeg-devel] [PATCH 6/6] lavc/cbs: Add tests for VP9

[James Almer]

On 4/30/2018 8:26 PM, Mark Thompson wrote:
> Uses the same mechanism as other codecs - conformance test files are
> passed through the metadata filter (which, with no options, reads the
> input and writes it back) and the output verified to match the input.
> ---
>  tests/fate/cbs.mak                                 | 34 ++++++++++++++++++----
>  tests/ref/fate/cbs-vp9-vp90-2-03-deltaq            |  1 +
>  tests/ref/fate/cbs-vp9-vp90-2-05-resize            |  1 +
>  tests/ref/fate/cbs-vp9-vp90-2-06-bilinear          |  1 +
>  tests/ref/fate/cbs-vp9-vp90-2-09-lf_deltas         |  1 +
>  .../ref/fate/cbs-vp9-vp90-2-10-show-existing-frame |  1 +
>  .../fate/cbs-vp9-vp90-2-10-show-existing-frame2    |  1 +
>  .../ref/fate/cbs-vp9-vp90-2-segmentation-aq-akiyo  |  1 +
>  .../ref/fate/cbs-vp9-vp90-2-segmentation-sf-akiyo  |  1 +
>  tests/ref/fate/cbs-vp9-vp90-2-tiling-pedestrian    |  1 +
>  tests/ref/fate/cbs-vp9-vp91-2-04-yuv440            |  1 +
>  tests/ref/fate/cbs-vp9-vp91-2-04-yuv444            |  1 +
>  tests/ref/fate/cbs-vp9-vp92-2-20-10bit-yuv420      |  1 +
>  tests/ref/fate/cbs-vp9-vp93-2-20-10bit-yuv422      |  1 +
>  tests/ref/fate/cbs-vp9-vp93-2-20-12bit-yuv444      |  1 +
>  15 files changed, 43 insertions(+), 5 deletions(-)
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-03-deltaq
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-05-resize
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-06-bilinear
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-09-lf_deltas
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-10-show-existing-frame
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-10-show-existing-frame2
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-segmentation-aq-akiyo
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-segmentation-sf-akiyo
>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-tiling-pedestrian
>  create mode 100644 tests/ref/fate/cbs-vp9-vp91-2-04-yuv440
>  create mode 100644 tests/ref/fate/cbs-vp9-vp91-2-04-yuv444
>  create mode 100644 tests/ref/fate/cbs-vp9-vp92-2-20-10bit-yuv420
>  create mode 100644 tests/ref/fate/cbs-vp9-vp93-2-20-10bit-yuv422
>  create mode 100644 tests/ref/fate/cbs-vp9-vp93-2-20-12bit-yuv444

LGTM.

Unrelated to this patch, but could you add -y to the command line for
all cbs tests? That way it will not fail/abort if the output file
already exists (Something that can happen if the test failed previously
as no cleanup takes place in that case).