[bug#70936] [PATCH gnome-team 3/3] gnu: qemu: Disable more tests.

[Maxim Cournoyer <maxim.cournoyer@gmail.com>]

Hello!

Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:

> Liliana Marie Prikler <liliana.prikler@gmail.com> writes:
>
>> * gnu/packages/virtualization.scm (qemu)[disable-unusable-tests]: Also disable
>> qtest/ahci-test and qemu-iotests/copy-before-write.
>> ---
>>  gnu/packages/virtualization.scm | 7 ++++++-
>>  1 file changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/gnu/packages/virtualization.scm b/gnu/packages/virtualization.scm
>> index 34cccd6550..5389af62f0 100644
>> --- a/gnu/packages/virtualization.scm
>> +++ b/gnu/packages/virtualization.scm
>> @@ -305,7 +305,12 @@ (define-public qemu
>>                (substitute* "tests/qtest/meson.build"
>>                  ;; These tests fail to get the expected number of tests
>>                  ;; on arm platforms.
>> -                (("'arm-cpu-features',") ""))))
>> +                (("'arm-cpu-features',") "")
>> +                ;; XXX: Why does this (sometimes) fail?
>
> Someone reported it already upstream:
> <https://gitlab.com/qemu-project/qemu/-/issues/2121>.  Please add a
> reference to it.
>
>> +                (("\\['ahci-test'\\]") "[]"))
>> +              ;; This test appears to be missing permissions.
>> +              ;; Very sus 🤔
>
> How can it be missing permissions if it is sometimes working?  Perhaps
> more something racy where the file hasn't been created yet.  I'd
> personally rather not see emojis or comments of dubious utility ("Very
> suspect") in the source.  Perhaps add a short excerpt of the error for
> the next curious person.
>
>> +              (delete-file "tests/qemu-iotests/tests/copy-before-write")))
>>            #$@(if (target-riscv64?)
>>                   '((add-after 'unpack 'disable-some-tests
>>                       (lambda _
>
> Otherwise, LGTM, with the added reference to the upstream issue.  Thanks
> for taking the time to address flaky tests!  They are a real drag when
> left unchecked.

Ping, in case it had fallen in cracks :-).

-- 
Thanks,
Maxim