tests/functional/alpha: Remove superfluous fetch() line from the clipper test

[PATCH] tests/functional/alpha: Remove superfluous fetch() line from the clipper test

Thomas Huth posted 1 patch 1 month ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20251010144525.842462-1-thuth@redhat.com

Maintainers: Richard Henderson <richard.henderson@linaro.org>

tests/functional/alpha/test_clipper.py | 1 -
1 file changed, 1 deletion(-)

Expand all Fold all

[PATCH] tests/functional/alpha: Remove superfluous fetch() line from the clipper test

Posted by Thomas Huth 1 month ago

From: Thomas Huth <thuth@redhat.com>

The kernel asset is retrieved automatically via the uncompress()
line below the fetch(), so the fetch() is simply not necessary here.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 tests/functional/alpha/test_clipper.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/functional/alpha/test_clipper.py b/tests/functional/alpha/test_clipper.py
index c5d71819531..d2a4c2a4ed9 100755
--- a/tests/functional/alpha/test_clipper.py
+++ b/tests/functional/alpha/test_clipper.py
@@ -17,7 +17,6 @@ class AlphaClipperTest(LinuxKernelTest):
 
     def test_alpha_clipper(self):
         self.set_machine('clipper')
-        kernel_path = self.ASSET_KERNEL.fetch()
 
         uncompressed_kernel = self.uncompress(self.ASSET_KERNEL, format="gz")
 
-- 
2.51.0

Re: [PATCH] tests/functional/alpha: Remove superfluous fetch() line from the clipper test

Posted by Richard Henderson 4 weeks, 1 day ago

On 10/10/25 07:45, Thomas Huth wrote:
> From: Thomas Huth<thuth@redhat.com>
> 
> The kernel asset is retrieved automatically via the uncompress()
> line below the fetch(), so the fetch() is simply not necessary here.
> 
> Signed-off-by: Thomas Huth<thuth@redhat.com>
> ---
>   tests/functional/alpha/test_clipper.py | 1 -
>   1 file changed, 1 deletion(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~