tests: do not reload the tags module (#1152)

Signed-off-by: Henry Schreiner <henryfs@princeton.edu>

Author: henryiii

src/packaging/tags.py

@@ -70,7 +70,13 @@ def __dir__() -> list[str]:
 }
 
 
-_32_BIT_INTERPRETER = struct.calcsize("P") == 4
+# This function can be unit tested without reloading the module
+# (Unlike _32_BIT_INTERPRETER)
+def _compute_32_bit_interpreter() -> bool:
+    return struct.calcsize("P") == 4
+
+
+_32_BIT_INTERPRETER = _compute_32_bit_interpreter()
 
 
 class Tag:

tests/test_tags.py

@@ -11,7 +11,6 @@
     import ctypes
 except ImportError:
     ctypes = None  # type: ignore[assignment]
-import importlib
 import os
 import pathlib
 import pickle
@@ -1806,9 +1805,6 @@ def test_pyemscripten(
 
 
 class TestBitness:
-    def teardown_method(self) -> None:
-        importlib.reload(tags)
-
     @pytest.mark.parametrize(
         ("maxsize", "sizeof_voidp", "expected"),
         [
@@ -1833,8 +1829,7 @@ def _calcsize(fmt: str) -> int:
 
         monkeypatch.setattr(sys, "maxsize", maxsize)
         monkeypatch.setattr(struct, "calcsize", _calcsize)
-        importlib.reload(tags)
-        assert expected == tags._32_BIT_INTERPRETER
+        assert expected == tags._compute_32_bit_interpreter()
 
 
 def test_pickle() -> None: