Improve PythonQt test cleanup consistency and add alternative cleanup order test

[jcfr]

PythonQtTestCleanup:

• Adds a new test to check the behavior when Python is finalized before calling PythonQt cleanup.
• Consistently indent test case and use helper (_helper->runScript) everywhere

---

The addition of the test PythonQtTestCleanup::testPyFinalizeThenPythonQtCleanup seems to indicate that some of the changes proposed in the following pull request may not be needed:

• Make it possible to use Py_Finalize before calling the PythonQt destructor #267

The new test also check that the cleanup order currently used in CTK¹ (and by extension Slicer²) works as expected.

Footnotes

1. https://github.com/commontk/CTK/blob/833ae56e69d403bd0c83f2c331a32a8fd9c70285/Libs/Scripting/Python/Core/ctkAbstractPythonManager.cpp#L85-L92 ↩

2. https://github.com/Slicer/Slicer/blob/b7ff98e7ce49d5f57ee67be3ee9b76dc03acbe95/Base/QTCore/qSlicerCorePythonManager.h ↩

[mrbean-bremen]

Looks good, thanks!

[jcfr]

So that we have a green CI, I suggest you rerun the failed test associated with https://github.com/MeVisLab/pythonqt/actions/runs/17634428359/job/50107927757. It looks this was a failure related to Qt installation.