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

[Fleet] Add unit tests for updating top-level fields for preconfigured policies #106606

Closed
jen-huang opened this issue Jul 22, 2021 · 2 comments · Fixed by #107759
Closed

[Fleet] Add unit tests for updating top-level fields for preconfigured policies #106606

jen-huang opened this issue Jul 22, 2021 · 2 comments · Fixed by #107759
Assignees
Labels
Team:Fleet Team label for Observability Data Collection Fleet team technical debt Improvement of the software architecture and operational architecture

Comments

@jen-huang
Copy link
Contributor

In #103722, we made preconfigured policies that are defined in kibana.yml that are managed (ones with is_managed: true) pick up new top-level field changes in kibana.yml.

We should add tests for this case alongside the rest of our unit tests for this feature in https://github.com/elastic/kibana/blob/master/x-pack/plugins/fleet/server/services/preconfiguration.test.ts

@jen-huang jen-huang added technical debt Improvement of the software architecture and operational architecture Team:Fleet Team label for Observability Data Collection Fleet team labels Jul 22, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@jen-huang
Copy link
Contributor Author

@hop-dev assigning this to you following our discussion of helping to improve our tests :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Fleet Team label for Observability Data Collection Fleet team technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants