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

[API keys] Improve functional tests for API keys management page #200110

Merged

Conversation

SiddharthMantri
Copy link
Contributor

@SiddharthMantri SiddharthMantri commented Nov 14, 2024

Closes #200756

Summary

Enhance existing functional tests for API Keys querying to test filters and toggles while querying API keys.

Notes

The following tests were added:

  • Toggling the following filters displays the correct keys
    • Personal, Managed, Cross cluster types
    • Active or expired keys
  • Filtering by username by clicking the dropdown list
  • Added test for querying using the search bar but it is skipped for now as we'd like to fix the behavior. Tracked by Improve search experience for API keys page #195795

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

  • Unit or functional tests were updated or added to match the most common scenarios=
  • The PR description includes the appropriate Release Notes section, and the correct release_node:* label is applied per the guidelines

@SiddharthMantri
Copy link
Contributor Author

@elasticmachine merge upstream

@SiddharthMantri SiddharthMantri added test Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! Feature:Users/Roles/API Keys release_note:skip Skip the PR/issue when compiling release notes backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development labels Nov 20, 2024
@SiddharthMantri SiddharthMantri marked this pull request as ready for review November 20, 2024 15:00
@SiddharthMantri SiddharthMantri requested a review from a team as a code owner November 20, 2024 15:00
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security (Team:Security)

@SiddharthMantri
Copy link
Contributor Author

@elasticmachine merge upstream

@SiddharthMantri
Copy link
Contributor Author

/ci

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
security 540.6KB 540.9KB +273.0B

History

@SiddharthMantri SiddharthMantri merged commit 889ce00 into elastic:main Nov 21, 2024
21 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.15, 8.16, 8.17, 8.x

https://github.com/elastic/kibana/actions/runs/11955043711

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 21, 2024
…stic#200110)

Closes elastic#200756

## Summary

Enhance existing functional tests for API Keys querying to test filters
and toggles while querying API keys.

### Notes

The following tests were added:

- Toggling the following filters displays the correct keys
    - Personal, Managed, Cross cluster types
    - Active or expired keys
- Filtering by username by clicking the dropdown list
- Added test for querying using the search bar **but it is skipped** for
now as we'd like to fix the behavior. Tracked by
elastic#195795

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [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=
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_node:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 889ce00)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 21, 2024
…stic#200110)

Closes elastic#200756

## Summary

Enhance existing functional tests for API Keys querying to test filters
and toggles while querying API keys.

### Notes

The following tests were added:

- Toggling the following filters displays the correct keys
    - Personal, Managed, Cross cluster types
    - Active or expired keys
- Filtering by username by clicking the dropdown list
- Added test for querying using the search bar **but it is skipped** for
now as we'd like to fix the behavior. Tracked by
elastic#195795

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [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=
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_node:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 889ce00)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 21, 2024
…stic#200110)

Closes elastic#200756

## Summary

Enhance existing functional tests for API Keys querying to test filters
and toggles while querying API keys.

### Notes

The following tests were added:

- Toggling the following filters displays the correct keys
    - Personal, Managed, Cross cluster types
    - Active or expired keys
- Filtering by username by clicking the dropdown list
- Added test for querying using the search bar **but it is skipped** for
now as we'd like to fix the behavior. Tracked by
elastic#195795

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [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=
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_node:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 889ce00)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 21, 2024
…stic#200110)

Closes elastic#200756

## Summary

Enhance existing functional tests for API Keys querying to test filters
and toggles while querying API keys.

### Notes

The following tests were added:

- Toggling the following filters displays the correct keys
    - Personal, Managed, Cross cluster types
    - Active or expired keys
- Filtering by username by clicking the dropdown list
- Added test for querying using the search bar **but it is skipped** for
now as we'd like to fix the behavior. Tracked by
elastic#195795

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [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=
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_node:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 889ce00)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.15
8.16
8.17
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 22, 2024
kibanamachine added a commit that referenced this pull request Nov 25, 2024
…ge (#200110) (#201204)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[API keys] Improve functional tests for API keys management page
(#200110)](#200110)

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

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

<!--BACKPORT
[{"author":{"name":"Sid","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-21T14:15:13Z","message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","Team:Security","Feature:Users/Roles/API
Keys","release_note:skip","v9.0.0","backport:prev-major"],"title":"[API
keys] Improve functional tests for API keys management
page","number":200110,"url":"https://github.com/elastic/kibana/pull/200110","mergeCommit":{"message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200110","number":200110,"mergeCommit":{"message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b"}}]}]
BACKPORT-->

Co-authored-by: Sid <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
kibanamachine added a commit that referenced this pull request Nov 25, 2024
#200110) (#201207)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[API keys] Improve functional tests for API keys management page
(#200110)](#200110)

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

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

<!--BACKPORT
[{"author":{"name":"Sid","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-21T14:15:13Z","message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","Team:Security","Feature:Users/Roles/API
Keys","release_note:skip","v9.0.0","backport:prev-major"],"title":"[API
keys] Improve functional tests for API keys management
page","number":200110,"url":"https://github.com/elastic/kibana/pull/200110","mergeCommit":{"message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200110","number":200110,"mergeCommit":{"message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b"}}]}]
BACKPORT-->

Co-authored-by: Sid <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
kibanamachine added a commit that referenced this pull request Nov 25, 2024
…ge (#200110) (#201206)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[API keys] Improve functional tests for API keys management page
(#200110)](#200110)

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

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

<!--BACKPORT
[{"author":{"name":"Sid","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-21T14:15:13Z","message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","Team:Security","Feature:Users/Roles/API
Keys","release_note:skip","v9.0.0","backport:prev-major"],"title":"[API
keys] Improve functional tests for API keys management
page","number":200110,"url":"https://github.com/elastic/kibana/pull/200110","mergeCommit":{"message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200110","number":200110,"mergeCommit":{"message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b"}}]}]
BACKPORT-->

Co-authored-by: Sid <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
kibanamachine added a commit that referenced this pull request Nov 25, 2024
…ge (#200110) (#201202)

# Backport

This will backport the following commits from `main` to `8.15`:
- [[API keys] Improve functional tests for API keys management page
(#200110)](#200110)

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

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

<!--BACKPORT
[{"author":{"name":"Sid","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-21T14:15:13Z","message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","Team:Security","Feature:Users/Roles/API
Keys","release_note:skip","v9.0.0","backport:prev-major"],"title":"[API
keys] Improve functional tests for API keys management
page","number":200110,"url":"https://github.com/elastic/kibana/pull/200110","mergeCommit":{"message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200110","number":200110,"mergeCommit":{"message":"[API
keys] Improve functional tests for API keys management page
(#200110)\n\nCloses
https://github.com/elastic/kibana/issues/200756\r\n\r\n##
Summary\r\n\r\nEnhance existing functional tests for API Keys querying
to test filters\r\nand toggles while querying API keys.\r\n\r\n###
Notes\r\n\r\nThe following tests were added:\r\n\r\n- Toggling the
following filters displays the correct keys\r\n - Personal, Managed,
Cross cluster types\r\n - Active or expired keys\r\n- Filtering by
username by clicking the dropdown list\r\n- Added test for querying
using the search bar **but it is skipped** for\r\nnow as we'd like to
fix the behavior. Tracked
by\r\nhttps://github.com//issues/195795\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\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- [ ] The PR
description includes the appropriate Release Notes section,\r\nand the
correct `release_node:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"889ce000eb18bf60d3dc65ec0b39dbf705c7184b"}}]}]
BACKPORT-->

Co-authored-by: Sid <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
@kibanamachine kibanamachine added v8.15.5 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Nov 25, 2024
@mistic
Copy link
Member

mistic commented Nov 26, 2024

This PR didn't make it on time for the latest v8.15.5 BC. Updating the labels.

@mistic mistic added v8.15.6 and removed v8.15.5 labels Nov 26, 2024
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this pull request Nov 26, 2024
…stic#200110)

Closes elastic#200756

## Summary

Enhance existing functional tests for API Keys querying to test filters
and toggles while querying API keys.

### Notes

The following tests were added:

- Toggling the following filters displays the correct keys
    - Personal, Managed, Cross cluster types
    - Active or expired keys
- Filtering by username by clicking the dropdown list
- Added test for querying using the search bar **but it is skipped** for
now as we'd like to fix the behavior. Tracked by
elastic#195795

### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [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=
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_node:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Elastic Machine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
…stic#200110)

Closes elastic#200756

## Summary

Enhance existing functional tests for API Keys querying to test filters
and toggles while querying API keys.

### Notes

The following tests were added:

- Toggling the following filters displays the correct keys
    - Personal, Managed, Cross cluster types
    - Active or expired keys
- Filtering by username by clicking the dropdown list
- Added test for querying using the search bar **but it is skipped** for
now as we'd like to fix the behavior. Tracked by
elastic#195795

### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [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=
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_node:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development Feature:Users/Roles/API Keys release_note:skip Skip the PR/issue when compiling release notes Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! test v8.15.6 v8.16.2 v8.17.0 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[API Keys] Update functional tests for API keys search page
5 participants