Merge pull request #62 from rossumai/stepan-s/fix-model-automation-bl…

…ocker-samples fix(models:automation_blocker): fix type of `samples`
rossumai · Mar 25, 2024 · e459253 · e459253
2 parents 2e5aeff + 06d2272
commit e459253
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -9,7 +9,7 @@ repos:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.940
+    rev: v0.981
     hooks:
       - id: mypy
         additional_dependencies: [

diff --git a/rossum_api/models/automation_blocker.py b/rossum_api/models/automation_blocker.py
@@ -10,7 +10,7 @@ class AutomationBlockerContent:
     type: str
     schema_id: Optional[str] = None
     samples_truncated: Optional[bool] = False
-    samples: Dict[str, Any] = field(default_factory=dict)
+    samples: List[Dict[str, Any]] = field(default_factory=list)
     details: Dict[str, Any] = field(default_factory=dict)
 
 

diff --git a/tests/elis_api_client/test_annotations.py b/tests/elis_api_client/test_annotations.py
@@ -108,7 +108,16 @@ def dummy_annotation_with_sideloads():
         "automation_blocker": {
             "id": 981916,
             "url": "https://elis.develop.r8.lol/api/v1/automation_blockers/981916",
-            "content": [{"type": "automation_disabled", "level": "annotation"}],
+            "content": [
+                {"type": "automation_disabled", "level": "annotation"},
+                {
+                    "level": "datapoint",
+                    "type": "error_message",
+                    "samples": [
+                        {"details": {"message_content": ["Total Amount is most likely not empty"]}}
+                    ],
+                },
+            ],
             "annotation": "https://elis.develop.r8.lol/api/v1/annotations/3232238",
         },
         "email_thread": None,