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

[Cases] Fix flaky test in user actions pagination #163474

Merged
merged 7 commits into from
Aug 9, 2023

Conversation

adcoelho
Copy link
Contributor

@adcoelho adcoelho commented Aug 9, 2023

Fixes #156750
Fixes #156747
Fixes #156744
Fixes #156743
Fixes #156748
Fixes #156746
Fixes #156745
Fixes #156749

Summary

Basically what you suggested offline @js-jankisalvi.

Scenario Old Test  Functional test where it is covered
Only shows one list of user actions when total_user_actions < 10 'renders only one action list when user actions are less than or equal to 10' 'initially renders user actions list correctly'
Show More button does not show up when total_user_actions <= page_size 'shows more button visible 21st user action added' 'initially renders user actions list correctly'
Shows 2 user action lists when total_user_actions > page_size 'renders two user actions list when user actions are more than 10' 'shows more actions on button click'
loading spinner 'Loading spinner when user actions loading' 'shows more actions on button click'

Flaky Test Runner

Let's make sure that the functional tests don't become flaky too.

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

@adcoelho adcoelho added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Feature:Cases Cases feature v8.10.0 labels Aug 9, 2023
@adcoelho adcoelho self-assigned this Aug 9, 2023
@adcoelho adcoelho requested a review from a team as a code owner August 9, 2023 10:16
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

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

@adcoelho adcoelho requested a review from js-jankisalvi August 9, 2023 10:24
@delanni delanni added release_note:skip Skip the PR/issue when compiling release notes and removed release_note:skip Skip the PR/issue when compiling release notes labels Aug 9, 2023
@adcoelho adcoelho removed the backport:skip This commit does not require backporting label Aug 9, 2023
Copy link
Contributor

@js-jankisalvi js-jankisalvi left a comment

Choose a reason for hiding this comment

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

looks good 🚀

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @adcoelho

@adcoelho adcoelho merged commit 49849cf into elastic:main Aug 9, 2023
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Aug 9, 2023
bryce-b pushed a commit to bryce-b/kibana that referenced this pull request Aug 9, 2023
Fixes elastic#156750
Fixes elastic#156747
Fixes elastic#156744
Fixes elastic#156743
Fixes elastic#156748
Fixes elastic#156746
Fixes elastic#156745
Fixes elastic#156749

## Summary

Basically what you suggested offline @js-jankisalvi.

| Scenario | Old Test | Functional test where it is covered |
| -------------   |  -------------   | ------------- |
| Only shows one list of user actions when `total_user_actions < 10` |
`'renders only one action list when user actions are less than or equal
to 10'` | `'initially renders user actions list correctly'`|
| `Show More` button does not show up when `total_user_actions <=
page_size` | `'shows more button visible 21st user action added'` |
`'initially renders user actions list correctly'` |
| Shows 2 user action lists when `total_user_actions > page_size` |
`'renders two user actions list when user actions are more than 10'` |
`'shows more actions on button click'` |
| loading spinner | `'Loading spinner when user actions loading'` |
`'shows more actions on button click'` |

## Flaky Test Runner

Let's make sure that the functional tests don't become flaky too.

https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2826
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Feature:Cases Cases feature release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.10.0
Projects
None yet
6 participants