I think it belongs in puppet/blah/TestHelper.rb. This way the individual version of puppet that has the dependency is specifying it and initializing it... puppetlabs_spec_helper doesn't need to know anything about it. It's only job is to say "Hey, puppet! Do whatever it is that you need to do to initialize yourself for testing!"
On Mon, May 14, 2012 at 2:44 PM, Jeff McCune <j...@puppetlabs.com> wrote: > On Mon, May 14, 2012 at 9:41 AM, Daniel Pittman <dan...@puppetlabs.com>wrote: > >> > My question here is - where should this be fixed now? >> >> This should be fixed in your code: Hiera is a mandatory component with >> the release of Telly, and disabling it for your tests is absolutely >> wrong. >> > > It seems like the puppetlabs_spec_helper should take care of this. Isn't > the spec helper the "right" place for this to be fixed, today? > > >> This is required to deliver the data/model separation, and will need >> to be installed everywhere that Telly is - for the master, and for >> apply. > > > Right, since it's required everywhere that Telly is required, it seems > like it should not live in every piece of code that requires telly but > instead the intermediate helper code. > > -Jeff > > -- > You received this message because you are subscribed to the Google Groups > "Puppet Developers" group. > To post to this group, send email to puppet-dev@googlegroups.com. > To unsubscribe from this group, send email to > puppet-dev+unsubscr...@googlegroups.com. > 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 puppet-dev@googlegroups.com. To unsubscribe from this group, send email to puppet-dev+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
