On Mon, Nov 25, 2024 at 8:51 PM Alex Bennée <[email protected]> wrote: > > From: Daniel P. Berrangé <[email protected]> > > The first comment is still relevant but should talk about our own test > harness instead. The second comment adds no value over reading the code > and can be removed.
I know I am late in the party (been away from QEMU for a bit) but ... > > Signed-off-by: Daniel P. Berrangé <[email protected]> Acked-by: Ani Sinha <[email protected]> > Message-Id: <[email protected]> > Reviewed-by: Pierrick Bouvier <[email protected]> > Signed-off-by: Alex Bennée <[email protected]> > Message-Id: <[email protected]> > > diff --git a/tests/functional/test_acpi_bits.py > b/tests/functional/test_acpi_bits.py > index ee40647d5b..4c192d95cc 100755 > --- a/tests/functional/test_acpi_bits.py > +++ b/tests/functional/test_acpi_bits.py > @@ -196,11 +196,12 @@ def copy_test_scripts(self): > for filename in os.listdir(bits_test_dir): > if os.path.isfile(os.path.join(bits_test_dir, filename)) and \ > filename.endswith('.py2'): > - # all test scripts are named with extension .py2 so that > - # avocado does not try to load them. These scripts are > - # written for python 2.7 not python 3 and hence if avocado > - # loaded them, it would complain about python 3 specific > - # syntaxes. > + # All test scripts are named with extension .py2 so that > + # they are not run by accident. > + # > + # These scripts are intended to run inside the test VM > + # and are written for python 2.7 not python 3, hence > + # would cause syntax errors if loaded ouside the VM. > newfilename = os.path.splitext(filename)[0] + '.py' > shutil.copy2(os.path.join(bits_test_dir, filename), > os.path.join(target_test_dir, newfilename)) > @@ -399,8 +400,6 @@ def test_acpi_smbios_bits(self): > > # biosbits has been configured to run all the specified test suites > # in batch mode and then automatically initiate a vm shutdown. > - # Set timeout to BITS_TIMEOUT for SHUTDOWN event from bits VM at par > - # with the avocado test timeout. > self._vm.event_wait('SHUTDOWN', timeout=BITS_TIMEOUT) > self._vm.wait(timeout=None) > self.logger.debug("Checking console output ...") > -- > 2.39.5 >
