From d9e97a483075b50c65410d1de2eb21d5d738b364 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tom=C3=A1=C5=A1=20Hozza?= Date: Thu, 20 Apr 2023 22:57:58 +0200 Subject: [PATCH] Test/assemblers/assertImageFile: don't set default argument values MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Do not specify the default value for 'expected_size' argument in assertImageFile() function declaration. Previously, it was set to `None`, which was never taken into account. Moreover, all callers of the function always provide an explicit value, so the default was never really used. Signed-off-by: Tomáš Hozza --- test/run/test_assemblers.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/run/test_assemblers.py b/test/run/test_assemblers.py index 87838661..2cb8efe1 100644 --- a/test/run/test_assemblers.py +++ b/test/run/test_assemblers.py @@ -23,7 +23,7 @@ def osbuild_fixture(): yield test.OSBuild() -def assertImageFile(filename, fmt, expected_size=None): +def assertImageFile(filename, fmt, expected_size): info = json.loads(subprocess.check_output(["qemu-img", "info", "--output", "json", filename])) assert info["format"] == fmt assert info["virtual-size"] == expected_size