iris-devtester 1.19.1__tar.gz → 1.19.2__tar.gz

{iris_devtester-1.19.1 → iris_devtester-1.19.2}/CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to iris-devtester will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.19.2] - 2026-07-30 - Fix connect_function ImportError when module absent from sys.modules
+
+### Fixed
+
+- **`DBAPIPackageInfo.connect_function` raises after `patch.dict` exit** (`utils/dbapi_compat.py`): `patch.dict(sys.modules, ...)` deletes keys that were absent before the patch when it exits — the common case for unit tests that mock `intersystems_iris` before it has been imported. The 1.19.1 `@property` resolved the MagicMock poisoning but left `sys.modules.get()` returning `None` in this scenario, raising `ImportError: IRIS DBAPI module 'intersystems_iris' is no longer in sys.modules`. The module was importable the whole time. Fixed by falling back to `importlib.import_module(import_path)` when `sys.modules.get()` returns `None`.
+
 ## [1.19.1] - 2026-07-30 - Fix dbapi_compat singleton poisoning under test mocks
 
 ### Fixed

{iris_devtester-1.19.1/iris_devtester.egg-info → iris_devtester-1.19.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iris-devtester
-Version: 1.19.1
+Version: 1.19.2
 Summary: Battle-tested InterSystems IRIS infrastructure utilities for Python development
 Author-email: InterSystems Community <community@intersystems.com>
 Maintainer-email: Thomas Dyar <thomas.dyar@intersystems.com>

{iris_devtester-1.19.1 → iris_devtester-1.19.2}/iris_devtester/__init__.py

@@ -26,7 +26,7 @@ LangChain Integration:
     ...     # Build your RAG app...
 """
 
-__version__ = "1.19.1"
+__version__ = "1.19.2"
 __author__ = "InterSystems Community"
 __license__ = "MIT"
 

{iris_devtester-1.19.1 → iris_devtester-1.19.2}/iris_devtester/utils/dbapi_compat.py

@@ -65,11 +65,17 @@ class DBAPIPackageInfo:
         Resolving at call time (not at detection time) ensures that test patches
         applied via patch.dict(sys.modules, ...) take effect for exactly the
         duration of the patch and do not poison the singleton after the patch exits.
+
+        patch.dict removes keys that were absent before the patch, so after exit
+        the module may be gone from sys.modules even though it is importable.
+        Fall back to importlib.import_module() in that case rather than raising.
         See IDT-BUG-dbapi-adapter-singleton.md.
         """
+        import importlib
+
         module = sys.modules.get(self.import_path)
         if module is None:
-            raise ImportError(f"IRIS DBAPI module '{self.import_path}' is no longer in sys.modules")
+            module = importlib.import_module(self.import_path)
         return getattr(module, self.connect_attr)
 
 

{iris_devtester-1.19.1 → iris_devtester-1.19.2/iris_devtester.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iris-devtester
-Version: 1.19.1
+Version: 1.19.2
 Summary: Battle-tested InterSystems IRIS infrastructure utilities for Python development
 Author-email: InterSystems Community <community@intersystems.com>
 Maintainer-email: Thomas Dyar <thomas.dyar@intersystems.com>

{iris_devtester-1.19.1 → iris_devtester-1.19.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "iris-devtester"
-version = "1.19.1"
+version = "1.19.2"
 description = "Battle-tested InterSystems IRIS infrastructure utilities for Python development"
 readme = "README.md"
 requires-python = ">=3.9"

{iris_devtester-1.19.1 → iris_devtester-1.19.2}/tests/unit/utils/test_dbapi_compat.py

@@ -21,6 +21,74 @@ def reset_adapter():
     dc.reset_adapter()
 
 
+class TestModuleAbsentAfterPatch:
+    """patch.dict removes keys that were absent before the patch on exit.
+
+    The module is importable but not in sys.modules. connect_function must
+    import it rather than raising ImportError.
+    """
+
+    def test_connect_function_importable_when_absent_from_sys_modules(self):
+        """If import_path was absent pre-patch, it's deleted post-patch; must still resolve."""
+        import importlib
+
+        # Ensure the module is absent from sys.modules, then mock it
+        real_module = sys.modules.pop("intersystems_iris", None)
+        fake = MagicMock()
+        fake.connect = object()  # sentinel
+
+        try:
+            assert "intersystems_iris" not in sys.modules, "pre-condition: key must be absent"
+            with patch.dict(sys.modules, {"intersystems_iris": fake}):
+                dc.reset_adapter()
+                dc.get_package_info()  # singleton built while mock is present
+
+            # After patch.dict exit the key is gone (was absent before)
+            assert "intersystems_iris" not in sys.modules
+
+            # connect_function must import the real module, not raise
+            info = dc.get_package_info()
+            # Should not raise; real module is importable
+            fn = info.connect_function
+            assert not isinstance(fn, MagicMock), (
+                "connect_function returned MagicMock after patch exit with absent key"
+            )
+        finally:
+            # Restore original state
+            if real_module is not None:
+                sys.modules["intersystems_iris"] = real_module
+            elif "intersystems_iris" in sys.modules:
+                del sys.modules["intersystems_iris"]
+
+    def test_connect_function_does_not_raise_when_module_absent_from_sys_modules(self):
+        """connect_function must not raise ImportError when module absent but importable."""
+        import importlib
+
+        real_module = sys.modules.pop("intersystems_iris", None)
+        fake = MagicMock()
+        fake.connect = MagicMock()
+
+        try:
+            with patch.dict(sys.modules, {"intersystems_iris": fake}):
+                dc.reset_adapter()
+                info = dc.get_package_info()
+
+            assert "intersystems_iris" not in sys.modules
+
+            # Must not raise
+            dc.reset_adapter()
+            try:
+                info2 = dc.get_package_info()
+                _ = info2.connect_function
+            except ImportError as e:
+                pytest.fail(f"connect_function raised ImportError for importable module: {e}")
+        finally:
+            if real_module is not None:
+                sys.modules["intersystems_iris"] = real_module
+            elif "intersystems_iris" in sys.modules:
+                del sys.modules["intersystems_iris"]
+
+
 class TestSingletonPoisoning:
     """Fix #1: connect resolved at call time, not detection time."""