Remove shutdown from most tests, disable the rest for now

filmor · Dec 8, 2025 · Oct 22, 2025 · Oct 23, 2025 · Oct 23, 2025 · Oct 23, 2025
commit e7b51d770d6ea9705139a17d8b23a8f506239964
diff --git a/src/embed_tests/Events.cs b/src/embed_tests/Events.cs
@@ -10,18 +10,6 @@ namespace Python.EmbeddingTest;
 
 public class Events
 {
-    [OneTimeSetUp]
-    public void SetUp()
-    {
-        PythonEngine.Initialize();
-    }
-
-    [OneTimeTearDown]
-    public void Dispose()
-    {
-        PythonEngine.Shutdown();
-    }
-
     [Test]
     public void UsingDoesNotLeak()
     {

diff --git a/src/embed_tests/Modules.cs b/src/embed_tests/Modules.cs
@@ -28,18 +28,6 @@ public void Dispose()
             }
         }
 
-        [OneTimeSetUp]
-        public void OneTimeSetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void OneTimeTearDown()
-        {
-            PythonEngine.Shutdown();
-        }
-
         /// <summary>
         /// Eval a Python expression and obtain its return value.
         /// </summary>

diff --git a/src/embed_tests/StateSerialization/MethodSerialization.cs b/src/embed_tests/StateSerialization/MethodSerialization.cs
@@ -20,7 +20,7 @@ public void GenericRoundtrip()
     }
 
     [Test]
-    public void ConstrctorRoundtrip()
+    public void ConstructorRoundtrip()
     {
         var ctor = typeof(MethodTestHost).GetConstructor(new[] { typeof(int) });
         var maybeConstructor = new MaybeMethodBase<MethodBase>(ctor);
@@ -33,6 +33,10 @@ static T SerializationRoundtrip<T>(T item)
     {
         using var buf = new MemoryStream();
         var formatter = RuntimeData.CreateFormatter();
+        if (typeof(NoopFormatter).IsAssignableFrom(formatter.GetType()))
+        {
+            Assert.Inconclusive("NoopFormatter in use, cannot perform serialization test.");
+        }
         formatter.Serialize(buf, item);
         buf.Position = 0;
         return (T)formatter.Deserialize(buf);

diff --git a/src/embed_tests/TestFinalizer.cs b/src/embed_tests/TestFinalizer.cs
@@ -86,6 +86,7 @@ public void CollectBasicObject()
         }
 
         [Test]
+        [Ignore("Requires explicit shutdown")]
         [Obsolete("GC tests are not guaranteed")]
         public void CollectOnShutdown()
         {

diff --git a/src/embed_tests/TestPyInt.cs b/src/embed_tests/TestPyInt.cs
@@ -10,18 +10,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPyInt
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void TestCtorInt()
         {

diff --git a/src/embed_tests/TestPyIter.cs b/src/embed_tests/TestPyIter.cs
@@ -9,18 +9,6 @@ namespace Python.EmbeddingTest
 {
     class TestPyIter
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void KeepOldObjects()
         {

diff --git a/src/embed_tests/TestPyList.cs b/src/embed_tests/TestPyList.cs
@@ -7,18 +7,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPyList
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void TestStringIsListType()
         {

diff --git a/src/embed_tests/TestPyNumber.cs b/src/embed_tests/TestPyNumber.cs
@@ -6,18 +6,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPyNumber
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void IsNumberTypeTrue()
         {

diff --git a/src/embed_tests/TestPyObject.cs b/src/embed_tests/TestPyObject.cs
@@ -8,18 +8,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPyObject
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void TestGetDynamicMemberNames()
         {

diff --git a/src/embed_tests/TestPySequence.cs b/src/embed_tests/TestPySequence.cs
@@ -6,18 +6,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPySequence
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void TestIsSequenceTrue()
         {

diff --git a/src/embed_tests/TestPyString.cs b/src/embed_tests/TestPyString.cs
@@ -6,18 +6,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPyString
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void TestStringCtor()
         {

diff --git a/src/embed_tests/TestPyTuple.cs b/src/embed_tests/TestPyTuple.cs
@@ -6,18 +6,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPyTuple
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         /// <summary>
         /// Test IsTupleType without having to Initialize a tuple.
         /// PyTuple constructor use IsTupleType. This decouples the tests.

diff --git a/src/embed_tests/TestPyType.cs b/src/embed_tests/TestPyType.cs
@@ -10,18 +10,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPyType
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void CanCreateHeapType()
         {

diff --git a/src/embed_tests/TestPyWith.cs b/src/embed_tests/TestPyWith.cs
@@ -6,18 +6,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPyWith
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         /// <summary>
         /// Test that exception is raised in context manager that ignores it.
         /// </summary>

diff --git a/src/embed_tests/TestPythonEngineProperties.cs b/src/embed_tests/TestPythonEngineProperties.cs
@@ -9,7 +9,6 @@ public class TestPythonEngineProperties
         [Test]
         public static void GetBuildinfoDoesntCrash()
         {
-            PythonEngine.Initialize();
             using (Py.GIL())
             {
                 string s = PythonEngine.BuildInfo;
@@ -22,7 +21,6 @@ public static void GetBuildinfoDoesntCrash()
         [Test]
         public static void GetCompilerDoesntCrash()
         {
-            PythonEngine.Initialize();
             using (Py.GIL())
             {
                 string s = PythonEngine.Compiler;
@@ -36,7 +34,6 @@ public static void GetCompilerDoesntCrash()
         [Test]
         public static void GetCopyrightDoesntCrash()
         {
-            PythonEngine.Initialize();
             using (Py.GIL())
             {
                 string s = PythonEngine.Copyright;
@@ -49,7 +46,6 @@ public static void GetCopyrightDoesntCrash()
         [Test]
         public static void GetPlatformDoesntCrash()
         {
-            PythonEngine.Initialize();
             using (Py.GIL())
             {
                 string s = PythonEngine.Platform;
@@ -62,7 +58,6 @@ public static void GetPlatformDoesntCrash()
         [Test]
         public static void GetVersionDoesntCrash()
         {
-            PythonEngine.Initialize();
             using (Py.GIL())
             {
                 string s = PythonEngine.Version;
@@ -75,21 +70,17 @@ public static void GetVersionDoesntCrash()
         [Test]
         public static void GetPythonPathDefault()
         {
-            PythonEngine.Initialize();
             string s = PythonEngine.PythonPath;
 
             StringAssert.Contains("python", s.ToLower());
-            PythonEngine.Shutdown();
         }
 
         [Test]
         public static void GetProgramNameDefault()
         {
-            PythonEngine.Initialize();
             string s = PythonEngine.ProgramName;
 
             Assert.NotNull(s);
-            PythonEngine.Shutdown();
         }
 
         /// <summary>
@@ -101,13 +92,12 @@ public static void GetPythonHomeDefault()
         {
             string envPythonHome = Environment.GetEnvironmentVariable("PYTHONHOME") ?? "";
 
-            PythonEngine.Initialize();
             string enginePythonHome = PythonEngine.PythonHome;
 
             Assert.AreEqual(envPythonHome, enginePythonHome);
-            PythonEngine.Shutdown();
         }
 
+        [Ignore("Only works if we can shutdown and re-init the interpreter")]
         [Test]
         public void SetPythonHome()
         {
@@ -130,6 +120,7 @@ public void SetPythonHome()
             PythonEngine.PythonHome = pythonHomeBackup;
         }
 
+        [Ignore("Only works if we can shutdown and re-init the interpreter")]
         [Test]
         public void SetPythonHomeTwice()
         {
@@ -172,6 +163,7 @@ public void SetPythonHomeEmptyString()
             PythonEngine.Shutdown();
         }
 
+        [Ignore("Only works if we can shutdown and re-init the interpreter")]
         [Test]
         public void SetProgramName()
         {
@@ -193,6 +185,7 @@ public void SetProgramName()
             PythonEngine.ProgramName = programNameBackup;
         }
 
+        [Ignore("Only works if we can shutdown and re-init the interpreter")]
         [Test]
         public void SetPythonPath()
         {

diff --git a/src/embed_tests/TestPythonException.cs b/src/embed_tests/TestPythonException.cs
@@ -6,18 +6,6 @@ namespace Python.EmbeddingTest
 {
     public class TestPythonException
     {
-        [OneTimeSetUp]
-        public void SetUp()
-        {
-            PythonEngine.Initialize();
-        }
-
-        [OneTimeTearDown]
-        public void Dispose()
-        {
-            PythonEngine.Shutdown();
-        }
-
         [Test]
         public void TestMessage()
         {

diff --git a/src/embed_tests/TestRuntime.cs b/src/embed_tests/TestRuntime.cs
@@ -5,6 +5,7 @@
 
 namespace Python.EmbeddingTest
 {
+    [Ignore("Only works if we can shutdown and re-initialize the Python runtime")]
     public class TestRuntime
     {
         [OneTimeSetUp]

diff --git a/src/embed_tests/pyimport.cs b/src/embed_tests/pyimport.cs
@@ -42,7 +42,8 @@ public void SetUp()
         [OneTimeTearDown]
         public void Dispose()
         {
-            // TODO Undo the above
+            using var _ = Py.GIL();
+            Py.Import("sys").GetAttr("path").InvokeMethod("remove", new PyString(TestPath));
         }
 
         /// <summary>
-Original file line number
+Diff line change
@@ Expand Up / @@ -86,6 +86,7 @@ public void CollectBasicObject() @@
             }
             [Test]
+            [Ignore("Requires explicit shutdown")]
             [Obsolete("GC tests are not guaranteed")]
             public void CollectOnShutdown()
             {
@@ Expand Down @@