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

proposal to remove Integration Tests #1404

Closed
puckpuck opened this issue Feb 21, 2024 · 0 comments · Fixed by #1406
Closed

proposal to remove Integration Tests #1404

puckpuck opened this issue Feb 21, 2024 · 0 comments · Fixed by #1406
Labels
bug Something isn't working

Comments

@puckpuck
Copy link
Contributor

The demo has 3 different tests defined:

  1. Frontend Tests
  2. Integration Tests
  3. Traced Based Tests

Only the Frontend Tests and the Trace Based Tests are run as part of our CI pipeline. The Integration Tests have been broken for several months, and are well covered by what we get with Trace Based Tests. Instead of fixing and maintaining multiple test frameworks that do the same thing, we should remove the Integration Tests completely.

@puckpuck puckpuck added the bug Something isn't working label Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant