Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
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".

  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