build: hide extension to test other cores

Author: nedbat

igor.py

@@ -52,52 +52,50 @@ def do_show_env():
         print(f"  {env} = {os.environ[env]!r}")
 
 
-def do_remove_extension(*args):
+def remove_extension(core):
     """Remove the compiled C extension, no matter what its name."""
 
+    if core == "ctrace":
+        return
+
     so_patterns = """
         tracer.so
         tracer.*.so
         tracer.pyd
         tracer.*.pyd
         """.split()
 
-    if "--from-install" in args:
-        # Get the install location using a subprocess to avoid
-        # locking the file we are about to delete
-        root = os.path.dirname(
-            subprocess.check_output(
-                [
-                    sys.executable,
-                    "-Xutf8",
-                    "-c",
-                    "import coverage; print(coverage.__file__)",
-                ],
-                encoding="utf-8",
-            ).strip(),
-        )
-        roots = [root]
-    else:
-        roots = [
-            "coverage",
-            "build/*/coverage",
-            ".tox/*/[Ll]ib/*/site-packages/coverage",
-            ".tox/*/[Ll]ib/site-packages/coverage",
-        ]
+    roots = [
+        "coverage",
+        "build/*/coverage",
+        ".tox/*/[Ll]ib/*/site-packages/coverage",
+        ".tox/*/[Ll]ib/site-packages/coverage",
+    ]
 
+    # On windows at least, we can't delete a loaded .pyd file. So move them
+    # out of the way into the tmp/ directory.
+    os.makedirs("tmp", exist_ok=True)
     for root, pattern in itertools.product(roots, so_patterns):
         pattern = os.path.join(root, pattern)
         if VERBOSITY > 1:
             print(f"Searching for {pattern} from {os.getcwd()}")
         for filename in glob.glob(pattern):
             if os.path.exists(filename):
+                hidden = f"tmp/{os.path.basename(filename)}"
                 if VERBOSITY > 1:
-                    print(f"Removing {os.path.abspath(filename)}")
+                    print(f"Moving {filename} to {hidden}")
                 try:
-                    os.remove(filename)
+                    if os.path.exists(hidden):
+                        os.remove(hidden)
                 except OSError as exc:
                     if VERBOSITY > 1:
-                        print(f"Couldn't remove {os.path.abspath(filename)}: {exc}")
+                        print(f"Couldn't remove {hidden}: {exc}")
+                else:
+                    try:
+                        os.rename(filename, hidden)
+                    except OSError as exc:
+                        if VERBOSITY > 1:
+                            print(f"Couldn't rename: {exc}")
 
 
 def label_for_core(core):
@@ -157,10 +155,10 @@ def make_env_id(core):
 
 def run_tests(core, *runner_args):
     """The actual running of tests."""
+    remove_extension(core)
     if "COVERAGE_TESTING" not in os.environ:
         os.environ["COVERAGE_TESTING"] = "True"
     print_banner(label_for_core(core))
-
     return pytest.main(list(runner_args))
 
 
@@ -209,6 +207,8 @@ def run_tests_with_coverage(core, *runner_args):
                 if getattr(mod, "__file__", "??").startswith(covdir):
                     covmods[name] = mod
                     del sys.modules[name]
+        remove_extension(core)
+
         import coverage  # pylint: disable=reimported
 
         sys.modules.update(covmods)

tox.ini

@@ -42,17 +42,15 @@ commands =
     # Create tests/zipmods.zip
     python igor.py zip_mods
 
-    # Remove the C extension so that we can test the PyTracer
-    python igor.py remove_extension
-
-    # Test with the PyTracer
-    python igor.py test_with_core pytrace {posargs}
-
     # Build the C extension and test with the CTracer
     python setup.py --quiet build_ext --inplace
     python -m pip install {env:COVERAGE_PIP_ARGS} -q -e .
     python igor.py test_with_core ctrace {posargs}
 
+    # Test with the PyTracer
+    python igor.py test_with_core pytrace {posargs}
+
+    # Test with sys.monitoring
     py3{12-15}{,t},anypy: python igor.py test_with_core sysmon {posargs}
 
 [testenv:anypy]