fix: move packages config to [tool.poetry] section and fix test UUID handling

- Move packages configuration from [project] to [tool.poetry] section
  to fix poetry install failure when using setuptools as build backend
- Fix UUID type handling in test fixtures (live_video, motion_capture, workflow_runs)
- Add restore_test_applicant fixture to prevent test failures from deleted applicants
- Fix document_id assertion to compare string representation in task tests

Author: dvacca-onfido

pyproject.toml

@@ -8,9 +8,6 @@ authors = [
 license = { text = "MIT" }
 readme = "README.md"
 keywords = ["OpenAPI", "OpenAPI-Generator", "onfido", "identity"]
-packages = [
-    { include = "onfido" }
-]
 requires-python = ">=3.9"
 
 dependencies = [
@@ -26,6 +23,9 @@ Repository = "https://github.com/onfido/onfido-python"
 
 [tool.poetry]
 requires-poetry = ">=2.0"
+packages = [
+    { include = "onfido" }
+]
 
 [tool.poetry.group.dev.dependencies]
 pytest = ">= 7.2.1"

tests/test_live_video.py

@@ -2,6 +2,17 @@
 
 from onfido import ApiException, LiveVideo, LiveVideosList
 from os import environ
+from uuid import UUID
+
+
+@pytest.fixture(scope="module", autouse=True)
+def restore_test_applicant(onfido_api):
+    """Restore the test applicant before running live video tests in case it's scheduled for deletion."""
+    try:
+        onfido_api.restore_applicant(environ["ONFIDO_SAMPLE_APPLICANT_ID"])
+    except ApiException:
+        # Applicant may not be deleted, ignore the error
+        pass
 
 
 @pytest.fixture(scope="function")
@@ -11,7 +22,7 @@ def applicant_id(onfido_api):
 
 @pytest.fixture(scope="function")
 def live_video_id(onfido_api):
-    return environ["ONFIDO_SAMPLE_VIDEO_ID_1"]
+    return UUID(environ["ONFIDO_SAMPLE_VIDEO_ID_1"])
 
 
 def test_list_live_videos(onfido_api, applicant_id):

tests/test_motion_capture.py

@@ -2,6 +2,7 @@
 
 from onfido import ApiException, MotionCapture, MotionCapturesList
 from os import environ
+from uuid import UUID
 
 
 @pytest.fixture(scope="function")
@@ -11,7 +12,7 @@ def applicant_id(onfido_api):
 
 @pytest.fixture(scope="function")
 def motion_id(onfido_api):
-    return environ["ONFIDO_SAMPLE_MOTION_ID_1"]
+    return UUID(environ["ONFIDO_SAMPLE_MOTION_ID_1"])
 
 
 def test_list_motion_captures(onfido_api, applicant_id):

tests/test_tasks.py

@@ -86,5 +86,5 @@ def test_complete_task(onfido_api, workflow_run_id, document_id):
         workflow_run_id,
         document_photo_task_id).output
 
-    assert document_photo_task_outputs == [{"id": document_id,
+    assert document_photo_task_outputs == [{"id": str(document_id),
                                             "type": "document"}]

tests/test_workflow_runs.py

@@ -2,6 +2,7 @@
 import zipfile
 import pytest
 
+from uuid import UUID
 from onfido import (
     TimelineFileReference,
     WorkflowRun,
@@ -23,7 +24,7 @@ def applicant_id(onfido_api):
 
 @pytest.fixture(scope="function")
 def workflow_id():
-    return "e8c921eb-0495-44fe-b655-bcdcaffdafe5"
+    return UUID("e8c921eb-0495-44fe-b655-bcdcaffdafe5")
 
 
 @pytest.fixture(scope="function")
@@ -41,7 +42,7 @@ def test_create_workflow_run(workflow_run, workflow_id):
 
 
 def test_create_workflow_run_with_custom_inputs(onfido_api, applicant_id):
-    workflow_id = "45092b29-f220-479e-aa6f-a6f989baac4c"
+    workflow_id = UUID("45092b29-f220-479e-aa6f-a6f989baac4c")
 
     workflow_run_builder = WorkflowRunBuilder(
         applicant_id=applicant_id,