diff mbox series

[FFmpeg-devel,1/3] fate/lavf-audio: Test writing AIFF-native tags

Message ID AS8P250MB0744763936DA411E5A4C088B8F292@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM
State New
Headers show
Series [FFmpeg-devel,1/3] fate/lavf-audio: Test writing AIFF-native tags | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 fail Make fate failed
andriy/make_x86 success Make finished
andriy/make_fate_x86 fail Make fate failed

Commit Message

Andreas Rheinhardt March 14, 2024, 12:35 a.m. UTC
In particular, test writing tags with odd strlen.
(These tags are zero-padded to even size.)

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/fate/lavf-audio.mak | 1 +
 tests/ref/lavf/aiff       | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

Comments

Michael Niedermayer March 14, 2024, 9:45 p.m. UTC | #1
On Thu, Mar 14, 2024 at 01:35:36AM +0100, Andreas Rheinhardt wrote:
> In particular, test writing tags with odd strlen.
> (These tags are zero-padded to even size.)
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  tests/fate/lavf-audio.mak | 1 +
>  tests/ref/lavf/aiff       | 4 ++--
>  2 files changed, 3 insertions(+), 2 deletions(-)

needs to update seek test too:


--- ./tests/ref/seek/lavf-aiff	2024-03-05 02:37:36.243300240 +0100
+++ tests/data/fate/seek-lavf-aiff	2024-03-14 22:24:47.357654173 +0100
@@ -1,53 +1,53 @@
-ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:     70 size:  4096
+ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:     84 size:  4096
 ret: 0         st:-1 flags:0  ts:-1.000000
-ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:     70 size:  4096
+ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:     84 size:  4096
 ret: 0         st:-1 flags:1  ts: 1.894167
 ret:-EOF
 ret: 0         st: 0 flags:0  ts: 0.788345
-ret: 0         st: 0 flags:1 dts: 0.788345 pts: 0.788345 pos:  69602 size:  4096
+ret: 0         st: 0 flags:1 dts: 0.788345 pts: 0.788345 pos:  69616 size:  4096
 ret: 0         st: 0 flags:1  ts:-0.317506
-ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:     70 size:  4096
+ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:     84 size:  4096
...

thx

[...]
Andreas Rheinhardt March 14, 2024, 9:46 p.m. UTC | #2
Michael Niedermayer:
> On Thu, Mar 14, 2024 at 01:35:36AM +0100, Andreas Rheinhardt wrote:
>> In particular, test writing tags with odd strlen.
>> (These tags are zero-padded to even size.)
>>
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
>> ---
>>  tests/fate/lavf-audio.mak | 1 +
>>  tests/ref/lavf/aiff       | 4 ++--
>>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> needs to update seek test too:
> 

Yeah, I already did it locally. Sorry for this.

- Andreas
Andreas Rheinhardt March 17, 2024, 2:48 p.m. UTC | #3
Andreas Rheinhardt:
> In particular, test writing tags with odd strlen.
> (These tags are zero-padded to even size.)
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  tests/fate/lavf-audio.mak | 1 +
>  tests/ref/lavf/aiff       | 4 ++--
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/fate/lavf-audio.mak b/tests/fate/lavf-audio.mak
> index d54cd107e0..7ea0c41da9 100644
> --- a/tests/fate/lavf-audio.mak
> +++ b/tests/fate/lavf-audio.mak
> @@ -28,6 +28,7 @@ $(FATE_LAVF_AUDIO): CMD = lavf_audio
>  $(FATE_LAVF_AUDIO): REF = $(SRC_PATH)/tests/ref/lavf/$(@:fate-lavf-%=%)
>  $(FATE_LAVF_AUDIO): $(AREF)
>  
> +fate-lavf-aiff: CMD = lavf_audio "" "-metadata copyright=noone"
>  fate-lavf-al fate-lavf-ul: CMD = lavf_audio "" "" "-ar 44100"
>  fate-lavf-dfpwm: CMD = lavf_audio "" "" "-sample_rate 44100"
>  fate-lavf-ogg: CMD = lavf_audio "" "-c:a flac"
> diff --git a/tests/ref/lavf/aiff b/tests/ref/lavf/aiff
> index d72ec85150..e208ff3e16 100644
> --- a/tests/ref/lavf/aiff
> +++ b/tests/ref/lavf/aiff
> @@ -1,3 +1,3 @@
> -2c129d88acef834e32869145fe792b9c *tests/data/lavf/lavf.aiff
> -88270 tests/data/lavf/lavf.aiff
> +655b5bd68e7a59599ab6663de0015324 *tests/data/lavf/lavf.aiff
> +88284 tests/data/lavf/lavf.aiff
>  tests/data/lavf/lavf.aiff CRC=0x3a1da17e

Will apply this patchset with the fate references updated.

- Andreas
diff mbox series

Patch

diff --git a/tests/fate/lavf-audio.mak b/tests/fate/lavf-audio.mak
index d54cd107e0..7ea0c41da9 100644
--- a/tests/fate/lavf-audio.mak
+++ b/tests/fate/lavf-audio.mak
@@ -28,6 +28,7 @@  $(FATE_LAVF_AUDIO): CMD = lavf_audio
 $(FATE_LAVF_AUDIO): REF = $(SRC_PATH)/tests/ref/lavf/$(@:fate-lavf-%=%)
 $(FATE_LAVF_AUDIO): $(AREF)
 
+fate-lavf-aiff: CMD = lavf_audio "" "-metadata copyright=noone"
 fate-lavf-al fate-lavf-ul: CMD = lavf_audio "" "" "-ar 44100"
 fate-lavf-dfpwm: CMD = lavf_audio "" "" "-sample_rate 44100"
 fate-lavf-ogg: CMD = lavf_audio "" "-c:a flac"
diff --git a/tests/ref/lavf/aiff b/tests/ref/lavf/aiff
index d72ec85150..e208ff3e16 100644
--- a/tests/ref/lavf/aiff
+++ b/tests/ref/lavf/aiff
@@ -1,3 +1,3 @@ 
-2c129d88acef834e32869145fe792b9c *tests/data/lavf/lavf.aiff
-88270 tests/data/lavf/lavf.aiff
+655b5bd68e7a59599ab6663de0015324 *tests/data/lavf/lavf.aiff
+88284 tests/data/lavf/lavf.aiff
 tests/data/lavf/lavf.aiff CRC=0x3a1da17e