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

[Reporting/Tests] Unskip "Download reports" test #195685

Merged
merged 5 commits into from
Oct 17, 2024

Conversation

tsullivan
Copy link
Member

@tsullivan tsullivan commented Oct 9, 2024

Closes #195144

The flaky test in question uses the default test login account for functional tests. I have moved it to the first test in the configuration to avoid having to re-login.

Other changes:

  • Reduced the size of the archived_reports archive by shortening the field values for output.content and payload.headers. These field values where long strings of text and aren't used for anything in tests.
  • Changed some report job titles in the archived_reports archive to make the tests easier to read and verify.

@tsullivan tsullivan added the release_note:skip Skip the PR/issue when compiling release notes label Oct 9, 2024
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/test-suites-xpack 729 730 +1

Total ESLint disabled count

id before after diff
@kbn/test-suites-xpack 754 755 +1

History

@tsullivan
Copy link
Member Author

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#7153

[✅] x-pack/test/reporting_functional/reporting_and_security.config.ts: 100/100 tests passed.

see run history

@tsullivan tsullivan marked this pull request as ready for review October 16, 2024 19:14
@tsullivan tsullivan requested a review from a team as a code owner October 16, 2024 19:14
@tsullivan tsullivan added the backport:skip This commit does not require backporting label Oct 16, 2024
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 v9.0.0
Projects
None yet
4 participants