On Tue, Dec 03, 2024 at 11:09:14AM +0530, Ani Sinha wrote:
> 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?
It is coming from patch 6 :-)
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
