[PATCH] usb: Fix test dependency on ext4write command
Marek Vasut
marex at denx.de
Tue Oct 15 17:41:30 CEST 2024
On 10/15/24 4:17 PM, Andrew Goodbody wrote:
> The tests test_usb_ext4load_ext4write, test_usb_ext2load and
> test_usb_load depend on the command ext4write being present not just
> the feature of being able to write to an ext4 file system. So update
> their dependencies to include the ext4write command itself. This will
> prevent spurious test failures when running the USB tests against USB
> storage with an ext2/ext4 partition but no ext4write command available.
>
> Fixes: 1c5b6edad381 ("test/py: usb: Add tests for USB device")
> Signed-off-by: Andrew Goodbody <andrew.goodbody at linaro.org>
> ---
>
> test/py/tests/test_usb.py | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/test/py/tests/test_usb.py b/test/py/tests/test_usb.py
> index fb3d20f082..b86fc290c7 100644
> --- a/test/py/tests/test_usb.py
> +++ b/test/py/tests/test_usb.py
> @@ -382,7 +382,7 @@ def test_usb_ext4ls(u_boot_console):
>
> @pytest.mark.buildconfigspec('cmd_usb')
> @pytest.mark.buildconfigspec('cmd_ext4')
> - at pytest.mark.buildconfigspec('ext4_write')
> + at pytest.mark.buildconfigspec('cmd_ext4_write')
> @pytest.mark.buildconfigspec('cmd_memory')
> def test_usb_ext4load_ext4write(u_boot_console):
> devices, controllers, storage_device = test_usb_part(u_boot_console)
> @@ -469,7 +469,7 @@ def test_usb_ext2ls(u_boot_console):
> @pytest.mark.buildconfigspec('cmd_usb')
> @pytest.mark.buildconfigspec('cmd_ext2')
> @pytest.mark.buildconfigspec('cmd_ext4')
> - at pytest.mark.buildconfigspec('ext4_write')
> + at pytest.mark.buildconfigspec('cmd_ext4_write')
> @pytest.mark.buildconfigspec('cmd_memory')
> def test_usb_ext2load(u_boot_console):
> devices, controllers, storage_device = test_usb_part(u_boot_console)
> @@ -543,6 +543,7 @@ def test_usb_ls(u_boot_console):
> pytest.skip('No partition detected')
>
> @pytest.mark.buildconfigspec('cmd_usb')
> + at pytest.mark.buildconfigspec('cmd_ext4_write')
> @pytest.mark.buildconfigspec('cmd_fs_generic')
> def test_usb_load(u_boot_console):
> devices, controllers, storage_device = test_usb_part(u_boot_console)
+CC test expert Simon
More information about the U-Boot
mailing list