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

[8.x] Functional tests - remove .empty file from screenshots dir (#200161) #200198

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…00161)

## Summary

This PR removes the `.empty` file from the `test/functional/screenshots`
directory as it's no longer needed.

### Details

The file has been introduced as part of elastic#14122 when the directory was
cleaned. Since then, new base line screenshots have been added, so the
directory is no longer empty.

(cherry picked from commit 79c5e84)
@kibanamachine kibanamachine merged commit 416e27d into elastic:8.x Nov 14, 2024
25 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #66 / APM API tests transactions/transactions_groups_alerts.spec.ts basic no archive when data is loaded Alerts Transaction groups with avg transaction duration alerts returns the correct number of alert counts

Metrics [docs]

✅ unchanged

cc @pheyos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants