Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(e2e): Improve e2e testing configuration and add more auth.spec.t… #158

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

ghost
Copy link

@ghost ghost commented Nov 13, 2024

This pull request introduces several changes to the end-to-end (E2E) testing setup and adds new test cases. The most significant changes include the creation of utility functions for managing server processes, updating the Playwright configuration to use these utilities, and adding new test cases for health checks and authentication.

Process Management Utilities:

  • Added Process utility to manage PID files and handle process start/stop operations (e2e/process.ts).

Playwright Configuration:

  • Updated Playwright configuration to use global setup and teardown scripts for starting and stopping servers (e2e/playwright.config.ts). [1] [2]
  • Added global setup and teardown scripts to manage server processes during tests (e2e/global-setup.ts, e2e/global-teardown.ts). [1] [2]

New Test Cases:

  • Added test case for signing in with invalid credentials (e2e/tests/auth/auth.spec.ts).
  • Added health check tests to verify application status when backend is up or down (e2e/tests/health/health.spec.ts).

Miscellaneous:

  • Added .pid files to .gitignore to ignore temporary PID files (e2e/.gitignore).
  • Created Application module to encapsulate logic for starting and stopping client and API servers (e2e/application.ts).

Copy link

vercel bot commented Nov 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
4-growth ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 13, 2024 9:14am

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