The main() method in the QemuBaseTest class has no parameters but is
defined as a regular method. Currently, this does not cause any issues
because in the functional tests main() is always called directly from
QemuBaseTest (never from instances), but the way this method is defined
makes its signature wrong, implying a 'self'. Hence, it's best practice
to define such a method as a static method, so decorate it with
@staticmethod.
Signed-off-by: Gustavo Romero <gustavo.rom...@linaro.org>
---
tests/functional/qemu_test/testcase.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/functional/qemu_test/testcase.py
b/tests/functional/qemu_test/testcase.py
index 5caf7b13fe..fbeb171058 100644
--- a/tests/functional/qemu_test/testcase.py
+++ b/tests/functional/qemu_test/testcase.py
@@ -235,6 +235,7 @@ def tearDown(self):
self.log.removeHandler(self._log_fh)
self._log_fh.close()
+ @staticmethod
def main():
warnings.simplefilter("default")
os.environ["PYTHONWARNINGS"] = "default"
--
2.34.1
