Skip to content

test(e2e): add playwright suite from the functional test cases - #83

Merged
Rl0007 merged 10 commits into
mainfrom
feat/playwright-suite
Oct 6, 2026
Merged

Rl0007 merged 10 commits into
mainfrom
feat/playwright-suite

Conversation

@Rl0007

@Rl0007 Rl0007 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor
  1. The 5 Oct functional test run was done by hand, so checking a fix against it meant repeating 55 cases.
  2. A Playwright suite in frontend/e2e/ now runs those cases. Bugs from that run are expected failures linked to their issues, so a fixed bug shows up as "expected to fail, but passed".

Copy frontend/e2e/.env.e2e.example to .env.e2e (a System Manager user and its API keys), then run from frontend/:

npx playwright install chromium
npx playwright test -c e2e --grep @smoke        # main journey, ~5 min
npx playwright test -c e2e --grep @functional   # every feature, ~35 min
npx playwright test -c e2e --grep @negative     # forbidden actions fail cleanly
npx playwright test -c e2e --grep "(?=.*@sanity)(?=.*@tree)"   # one area after a fix
npx playwright test -c e2e --grep @llm          # AI tests only

The AI tests were run with Settings > Models > Local models on (Sonnet through the Claude CLI), which is not identical to production.

Verification

wikify.localhost, one background worker:

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@Rl0007
Rl0007 merged commit 4166c75 into main Oct 6, 2026
7 checks passed
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