Skip to content

[9.3] [Index Management] Migrate flaky integration tests to unit tests (#258942)#260061

Merged
kapral18 merged 1 commit intoelastic:9.3from
kapral18:backport/9.3/pr-258942
Mar 31, 2026
Merged

[9.3] [Index Management] Migrate flaky integration tests to unit tests (#258942)#260061
kapral18 merged 1 commit intoelastic:9.3from
kapral18:backport/9.3/pr-258942

Conversation

@kapral18
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.3:

Questions ?

Please refer to the Backport tool documentation

@kapral18 kapral18 requested a review from kibanamachine as a code owner March 27, 2026 17:12
@kapral18 kapral18 added the backport This PR is a backport of another PR label Mar 27, 2026
@kapral18 kapral18 enabled auto-merge (squash) March 27, 2026 17:12
@kapral18 kapral18 force-pushed the backport/9.3/pr-258942 branch 3 times, most recently from c1c60ff to 7f43af1 Compare March 30, 2026 13:01
…stic#258942)

- Migrates flaky Index Management client integration coverage to focused
unit/section tests (mappings editor + template wizard).
- Speeds up `mappings_editor.test.tsx` by mocking UI-heavy bits and adds
compensation unit tests for `TypeParameter` and `ReferenceFieldSelects`.
- Removes legacy `__jest__/client_integration` specs/helpers for these
areas.

- Closes elastic#257682
- Closes elastic#256305
- Closes elastic#253534
- Closes elastic#239816
- Closes elastic#239817
- Closes elastic#239818
- Closes elastic#254922
- Closes elastic#254951

- [Integration → unit test mapping
(gist)](https://gist.github.com/kapral18/24d218baaddaa374822493dd812cfe7e)

- [x] `node scripts/check_changes.ts`
- [x] `yarn test:jest
x-pack/platform/plugins/shared/index_management/public/application/components/mappings_editor/mappings_editor.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/components/mappings_editor/mappings_editor.edit_field.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/components/mappings_editor/mappings_editor.text_datatype.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/components/mappings_editor/components/document_fields/field_parameters/select_inference_id.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/components/mappings_editor/components/document_fields/field_parameters/type_parameter.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/components/mappings_editor/components/document_fields/field_parameters/reference_field_selects.test.tsx`
- [x] `yarn test:jest
x-pack/platform/plugins/shared/index_management/public/application/sections/template_clone/template_clone.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/sections/template_create/template_create.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/sections/template_edit/template_edit.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/components/template_form/utils/build_template_from_wizard_data.test.ts
x-pack/platform/plugins/shared/index_management/public/application/components/template_form/steps/step_components.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/components/template_form/steps/step_logistics.test.tsx
x-pack/platform/plugins/shared/index_management/public/application/components/template_form/steps/step_review.test.tsx`

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
(cherry picked from commit a31fae8)
@kapral18 kapral18 force-pushed the backport/9.3/pr-258942 branch from 7f43af1 to 854f47c Compare March 30, 2026 13:19
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
indexManagement 699 700 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
indexManagement 739.9KB 740.0KB +112.0B
Unknown metric groups

ESLint disabled in files

id before after diff
indexManagement 2 1 -1

ESLint disabled line counts

id before after diff
indexManagement 15 11 -4

Total ESLint disabled count

id before after diff
indexManagement 17 12 -5

History

@kapral18 kapral18 merged commit 73fbf7d into elastic:9.3 Mar 31, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants