fix(requests): approvedBy/rejectedBy are lists for multi-approver requests (BLDX-1611)

Live kill-argo testing: listing a page containing already-actioned
requests crashed with a pydantic ValidationError — the server returns
rejectedBy/approvedBy as arrays (multi-approver), while the model (copied
from atlan-java, which is stale here) typed them str. Now
Union[str, List[str]] with a regression test parsing an actioned record.

Author: Aryamanz29

pyatlan/model/atlan_request.py

@@ -4,7 +4,7 @@
 from __future__ import annotations
 
 import json
-from typing import Any, Dict, Generator, List, Optional
+from typing import Any, Dict, Generator, List, Optional, Union
 
 from pydantic.v1 import Field, PrivateAttr, ValidationError, parse_obj_as
 
@@ -132,11 +132,19 @@ class AtlanRequest(AtlanObject):
         default=None,
         description="How the request must be approved: `single`, `unanimous` or `consesus`.",
     )
-    approved_by: Optional[str] = Field(
-        default=None, description="User who approved the request, if approved."
+    approved_by: Optional[Union[str, List[str]]] = Field(
+        default=None,
+        description=(
+            "User(s) who approved the request — a list when the request "
+            "has multiple approvers."
+        ),
     )
-    rejected_by: Optional[str] = Field(
-        default=None, description="User who rejected the request, if rejected."
+    rejected_by: Optional[Union[str, List[str]]] = Field(
+        default=None,
+        description=(
+            "User(s) who rejected the request — a list when the request "
+            "has multiple approvers."
+        ),
     )
     status: Optional[str] = Field(
         default=None,

tests/unit/test_requests_client.py

@@ -82,6 +82,21 @@ def test_list_with_typed_filter(client, mock_api_caller):
     mock_api_caller.reset_mock()
 
 
+def test_actioned_records_with_list_approvers_parse():
+    """rejectedBy/approvedBy come back as LISTS for multi-approver requests —
+    a page containing already-actioned records must parse (regression: the
+    str-typed fields crashed list() on any tenant with actioned requests)."""
+    actioned = {
+        **RAW_REQUEST,
+        "status": "rejected",
+        "rejectedBy": ["admin-one", "admin-two"],
+        "approvedBy": [],
+    }
+    parsed = AtlanRequest(**actioned)
+    assert parsed.rejected_by == ["admin-one", "admin-two"]
+    assert parsed.approved_by == []
+
+
 def test_filter_builder_combines_with_and():
     """Multiple typed filters combine with AND; enums serialize to their
     wire values."""