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

Functional tests for execution context propagation in alerting and task manager plugins #111179

Merged

Conversation

mshustov
Copy link
Contributor

@mshustov mshustov commented Sep 3, 2021

Summary

Functionals tests for Alerts and Task manager to propagate execution context data.

@mshustov mshustov added chore Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.0.0 release_note:skip Skip the PR/issue when compiling release notes auto-backport Deprecated - use backport:version if exact versions are needed v7.16.0 labels Sep 3, 2021
@mshustov mshustov marked this pull request as ready for review September 6, 2021 07:03
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

},
});

alerting.registerType({
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The plugin registers a custom alert. Test suit will create an instance of this alert.


export default function ({ loadTestFile }: FtrProviderContext) {
describe('Execution context', function () {
this.tags('ciGroup1');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I separated tests to avoid skipping both in case of flakiness

@mshustov mshustov requested a review from a team September 6, 2021 07:18
.send({
enabled: true,
name: 'abc',
tags: ['foo'],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

scripts/functional_tests.js Show resolved Hide resolved
x-pack/test/functional_execution_context/tests/server.ts Outdated Show resolved Hide resolved
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@mshustov mshustov merged commit 4a2acad into elastic:master Sep 14, 2021
@mshustov mshustov deleted the functional-execution-context-alerting branch September 14, 2021 09:52
@kibanamachine
Copy link
Contributor

💔 Backport failed

Status Branch Result
7.x Commit could not be cherrypicked due to conflicts

To backport manually run:
node scripts/backport --pr 111179

mshustov added a commit to mshustov/kibana that referenced this pull request Sep 14, 2021
…sk manager plugins (elastic#111179)

* move execution context tests to x-pack folder

* add execution context tests for alerts and tasks

* cleanup tests

* remove obsolete test folder

* use ToolingLog instead of console.log
# Conflicts:
#	scripts/functional_tests.js
#	x-pack/scripts/functional_tests.js
mshustov added a commit that referenced this pull request Sep 14, 2021
…sk manager plugins (#111179) (#112066)

* move execution context tests to x-pack folder

* add execution context tests for alerts and tasks

* cleanup tests

* remove obsolete test folder

* use ToolingLog instead of console.log
# Conflicts:
#	scripts/functional_tests.js
#	x-pack/scripts/functional_tests.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed chore release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v7.16.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants