daniel-pittman/puppet — comment on commit 48726b662d2fb5cc3532fca12eb6aa5a3c1622cd
File:  line 0
By mmrobins:

Shouldn't this commit have included some specs? I see that there are no specs for the insync method at all in the property_spec.rb. While there should have been some before this change, not adding any with this change also seems bad.

I'm also curious why you're doing a puts right before the fail. I suppose it can't hurt, but shouldn't the fail be enough?