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

[Metricbeat] Add http endpoint tests for php_fpm module #11088

Merged
merged 1 commit into from
Mar 6, 2019

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Mar 5, 2019

This change is based on #10648 to migrate to golden files instead of the dynamically generated data files.

This adds also support for query params to the testing framework.

This adds also support for query params to the testing framework.
@ruflin ruflin added module review Metricbeat Metricbeat Team:Integrations Label for the Integrations team labels Mar 5, 2019
@ruflin ruflin requested review from a team as code owners March 5, 2019 14:24
Copy link
Contributor

@sayden sayden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While I see that "endpoint tests for php_fpm module" have been added, I don't understand why the TestData functions have been removed. I don't think it's an error but maybe you can update the PR description so we can give you a better review 🙂

@ruflin
Copy link
Contributor Author

ruflin commented Mar 6, 2019

This change is based on #10648 The data.json files are now generated on each test run and should stay the same (like in Filebeat). The dynamic generation is not needed anymore.

Updated also the PR description.

@ruflin ruflin merged commit af9e048 into elastic:master Mar 6, 2019
@ruflin ruflin deleted the add-php_fpm-tests branch March 6, 2019 09:38
@ruflin ruflin self-assigned this Mar 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Metricbeat Metricbeat module review Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants