fix(cleanup): Skip GIL scope when Python is finalized

Avoid crash during cleanup with threading enabled if
Python has been finalized before cleanup.

Author: Thibault-Pelletier

src/PythonQtThreadSupport.cpp

@@ -47,7 +47,7 @@ bool PythonQtGILScope::_enableGILScope = false;
 PythonQtGILScope::PythonQtGILScope()
   : _ensured(false)
 {
-  if (_enableGILScope) {
+  if (_enableGILScope && Py_IsInitialized()) {
     _state = PyGILState_Ensure();
     _ensured = true;
   }

tests/PythonQtTestCleanup.cpp

@@ -79,6 +79,11 @@ void PythonQtTestCleanup::testSignalReceiverCleanup()
 
 void PythonQtTestCleanup::testPyFinalizeThenPythonQtCleanup()
 {
+  QVERIFY(_helper->runScript("import PythonQt.QtCore\n"
+                             "timer = PythonQt.QtCore.QTimer(obj)\n"
+                             "timer.connect('destroyed()', obj.onDestroyed)\n"
+                             "obj.setPassed()\n"));
+
   if (Py_IsInitialized()) {
     Py_Finalize();
   }

tests/PythonQtTestMain.cpp

@@ -49,16 +49,16 @@ int main(int argc, char* argv[])
 {
   QApplication qapp(argc, argv);
 
+  PythonQt::setEnableThreadSupport(QProcessEnvironment::systemEnvironment().contains("PYTHONQT_ENABLE_THREAD_SUPPORT"));
+
   if (QProcessEnvironment::systemEnvironment().contains("PYTHONQT_RUN_ONLY_MEMORY_TESTS")) {
     PythonQtMemoryTests test;
-    QTest::qExec(&test, argc, argv);
-    return 0;
+    return QTest::qExec(&test, argc, argv);
   }
 
   if (QProcessEnvironment::systemEnvironment().contains("PYTHONQT_RUN_ONLY_CLEANUP_TESTS")) {
     PythonQtTestCleanup cleanup;
-    QTest::qExec(&cleanup, argc, argv);
-    return 0;
+    return QTest::qExec(&cleanup, argc, argv);
   }
 
   PythonQt::init(PythonQt::IgnoreSiteModule | PythonQt::RedirectStdOut);