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

[feature] Update E2E tests (or create new ones) to test guaccollect and the other components #1953

Closed
pxp928 opened this issue Jun 10, 2024 · 1 comment · Fixed by #1998
Closed
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@pxp928
Copy link
Collaborator

pxp928 commented Jun 10, 2024

Is your feature request related to a problem? Please describe.
The current e2e tests most test via calls made from guacone. This is good to test from a demo deployment standpoint but a persistent deployment (via ENT, guaccollect and guacingest) is not currently tested. We need to add an e2e integration tests for all these to ensure functionality.

Describe the solution you'd like
Create a new e2e that tests via ENT, guaccollect, guaccollect osv and guacingest(with collector subscriber) all running. This will give a much better representation of an actual production-type deployment.

@pxp928 pxp928 added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Jun 10, 2024
@pxp928
Copy link
Collaborator Author

pxp928 commented Jun 11, 2024

The current e2e test is located here: https://github.com/guacsec/guac/tree/main/internal/testing/e2e

The new tests (not replacing the existing) should be in a similar format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant