puppetlabs/puppet — pull request #973 by djmitche (2012-07-26 19:23:33+00:00)
Title: Better error messages for failed parameter validation

This is a pretty basic change to go from

err: Failed to apply catalog: Parameter group failed: Invalid group name '""'

to

err: Failed to apply catalog: Parameter group for file /etc/foo failed: Invalid group name '""'

This doesn't have tests yet, because I still haven't figured out how to get them running on my laptop. But it's also most likely the wrong way to specify a resource. I'd rather have it say "..for File['/etc/foo']..", but I'm not sure how best to generate that string. So, advice appreciated (ping me in IRC if you have a few minutes to help with running the tests), and then I'll revise this pull req.