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.

Sign up for GitHub

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

Jump to bottom

Failing test: Jest Tests.x-pack/plugins/cases/public/components/case_form_fields - SyncAlertsToggle it renders #190270

Open
kibanamachine opened this issue Aug 9, 2024 · 17 comments · Fixed by #200870
Open

Failing test: Jest Tests.x-pack/plugins/cases/public/components/case_form_fields - SyncAlertsToggle it renders #190270

kibanamachine opened this issue Aug 9, 2024 · 17 comments · Fixed by #200870
Assignees
@adcoelho
Labels
failed-test A test failure on a tracked branch, potentially flaky-test Feature:Cases Cases feature Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@kibanamachine
Copy link
Contributor

kibanamachine commented Aug 9, 2024 •
edited
Loading

A test failed on a tracked branch

Error: thrown: "Exceeded timeout of 5000 ms for a test.
Add a timeout value to this test to increase the timeout, if this is a long-running test. See https://jestjs.io/docs/api#testname-fn-timeout."
    at it (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/x-pack/plugins/cases/public/components/case_form_fields/sync_alerts_toggle.test.tsx:33:3)
    at _dispatchDescribe (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/node_modules/jest-circus/build/index.js:91:26)
    at describe (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/node_modules/jest-circus/build/index.js:55:5)
    at Object.describe (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/x-pack/plugins/cases/public/components/case_form_fields/sync_alerts_toggle.test.tsx:17:1)
    at Runtime._execModule (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/node_modules/jest-runtime/build/index.js:1430:24)
    at Runtime._loadModule (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/node_modules/jest-runtime/build/index.js:1013:12)
    at Runtime.requireModule (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/node_modules/jest-runtime/build/index.js:873:12)
    at jestAdapter (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:77:13)
    at runTestInternal (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/node_modules/jest-runner/build/runTest.js:367:16)
    at runTest (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1723227486306085599/elastic/kibana-on-merge/kibana/node_modules/jest-runner/build/runTest.js:444:34)

First failure: kibana-on-merge - main
@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Aug 9, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Aug 9, 2024
@kibanamachine kibanamachine added the Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) label Aug 9, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Aug 9, 2024
@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

jbudz added a commit that referenced this issue Aug 15, 2024
@jbudz
skip failing test suite (#190270)
dde88c8
@jbudz
Copy link
Member

jbudz commented Aug 15, 2024

Skipped

main: dde88c8

@cnasikas cnasikas added the Feature:Cases Cases feature label Sep 3, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops-cases (Feature:Cases)

@js-jankisalvi
Copy link
Contributor

Fixed by #191212

@kibanamachine kibanamachine reopened this Sep 26, 2024
@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.15

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

jbudz added a commit that referenced this issue Oct 11, 2024
@jbudz
skip failing test suite (#192997, #190270)
e10b330
jbudz added a commit that referenced this issue Oct 11, 2024
@jbudz
skip failing test suite (#192997, #190270)
6903095
@jbudz
Copy link
Member

jbudz commented Oct 11, 2024

Skipped

main: e10b330
8.x: 6903095

@jbudz jbudz added the v8.16.0 label Oct 11, 2024
@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

delanni added a commit to delanni/kibana that referenced this issue Oct 17, 2024
@delanni
skip SyncAlertsToggle tests for elastic#190270
d4ceeba
@delanni delanni mentioned this issue Oct 17, 2024
Merged
delanni added a commit that referenced this issue Oct 18, 2024
@delanni
[tests] skip SyncAlertsToggle tests (#196686)
7178062 
## Summary
It seems the flaky unit-tests cases from
#190270 were skipped in the
wrong file with a similar component name.
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Oct 18, 2024
@delanni
[tests] skip SyncAlertsToggle tests (elastic#196686)
f8437d0 
## Summary
It seems the flaky unit-tests cases from
elastic#190270 were skipped in the
wrong file with a similar component name.

(cherry picked from commit 7178062)
kibanamachine added a commit that referenced this issue Oct 18, 2024
@kibanamachine @delanni
[8.x] [tests] skip SyncAlertsToggle tests (#196686) (#196805)
2f65de0 
# Backport

This will backport the following commits from `main` to `8.x`:
- [[tests] skip SyncAlertsToggle tests
(#196686)](#196686)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Alex
Szabo","email":"alex.szabo@elastic.co"},"sourceCommit":{"committedDate":"2024-10-18T07:49:36Z","message":"[tests]
skip SyncAlertsToggle tests (#196686)\n\n## Summary\r\nIt seems the
flaky unit-tests cases
from\r\nhttps://github.com//issues/190270 were skipped in
the\r\nwrong file with a similar component
name.","sha":"717806263b06aca6d655a3edac1aa9c62c042b07","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor"],"title":"[tests]
skip SyncAlertsToggle
tests","number":196686,"url":"https://github.com/elastic/kibana/pull/196686","mergeCommit":{"message":"[tests]
skip SyncAlertsToggle tests (#196686)\n\n## Summary\r\nIt seems the
flaky unit-tests cases
from\r\nhttps://github.com//issues/190270 were skipped in
the\r\nwrong file with a similar component
name.","sha":"717806263b06aca6d655a3edac1aa9c62c042b07"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196686","number":196686,"mergeCommit":{"message":"[tests]
skip SyncAlertsToggle tests (#196686)\n\n## Summary\r\nIt seems the
flaky unit-tests cases
from\r\nhttps://github.com//issues/190270 were skipped in
the\r\nwrong file with a similar component
name.","sha":"717806263b06aca6d655a3edac1aa9c62c042b07"}}]}] BACKPORT-->

Co-authored-by: Alex Szabo <alex.szabo@elastic.co>
@cnasikas cnasikas removed the v8.16.0 label Oct 29, 2024
@adcoelho adcoelho self-assigned this Nov 20, 2024
@adcoelho adcoelho mentioned this issue Nov 20, 2024
Merged
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 21, 2024
@adcoelho
[ResponseOps][Cases] Fix SyncAlertsToggle flaky test (elastic#200870)
639779c 
Fixes elastic#190270

## Summary

We already wrap `SyncAlertsToggle` in `<FormTestComponent>` so I removed
`createAppMockRenderer` which wasn't necessary.

(cherry picked from commit d91d82c)
kibanamachine added a commit that referenced this issue Nov 21, 2024
@kibanamachine @adcoelho
[8.x] [ResponseOps][Cases] Fix &#x60;SyncAlertsToggle&#x60; flaky test (
4e2dec9 
#200870) (#201093)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ResponseOps][Cases] Fix &#x60;SyncAlertsToggle&#x60; flaky test
(#200870)](#200870)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Antonio","email":"antonio.coelho@elastic.co"},"sourceCommit":{"committedDate":"2024-11-21T09:26:10Z","message":"[ResponseOps][Cases]
Fix `SyncAlertsToggle` flaky test (#200870)\n\nFixes #190270\r\n\r\n##
Summary\r\n\r\nWe already wrap `SyncAlertsToggle` in
`<FormTestComponent>` so I removed\r\n`createAppMockRenderer` which
wasn't
necessary.","sha":"d91d82c2b9581821cf5ee30e8987a13f2d33b907","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["technical
debt","release_note:skip","Team:ResponseOps","v9.0.0","Feature:Cases","backport:prev-minor"],"title":"[ResponseOps][Cases]
Fix `SyncAlertsToggle` flaky
test","number":200870,"url":"https://github.com/elastic/kibana/pull/200870","mergeCommit":{"message":"[ResponseOps][Cases]
Fix `SyncAlertsToggle` flaky test (#200870)\n\nFixes #190270\r\n\r\n##
Summary\r\n\r\nWe already wrap `SyncAlertsToggle` in
`<FormTestComponent>` so I removed\r\n`createAppMockRenderer` which
wasn't
necessary.","sha":"d91d82c2b9581821cf5ee30e8987a13f2d33b907"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200870","number":200870,"mergeCommit":{"message":"[ResponseOps][Cases]
Fix `SyncAlertsToggle` flaky test (#200870)\n\nFixes #190270\r\n\r\n##
Summary\r\n\r\nWe already wrap `SyncAlertsToggle` in
`<FormTestComponent>` so I removed\r\n`createAppMockRenderer` which
wasn't necessary.","sha":"d91d82c2b9581821cf5ee30e8987a13f2d33b907"}}]}]
BACKPORT-->

Co-authored-by: Antonio <antonio.coelho@elastic.co>
@kibanamachine kibanamachine reopened this Nov 21, 2024
@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

@cnasikas
Copy link
Member

I am closing the issue because it is possible that the backport was merged to 8.x after the new failure.

@kibanamachine kibanamachine reopened this Nov 24, 2024
@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this issue Nov 26, 2024
@adcoelho @paulinashakirova
[ResponseOps][Cases] Fix SyncAlertsToggle flaky test (elastic#200870)
74ef7da 
Fixes elastic#190270

## Summary

We already wrap `SyncAlertsToggle` in `<FormTestComponent>` so I removed
`createAppMockRenderer` which wasn't necessary.
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Dec 12, 2024
@adcoelho @CAWilson94
[ResponseOps][Cases] Fix SyncAlertsToggle flaky test (elastic#200870)
9beca40 
Fixes elastic#190270

## Summary

We already wrap `SyncAlertsToggle` in `<FormTestComponent>` so I removed
`createAppMockRenderer` which wasn't necessary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adcoelho adcoelho

Labels
failed-test A test failure on a tracked branch, potentially flaky-test Feature:Cases Cases feature Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ResponseOps][Cases] Fix SyncAlertsToggle flaky test adcoelho/kibana
6 participants
@adcoelho @jbudz @cnasikas @elasticmachine @kibanamachine @js-jankisalvi