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

Failing test: Jest Tests.x-pack/plugins/cases/public/components/add_comment - AddComment should post comment on submit click #168505

Open
kibanamachine opened this issue Oct 10, 2023 · 6 comments · Fixed by #170400 or #170394
Assignees
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 Oct 10, 2023

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 (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/x-pack/plugins/cases/public/components/add_comment/index.test.tsx:99:3)
    at _dispatchDescribe (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/node_modules/jest-circus/build/index.js:91:26)
    at describe (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/node_modules/jest-circus/build/index.js:55:5)
    at Object.describe (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/x-pack/plugins/cases/public/components/add_comment/index.test.tsx:54:1)
    at Runtime._execModule (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/node_modules/jest-runtime/build/index.js:1430:24)
    at Runtime._loadModule (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/node_modules/jest-runtime/build/index.js:1013:12)
    at Runtime.requireModule (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/node_modules/jest-runtime/build/index.js:873:12)
    at jestAdapter (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:77:13)
    at runTestInternal (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/node_modules/jest-runner/build/runTest.js:367:16)
    at runTest (/var/lib/buildkite-agent/builds/kb-n2-4-spot-4fc618f967d526e5/elastic/kibana-on-merge/kibana/node_modules/jest-runner/build/runTest.js:444:34)

First failure: CI Build - 8.11

@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Oct 10, 2023
@botelastic botelastic bot added the needs-team Issues missing a team label label Oct 10, 2023
@kibanamachine kibanamachine added the Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) label Oct 10, 2023
@elasticmachine
Copy link
Contributor

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

@botelastic botelastic bot removed the needs-team Issues missing a team label label Oct 10, 2023
@cnasikas cnasikas added the Feature:Cases Cases feature label Oct 10, 2023
@elasticmachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor Author

New failure: CI Build - main

@kibanamachine
Copy link
Contributor Author

New failure: CI Build - main

@mistic
Copy link
Member

mistic commented Oct 25, 2023

Skipped.

main: e7dbc9e
8.11: 350492e

mistic added a commit that referenced this issue Oct 25, 2023
@mistic mistic added the v8.11.1 label Oct 25, 2023
bryce-b pushed a commit to bryce-b/kibana that referenced this issue Oct 30, 2023
@adcoelho adcoelho self-assigned this Nov 2, 2023
adcoelho added a commit that referenced this issue Nov 6, 2023
Fixes #168505
Fixes #168506
Fixes #168507
Fixes #168508
Fixes #168509
(I hope)

## Summary

Covered most of the logic in e2e tests.

Flaky test runner:
-
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3869

## Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 6, 2023
Fixes elastic#168505
Fixes elastic#168506
Fixes elastic#168507
Fixes elastic#168508
Fixes elastic#168509
(I hope)

## Summary

Covered most of the logic in e2e tests.

Flaky test runner:
-
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3869

## Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 96a4d5b)
kibanamachine added a commit that referenced this issue Nov 6, 2023
# Backport

This will backport the following commits from `main` to `8.11`:
- [[Cases] Fix flaky "add comment" tests.
(#170400)](#170400)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT
[{"author":{"name":"Antonio","email":"antonio.coelho@elastic.co"},"sourceCommit":{"committedDate":"2023-11-06T10:45:16Z","message":"[Cases]
Fix flaky \"add comment\" tests. (#170400)\n\nFixes #168505\r\nFixes
#168506\r\nFixes #168507\r\nFixes #168508\r\nFixes #168509\r\n(I
hope)\r\n\r\n## Summary\r\n\r\nCovered most of the logic in e2e
tests.\r\n\r\nFlaky test
runner:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3869\r\n\r\n##
Checklist\r\n\r\nDelete any items that are not applicable to this
PR.\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"96a4d5b63fd2fedc94905729edf06e5df7cac2aa","branchLabelMapping":{"^v8.12.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:ResponseOps","Feature:Cases","v8.11.0","v8.12.0"],"number":170400,"url":"https://github.com/elastic/kibana/pull/170400","mergeCommit":{"message":"[Cases]
Fix flaky \"add comment\" tests. (#170400)\n\nFixes #168505\r\nFixes
#168506\r\nFixes #168507\r\nFixes #168508\r\nFixes #168509\r\n(I
hope)\r\n\r\n## Summary\r\n\r\nCovered most of the logic in e2e
tests.\r\n\r\nFlaky test
runner:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3869\r\n\r\n##
Checklist\r\n\r\nDelete any items that are not applicable to this
PR.\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"96a4d5b63fd2fedc94905729edf06e5df7cac2aa"}},"sourceBranch":"main","suggestedTargetBranches":["8.11"],"targetPullRequestStates":[{"branch":"8.11","label":"v8.11.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.12.0","labelRegex":"^v8.12.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/170400","number":170400,"mergeCommit":{"message":"[Cases]
Fix flaky \"add comment\" tests. (#170400)\n\nFixes #168505\r\nFixes
#168506\r\nFixes #168507\r\nFixes #168508\r\nFixes #168509\r\n(I
hope)\r\n\r\n## Summary\r\n\r\nCovered most of the logic in e2e
tests.\r\n\r\nFlaky test
runner:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3869\r\n\r\n##
Checklist\r\n\r\nDelete any items that are not applicable to this
PR.\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"96a4d5b63fd2fedc94905729edf06e5df7cac2aa"}}]}]
BACKPORT-->

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

New failure: CI Build - 8.12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
5 participants