Skip to content

Commit 2a75532

Browse files
committed
fix: tests fix and ruff fix
1 parent 292c480 commit 2a75532

File tree

7 files changed

+26
-40
lines changed

7 files changed

+26
-40
lines changed

example/example1/requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,4 +8,4 @@ openfga-sdk >= 0.9.7
88
python-dateutil >= 2.9.0.post0
99
urllib3 >= 1.26.19, != 2.0.*, != 2.1.*, != 2.2.0, != 2.2.1, < 3
1010
yarl >= 1.20.1
11-
python-dotenv >= 1, <2
11+
python-dotenv >= 1, <2

openfga_sdk/client/client.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,6 @@
3939
from openfga_sdk.client.models.list_users_request import ClientListUsersRequest
4040
from openfga_sdk.client.models.read_changes_request import ClientReadChangesRequest
4141
from openfga_sdk.client.models.tuple import ClientTuple, convert_tuple_keys
42-
from openfga_sdk.client.models.write_conflict_opts import (
43-
ClientWriteRequestOnDuplicateWrites,
44-
ClientWriteRequestOnMissingDeletes,
45-
ConflictOptions,
46-
)
47-
from openfga_sdk.client.models.write_options import ClientWriteOptions
4842
from openfga_sdk.client.models.write_request import ClientWriteRequest
4943
from openfga_sdk.client.models.write_response import ClientWriteResponse
5044
from openfga_sdk.client.models.write_single_response import (

openfga_sdk/models/write_request_deletes.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,7 @@ def on_missing(self, on_missing):
106106
and on_missing not in allowed_values
107107
):
108108
raise ValueError(
109-
"Invalid value for `on_missing` ({0}), must be one of {1}".format(
110-
on_missing, allowed_values
111-
)
109+
f"Invalid value for `on_missing` ({on_missing}), must be one of {allowed_values}"
112110
)
113111

114112
self._on_missing = on_missing

openfga_sdk/models/write_request_writes.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,7 @@ def on_duplicate(self, on_duplicate):
106106
and on_duplicate not in allowed_values
107107
):
108108
raise ValueError(
109-
"Invalid value for `on_duplicate` ({0}), must be one of {1}".format(
110-
on_duplicate, allowed_values
111-
)
109+
f"Invalid value for `on_duplicate` ({on_duplicate}), must be one of {allowed_values}"
112110
)
113111

114112
self._on_duplicate = on_duplicate

openfga_sdk/sync/client/client.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,6 @@
3838
from openfga_sdk.client.models.list_users_request import ClientListUsersRequest
3939
from openfga_sdk.client.models.read_changes_request import ClientReadChangesRequest
4040
from openfga_sdk.client.models.tuple import ClientTuple, convert_tuple_keys
41-
from openfga_sdk.client.models.write_conflict_opts import (
42-
ClientWriteRequestOnDuplicateWrites,
43-
ClientWriteRequestOnMissingDeletes,
44-
ConflictOptions,
45-
)
46-
from openfga_sdk.client.models.write_options import ClientWriteOptions
4741
from openfga_sdk.client.models.write_request import ClientWriteRequest
4842
from openfga_sdk.client.models.write_response import ClientWriteResponse
4943
from openfga_sdk.client.models.write_single_response import (

test/client/client_test.py

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3901,17 +3901,9 @@ def client_configuration():
39013901
)
39023902

39033903

3904-
class TestClientConfigurationHeaders(IsolatedAsyncioTestCase):
3904+
class TestClientConfigurationHeaders:
39053905
"""Tests for ClientConfiguration headers parameter"""
39063906

3907-
def setUp(self):
3908-
self.configuration = ClientConfiguration(
3909-
api_url="http://api.fga.example",
3910-
)
3911-
3912-
def tearDown(self):
3913-
pass
3914-
39153907
def test_client_configuration_headers_default_none(self, client_configuration):
39163908
"""Test that headers default to an empty dict in ClientConfiguration"""
39173909
assert client_configuration.headers == {}
@@ -3975,6 +3967,7 @@ def test_client_configuration_headers_deepcopy(self):
39753967
assert "X-New" not in copied_config.headers
39763968

39773969
@patch.object(rest.RESTClientObject, "request")
3970+
@pytest.mark.asyncio
39783971
async def test_write_with_conflict_options_ignore_duplicates(self, mock_request):
39793972
"""Test case for write with conflict options - ignore duplicates"""
39803973
from openfga_sdk.client.models.write_conflict_opts import (
@@ -3984,6 +3977,9 @@ async def test_write_with_conflict_options_ignore_duplicates(self, mock_request)
39843977

39853978
response_body = "{}"
39863979
mock_request.return_value = mock_response(response_body, 200)
3980+
self.configuration = ClientConfiguration(
3981+
api_url="http://api.fga.example",
3982+
)
39873983
configuration = self.configuration
39883984
configuration.store_id = store_id
39893985
async with OpenFgaClient(configuration) as api_client:
@@ -4029,6 +4025,7 @@ async def test_write_with_conflict_options_ignore_duplicates(self, mock_request)
40294025
)
40304026

40314027
@patch.object(rest.RESTClientObject, "request")
4028+
@pytest.mark.asyncio
40324029
async def test_write_with_conflict_options_ignore_missing_deletes(
40334030
self, mock_request
40344031
):
@@ -4040,6 +4037,9 @@ async def test_write_with_conflict_options_ignore_missing_deletes(
40404037

40414038
response_body = "{}"
40424039
mock_request.return_value = mock_response(response_body, 200)
4040+
self.configuration = ClientConfiguration(
4041+
api_url="http://api.fga.example",
4042+
)
40434043
configuration = self.configuration
40444044
configuration.store_id = store_id
40454045
async with OpenFgaClient(configuration) as api_client:
@@ -4085,6 +4085,7 @@ async def test_write_with_conflict_options_ignore_missing_deletes(
40854085
)
40864086

40874087
@patch.object(rest.RESTClientObject, "request")
4088+
@pytest.mark.asyncio
40884089
async def test_write_with_conflict_options_both(self, mock_request):
40894090
"""Test case for write with both conflict options"""
40904091
from openfga_sdk.client.models.write_conflict_opts import (
@@ -4095,6 +4096,9 @@ async def test_write_with_conflict_options_both(self, mock_request):
40954096

40964097
response_body = "{}"
40974098
mock_request.return_value = mock_response(response_body, 200)
4099+
self.configuration = ClientConfiguration(
4100+
api_url="http://api.fga.example",
4101+
)
40984102
configuration = self.configuration
40994103
configuration.store_id = store_id
41004104
async with OpenFgaClient(configuration) as api_client:

test/sync/client/client_test.py

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3880,17 +3880,9 @@ def client_configuration():
38803880
)
38813881

38823882

3883-
class TestSyncClientConfigurationHeaders(IsolatedAsyncioTestCase):
3883+
class TestSyncClientConfigurationHeaders:
38843884
"""Tests for ClientConfiguration headers parameter in sync client"""
38853885

3886-
def setUp(self):
3887-
self.configuration = ClientConfiguration(
3888-
api_url="http://api.fga.example",
3889-
)
3890-
3891-
def tearDown(self):
3892-
pass
3893-
38943886
def test_sync_client_configuration_headers_default_none(self, client_configuration):
38953887
"""Test that headers default to an empty dict in ClientConfiguration"""
38963888
assert client_configuration.headers == {}
@@ -3971,7 +3963,9 @@ def test_sync_write_with_conflict_options_ignore_duplicates(self, mock_request):
39713963

39723964
response_body = "{}"
39733965
mock_request.return_value = mock_response(response_body, 200)
3974-
configuration = self.configuration
3966+
configuration = ClientConfiguration(
3967+
api_url="http://api.fga.example",
3968+
)
39753969
configuration.store_id = store_id
39763970
with OpenFgaClient(configuration) as api_client:
39773971
body = ClientWriteRequest(
@@ -4027,7 +4021,9 @@ def test_sync_write_with_conflict_options_ignore_missing_deletes(
40274021

40284022
response_body = "{}"
40294023
mock_request.return_value = mock_response(response_body, 200)
4030-
configuration = self.configuration
4024+
configuration = ClientConfiguration(
4025+
api_url="http://api.fga.example",
4026+
)
40314027
configuration.store_id = store_id
40324028
with OpenFgaClient(configuration) as api_client:
40334029
body = ClientWriteRequest(
@@ -4082,7 +4078,9 @@ def test_sync_write_with_conflict_options_both(self, mock_request):
40824078

40834079
response_body = "{}"
40844080
mock_request.return_value = mock_response(response_body, 200)
4085-
configuration = self.configuration
4081+
configuration = ClientConfiguration(
4082+
api_url="http://api.fga.example",
4083+
)
40864084
configuration.store_id = store_id
40874085
with OpenFgaClient(configuration) as api_client:
40884086
body = ClientWriteRequest(

0 commit comments

Comments
 (0)