Skip to content

Commit 87eb732

Browse files
final nit
1 parent 0e5eaab commit 87eb732

File tree

5 files changed

+15
-143
lines changed

5 files changed

+15
-143
lines changed

src/workos/fga.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@ class WarrantQueryListResource(
5353
warnings: Optional[Sequence[FGAWarning]] = None
5454

5555

56+
# Deprecated: Use the Authorization module instead.
57+
# See: workos.authorization
5658
class FGAModule(Protocol):
5759
def get_resource(
5860
self, *, resource_type: str, resource_id: str

tests/test_async_http_client.py

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ async def test_request_without_body(
7777
"method,status_code,expected_response",
7878
[
7979
("POST", 201, {"message": "Success!"}),
80+
("PUT", 200, {"message": "Success!"}),
8081
("PATCH", 200, {"message": "Success!"}),
8182
],
8283
)
@@ -272,7 +273,6 @@ async def test_request_includes_base_headers(
272273

273274
await self.http_client.request("ok_place")
274275

275-
assert request_kwargs["url"].endswith("ok_place")
276276
default_headers = set(
277277
(header[0].lower(), header[1])
278278
for header in self.http_client.default_headers.items()
@@ -314,7 +314,6 @@ async def test_request_removes_none_parameter_values(
314314
params={"organization_id": None, "test": "value"},
315315
)
316316

317-
assert request_kwargs["url"].endswith("/test")
318317
assert request_kwargs["params"] == {"test": "value"}
319318

320319
async def test_request_removes_none_json_values(
@@ -338,12 +337,11 @@ async def test_delete_with_body_sends_json(
338337

339338
await self.http_client.delete_with_body(
340339
path="/test",
341-
json={"resource_id": "res_01ABC"},
340+
json={"obj": "json"},
342341
)
343342

344343
assert request_kwargs["method"] == "delete"
345-
assert request_kwargs["url"].endswith("/test")
346-
assert request_kwargs["json"] == {"resource_id": "res_01ABC"}
344+
assert request_kwargs["json"] == {"obj": "json"}
347345

348346
async def test_delete_with_body_sends_params(
349347
self, capture_and_mock_http_client_request
@@ -352,14 +350,12 @@ async def test_delete_with_body_sends_params(
352350

353351
await self.http_client.delete_with_body(
354352
path="/test",
355-
json={"resource_id": "res_01ABC"},
356-
params={"org_id": "org_01ABC"},
353+
json={"obj1": "json"},
354+
params={"obj2": "params"},
357355
)
358356

359-
assert request_kwargs["method"] == "delete"
360-
assert request_kwargs["url"].endswith("/test")
361-
assert request_kwargs["params"] == {"org_id": "org_01ABC"}
362-
assert request_kwargs["json"] == {"resource_id": "res_01ABC"}
357+
assert request_kwargs["json"] == {"obj1": "json"}
358+
assert request_kwargs["params"] == {"obj2": "params"}
363359

364360
async def test_delete_without_body_raises_value_error(self):
365361
with pytest.raises(

tests/test_authorization_resource_external_id.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ def mock_resource(self):
2323
return MockAuthorizationResource().dict()
2424

2525
# --- get_resource_by_external_id ---
26-
2726
def test_get_resource_by_external_id(
2827
self, mock_resource, capture_and_mock_http_client_request
2928
):

tests/test_authorization_types.py

Lines changed: 0 additions & 125 deletions
This file was deleted.

tests/test_sync_http_client.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -378,23 +378,23 @@ def test_delete_with_body_sends_json(self, capture_and_mock_http_client_request)
378378

379379
self.http_client.delete_with_body(
380380
path="/test",
381-
json={"resource_id": "res_01ABC"},
381+
json={"obj": "json"},
382382
)
383383

384384
assert request_kwargs["method"] == "delete"
385-
assert request_kwargs["json"] == {"resource_id": "res_01ABC"}
385+
assert request_kwargs["json"] == {"obj": "json"}
386386

387387
def test_delete_with_body_sends_params(self, capture_and_mock_http_client_request):
388388
request_kwargs = capture_and_mock_http_client_request(self.http_client, {}, 200)
389389

390390
self.http_client.delete_with_body(
391391
path="/test",
392-
json={"resource_id": "res_01ABC"},
393-
params={"org_id": "org_01ABC"},
392+
json={"obj1": "json"},
393+
params={"obj2": "params"},
394394
)
395395

396-
assert request_kwargs["params"] == {"org_id": "org_01ABC"}
397-
assert request_kwargs["json"] == {"resource_id": "res_01ABC"}
396+
assert request_kwargs["json"] == {"obj1": "json"}
397+
assert request_kwargs["params"] == {"obj2": "params"}
398398

399399
def test_delete_without_body_raises_value_error(self):
400400
with pytest.raises(

0 commit comments

Comments
 (0)