On Mon, Dec 2, 2024 at 3:02 PM Thomas Huth <th...@redhat.com> wrote: > > On 29/11/2024 18.31, Daniel P. Berrangé wrote: > > This removes direct path manipulation to figure out the source dir > > > > Signed-off-by: Daniel P. Berrangé <berra...@redhat.com> > > --- > > tests/functional/test_acpi_bits.py | 22 +++++++--------------- > > 1 file changed, 7 insertions(+), 15 deletions(-) > > > > diff --git a/tests/functional/test_acpi_bits.py > > b/tests/functional/test_acpi_bits.py > > index 4de5fae37e..948b134b16 100755 > > --- a/tests/functional/test_acpi_bits.py > > +++ b/tests/functional/test_acpi_bits.py > > @@ -41,7 +41,6 @@ > > import tempfile > > import zipfile > > > > -from pathlib import Path > > from typing import ( > > List, > > Optional, > > @@ -122,7 +121,6 @@ class AcpiBitsTest(QemuBaseTest): #pylint: > > disable=too-many-instance-attributes > > def __init__(self, *args, **kwargs): > > super().__init__(*args, **kwargs) > > self._vm = None > > - self._baseDir = None > > > > self._debugcon_addr = '0x403' > > self._debugcon_log = 'debugcon-log.txt' > > @@ -137,26 +135,22 @@ def _print_log(self, log): > > def copy_bits_config(self): > > """ copies the bios bits config file into bits. > > """ > > - config_file = 'bits-cfg.txt' > > - bits_config_dir = os.path.join(self._baseDir, 'acpi-bits', > > - 'bits-config') > > + bits_config_file = self.data_file('acpi-bits',
I do not understand, Where is data_file coming from? AFAICS its not there in QemuBaseTest or QemuSystemTest or unittest.TestCase https://docs.python.org/3/library/unittest.html#unittest.TestCase what am I missing or failing to see?