Skip to content

[scout tests] fix flaky FieldName typing for MKI#249754

Merged
dmlemeshko merged 4 commits intoelastic:mainfrom
dmlemeshko:scout/update-fieldName-selection
Jan 22, 2026
Merged

[scout tests] fix flaky FieldName typing for MKI#249754
dmlemeshko merged 4 commits intoelastic:mainfrom
dmlemeshko:scout/update-fieldName-selection

Conversation

@dmlemeshko
Copy link
Copy Markdown
Contributor

@dmlemeshko dmlemeshko commented Jan 20, 2026

Summary

While running tests on MKI we see flakiness related to typing Field Name:

Error: expect(locator).toHaveValue(expected) failed

Locator:  locator('.euiSuperSelect').filter({ has: locator('[data-test-subj="streamsAppFieldFormTypeSelect"]').first() }).locator('input[type="hidden"]')
Expected: "ip"
Received: ""
Timeout:  10000ms

Call log:
  - Expect "toHaveValue" with timeout 10000ms
  - waiting for locator('.euiSuperSelect').filter({ has: locator('[data-test-subj="streamsAppFieldFormTypeSelect"]').first() }).locator('input[type="hidden"]')
    14 × locator resolved to <input value="" type="hidden"/>
       - unexpected value ""

    at x-pack/platform/plugins/shared/streams_app/test/scout/ui/tests/data_management/data_mapping/wired_streams_schema.spec.ts:111:80
image

This PR uses typing with delay that is closer to end-user experience and should be more stable (reasonably slower)

@dmlemeshko dmlemeshko changed the title add typeFieldName in StreamsApp [scout tests] fix flaky FieldName typing for MKI Jan 20, 2026
@dmlemeshko dmlemeshko added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting test:scout labels Jan 20, 2026
@dmlemeshko dmlemeshko marked this pull request as ready for review January 21, 2026 10:17
@dmlemeshko dmlemeshko requested review from a team as code owners January 21, 2026 10:17
dmlemeshko
dmlemeshko commented Jan 21, 2026
View reviewed changes
Comment thread x-pack/platform/plugins/shared/streams_app/test/scout/ui/fixtures/page_objects/streams_app.ts

async typeFieldName(value: string) {
await this.page.testSubj.typeWithDelay('streamsAppSchemaEditorAddFieldFlyoutFieldName', value, {
delay: 75,
Copy link
Copy Markdown
Contributor Author

@dmlemeshko dmlemeshko Jan 21, 2026

Choose a reason for hiding this comment

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

75 ms simulates user typing experience

mohamedhamed-ahmed
mohamedhamed-ahmed approved these changes Jan 21, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@mohamedhamed-ahmed mohamedhamed-ahmed left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for fixing and cleaning this :)

@CoenWarmer
Copy link
Copy Markdown
Contributor

CoenWarmer commented Jan 21, 2026

Hi @dmlemeshko, is this similar to changes made in #249617?

@dmlemeshko dmlemeshko enabled auto-merge (squash) January 21, 2026 10:25
@dmlemeshko
Copy link
Copy Markdown
Contributor Author

dmlemeshko commented Jan 21, 2026 •
edited
Loading

Hi @dmlemeshko, is this similar to changes made in #249617?

Hi @CoenWarmer , not really. My changes are scoped to particular input field and in your PR it was untouched:

kibana/x-pack/platform/plugins/shared/streams_app/test/scout/ui/tests/data_management/data_mapping/wired_streams_schema.spec.ts

Lines 63 to 65 in 88b9530

await page.getByTestId('streamsAppSchemaEditorAddFieldFlyoutFieldName').click();
await page.keyboard.type(parentFieldName);
await page.keyboard.press('Enter');

Btw great to see API scout tests were addede, hope to see more UI -> API migration whenever possible.

@dmlemeshko dmlemeshko merged commit 2a5a66a into elastic:main Jan 22, 2026
16 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jan 22, 2026

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

yuliia-fryshko pushed a commit to yuliia-fryshko/kibana that referenced this pull request Jan 22, 2026
@dmlemeshko @yuliia-fryshko
[scout tests] fix flaky FieldName typing for MKI (elastic#249754)
8a46788 
## Summary

While running tests on MKI we see flakiness related to typing Field
Name:

```
Error: expect(locator).toHaveValue(expected) failed

Locator:  locator('.euiSuperSelect').filter({ has: locator('[data-test-subj="streamsAppFieldFormTypeSelect"]').first() }).locator('input[type="hidden"]')
Expected: "ip"
Received: ""
Timeout:  10000ms

Call log:
  - Expect "toHaveValue" with timeout 10000ms
  - waiting for locator('.euiSuperSelect').filter({ has: locator('[data-test-subj="streamsAppFieldFormTypeSelect"]').first() }).locator('input[type="hidden"]')
    14 × locator resolved to <input value="" type="hidden"/>
       - unexpected value ""

    at x-pack/platform/plugins/shared/streams_app/test/scout/ui/tests/data_management/data_mapping/wired_streams_schema.spec.ts:111:80
```

<img width="1280" height="720" alt="image"
src="https://github.com/user-attachments/assets/ce6b7ffe-6a72-405f-a76d-4cc8da9c3d53"
/>

This PR uses typing with delay that is closer to end-user experience and
should be more stable (reasonably slower)
qn895 pushed a commit to qn895/kibana that referenced this pull request Jan 22, 2026
@dmlemeshko @qn895
[scout tests] fix flaky FieldName typing for MKI (elastic#249754)
6309bef 
## Summary

While running tests on MKI we see flakiness related to typing Field
Name:

```
Error: expect(locator).toHaveValue(expected) failed

Locator:  locator('.euiSuperSelect').filter({ has: locator('[data-test-subj="streamsAppFieldFormTypeSelect"]').first() }).locator('input[type="hidden"]')
Expected: "ip"
Received: ""
Timeout:  10000ms

Call log:
  - Expect "toHaveValue" with timeout 10000ms
  - waiting for locator('.euiSuperSelect').filter({ has: locator('[data-test-subj="streamsAppFieldFormTypeSelect"]').first() }).locator('input[type="hidden"]')
    14 × locator resolved to <input value="" type="hidden"/>
       - unexpected value ""

    at x-pack/platform/plugins/shared/streams_app/test/scout/ui/tests/data_management/data_mapping/wired_streams_schema.spec.ts:111:80
```

<img width="1280" height="720" alt="image"
src="https://github.com/user-attachments/assets/ce6b7ffe-6a72-405f-a76d-4cc8da9c3d53"
/>

This PR uses typing with delay that is closer to end-user experience and
should be more stable (reasonably slower)
dennis-tismenko pushed a commit to dennis-tismenko/kibana that referenced this pull request Jan 22, 2026
@dmlemeshko @dennis-tismenko
[scout tests] fix flaky FieldName typing for MKI (elastic#249754)
64efaf1 
## Summary

While running tests on MKI we see flakiness related to typing Field
Name:

```
Error: expect(locator).toHaveValue(expected) failed

Locator:  locator('.euiSuperSelect').filter({ has: locator('[data-test-subj="streamsAppFieldFormTypeSelect"]').first() }).locator('input[type="hidden"]')
Expected: "ip"
Received: ""
Timeout:  10000ms

Call log:
  - Expect "toHaveValue" with timeout 10000ms
  - waiting for locator('.euiSuperSelect').filter({ has: locator('[data-test-subj="streamsAppFieldFormTypeSelect"]').first() }).locator('input[type="hidden"]')
    14 × locator resolved to <input value="" type="hidden"/>
       - unexpected value ""

    at x-pack/platform/plugins/shared/streams_app/test/scout/ui/tests/data_management/data_mapping/wired_streams_schema.spec.ts:111:80
```

<img width="1280" height="720" alt="image"
src="https://github.com/user-attachments/assets/ce6b7ffe-6a72-405f-a76d-4cc8da9c3d53"
/>

This PR uses typing with delay that is closer to end-user experience and
should be more stable (reasonably slower)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shahzad31 shahzad31 shahzad31 approved these changes

@mohamedhamed-ahmed mohamedhamed-ahmed mohamedhamed-ahmed approved these changes

Assignees

No one assigned

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes test:scout v9.4.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@dmlemeshko @CoenWarmer @elasticmachine @shahzad31 @mohamedhamed-ahmed @kibanamachine