Fix workspace bounding box parsing (#191)

This pr adds a fix to the workspace bounding box parsing as well as adds
a test adding coverage to that parsing

## Checklist

- [ ] I have read the contributing guide and the code of conduct

Author: sachinseizer

packages/evo-sdk-common/pyproject.toml

@@ -1,7 +1,7 @@
 [project]
 name = "evo-sdk-common"
 description = "Python package that establishes a common framework for use by client libraries that interact with Seequent Evo APIs"
-version = "0.5.14"
+version = "0.5.15"
 requires-python = ">=3.10"
 license-files = ["LICENSE.md"]
 dynamic = ["readme"]

packages/evo-sdk-common/src/evo/workspaces/parse.py

@@ -83,9 +83,9 @@ def workspace_model(
     :param base_url: The base URL of the hub.
     :return: A Workspace instance.
     """
-    bounding_box = None
+    parsed_bounding_box = None
     if model.bounding_box:
-        bounding_box = bounding_box(model.bounding_box)
+        parsed_bounding_box = bounding_box(model.bounding_box)
 
     return Workspace(
         id=model.id,
@@ -94,11 +94,11 @@ def workspace_model(
         display_name=model.name,
         description=model.description,
         user_role=WorkspaceRole[str(model.current_user_role.value)] if model.current_user_role else None,
-        created_at=model.created_at,
+        created_at=model.created_at.replace(tzinfo=timezone.utc),
         created_by=ServiceUser.from_model(model.created_by),
-        updated_at=model.updated_at,
+        updated_at=model.updated_at.replace(tzinfo=timezone.utc),
         updated_by=ServiceUser.from_model(model.updated_by),
-        bounding_box=bounding_box,
+        bounding_box=parsed_bounding_box,
         default_coordinate_system=model.default_coordinate_system,
         labels=model.labels,
     )

packages/evo-sdk-common/tests/data/new_workspace_with_bounding_box.json

@@ -0,0 +1,49 @@
+{
+    "id": "00000000-0000-0000-0000-00000000000d",
+    "name": "Test Workspace D",
+    "description": "test workspace d",
+    "current_user_role": "owner",
+    "bounding_box": {
+        "coordinates": [
+            [
+                [
+                100,
+                0
+                ],
+                [
+                101,
+                0
+                ],
+                [
+                101,
+                1
+                ],
+                [
+                100,
+                1
+                ],
+                [
+                100,
+                0
+                ]
+            ]
+        ],
+        "type": "Polygon"
+    },
+    "created_at": "2020-01-01T00:00:00.000000+00:00",
+    "updated_at": "2020-01-01T00:00:00.000000+00:00",
+    "created_by": {
+        "id": "00000000-0000-0000-0000-000000000002",
+        "name": "Test User",
+        "email": "test.user@unit.test"
+    },
+    "updated_by": {
+        "id": "00000000-0000-0000-0000-000000000002",
+        "name": "Test User",
+        "email": "test.user@unit.test"
+    },
+    "ml_enabled": false,
+    "default_coordinate_system": "",
+    "labels": [],
+    "self_link": "https://unittest.localhost/workspace/orgs/00000000-0000-0000-0000-000000000000/workspaces"
+}

packages/evo-sdk-common/tests/workspaces/test_workspace_client.py

@@ -20,6 +20,8 @@
 from evo.workspaces import (
     AddedInstanceUsers,
     BasicWorkspace,
+    BoundingBox,
+    Coordinate,
     InstanceRole,
     InstanceRoleWithPermissions,
     InstanceUser,
@@ -44,7 +46,7 @@
 TEST_USER = ServiceUser(id=USER_ID, name="Test User", email="test.user@unit.test")
 
 
-def _test_workspace(ws_id: UUID, name: str) -> Workspace:
+def _test_workspace(ws_id: UUID, name: str, bounding_box: BoundingBox | None = None) -> Workspace:
     """Factory method to create test workspace objects."""
     return Workspace(
         id=ws_id,
@@ -57,6 +59,7 @@ def _test_workspace(ws_id: UUID, name: str) -> Workspace:
         created_by=TEST_USER,
         updated_at=utc_datetime(2020, 1, 1),
         updated_by=TEST_USER,
+        bounding_box=bounding_box,
     )
 
 
@@ -116,6 +119,23 @@ def _test_instance_user_invitation(
 TEST_WORKSPACE_A = _test_workspace(UUID(int=0xA), "Test Workspace A")
 TEST_WORKSPACE_B = _test_workspace(UUID(int=0xB), "Test Workspace B")
 TEST_WORKSPACE_C = _test_workspace(UUID(int=0xC), "Test Workspace C")
+TEST_WORKSPACE_D = _test_workspace(
+    UUID(int=0xD),
+    "Test Workspace D",
+    bounding_box=BoundingBox(
+        coordinates=[
+            [
+                Coordinate(longitude=100, latitude=0),
+                Coordinate(longitude=101, latitude=0),
+                Coordinate(longitude=101, latitude=1),
+                Coordinate(longitude=100, latitude=1),
+                Coordinate(longitude=100, latitude=0),
+            ]
+        ],
+        type="Polygon",
+    ),
+)
+
 TEST_BASIC_WORKSPACE_A = _test_basic_workspace(UUID(int=0xA), "Test Workspace A")
 TEST_BASIC_WORKSPACE_B = _test_basic_workspace(UUID(int=0xB), "Test Workspace B")
 TEST_BASIC_WORKSPACE_C = _test_basic_workspace(UUID(int=0xC), "Test Workspace C")
@@ -204,6 +224,33 @@ async def test_create_workspace(self):
         )
         self.assertEqual(TEST_WORKSPACE_A, new_workspace)
 
+    async def test_create_workspace_with_bounding_box(self):
+        with self.transport.set_http_response(201, json.dumps(load_test_data("new_workspace_with_bounding_box.json"))):
+            new_workspace = await self.workspace_client.create_workspace(
+                name="Test Workspace",
+                description="test workspace",
+                bounding_box_coordinates=[(100, 0), (101, 0), (101, 1), (100, 1), (100, 0)],
+            )
+        self.assert_request_made(
+            method=RequestMethod.POST,
+            path=f"{BASE_PATH}/workspaces",
+            headers={
+                "Accept": "application/json",
+                "Content-Type": "application/json",
+            },
+            body={
+                "bounding_box": {
+                    "coordinates": [[[100, 0], [101, 0], [101, 1], [100, 1], [100, 0]]],
+                    "type": "Polygon",
+                },
+                "default_coordinate_system": "",
+                "description": "test workspace",
+                "labels": None,
+                "name": "Test Workspace",
+            },
+        )
+        self.assertEqual(TEST_WORKSPACE_D, new_workspace)
+
     async def test_update_workspace(self):
         with self.transport.set_http_response(200, json.dumps(load_test_data("new_workspace.json"))):
             updated_workspace = await self.workspace_client.update_workspace(