acceptance tests in rough state

Do not merge 
No, zero isn't the correct exit code.  Ideally 0 would be the exit code after a second run (but the exit code still wouldn't be zero for the first run since there are changes), but puppet currently has deprecation warnings that will make the exit code zero.