Skip to content

Conversation

@karpikpl
Copy link
Collaborator

@karpikpl karpikpl commented Jan 3, 2025

This pull request includes several significant changes to integrate Playwright for testing, add mock data handling, and update Docker configurations. The most important changes include adding Playwright configuration and tests, updating Docker and environment configurations, and modifying API handling for mock data.

Playwright Integration:

  • Added playwright.config.ts to configure Playwright with multiple browser projects and environment variables.
  • Updated package.json to include Playwright dependencies and scripts for running tests. [1] [2]
  • Created Playwright test files tests/copilot.ent.spec.ts and tests/copilot.org.spec.ts to validate enterprise and organization views respectively. [1] [2]
  • Added GitHub Actions workflow playwright.yml to run Playwright tests on push and pull request events.

Docker and Environment Configuration:

  • Updated docker-compose.yml and docker-compose.ent.override.yml to include Playwright service and environment variables for API scope and GitHub settings. [1] [2]
  • Modified .dockerignore to include directories for deployment and test results.
  • Updated api.Dockerfile to copy mock data into the Docker image.

API Mock Data Handling:

  • Added middleware functions in api/server.mjs to log requests and handle mock responses based on environment variables.
  • Updated imports in src/api/ExtractSeats.ts and src/api/GitHubApi.ts to use mock data from the mock-data directory. [1] [2]

@karpikpl
Copy link
Collaborator Author

karpikpl commented Jan 5, 2025

merged into #134

@karpikpl karpikpl closed this Jan 5, 2025
@karpikpl karpikpl deleted the playwright branch January 13, 2025 16:58
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.

2 participants