Wrap response operation-name empty-check length failures

Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>

Author: cursoragent

hyperbrowser/client/managers/response_utils.py

@@ -64,14 +64,15 @@ def parse_response_model(
         normalized_operation_name_input = operation_name.strip()
         if not isinstance(normalized_operation_name_input, str):
             raise TypeError("normalized operation_name must be a string")
+        is_empty_operation_name = len(normalized_operation_name_input) == 0
     except HyperbrowserError:
         raise
     except Exception as exc:
         raise HyperbrowserError(
             "Failed to normalize operation_name",
             original_error=exc,
         ) from exc
-    if not normalized_operation_name_input:
+    if is_empty_operation_name:
         raise HyperbrowserError("operation_name must be a non-empty string")
     normalized_operation_name = _normalize_operation_name_for_error(operation_name)
     if not isinstance(response_data, Mapping):

tests/test_response_utils.py

@@ -256,6 +256,48 @@ def strip(self, chars=None):  # type: ignore[override]
     assert isinstance(exc_info.value.original_error, TypeError)
 
 
+def test_parse_response_model_wraps_operation_name_empty_check_length_failures():
+    class _BrokenOperationName(str):
+        class _NormalizedName(str):
+            def __len__(self):
+                raise RuntimeError("operation name length exploded")
+
+        def strip(self, chars=None):  # type: ignore[override]
+            _ = chars
+            return self._NormalizedName("basic operation")
+
+    with pytest.raises(HyperbrowserError, match="Failed to normalize operation_name") as exc_info:
+        parse_response_model(
+            {"success": True},
+            model=BasicResponse,
+            operation_name=_BrokenOperationName("basic operation"),
+        )
+
+    assert isinstance(exc_info.value.original_error, RuntimeError)
+
+
+def test_parse_response_model_preserves_hyperbrowser_operation_name_empty_check_length_failures():
+    class _BrokenOperationName(str):
+        class _NormalizedName(str):
+            def __len__(self):
+                raise HyperbrowserError("custom operation name length failure")
+
+        def strip(self, chars=None):  # type: ignore[override]
+            _ = chars
+            return self._NormalizedName("basic operation")
+
+    with pytest.raises(
+        HyperbrowserError, match="custom operation name length failure"
+    ) as exc_info:
+        parse_response_model(
+            {"success": True},
+            model=BasicResponse,
+            operation_name=_BrokenOperationName("basic operation"),
+        )
+
+    assert exc_info.value.original_error is None
+
+
 def test_parse_response_model_truncates_operation_name_in_errors():
     long_operation_name = "basic operation " + ("x" * 200)