Skip to content

Commit e487e0c

Browse files
Remove rationale comment from WIF e2e teardown
The behavior is self-evident from a bare `delete()` with no try/except; the rationale belongs in the PR / commit history, not the test body. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent f71cf38 commit e487e0c

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

‎test_e2e/test_auth_federation.py‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,10 +60,6 @@ def wif_site(server):
6060

6161
yield server
6262

63-
# Let any ServerResponseError propagate: a failed teardown delete must
64-
# surface as a pytest teardown error. Swallowing it would leave WIF
65-
# enabled on the site, so the next run would skip (fixture guard above)
66-
# and silently stop exercising the destructive paths.
6763
server.auth_federation.delete()
6864

6965

0 commit comments

Comments
 (0)