Skip to content

refactor: rewrite e2e tests for clarity and correctness#61

Merged
Bentlybro merged 1 commit intoSignificant-Gravitas:masterfrom
Bentlybro:fix/e2e-tests-rewrite
Feb 17, 2026
Merged

refactor: rewrite e2e tests for clarity and correctness#61
Bentlybro merged 1 commit intoSignificant-Gravitas:masterfrom
Bentlybro:fix/e2e-tests-rewrite

Conversation

@Bentlybro
Copy link
Copy Markdown
Member

What

Rewrote the e2e test suite — the old one was broken (login test failing because registration auto-logs you in 😂).

Changes

  • Fix the actual bug: Remove redundant login test that fails after registration auto-authenticates
  • Modular structure: Separate client class, individual test functions with clear names
  • Better output: Shows exactly what passed/failed and why
  • Smart CI handling: Detects test-key OpenAI key and skips moderation tests automatically
  • Actually runs all tests: Some test functions were defined but never called
  • Early exit: Stops if critical setup fails instead of running useless tests

Before

..F..
4/5 tests passed
❌ Some E2E tests failed!

After

▸ Platform Core
  ✓ health check
  ✓ register + auto-login
  ✓ create project
  ✓ get API key

▸ Moderation API (skipped — no OpenAI key)

4/4 tests passed
✅ All tests passed!

- Fix login test that fails because registration auto-authenticates
- Add modular test structure with clear pass/fail reporting
- Skip moderation tests in CI (detects test-key, no real OpenAI)
- Actually run all test functions (some were defined but never called)
- Early exit on critical failures instead of running useless tests
- Clean up CI workflow env vars
@Bentlybro Bentlybro merged commit 2aa36ed into Significant-Gravitas:master Feb 17, 2026
4 of 5 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