Set up Playwright E2E testing framework with CI integration #301
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
coreyja
force-pushed
the
push-mzwpvszromzp
branch
from
4fc4ec9 to
cf8089f
Compare
- Add Playwright configuration with OAuth mocking for GitHub auth - Create reusable GitHub Actions workflow for E2E tests - Add basic smoke tests as a starting point - Configure test infrastructure with fixtures and helpers Add required environment variables to Playwright CI workflow The server requires several environment variables to start up. Added fake/test values for all required env vars so the server can start during E2E tests.
coreyja
force-pushed
the
push-mzwpvszromzp
branch
from
cf8089f to
7cdfad4
Compare
Added DISCORD_BOT_DISABLED=true to the playwright_tests.yml workflow to prevent the server from attempting to connect to Discord during E2E tests. This eliminates the need for a real Discord token in CI.
Refactored Discord setup to be optional, allowing the server to run without a Discord token configured. This enables CI/E2E tests to run without requiring Discord integration. Changes: - discord::setup() now returns Option<DiscordSetup> instead of Result<DiscordSetup> - AppState.discord changed from DiscordClient to Option<DiscordClient> - All Discord-dependent code now checks if Discord is configured before use - Jobs and tools gracefully skip Discord operations when not configured - Fixed test helpers to handle optional Discord client
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Changes
.github/workflows/playwright_tests.ymlthat runs E2E tests on push/PRe2e/directory with dependencies and test scriptsTest Plan
cd e2e && npm test🤖 Generated with Claude Code