From c04a0063f4d7493a1ce33b080c6485e601f5701d Mon Sep 17 00:00:00 2001 From: Eugene Yurtsev Date: Mon, 9 Feb 2026 16:46:24 -0500 Subject: [PATCH] x --- .../integration_tests/sandboxes.py | 29 +++++-------------- 1 file changed, 8 insertions(+), 21 deletions(-) diff --git a/libs/standard-tests/langchain_tests/integration_tests/sandboxes.py b/libs/standard-tests/langchain_tests/integration_tests/sandboxes.py index 0f8c27339b5..64221337929 100644 --- a/libs/standard-tests/langchain_tests/integration_tests/sandboxes.py +++ b/libs/standard-tests/langchain_tests/integration_tests/sandboxes.py @@ -57,9 +57,6 @@ class SandboxIntegrationTests(BaseStandardTests): sandbox.execute("rm -rf /tmp/test_sandbox_ops && mkdir -p /tmp/test_sandbox_ops") yield sandbox - @property - def supports_distinct_download_errors(self) -> bool: - return True @abstractmethod @pytest.fixture(scope="class") @@ -263,15 +260,10 @@ class SandboxIntegrationTests(BaseStandardTests): responses = sandbox_backend.download_files([dir_path]) - if not self.supports_distinct_download_errors: - assert responses == [ - FileDownloadResponse(path=dir_path, content=None, error="file_not_found") - ] - return - - assert responses == [ - FileDownloadResponse(path=dir_path, content=None, error="is_directory") - ] + assert len(responses) == 1 + assert responses[0].path == dir_path + assert responses[0].content is None + assert responses[0].error in {"is_directory", "file_not_found", "invalid_path"} def test_download_error_permission_denied(self, sandbox_backend: SandboxBackendProtocol) -> None: if not self.has_sync: @@ -287,15 +279,10 @@ class SandboxIntegrationTests(BaseStandardTests): finally: sandbox_backend.execute(f"chmod 644 {test_path} || true") - if not self.supports_distinct_download_errors: - assert responses == [ - FileDownloadResponse(path=test_path, content=None, error="file_not_found") - ] - return - - assert responses == [ - FileDownloadResponse(path=test_path, content=None, error="permission_denied") - ] + assert len(responses) == 1 + assert responses[0].path == test_path + assert responses[0].content is None + assert responses[0].error in {"permission_denied", "file_not_found", "invalid_path"} def test_download_error_invalid_path_relative( self,