From 4b0b9787af75e8fa76b85c63de186da413a5b4e0 Mon Sep 17 00:00:00 2001
From: crisfiorini-oss <cristopher.fiorini@obin.ai>
Date: Tue, 27 Oct 2026 17:48:48 +0000
Subject: [PATCH] fix(python): handle translated Azure missing-blob errors in
 exists

---
 .../apache_beam/io/azure/blobstorageio.py     |  4 +-
 .../io/azure/blobstorageio_test.py            | 40 +++++++++++++++++++
 2 files changed, 42 insertions(+), 2 deletions(-)

diff --git a/sdks/python/apache_beam/io/azure/blobstorageio.py b/sdks/python/apache_beam/io/azure/blobstorageio.py
index bc017ff7b214..839d69194b7d 100644
--- a/sdks/python/apache_beam/io/azure/blobstorageio.py
+++ b/sdks/python/apache_beam/io/azure/blobstorageio.py
@@ -336,8 +336,8 @@ def exists(self, path):
     try:
       self._blob_properties(path)
       return True
-    except ResourceNotFoundError as e:
-      if e.status_code == 404:
+    except BlobStorageError as e:
+      if e.code == 404:
         # HTTP 404 indicates that the file did not exist.
         return False
       else:
diff --git a/sdks/python/apache_beam/io/azure/blobstorageio_test.py b/sdks/python/apache_beam/io/azure/blobstorageio_test.py
index 262a75b756a8..91920034e59e 100644
--- a/sdks/python/apache_beam/io/azure/blobstorageio_test.py
+++ b/sdks/python/apache_beam/io/azure/blobstorageio_test.py
@@ -21,10 +21,13 @@
 
 import logging
 import unittest
+from unittest import mock
 
 # Protect against environments where azure library is not available.
 # pylint: disable=wrong-import-order, wrong-import-position
 try:
+  from azure.core.exceptions import HttpResponseError
+  from azure.core.exceptions import ResourceNotFoundError
   from apache_beam.io.azure import blobstorageio
 except ImportError:
   blobstorageio = None  # type: ignore[assignment]
@@ -86,6 +89,43 @@ def test_bad_azfs_path_blob_optional(self):
       self.assertRaises(ValueError, blobstorageio.parse_azfs_path, path, True)
 
 
+@unittest.skipIf(
+    blobstorageio is None or not blobstorageio.AZURE_DEPS_INSTALLED,
+    'Azure dependencies are not installed')
+class TestBlobStorageExists(unittest.TestCase):
+  def setUp(self):
+    self.client = mock.Mock()
+    self.blob = self.client.get_blob_client.return_value
+    self.io = blobstorageio.BlobStorageIO(client=self.client)
+    self.path = "azfs://storageaccount/container/new-output.txt"
+
+  def test_existing_blob(self):
+    self.assertTrue(self.io.exists(self.path))
+    self.client.get_blob_client.assert_called_once_with(
+        "container", "new-output.txt")
+
+  def test_missing_blob_returns_false_after_error_translation(self):
+    error = ResourceNotFoundError("The specified blob does not exist.")
+    error.status_code = 404
+    self.blob.get_blob_properties.side_effect = error
+    self.assertFalse(self.io.exists(self.path))
+
+  def test_authorization_failure_is_not_reported_as_missing(self):
+    error = HttpResponseError("AuthorizationPermissionMismatch")
+    error.status_code = 403
+    self.blob.get_blob_properties.side_effect = error
+    with self.assertRaises(HttpResponseError) as caught:
+      self.io.exists(self.path)
+    self.assertIs(caught.exception, error)
+
+  def test_other_translated_errors_are_not_reported_as_missing(self):
+    error = blobstorageio.BlobStorageError("unexpected status", 500)
+    with mock.patch.object(self.io, "_blob_properties", side_effect=error), \
+        self.assertRaises(blobstorageio.BlobStorageError) as caught:
+      self.io.exists(self.path)
+    self.assertIs(caught.exception, error)
+
+
 if __name__ == '__main__':
   logging.getLogger().setLevel(logging.INFO)
   unittest.main()