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
apache: sync options and add tests #798
Conversation
content: content_instance_file, | ||
path: "#{options[:plugin_conf_dir]}/25-apache-instance-site2.conf" | ||
) | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you add another test to verify that it compiles with all dependencies?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added check that required classes are included, is that enough ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a is_expected.to compile.with_all_deps
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gotcha, done
|
||
it 'Will not manage collectd-apache' do | ||
is_expected.not_to contain_package('collectd-apache').with(ensure: 'present', | ||
name: 'collectd-apache') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Hi @sileht, thanks for the PR. Can you take a look at the inline comments and the failing travis job? |
cf1287a
to
01f9211
Compare
Any update ? |
This change adds test coverage for the apache plugins. And two missing options SSLCiphers and Timeout.
01f9211
to
8c189e0
Compare
apache: sync options and add tests
This change adds test coverage for the apache plugins.
And two missing options SSLCiphers and Timeout.