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

[RAM] Decrease repeated code in functional and unit tests for APIs #132464

Open
Zacqary opened this issue May 18, 2022 · 1 comment
Open

[RAM] Decrease repeated code in functional and unit tests for APIs #132464

Zacqary opened this issue May 18, 2022 · 1 comment
Labels
Feature:Alerting/RulesManagement Issues related to the Rules Management UX Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@Zacqary
Copy link
Contributor

Zacqary commented May 18, 2022

We need to create utility functions that generate API response objects for rules. These are repeated in lots of different files, so changing a single field in the API response breaks many different tests.
@Zacqary Zacqary added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Feature:Alerting/RulesManagement Issues related to the Rules Management UX labels May 18, 2022
@elasticmachine
Copy link
Contributor

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

@Zacqary Zacqary self-assigned this Feb 14, 2023
@Zacqary Zacqary mentioned this issue Feb 15, 2023
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature:Alerting/RulesManagement Issues related to the Rules Management UX Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
No open projects
AppEx: ResponseOps - Rules & Alerts Management
Status: Up for grabs
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[RAM] Decrease repeated code in APIs and functional tests Zacqary/kibana
2 participants
@Zacqary @elasticmachine