It's unfortunate that the existing up2date provider doesn't already have unit tests. They would live in spec/unit/provider/package/up2date.rb . There are already a few files in that directory for some other providers. We do have a little bit of documentation about our test standards at http://reductivelabs.com/trac/puppet/wiki/Development/WritingTests A new patch is only expected to contain tests to exercise the code that it changes.
~Jesse On Thu, Feb 25, 2010 at 12:54 PM, James Cammarata <[email protected]> wrote: > > On Thu, 25 Feb 2010 12:26:33 -0800, Jesse Wolfe <[email protected]> > wrote: > > I'd like to see this patch included in the next testing/release cycle, > and > > the code looks okay, but it's going to need unit tests for the new > > functionality. Would you be interested in writing those? > > > > ~Jesse Wolfe > > Sure, if you can aim me at any guidelines, examples, etc. This is my first > patch submitted for puppet so I'm not exactly 100% up on your > practices/procedures. > > > -- > This message has been scanned for viruses and > dangerous content by MailScanner, and is > believed to be clean. > > -- > 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]<puppet-dev%[email protected]> > . > For more options, visit this group at > http://groups.google.com/group/puppet-dev?hl=en. > > -- 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.
