Skip to content

test: authenticate the e2e API client as the forum service user (0.2.x) - #295

Merged
feanil merged 1 commit into
release/0.2.xfrom
feanil/fix-e2e-api-client-0.2.x
Oct 14, 2026
Merged

feanil merged 1 commit into
release/0.2.xfrom
feanil/fix-e2e-api-client-0.2.x

Conversation

@feanil

@feanil feanil commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Backport of #294 to release/0.2.x. The forum HTTP API now requires a staff caller (GHSA-c6fp-f54c-6cv4), and tests/e2e/conftest.py overrode the authenticated api_client fixture with an anonymous client, so every e2e request got a 403. Locally pytest tests/e2e gives 74 passed with this change.

🤖 Generated with Claude Code

The forum HTTP API now requires a staff caller (GHSA-c6fp-f54c-6cv4), and
tests/conftest.py's api_client fixture logs in as a staff service account
for that reason. tests/e2e/conftest.py overrode that fixture with an
anonymous client, so every e2e request got a 403.

Drop the override so the e2e suite uses the authenticated fixture from
tests/conftest.py.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@feanil
feanil marked this pull request as ready for review October 14, 2026 14:12
@feanil
feanil merged commit 6069a1b into release/0.2.x Oct 14, 2026
10 checks passed
@feanil
feanil deleted the feanil/fix-e2e-api-client-0.2.x branch October 14, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant