On Fri, May 13, 2011 at 03:57:33PM -0700, Max Martin wrote: > Paul, > > We really appreciate your work on adding testing for this provider; > unfortunately, we are unable to run the tests you wrote on our development > machines, as they directly call the pacman command, which is not available on > our computers (currently, 15/19 of the tests fail with the message "Command > pacman is missing" or some variant thereof). You will need to stub these > calls rather than calling the actual pacman command. You may want to check > out the specs for other package providers such as pip or apt to see how they > stub out calls to the actual executable to prevent it from being run. If you > need more information or guidance on how to proceed, please feel free to > e-mail the list, or check out our Wiki documentation on writing tests: > > https://projects.puppetlabs.com/projects/puppet/wiki/Development_Writing_Tests > > -- > Max Martin > On Wednesday, May 11, 2011 at 6:35 PM, Paul Boyd wrote:
Thanks for looking at this. I was stubbing the calls, but I didn't stub the method that checks for the pacman command. I'll send another patch momentarily (it runs on my ubuntu machine now, so I think it's fixed). -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
