From: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
To: ffmpeg-devel@ffmpeg.org
Cc: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Subject: [FFmpeg-devel] [PATCH 17/32] fate/fits: Use transcode for transcode-like test
Date: Sun, 24 Apr 2022 05:34:02 +0200
Message-ID: <AS8PR01MB794414A62A3367140CC96AA48FF99@AS8PR01MB7944.eurprd01.prod.exchangelabs.com> (raw)
In-Reply-To: <AS8PR01MB794466793DDE565DD5B8A1DD8FF99@AS8PR01MB7944.eurprd01.prod.exchangelabs.com>
Each of the intermediately generated lena-*.fits files is only used
for exactly one test; so it could be deleted right after the test.
Switching to a transcode test (which is also more natural) achieves
this. It also adds checksums of the intermediate files to the ref-file.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
tests/fate/fits.mak | 19 +++++--------------
tests/ref/fate/fitsdec-gbrap16le | 2 ++
tests/ref/fate/fitsdec-gbrp | 2 ++
tests/ref/fate/fitsdec-gbrp16 | 2 ++
tests/ref/fate/fitsdec-gray | 2 ++
5 files changed, 13 insertions(+), 14 deletions(-)
diff --git a/tests/fate/fits.mak b/tests/fate/fits.mak
index ea471d6654..b9e99d97ee 100644
--- a/tests/fate/fits.mak
+++ b/tests/fate/fits.mak
@@ -6,17 +6,10 @@ tests/data/fits-multi.fits: ffmpeg$(PROGSSUF)$(EXESUF) | tests/data
#mapping of fits file formats to png filenames
# TODO: Use an actual 64bit input file and fix the gbrp16 test on big-endian
-map.tests/data/lena-gray.fits := gray8
-map.tests/data/lena-gbrp.fits := rgb24
-map.tests/data/lena-gbrp16.fits := rgb48
-map.tests/data/lena-gbrap16le.fits := rgba64
-
-tests/data/lena%.fits: TAG = GEN
-tests/data/lena%.fits: NAME = $(map.$(@))
-tests/data/lena%.fits: ffmpeg$(PROGSSUF)$(EXESUF) | tests/data
- $(M)$(TARGET_EXEC) $(TARGET_PATH)/$< -nostdin \
- -i $(TARGET_SAMPLES)/png1/lena-$(map.$(@)).png \
- -y $(TARGET_PATH)/$(@) 2>/dev/null
+fits-png-map-gray := gray8
+fits-png-map-gbrp := rgb24
+fits-png-map-gbrp16 := rgb48
+fits-png-map-gbrap16le := rgba64
FATE_FITS_DEC-$(call FRAMECRC, FITS, FITS, SCALE_FILTER) += fate-fitsdec-ext_data_min_max
fate-fitsdec-ext_data_min_max: CMD = framecrc -i $(TARGET_SAMPLES)/fits/x0cj010ct_d0h.fit -pix_fmt gray16le -vf scale
@@ -35,11 +28,9 @@ fate-fitsdec-multi: tests/data/fits-multi.fits
fate-fitsdec-multi: CMD = framecrc -i $(TARGET_PATH)/tests/data/fits-multi.fits -pix_fmt gbrap
fate-fitsdec%: PIXFMT = $(word 3, $(subst -, ,$(@)))
-fate-fitsdec%: SRC = $(TARGET_PATH)/tests/data/lena-$(PIXFMT).fits
-fate-fitsdec%: CMD = framecrc -i $(SRC) -pix_fmt $(PIXFMT)
+fate-fitsdec%: CMD = transcode image2 $(TARGET_SAMPLES)/png1/lena-$(fits-png-map-$(PIXFMT)).png fits "-vf scale -pix_fmt $(PIXFMT)"
FATE_FITS_DEC_PIXFMT = gray gbrp gbrp16 gbrap16le
-$(FATE_FITS_DEC_PIXFMT:%=fate-fitsdec-%): fate-fitsdec-%: tests/data/lena-%.fits
FATE_FITS_DEC-$(call TRANSCODE, FITS, FITS, IMAGE2_DEMUXER PNG_DECODER SCALE_FILTER) += $(FATE_FITS_DEC_PIXFMT:%=fate-fitsdec-%)
FATE_FITS += $(FATE_FITS_DEC-yes)
diff --git a/tests/ref/fate/fitsdec-gbrap16le b/tests/ref/fate/fitsdec-gbrap16le
index 78abb5cde7..53ef980b13 100644
--- a/tests/ref/fate/fitsdec-gbrap16le
+++ b/tests/ref/fate/fitsdec-gbrap16le
@@ -1,3 +1,5 @@
+64526d8da12d1fa07ceea5725647076f *tests/data/fate/fitsdec-gbrap16le.fits
+135360 tests/data/fate/fitsdec-gbrap16le.fits
#tb 0: 1/1
#media_type 0: video
#codec_id 0: rawvideo
diff --git a/tests/ref/fate/fitsdec-gbrp b/tests/ref/fate/fitsdec-gbrp
index 8767b6715f..23249f492b 100644
--- a/tests/ref/fate/fitsdec-gbrp
+++ b/tests/ref/fate/fitsdec-gbrp
@@ -1,3 +1,5 @@
+38e232e4c1ca57b5866efa01da70359c *tests/data/fate/fitsdec-gbrp.fits
+54720 tests/data/fate/fitsdec-gbrp.fits
#tb 0: 1/1
#media_type 0: video
#codec_id 0: rawvideo
diff --git a/tests/ref/fate/fitsdec-gbrp16 b/tests/ref/fate/fitsdec-gbrp16
index f6368f2c02..9250690e9b 100644
--- a/tests/ref/fate/fitsdec-gbrp16
+++ b/tests/ref/fate/fitsdec-gbrp16
@@ -1,3 +1,5 @@
+2078208c93ba417d3fe150ba42bf5a30 *tests/data/fate/fitsdec-gbrp16.fits
+103680 tests/data/fate/fitsdec-gbrp16.fits
#tb 0: 1/1
#media_type 0: video
#codec_id 0: rawvideo
diff --git a/tests/ref/fate/fitsdec-gray b/tests/ref/fate/fitsdec-gray
index 488ee71022..b7992bfda3 100644
--- a/tests/ref/fate/fitsdec-gray
+++ b/tests/ref/fate/fitsdec-gray
@@ -1,3 +1,5 @@
+c02ac3e0a9f2ceeb1e6e66c085a63671 *tests/data/fate/fitsdec-gray.fits
+20160 tests/data/fate/fitsdec-gray.fits
#tb 0: 1/1
#media_type 0: video
#codec_id 0: rawvideo
--
2.32.0
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2022-04-24 3:37 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-24 3:29 [FFmpeg-devel] [PATCH 01/32] fate/matroska: Use REMUX and TRANSCODE where appropriate Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 02/32] fate/id3v2: Use REMUX " Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 03/32] fate/amr[nw]b: " Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 04/32] fate/caf: " Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 05/32] fate/mov: Use REMUX and TRANSCODE " Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 06/32] fate/lavf-image: Fix requirements of tests Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 07/32] tests/Makefile: Make DEMDEC etc. auxiliary functions more flexible Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 08/32] fate/image: Fix requirements of tests Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 09/32] tests/Makefile: Add FRAMECRC function Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 10/32] fate/screen: Fix test requirements Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 11/32] fate/video: " Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 12/32] fate/dfa: " Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 13/32] fate/cdxl: " Andreas Rheinhardt
2022-04-24 3:33 ` [FFmpeg-devel] [PATCH 14/32] fate/bmp: " Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 15/32] fate/utvideo: " Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 16/32] fate/fits: " Andreas Rheinhardt
2022-04-24 3:34 ` Andreas Rheinhardt [this message]
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 18/32] avformat/fitsenc: Simplify writing header padding Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 19/32] fate/speedhq: Fix test requirements Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 20/32] tests/fate-run: Remove intermediate lavf_container_fate files Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 21/32] fate/seek: Avoid list of source files Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 22/32] fate/seek: Don't duplicate test requirements Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 23/32] fate/vcodec: Don't add scale filter unnecessarily Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 24/32] fate/vcodec: Fix test requirements Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 25/32] fate/acodec: " Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 26/32] fate/acodec: Remove acodec-adpcm-adx-trellis test Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 27/32] avformat/format: Also search for image2-codecs for image2pipe Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 28/32] fate/lavf-image2pipe: Fix test requirements Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 29/32] fate/lavf-video: " Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 30/32] fate/lavf-audio: Fix requirements of tests Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 31/32] fate/lavf-container: Fix test requirements Andreas Rheinhardt
2022-04-24 3:34 ` [FFmpeg-devel] [PATCH 32/32] fate/concatdec: Don't duplicate " Andreas Rheinhardt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=AS8PR01MB794414A62A3367140CC96AA48FF99@AS8PR01MB7944.eurprd01.prod.exchangelabs.com \
--to=andreas.rheinhardt@outlook.com \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git