diff mbox series

[FFmpeg-devel,2/2] FATE: add a test for PNG frame metadata

Message ID 20210321101518.10920-2-anton@khirnov.net
State New
Headers show
Series [FFmpeg-devel,1/2] lavc/pngdec: fix exporting frame metadata after 5663301560
Related show

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Anton Khirnov March 21, 2021, 10:15 a.m. UTC
---
 tests/fate/apng.mak                |   4 +
 tests/ref/fate/apng-frame-metadata | 270 +++++++++++++++++++++++++++++
 2 files changed, 274 insertions(+)
 create mode 100644 tests/ref/fate/apng-frame-metadata

Comments

Andreas Rheinhardt March 21, 2021, 10:20 a.m. UTC | #1
Anton Khirnov:
> ---
>  tests/fate/apng.mak                |   4 +
>  tests/ref/fate/apng-frame-metadata | 270 +++++++++++++++++++++++++++++
>  2 files changed, 274 insertions(+)
>  create mode 100644 tests/ref/fate/apng-frame-metadata
> 
> diff --git a/tests/fate/apng.mak b/tests/fate/apng.mak
> index 0a5f542f19..0d8f191359 100644
> --- a/tests/fate/apng.mak
> +++ b/tests/fate/apng.mak
> @@ -7,6 +7,10 @@ fate-apng-osample: CMD = framecrc -i $(TARGET_SAMPLES)/apng/o_sample.png
>  FATE_APNG += fate-apng-dispose-previous
>  fate-apng-dispose-previous: CMD = framecrc -i $(TARGET_SAMPLES)/apng/apng_out_of_order_frames.png
>  
> +FATE_APNG += fate-apng-frame-metadata
> +fate-apng-frame-metadata: CMD = run ffprobe$(PROGSSUF)$(EXESUF) -show_entries frame_tags \
> +    -i $(TARGET_SAMPLES)/apng/apng_out_of_order_frames.png
> +
>  FATE_APNG-$(call DEMDEC, APNG, APNG) += $(FATE_APNG)
>  
>  FATE_SAMPLES_FFMPEG += $(FATE_APNG-yes)
> diff --git a/tests/ref/fate/apng-frame-metadata b/tests/ref/fate/apng-frame-metadata
> new file mode 100644
> index 0000000000..559562c425
> --- /dev/null
> +++ b/tests/ref/fate/apng-frame-metadata
> @@ -0,0 +1,270 @@
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +[/FRAME]
> +[FRAME]
> +TAG:Software=ezgif.com
> +TAG:Comment=Resized on https://ezgif.com/resize
> +[/FRAME]
> 
This metadata is also reported now; it is not affected by the
regression. But the files fate-suite/filter/pixelart[01].png are.

- Andreas
diff mbox series

Patch

diff --git a/tests/fate/apng.mak b/tests/fate/apng.mak
index 0a5f542f19..0d8f191359 100644
--- a/tests/fate/apng.mak
+++ b/tests/fate/apng.mak
@@ -7,6 +7,10 @@  fate-apng-osample: CMD = framecrc -i $(TARGET_SAMPLES)/apng/o_sample.png
 FATE_APNG += fate-apng-dispose-previous
 fate-apng-dispose-previous: CMD = framecrc -i $(TARGET_SAMPLES)/apng/apng_out_of_order_frames.png
 
+FATE_APNG += fate-apng-frame-metadata
+fate-apng-frame-metadata: CMD = run ffprobe$(PROGSSUF)$(EXESUF) -show_entries frame_tags \
+    -i $(TARGET_SAMPLES)/apng/apng_out_of_order_frames.png
+
 FATE_APNG-$(call DEMDEC, APNG, APNG) += $(FATE_APNG)
 
 FATE_SAMPLES_FFMPEG += $(FATE_APNG-yes)
diff --git a/tests/ref/fate/apng-frame-metadata b/tests/ref/fate/apng-frame-metadata
new file mode 100644
index 0000000000..559562c425
--- /dev/null
+++ b/tests/ref/fate/apng-frame-metadata
@@ -0,0 +1,270 @@ 
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+[/FRAME]
+[FRAME]
+TAG:Software=ezgif.com
+TAG:Comment=Resized on https://ezgif.com/resize
+[/FRAME]