Extend testing (#107)

* Test headers

* ci: Format code

* Update test/headers_test.py

* ci: Format code

* Fix headers test

* Fix header definition

* Fix lint

* Rename client_test.py

* Test waiting for action attempt feature

* ci: Format code

* Remove Seam class Routes inheritence

* Use Seam().client in headers test

* Remove duplicate code

* Don't adjust client to accommodate the test, test should replicate the behaviour of the lib

---------

Co-authored-by: Seam Bot <devops@getseam.com>

Author: andrii-balitskyi

seam/seam_multi_workspace.py

@@ -60,11 +60,6 @@ def __init__(
             auth_headers=auth_headers,
         )
 
-        self.client = SeamHttpClient(
-            base_url=endpoint,
-            auth_headers=auth_headers,
-        )
-
         defaults = {"wait_for_action_attempt": wait_for_action_attempt}
 
         self._workspaces = Workspaces(client=self.client, defaults=defaults)

test/api_key_test.py

@@ -30,7 +30,7 @@ def test_seam_client_constructor_interprets_single_string_argument_as_api_key(se
     assert seam is not None
 
     with pytest.raises(SeamInvalidTokenError, match=r"api_key"):
-        Seam(api_key="some-invalid-key-format")
+        Seam("some-invalid-key-format")
 
 
 def test_seam_client_checks_api_key_format():

test/headers_test.py

@@ -0,0 +1,41 @@
+import niquests
+import uuid
+from unittest.mock import patch, Mock
+from seam import Seam
+from importlib.metadata import version
+from seam.constants import LTS_VERSION
+
+
+def test_seam_http_client_request(server):
+    endpoint, seed = server
+    seam = Seam.from_api_key(seed["seam_apikey1_token"], endpoint=endpoint)
+    device_id = str(uuid.uuid4())
+
+    mock_response = Mock()
+    mock_response.status_code = 200
+    mock_response.headers = {"content-type": "application/json"}
+    mock_response_data = {"device": {"device_id": device_id}}
+    mock_response.json.return_value = mock_response_data
+
+    with patch.object(
+        niquests.Session, "request", return_value=mock_response
+    ) as mock_request:
+        response = seam.client.post("/devices/get", json={"device_id": device_id})
+
+        mock_request.assert_called_once()
+        args, _ = mock_request.call_args
+
+        assert args[0] == "POST"
+        assert args[1] == f"{endpoint}/devices/get"
+
+        passed_headers = mock_request.call_args.kwargs["headers"] or {}
+        request_headers = {
+            **seam.client.headers,
+            **passed_headers,
+        }
+
+        assert request_headers["seam-sdk-name"] == "seamapi/python"
+        assert request_headers["seam-sdk-version"] == version("seam")
+        assert request_headers["seam-lts-version"] == LTS_VERSION
+
+        assert response == mock_response_data

test/client_test.py test/http_error_test.pytest/client_test.py renamed to test/http_error_test.py

@@ -0,0 +1,48 @@
+import pytest
+from seam import Seam
+
+
+def test_wait_for_action_attempt_directly_on_returned_action_attempt(server):
+    endpoint, seed = server
+    seam = Seam.from_api_key(
+        seed["seam_apikey1_token"], endpoint=endpoint, wait_for_action_attempt=False
+    )
+
+    action_attempt = seam.locks.unlock_door(
+        device_id=seed["august_device_1"], wait_for_action_attempt=True
+    )
+
+    assert action_attempt.status == "success"
+
+
+def test_wait_for_action_attempt_by_default(server):
+    endpoint, seed = server
+    seam = Seam.from_api_key(seed["seam_apikey1_token"], endpoint=endpoint)
+
+    action_attempt = seam.locks.unlock_door(device_id=seed["august_device_1"])
+
+    assert action_attempt.status == "success"
+
+
+def test_wait_for_action_attempt_can_set_class_default(server):
+    endpoint, seed = server
+    seam = Seam.from_api_key(
+        seed["seam_apikey1_token"], endpoint=endpoint, wait_for_action_attempt=False
+    )
+
+    action_attempt = seam.locks.unlock_door(device_id=seed["august_device_1"])
+
+    assert action_attempt.status == "pending"
+
+
+def test_wait_for_action_attempt_can_set_class_default_with_object(server):
+    endpoint, seed = server
+    seam = Seam.from_api_key(
+        seed["seam_apikey1_token"],
+        endpoint=endpoint,
+        wait_for_action_attempt={"timeout": 5000},
+    )
+
+    action_attempt = seam.locks.unlock_door(device_id=seed["august_device_1"])
+
+    assert action_attempt.status == "success"