refactor: improve code readability by formatting arguments across multiple files
Test / test (push) Successful in 15s
Test / test (push) Successful in 15s
This commit is contained in:
@@ -28,7 +28,9 @@ class StubOrganizationRepository(OrganizationRepository):
|
||||
self._membership = membership
|
||||
|
||||
async def get_membership(
|
||||
self, organization_id: int, user_id: int,
|
||||
self,
|
||||
organization_id: int,
|
||||
user_id: int,
|
||||
) -> OrganizationMember | None: # pragma: no cover - helper
|
||||
if (
|
||||
self._membership
|
||||
@@ -40,7 +42,10 @@ class StubOrganizationRepository(OrganizationRepository):
|
||||
|
||||
|
||||
def make_membership(
|
||||
role: OrganizationRole, *, organization_id: int = 1, user_id: int = 10,
|
||||
role: OrganizationRole,
|
||||
*,
|
||||
organization_id: int = 1,
|
||||
user_id: int = 10,
|
||||
) -> OrganizationMember:
|
||||
organization = Organization(name="Acme Inc")
|
||||
organization.id = organization_id
|
||||
@@ -75,7 +80,8 @@ class MembershipRepositoryStub(OrganizationRepository):
|
||||
"""Repository stub that can emulate duplicate checks for add_member."""
|
||||
|
||||
def __init__(
|
||||
self, memberships: dict[tuple[int, int], OrganizationMember] | None = None,
|
||||
self,
|
||||
memberships: dict[tuple[int, int], OrganizationMember] | None = None,
|
||||
) -> None:
|
||||
self._session_stub = SessionStub()
|
||||
super().__init__(session=cast(AsyncSession, self._session_stub))
|
||||
@@ -95,7 +101,8 @@ async def test_get_context_success() -> None:
|
||||
service = OrganizationService(StubOrganizationRepository(membership))
|
||||
|
||||
context = await service.get_context(
|
||||
user_id=membership.user_id, organization_id=membership.organization_id,
|
||||
user_id=membership.user_id,
|
||||
organization_id=membership.organization_id,
|
||||
)
|
||||
|
||||
assert context.organization_id == membership.organization_id
|
||||
@@ -183,7 +190,9 @@ async def test_add_member_rejects_duplicate_membership() -> None:
|
||||
|
||||
with pytest.raises(OrganizationMemberAlreadyExistsError):
|
||||
await service.add_member(
|
||||
context=context, user_id=duplicate_user_id, role=OrganizationRole.MANAGER,
|
||||
context=context,
|
||||
user_id=duplicate_user_id,
|
||||
role=OrganizationRole.MANAGER,
|
||||
)
|
||||
|
||||
|
||||
|
||||
Reference in New Issue
Block a user