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

ci: setup buildkite test analytics for cypress and jest tests #7953

Merged
merged 10 commits into from
Aug 22, 2024

Conversation

tkajtoch
Copy link
Member

@tkajtoch tkajtoch commented Aug 8, 2024

Summary

This PR configures Buildkite Test Analytics for EUI's cypress and jest tests.

QA

  • Ensure the CI passes and both cypress and jest test reports are visible in Buildkite

@tkajtoch tkajtoch self-assigned this Aug 8, 2024
? 'buildkite-test-collector/cypress/reporter'
: undefined,
reporterOptions: {
token_name: 'BUILDKITE_ANALYTICS_CYPRESS_TOKEN',
Copy link
Member Author

Choose a reason for hiding this comment

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

Cypress reporter expects token_name to be passed and reads the token automatically. Jest expects the token to be passed directly in the token property. Not a perfect API, but it works!

@tkajtoch tkajtoch force-pushed the feat/setup-buildkite-test-analytics branch from 0fb8637 to b212a51 Compare August 9, 2024 17:58
@tkajtoch tkajtoch force-pushed the feat/setup-buildkite-test-analytics branch from b212a51 to b0df47c Compare August 9, 2024 18:23
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @tkajtoch

@tkajtoch tkajtoch marked this pull request as ready for review August 9, 2024 19:00
@tkajtoch tkajtoch requested a review from a team as a code owner August 9, 2024 19:00
@tkajtoch tkajtoch requested a review from 1Copenut August 12, 2024 21:17
Copy link
Contributor

@1Copenut 1Copenut left a comment

Choose a reason for hiding this comment

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

LGTM! Changeset was easy to follow and I manually verified the analytics ran on the Buildkite UI. This will be a great way to track regressions or flaky tests over time!

@tkajtoch tkajtoch merged commit 37bf78b into elastic:main Aug 22, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants