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
(FM-3924) Create Master/Agent automation test for Puppetlabs-motd #29
(FM-3924) Create Master/Agent automation test for Puppetlabs-motd #29
Conversation
Why is there so many Beaker host files included? I feel like we could use a much smaller subset. |
@@ -0,0 +1,15 @@ | |||
require 'master_manipulator' | |||
test_name 'FM-3804 - C61700 - Install Puppet Enterprise' |
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.
The FM-3804
ticket referenced here is associated with Websphere.
You should add a README for the tests folder with some information about running the tests. Especially given the fact that the tests are only runnable inside Puppet Labs. Here is an example from DSC: https://github.com/puppetlabs/puppetlabs-dsc/tree/master/tests |
|
||
# Init | ||
step 'Install PE' | ||
install_pe |
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.
There is no requirement that PE be used for the master. You could use the FOSS stuff instead.
a9e7217
to
7805160
Compare
7805160
to
2600a9e
Compare
else | ||
gem 'beaker-rspec', :require => false | ||
end | ||
gem 'serverspec', :require => false |
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.
You have duplicate entry for "serverspec". Only specify the dependency one time.
…tlabs-motd_module (FM-3924) Create Master/Agent automation test for Puppetlabs-motd
This PR has one test case of creating /etc/motd file using content parameter, the test case is located at puppetlabs-motd/tests/beaker/tests/01_create_motd_content.rb
The PR also uses Test Organization in Module Repositories as specified in https://confluence.puppetlabs.com/pages/viewpage.action?spaceKey=ENG&title=Test+Tiering+for+the+Modules+Team document written by @cowofevil