On 9/2/21 11:58 AM, Vladimir Sementsov-Ogievskiy wrote: > 02.09.2021 12:40, Hanna Reitz wrote: >> The AbnormalShutdown exception class is not in qemu.machine, but in >> qemu.machine.machine. (qemu.machine.AbnormalShutdown was enough for >> Python to find it in order to run this test, but pylint complains about >> it.) >> >> Signed-off-by: Hanna Reitz <hre...@redhat.com> >> --- >> tests/qemu-iotests/tests/mirror-top-perms | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tests/qemu-iotests/tests/mirror-top-perms >> b/tests/qemu-iotests/tests/mirror-top-perms >> index 451a0666f8..2fc8dd66e0 100755 >> --- a/tests/qemu-iotests/tests/mirror-top-perms >> +++ b/tests/qemu-iotests/tests/mirror-top-perms >> @@ -47,7 +47,7 @@ class TestMirrorTopPerms(iotests.QMPTestCase): >> def tearDown(self): >> try: >> self.vm.shutdown() >> - except qemu.machine.AbnormalShutdown: >> + except qemu.machine.machine.AbnormalShutdown: >> pass >> if self.vm_b is not None: >> > > Hmm, interesting.. May be that bad that module has same name as subpackage?
Confusing indeed. Could this be improved?
