This is an automated email from the ASF dual-hosted git repository.
pankajastro pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 249e5cbcbc1 Fix Modal contract test importing a private class from its
old location (#75188)
249e5cbcbc1 is described below
commit 249e5cbcbc197947eb1bdac74bfd7c427d5503fe
Author: Pankaj Singh <[email protected]>
AuthorDate: Tue Oct 20 15:55:25 2026 +0530
Fix Modal contract test importing a private class from its old location
(#75188)
* Fix Modal contract test importing a private class from its old location
modal 1.6 turned modal.sandbox into a package and moved _SandboxFilesystem
to modal.sandbox._filesystem, so the vendor contract test failed with an
ImportError on any job that resolved the latest modal.
Co-Authored-By: Claude <[email protected]>
* Update providers/common/ai/tests/unit/common/ai/sandbox/test_modal.py
Co-authored-by: Kaxil Naik <[email protected]>
---------
Co-authored-by: Claude <[email protected]>
Co-authored-by: Kaxil Naik <[email protected]>
---
providers/common/ai/tests/unit/common/ai/sandbox/test_modal.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/providers/common/ai/tests/unit/common/ai/sandbox/test_modal.py
b/providers/common/ai/tests/unit/common/ai/sandbox/test_modal.py
index 348f3865563..6987a8262f3 100644
--- a/providers/common/ai/tests/unit/common/ai/sandbox/test_modal.py
+++ b/providers/common/ai/tests/unit/common/ai/sandbox/test_modal.py
@@ -144,7 +144,8 @@ class TestVendorContract:
def test_the_calls_behind_the_tools_still_exist(self, real_modal):
import inspect
- from modal.sandbox import _SandboxFilesystem
+ # Not imported: the private module path changed in modal 1.6.2.
+ _SandboxFilesystem =
inspect.signature(real_modal.Sandbox.filesystem.fget).return_annotation
assert
list(inspect.signature(_SandboxFilesystem.write_bytes).parameters)[1:3] == [
"data",
