diff options
| author | Philippe Mathieu-Daudé <philmd@linaro.org> | 2023-11-15 21:51:49 +0100 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2023-11-16 07:13:41 +0100 |
| commit | 9d72dd100ced090f7708b2189a8ef4e855ac3314 (patch) | |
| tree | 461f55bcf86c8952e20fdc6bdf69e20436208170 | |
| parent | cfe52e91c4020cacf6f95f907c06bf5c2deeec09 (diff) | |
| download | focaccia-qemu-9d72dd100ced090f7708b2189a8ef4e855ac3314.tar.gz focaccia-qemu-9d72dd100ced090f7708b2189a8ef4e855ac3314.zip | |
tests/avocado: Make fetch_asset() unconditionally require a crypto hash
In a perfect world we'd have reproducible tests, but then we'd be sure we run the same binaries. If a binary artifact isn't hashed, we have no idea what we are running. Therefore enforce hashing for all our artifacts. With this change, unhashed artifacts produce: $ avocado run tests/avocado/multiprocess.py (1/2) tests/avocado/multiprocess.py:Multiprocess.test_multiprocess_x86_64: ERROR: QemuBaseTest.fetch_asset() missing 1 required positional argument: 'asset_hash' (0.19 s) Inspired-by: Thomas Huth <thuth@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Thomas Huth <thuth@redhat.com> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Message-ID: <20231115205149.90765-1-philmd@linaro.org> Signed-off-by: Thomas Huth <thuth@redhat.com>
| -rw-r--r-- | tests/avocado/avocado_qemu/__init__.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/avocado/avocado_qemu/__init__.py b/tests/avocado/avocado_qemu/__init__.py index d71e989db6..304c428168 100644 --- a/tests/avocado/avocado_qemu/__init__.py +++ b/tests/avocado/avocado_qemu/__init__.py @@ -254,7 +254,7 @@ class QemuBaseTest(avocado.Test): self.cancel("No QEMU binary defined or found in the build tree") def fetch_asset(self, name, - asset_hash=None, algorithm=None, + asset_hash, algorithm=None, locations=None, expire=None, find_only=False, cancel_on_missing=True): return super().fetch_asset(name, |