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

Fix ess deployment agnostic tests #199341

Merged

Conversation

fake-haris
Copy link
Contributor

Summary

Fixed assertions in integrations.ts. #198894

Details

For installed and custom integrations: We fetch integrations in before step and concat them with the additional expected integrations.
For integrations datasets map: We fetch the proper entry from the integrations list.

@fake-haris fake-haris added test_xpack_functional release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting labels Nov 7, 2024
@fake-haris fake-haris self-assigned this Nov 7, 2024
@fake-haris fake-haris requested a review from a team as a code owner November 7, 2024 15:41
Copy link
Member

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

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

Good job! few nits

@fake-haris fake-haris requested a review from dmlemeshko November 8, 2024 13:23
Copy link
Member

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@pheyos pheyos left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #5 / @ess @serverless SecuritySolution Explore Network Tls Test with Packetbeat Tls Overview Test Ensure data is returned for FlowTarget.Source

Metrics [docs]

✅ unchanged

History

cc @fake-haris

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes test_xpack_functional v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants