[cross-repo from server#351] Conformance blocker: complete namespaces parity coverage (#150)

Author: durable-workflow-ops

src/durable_workflow/sync.py

@@ -390,6 +390,10 @@ def update_namespace(
         )
         return result
 
+    def delete_namespace(self, name: str) -> NamespaceDescription:
+        result: NamespaceDescription = _run(self._async.delete_namespace(name))
+        return result
+
     def set_namespace_external_storage(
         self,
         name: str | None = None,

tests/test_client.py

@@ -1090,6 +1090,7 @@ async def test_delete_namespace_matches_polyglot_fixture(self, client: Client) -
             result = await client.delete_namespace(**fixture["sdk_python"]["args"])
 
         assert mock.call_args.args[:2] == (fixture["request"]["method"], f"/api{fixture['request']['path']}")
+        assert mock.call_args.kwargs.get("json") is None
         assert result.name == fixture["semantic_body"]["name"]
         assert result.status == fixture["semantic_body"]["status"]
         assert result.deleted is not None

tests/test_sync.py

@@ -320,6 +320,32 @@ def test_resume_task_queue_build_id_for_unversioned_cohort(self) -> None:
             assert mock.call_args.kwargs.get("json") == {"build_id": None}
 
 
+class TestSyncClientNamespaces:
+    def test_delete_namespace(self) -> None:
+        client = Client("http://localhost:8080")
+        resp = _mock_response(
+            200,
+            {
+                "name": "billing reports",
+                "status": "deleted",
+                "deleted": {"workflow_runs": 2},
+            },
+        )
+        with patch.object(
+            client._async._http, "request", new_callable=AsyncMock, return_value=resp
+        ) as mock:
+            result = client.delete_namespace("billing reports")
+
+        assert result.name == "billing reports"
+        assert result.status == "deleted"
+        assert result.deleted == {"workflow_runs": 2}
+        assert mock.call_args.args[:2] == (
+            "DELETE",
+            "/api/namespaces/billing%20reports",
+        )
+        assert mock.call_args.kwargs.get("json") is None
+
+
 class TestSyncClientRunVisibility:
     def test_list_workflow_runs(self) -> None:
         client = Client("http://localhost:8080")