diff options
| author | Daniel P. Berrangé <berrange@redhat.com> | 2025-03-17 12:43:00 +0000 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2025-03-19 07:41:32 +0100 |
| commit | 23686dfb763d22d95c05c49382116d77133637b7 (patch) | |
| tree | dc9a84a3d6a223d7ef3636ca1621b38bd37657fa /tests/functional/test_acpi_bits.py | |
| parent | 6d19d095470221684afe065cffd40adebe644d04 (diff) | |
| download | focaccia-qemu-23686dfb763d22d95c05c49382116d77133637b7.tar.gz focaccia-qemu-23686dfb763d22d95c05c49382116d77133637b7.zip | |
tests/functional: remove all class level fields
A number of fields are set at the class level on QemuBaseTest, even though the exact same named field is then set at the object level later in most cases. The 'self.logger' initialization in ACPI bits test needs to be removed since 'self.log' won't exist at that point in the flow. It already initialized 'self.logger' later in the setUp() method, so the __init__ method was redundant. Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> Message-ID: <20250317124300.84266-1-berrange@redhat.com> Reviewed-by: Thomas Huth <thuth@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'tests/functional/test_acpi_bits.py')
| -rwxr-xr-x | tests/functional/test_acpi_bits.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tests/functional/test_acpi_bits.py b/tests/functional/test_acpi_bits.py index 20da435687..8e0563a97b 100755 --- a/tests/functional/test_acpi_bits.py +++ b/tests/functional/test_acpi_bits.py @@ -119,7 +119,6 @@ class AcpiBitsTest(QemuSystemTest): #pylint: disable=too-many-instance-attribute self._debugcon_addr = '0x403' self._debugcon_log = 'debugcon-log.txt' - self.logger = self.log def _print_log(self, log): self.logger.info('\nlogs from biosbits follows:') |