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] Promote selected tests as integration tests #121903

Merged
merged 6 commits into from
Dec 27, 2021

Conversation

tsullivan
Copy link
Member

@tsullivan tsullivan commented Dec 22, 2021

Summary

This PR is to make Reporting long-running tests more stable. Certain tests integrate with other dependencies and need to be classified as "integration tests" by making them in a folder named integration_tests.

Closes #87021
Closes #115881

@tsullivan tsullivan requested review from a team as code owners December 22, 2021 17:41
@tsullivan tsullivan added release_note:skip Skip the PR/issue when compiling release notes v7.17.0 v8.0.0 v8.1.0 labels Dec 22, 2021
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@tsullivan tsullivan merged commit d2fd150 into elastic:main Dec 27, 2021
@tsullivan tsullivan deleted the reporting/integration-tests branch December 27, 2021 18:02
tsullivan added a commit to tsullivan/kibana that referenced this pull request Dec 27, 2021
* move test files to integration_tests

* cleanup

* fix2

* [Reporting] Fix skipped integrations test
# Conflicts:
#	x-pack/plugins/reporting/server/routes/diagnostic/integration_tests/browser.test.ts
#	x-pack/plugins/reporting/server/routes/generate/integration_tests/generation_from_jobparams.test.ts
#	x-pack/plugins/reporting/server/routes/management/integration_tests/jobs.test.ts
tsullivan added a commit that referenced this pull request Dec 27, 2021
…22040)

* move test files to integration_tests

* cleanup

* fix2

* [Reporting] Fix skipped integrations test
# Conflicts:
#	x-pack/plugins/reporting/server/routes/diagnostic/integration_tests/browser.test.ts
#	x-pack/plugins/reporting/server/routes/generate/integration_tests/generation_from_jobparams.test.ts
#	x-pack/plugins/reporting/server/routes/management/integration_tests/jobs.test.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v7.17.0 v8.0.0 v8.1.0
Projects
None yet
3 participants