-
Notifications
You must be signed in to change notification settings - Fork 262
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
tests for integration classes #145
Conversation
I made each test into a separate commit for easier perusal of the changes related to that test. I can squash this before merging if necessary. |
@kitchen looks great, thanks for your efforts! We'll be reviewing this more thoroughly shortly, and have it slated for an upcoming release |
@kitchen just wanted to let you know there was a typo in the changelog that shipped with the released puppet module |
these are tests for the integration classes.
I have left out 2 integrations:
datadog_agent::integration::directory
This integration doesn't appear to be functional. Catalogs which include it cannot compile. I've opened an issue at datadog_agent::integrations::directory is not functional? #144 for this.
datadog_agent::integration::generic
The work I plan to do next on this will obviate the need for the 'generic' integration, so it didn't make sense adding tests for something that's going to go away Real Soon Now™