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.

Sign up for GitHub

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

Jump to bottom

[Cases] Serverless tests for Cases detail view #164827

Merged
merged 25 commits into from
Sep 7, 2023
Merged

[Cases] Serverless tests for Cases detail view #164827

merged 25 commits into from
Sep 7, 2023

Conversation

adcoelho
Copy link
Contributor

@adcoelho adcoelho commented Aug 25, 2023 •
edited
Loading

Connected with #164552

Summary

This PR creates a serverless version of the tests currently in x-pack/test/functional_with_es_ssl/apps/cases/group1/view_case.ts.

@adcoelho adcoelho added technical debt Improvement of the software architecture and operational architecture Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Feature:Cases Cases feature labels Aug 25, 2023
@adcoelho adcoelho self-assigned this Aug 25, 2023
@adcoelho adcoelho requested a review from a team as a code owner August 25, 2023 10:57
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

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

@adcoelho adcoelho added the release_note:skip Skip the PR/issue when compiling release notes label Aug 25, 2023
@adcoelho adcoelho mentioned this pull request Aug 25, 2023
Open
6 tasks
@adcoelho adcoelho requested a review from pheyos August 25, 2023 13:22
@pheyos
Copy link
Member

pheyos commented Sep 1, 2023

With the serverless test infrastructure changes we recently made, I think this PR needs a merge from main before looking into any more test failures.

js-jankisalvi
js-jankisalvi approved these changes Sep 5, 2023
View reviewed changes
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.

LGTM 👍

@kibana-ci
Copy link
Collaborator

kibana-ci commented Sep 7, 2023 •
edited
Loading

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Serverless Security Investigations Cypress Tests #3 / Row renderers "before all" hook for "Row renderers should be enabled by default" "before all" hook for "Row renderers should be enabled by default"

Metrics [docs]

✅ unchanged

History

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

cc @adcoelho

@adcoelho adcoelho merged commit 3f18975 into elastic:main Sep 7, 2023
@kibanamachine kibanamachine added v8.11.0 backport:skip This commit does not require backporting labels Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@js-jankisalvi js-jankisalvi js-jankisalvi approved these changes

@cnasikas cnasikas cnasikas approved these changes

@pheyos pheyos Awaiting requested review from pheyos

Assignees

@adcoelho adcoelho

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) technical debt Improvement of the software architecture and operational architecture v8.11.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@adcoelho @elasticmachine @pheyos @kibana-ci @cnasikas @js-jankisalvi @kibanamachine