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

[Infra UI] Serverless e2e infra security testing #168931

Closed
jennypavlova opened this issue Oct 16, 2023 · 3 comments
Closed

[Infra UI] Serverless e2e infra security testing #168931

jennypavlova opened this issue Oct 16, 2023 · 3 comments
Labels
blocked Feature:Metrics UI Metrics UI feature Feature:ObsHosts Hosts feature within Observability needs-refinement A reason and acceptance criteria need to be defined for this issue Project:Serverless Work as part of the Serverless project for its initial release Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team technical debt Improvement of the software architecture and operational architecture

Comments

@jennypavlova
Copy link
Member

jennypavlova commented Oct 16, 2023

We want to add functional tests in serverless for the cases when the infra plugin is used by different users with different roles. We have similar tests for some of the infra pages as part of our functional tests (https://github.com/elastic/kibana/blob/main/x-pack/test/functional/apps/infra/feature_controls/infrastructure_security.ts.).
In the serverless environment, the roles work differently and currently testing using different roles/users is not available. This issue tracks the progress of adding this option for the Cypress tests.
We have several options: Use Cypress for implementing role/user-based testing or wait until this is available: Maybe the better option should be to wait.

AC:

Image

@jennypavlova jennypavlova added Feature:Metrics UI Metrics UI feature Feature:ObsHosts Hosts feature within Observability needs-refinement A reason and acceptance criteria need to be defined for this issue Project:Serverless Work as part of the Serverless project for its initial release Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Oct 16, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

jennypavlova added a commit that referenced this issue Oct 17, 2023
Closes #166749
## Summary

This PR adds serverless functional tests for the infra plugin. The main
goal is to make sure that the app loads all the pages and successfully
navigates between them. As we already have functional tests covering the
whole functionality here we want to cover the differences with
serverless and some basic checks ensuring that base functionality works
correctly.

- Update the test to cover disabling the create rule button added
[here](#167978):
([x-pack/test_serverless/functional/test_suites/observability/infra/hosts_page.ts](87fb991#diff-76bd05d3fed2c44187682e27b4030ddb9cba1a36235321e0db1b371ef6739cc9))

⚠️ The tests for the second AC point related to user roles will go in a[
separate issue ](#168931)

Test server run:
`node scripts/functional_tests_server.js
--config=x-pack/test_serverless/functional/test_suites/observability/config.ts`
 Tests run: 
`node scripts/functional_test_runner
--config=x-pack/test_serverless/functional/test_suites/observability/config.ts
--grep "Observability Infra"`
dej611 pushed a commit to dej611/kibana that referenced this issue Oct 17, 2023
Closes elastic#166749
## Summary

This PR adds serverless functional tests for the infra plugin. The main
goal is to make sure that the app loads all the pages and successfully
navigates between them. As we already have functional tests covering the
whole functionality here we want to cover the differences with
serverless and some basic checks ensuring that base functionality works
correctly.

- Update the test to cover disabling the create rule button added
[here](elastic#167978):
([x-pack/test_serverless/functional/test_suites/observability/infra/hosts_page.ts](elastic@87fb991#diff-76bd05d3fed2c44187682e27b4030ddb9cba1a36235321e0db1b371ef6739cc9))

⚠️ The tests for the second AC point related to user roles will go in a[
separate issue ](elastic#168931)

Test server run:
`node scripts/functional_tests_server.js
--config=x-pack/test_serverless/functional/test_suites/observability/config.ts`
 Tests run: 
`node scripts/functional_test_runner
--config=x-pack/test_serverless/functional/test_suites/observability/config.ts
--grep "Observability Infra"`
@smith smith added the technical debt Improvement of the software architecture and operational architecture label Oct 30, 2023
@smith smith added blocked Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Nov 8, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@smith smith removed the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Nov 14, 2023
@smith
Copy link
Contributor

smith commented Nov 15, 2023

@jennypavlova got this covered.

@smith smith closed this as completed Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Feature:Metrics UI Metrics UI feature Feature:ObsHosts Hosts feature within Observability needs-refinement A reason and acceptance criteria need to be defined for this issue Project:Serverless Work as part of the Serverless project for its initial release Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Development

No branches or pull requests

3 participants