Skip to content

Convert the rspec-system tests over to beaker. #114

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

Merged
merged 1 commit into from
Dec 10, 2013
Merged

Convert the rspec-system tests over to beaker. #114

merged 1 commit into from
Dec 10, 2013

Conversation

apenney
Copy link
Contributor

@apenney apenney commented Dec 9, 2013

This converts over the spec tests to be beaker tests. It adds tests for
all parameters that were untested previously.

@puppetlabs-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

expect(r.stderr).to eq("")
end
apply_manifest(pp, :catch_failures => true) do |r|
expect(r.exit_code).to be_zero
Copy link
Contributor

Choose a reason for hiding this comment

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

You could add another expect(r.stderr).to eq("") here if you want

This converts over the spec tests to be beaker tests.  It adds tests for
all parameters that were untested previously.
@apenney
Copy link
Contributor Author

apenney commented Dec 10, 2013

Updated with those changes, and the missing ntp_parameters_spec.rb that contains most of the work, whoops.

apenney pushed a commit that referenced this pull request Dec 10, 2013
Convert the rspec-system tests over to beaker.
@apenney apenney merged commit d8449e5 into puppetlabs:master Dec 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants