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/transform/public/app/sections/create_transform/components/step_define - Transform: <DefinePivotSummary /> Minimal initialization #195992

Closed
kibanamachine opened this issue Oct 11, 2024 · 13 comments · Fixed by #195998
Closed

Failing test: Jest Tests.x-pack/plugins/transform/public/app/sections/create_transform/components/step_define - Transform: <DefinePivotSummary /> Minimal initialization #195992

kibanamachine opened this issue Oct 11, 2024 · 13 comments · Fixed by #195998
Assignees
@walterra
Labels
failed-test A test failure on a tracked branch, potentially flaky-test :ml

Comments

@kibanamachine
Copy link
Contributor

kibanamachine commented Oct 11, 2024 •
edited
Loading

A test failed on a tracked branch

TypeError: Cannot read properties of undefined (reading 'map')
    at map (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/x-pack/plugins/transform/public/app/hooks/use_transform_config_data.ts:186:40)
    at invokePassiveEffectCreate (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom.development.js:23487:20)
    at HTMLUnknownElement.callCallback (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom.development.js:3945:14)
    at HTMLUnknownElement.callTheUserObjectsOperation (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/jsdom/lib/jsdom/living/generated/EventListener.js:26:30)
    at innerInvokeEventListeners (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:350:25)
    at invokeEventListeners (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:286:3)
    at HTMLUnknownElementImpl._dispatch (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:233:9)
    at HTMLUnknownElementImpl.dispatchEvent (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:104:17)
    at HTMLUnknownElement.dispatchEvent (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/jsdom/lib/jsdom/living/generated/EventTarget.js:241:34)
    at Object.invokeGuardedCallbackDev (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom.development.js:3994:16)
    at invokeGuardedCallback (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom.development.js:4056:31)
    at flushPassiveEffectsImpl (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom.development.js:23574:9)
    at unstable_runWithPriority (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/scheduler/cjs/scheduler.development.js:468:12)
    at runWithPriority$1 (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom.development.js:11276:10)
    at flushPassiveEffects (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom.development.js:23447:14)
    at Object.<anonymous>.flushWork (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom-test-utils.development.js:992:10)
    at Immediate._onImmediate (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1728674988532779446/elastic/kibana-on-merge/kibana/node_modules/react-dom/cjs/react-dom-test-utils.development.js:1003:11)
    at processImmediate (node:internal/timers:478:21)

First failure: kibana-on-merge - main
@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Oct 11, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Oct 11, 2024
@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

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

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

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

@kibanamachine
Copy link
Contributor Author

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

@kibanamachine
Copy link
Contributor Author

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

@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 (#195992)
24f523b
jbudz added a commit that referenced this issue Oct 11, 2024
@jbudz
skip failing test suite (#195992)
08367e0
@jbudz
Copy link
Member

jbudz commented Oct 11, 2024

Skipped

main: 24f523b
8.x: 08367e0

@kibanamachine
Copy link
Contributor Author

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

@kibanamachine
Copy link
Contributor Author

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

@kibanamachine
Copy link
Contributor Author

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

@walterra
Copy link
Contributor

walterra commented Oct 11, 2024 •
edited
Loading

I'm removing the blocker label, it seems the failing is caused by a timeout, the tests pass locally. Will continue to investigate here: #195998

@walterra walterra removed the blocker label Oct 11, 2024
@walterra
Copy link
Contributor

Update: I was not able to reproduce this locally yet. I set something up locally to run the test continuously over night on my local machine and it never timed out. Since this code is also covered by functional tests, I'm moving the version label from 8.16 to 8.17.

@walterra walterra added v8.17.0 and removed v8.16.0 labels Oct 25, 2024
@walterra walterra mentioned this issue Dec 2, 2024
Merged
2 tasks
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Dec 5, 2024
@walterra
[ML] Transform: Fix <DefinePivotSummary /> jest test. (elastic#195998)
c65b935 
## Summary

Fixes elastic#195992 (failing test).

Updates the mock for the preview endpoint to make the test pass again.

### Checklist

- [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
- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)

(cherry picked from commit c4d6403)
kibanamachine added a commit that referenced this issue Dec 5, 2024
@kibanamachine @walterra
[8.x] [ML] Transform: Fix &lt;DefinePivotSummary /&gt; jest test. (#1…
627d84b 
…95998) (#203129)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ML] Transform: Fix &lt;DefinePivotSummary /&gt; jest test.
(#195998)](#195998)

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

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

<!--BACKPORT [{"author":{"name":"Walter
Rafelsberger","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-05T15:59:00Z","message":"[ML]
Transform: Fix <DefinePivotSummary /> jest test. (#195998)\n\n##
Summary\r\n\r\nFixes #195992 (failing test).\r\n\r\nUpdates the mock for
the preview endpoint to make the test pass again.\r\n\r\n###
Checklist\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- [x] This was
checked for breaking API changes and was
[labeled\r\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)","sha":"c4d64039b3c23433eb643f3a772357407695d024","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":[":ml","release_note:skip","Feature:Transforms","v9.0.0","backport:version","v8.18.0"],"title":"[ML]
Transform: Fix <DefinePivotSummary /> jest
test.","number":195998,"url":"https://github.com/elastic/kibana/pull/195998","mergeCommit":{"message":"[ML]
Transform: Fix <DefinePivotSummary /> jest test. (#195998)\n\n##
Summary\r\n\r\nFixes #195992 (failing test).\r\n\r\nUpdates the mock for
the preview endpoint to make the test pass again.\r\n\r\n###
Checklist\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- [x] This was
checked for breaking API changes and was
[labeled\r\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)","sha":"c4d64039b3c23433eb643f3a772357407695d024"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195998","number":195998,"mergeCommit":{"message":"[ML]
Transform: Fix <DefinePivotSummary /> jest test. (#195998)\n\n##
Summary\r\n\r\nFixes #195992 (failing test).\r\n\r\nUpdates the mock for
the preview endpoint to make the test pass again.\r\n\r\n###
Checklist\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- [x] This was
checked for breaking API changes and was
[labeled\r\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)","sha":"c4d64039b3c23433eb643f3a772357407695d024"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Walter Rafelsberger <[email protected]>
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this issue Dec 9, 2024
@walterra @SoniaSanzV
[ML] Transform: Fix <DefinePivotSummary /> jest test. (elastic#195998)
daec1ef 
## Summary

Fixes elastic#195992 (failing test).

Updates the mock for the preview endpoint to make the test pass again.

### Checklist

- [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
- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this issue Dec 9, 2024
@walterra @SoniaSanzV
[ML] Transform: Fix <DefinePivotSummary /> jest test. (elastic#195998)
b36baeb 
## Summary

Fixes elastic#195992 (failing test).

Updates the mock for the preview endpoint to make the test pass again.

### Checklist

- [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
- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Dec 9, 2024
@walterra @CAWilson94
[ML] Transform: Fix <DefinePivotSummary /> jest test. (elastic#195998)
1bde52e 
## Summary

Fixes elastic#195992 (failing test).

Updates the mock for the preview endpoint to make the test pass again.

### Checklist

- [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
- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
Samiul-TheSoccerFan pushed a commit to Samiul-TheSoccerFan/kibana that referenced this issue Dec 10, 2024
@walterra @Samiul-TheSoccerFan
[ML] Transform: Fix <DefinePivotSummary /> jest test. (elastic#195998)
c106f2c 
## Summary

Fixes elastic#195992 (failing test).

Updates the mock for the preview endpoint to make the test pass again.

### Checklist

- [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
- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
mykolaharmash pushed a commit to mykolaharmash/kibana that referenced this issue Dec 11, 2024
@walterra @mykolaharmash
[ML] Transform: Fix <DefinePivotSummary /> jest test. (elastic#195998)
96fc387 
## Summary

Fixes elastic#195992 (failing test).

Updates the mock for the preview endpoint to make the test pass again.

### Checklist

- [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
- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Dec 12, 2024
@walterra @CAWilson94
[ML] Transform: Fix <DefinePivotSummary /> jest test. (elastic#195998)
d16e2cc 
## Summary

Fixes elastic#195992 (failing test).

Updates the mock for the preview endpoint to make the test pass again.

### Checklist

- [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
- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@walterra walterra

Labels
failed-test A test failure on a tracked branch, potentially flaky-test :ml
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ML] Transform: Fix <DefinePivotSummary /> jest test. walterra/kibana
5 participants
@walterra @jbudz @peteharverson @elasticmachine @kibanamachine