Release the GIL

BadSingleton · BadSingleton · commit 7ec9a6c677cd · 2020-08-20T12:00:46.000-04:00
Addresses comment: #958
diff --git a/src/embed_tests/TestRuntime.cs b/src/embed_tests/TestRuntime.cs
@@ -40,16 +40,27 @@ public static void PlatformCache()
         [Test]
         public static void Py_IsInitializedValue()
         {
-            if (Runtime.Runtime.Py_IsInitialized() == 1)
+            IntPtr state = IntPtr.Zero;
+            try
             {
-                Runtime.Runtime.PyGILState_Ensure();
+                if (Runtime.Runtime.Py_IsInitialized() == 1)
+                {
+                    state = Runtime.Runtime.PyGILState_Ensure();
+                }
+                Runtime.Runtime.Py_Finalize();
+                Assert.AreEqual(0, Runtime.Runtime.Py_IsInitialized());
+                Runtime.Runtime.Py_Initialize();
+                Assert.AreEqual(1, Runtime.Runtime.Py_IsInitialized());
+                Runtime.Runtime.Py_Finalize();
+                Assert.AreEqual(0, Runtime.Runtime.Py_IsInitialized());
+            }
+            finally
+            {
+                if(state != IntPtr.Zero)
+                {
+                    Runtime.Runtime.PyGILState_Release(state);
+                }
             }
-            Runtime.Runtime.Py_Finalize();
-            Assert.AreEqual(0, Runtime.Runtime.Py_IsInitialized());
-            Runtime.Runtime.Py_Initialize();
-            Assert.AreEqual(1, Runtime.Runtime.Py_IsInitialized());
-            Runtime.Runtime.Py_Finalize();
-            Assert.AreEqual(0, Runtime.Runtime.Py_IsInitialized());
         }
 
         [Test]
