Skip to content

feat: add end-to-end tests #179

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

Merged
merged 11 commits into from
Mar 27, 2025
Merged

feat: add end-to-end tests #179

merged 11 commits into from
Mar 27, 2025

Conversation

kai687
Copy link
Contributor

@kai687 kai687 commented Mar 18, 2025

This PR adds a few simple end-to-end tests, based on Go's testscript package.

This works best with an empty Algolia application, but otherwise nothing special is required.
To run them, just add a .env file with ALGOLIA_APPLICATION_ID and ALGOLIA_API_KEY and run task e2e.

This is probably very flaky, that's why I didn't add anything to the CI yet.

@kai687 kai687 marked this pull request as ready for review March 18, 2025 15:49
Copy link
Collaborator

@dylantientcheu dylantientcheu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome job! I left a small remark on one of the e2e tests.

Copy link
Collaborator

@dylantientcheu dylantientcheu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome job! I left a small remark on one of the e2e tests.

@kai687 kai687 requested a review from dylantientcheu March 27, 2025 09:16
Copy link
Collaborator

@dylantientcheu dylantientcheu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@dylantientcheu dylantientcheu merged commit 330ca28 into algolia:main Mar 27, 2025
2 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.

2 participants