On Mon, Apr 11, 2011 at 20:00, Luke Kanies <[email protected]> wrote:
> On Apr 11, 2011, at 7:40 PM, Daniel Pittman <[email protected]> wrote:
> On Mon, Apr 11, 2011 at 18:57, Nigel Kersten <[email protected]> wrote:

[…]
> You have to recognize that you're talking about a brand spanking new change
> in how tests work, and one that apparently hasn't even been ported to all
> tests.  At the least, 'no longer used' is a better term than 'incorrect'.

I think that is fair, and I will keep that in mind for the future.

> Even worse, the change doesn't even work for all cases.  :/ I agree it's
> better over all, but you can't directly execute specs any more.

As far as I can tell, rspec .../foo_spec.rb still works, and runs just
that spec directly?

> And yes, the reason for this complicated line is that you can copy it
> directly into any file at any depth and it works.  And I actually don't
> think this is the code that caused the spec_helper file to get loaded eery
> time - that was the one that loaded it from parent dirs without recursively
> chdir'ing.

You are right; it would conflict with the plain require of
'spec_helper', but absent that it would have done the right thing, if
it was used uniformly, just like the plain version will when used
uniformly.

Daniel
-- 
⎋ Puppet Labs Developer – http://puppetlabs.com
✉ Daniel Pittman <[email protected]>
✆ Contact me via gtalk, email, or phone: +1 (877) 575-9775
♲ Made with 100 percent post-consumer electrons

-- 
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.

Reply via email to