Skip to content

[Alerting] Fixed Jest test suites with unhandled promise rejections#113213

Merged
YulNaumenko merged 2 commits intoelastic:masterfrom
YulNaumenko:alerting-fix-jest-test-promise-rej
Sep 28, 2021
Merged

[Alerting] Fixed Jest test suites with unhandled promise rejections#113213
YulNaumenko merged 2 commits intoelastic:masterfrom
YulNaumenko:alerting-fix-jest-test-promise-rej

Conversation

@YulNaumenko
Copy link
Contributor

Summary

Current PR fixed UnhandledPromiseRejectionWarning: ... which caused in the next Alerting jest tests:
x-pack/plugins/stack_alerts/server/feature.test.ts
x-pack/plugins/stack_alerts/server/plugin.test.ts
x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_add_modal.test.tsx
x-pack/plugins/triggers_actions_ui/server/data/lib/time_series_query.test.ts

To test it just run each suite using command:

node scripts/jest <suite>

Resolves #112703

@YulNaumenko YulNaumenko added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v7.16.0 Feature:Alerting/RulesFramework Issues related to the Alerting Rules Framework Feature:Alerting/RulesManagement Issues related to the Rules Management UX labels Sep 28, 2021
@YulNaumenko YulNaumenko self-assigned this Sep 28, 2021
@YulNaumenko YulNaumenko requested a review from a team as a code owner September 28, 2021 05:00
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

Copy link
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

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

LGTM! Nice job :)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

cc @YulNaumenko

Copy link
Contributor

@chrisronline chrisronline left a comment

Choose a reason for hiding this comment

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

LGTM!

@YulNaumenko YulNaumenko merged commit 3dcfd44 into elastic:master Sep 28, 2021
YulNaumenko added a commit to YulNaumenko/kibana that referenced this pull request Sep 28, 2021
…lastic#113213)

* [Alerting] Fixed Jest test suites with unhandled promise rejections

* fixed typecheck
YulNaumenko added a commit that referenced this pull request Sep 28, 2021
…113213) (#113310)

* [Alerting] Fixed Jest test suites with unhandled promise rejections

* fixed typecheck
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Alerting/RulesFramework Issues related to the Alerting Rules Framework Feature:Alerting/RulesManagement Issues related to the Rules Management UX release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v7.16.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Alerting] Jest test suites with unhandled promise rejections

5 participants