Skip to content

Commit 1350d0b

Browse files
committed
formatting
1 parent ddd60c2 commit 1350d0b

File tree

1 file changed

+6
-11
lines changed

1 file changed

+6
-11
lines changed

integration/tests/posit/connect/test_permissions_upsert.py

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -335,9 +335,7 @@ def update_permission(thread_id: int):
335335
successful = [r for r in results if r["success"]]
336336
failed = [r for r in results if not r["success"]]
337337

338-
print(
339-
f"\nConcurrent updates - Successful: {len(successful)}, Failed: {len(failed)}"
340-
)
338+
print(f"\nConcurrent updates - Successful: {len(successful)}, Failed: {len(failed)}")
341339

342340
# All requests should succeed
343341
assert len(successful) == num_concurrent_requests, (
@@ -348,8 +346,7 @@ def update_permission(thread_id: int):
348346
# Verify still only ONE permission exists
349347
permissions = self.content.permissions.find(principal_guid=self.user["guid"])
350348
assert len(permissions) == 1, (
351-
f"Expected exactly 1 permission after concurrent updates, "
352-
f"but found {len(permissions)}"
349+
f"Expected exactly 1 permission after concurrent updates, but found {len(permissions)}"
353350
)
354351

355352
# Verify final state is correct
@@ -416,16 +413,15 @@ def update_permission(thread_id: int, target_role: str):
416413
viewer_attempts = [r for r in results if r["target_role"] == "viewer"]
417414
owner_attempts = [r for r in results if r["target_role"] == "owner"]
418415

419-
print(f"\nConcurrent conflicting updates:")
416+
print("\nConcurrent conflicting updates:")
420417
print(f" Total threads: {num_threads}")
421418
print(f" Viewer update attempts: {len(viewer_attempts)}")
422419
print(f" Owner update attempts: {len(owner_attempts)}")
423420
print(f" Successful: {len(successful)}, Failed: {len(failed)}")
424421

425422
# All requests should succeed (upsert behavior)
426423
assert len(successful) == num_threads, (
427-
f"Expected all {num_threads} requests to succeed, "
428-
f"but {len(failed)} failed: {failed}"
424+
f"Expected all {num_threads} requests to succeed, but {len(failed)} failed: {failed}"
429425
)
430426

431427
# Critical check: Verify only ONE permission exists
@@ -443,7 +439,7 @@ def update_permission(thread_id: int, target_role: str):
443439
)
444440

445441
print(f" Final state: 1 permission with role='{final_role}'")
446-
print(f" Result: Last-write-wins behavior (acceptable)")
442+
print(" Result: Last-write-wins behavior (acceptable)")
447443

448444
# Clean up
449445
self.content.permissions.destroy(self.group)
@@ -613,8 +609,7 @@ def create_permission(thread_id: int):
613609

614610
# All requests should succeed
615611
assert len(successful) == num_concurrent_requests, (
616-
f"Expected all {num_concurrent_requests} requests to succeed, "
617-
f"but {len(failed)} failed"
612+
f"Expected all {num_concurrent_requests} requests to succeed, but {len(failed)} failed"
618613
)
619614

620615
# Critical check: Verify only ONE permission exists

0 commit comments

Comments
 (0)