Skip to content

Conversation

ptomulik
Copy link
Contributor

Just caught it.

@@ -13,7 +13,7 @@
end

it "should raise a ParseError if there are greater than 2 arguments" do
lambda { scope.function_delete([[], 'foo', 'bar']) }.should( raise_error(Puppet::ParseError))
lambda { scope.function_delete_values([[], 'foo', 'bar']) }.should( raise_error(Puppet::ParseError))
Copy link
Contributor

Choose a reason for hiding this comment

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

Errors like this can creep in if there's no expectation of the errror message itself. I'll merge this in, but future tests should add expectations on both that an error was raised, and that the right error message was raised as well.

adrienthebo added a commit that referenced this pull request Aug 28, 2013
small fix to delete_values_spec.rb and README.markdown
@adrienthebo adrienthebo merged commit 6dd6d06 into puppetlabs:master Aug 28, 2013
@adrienthebo
Copy link
Contributor

summary: merged into master in 6dd6d06; thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants