We should limit the number of methods we add onto objects we don't own.
- No need to stub the private formatted_location method. - More detailed message. - Changelog entry.
...even if an error occurs in before(:all). Fixes #558.
* args was being double wrapped in an Array after being splatted twice
Signed-off-by: Sam Phippen <firstname.lastname@example.org>
I think that it's cleaner to have this glob of functionality in it's own method rather than starting a new block halfway through the function. Signed-off-by: Sam Phippen <email@example.com>
Before we were creating instance variables both before and after yielding to the user. Now we set all defaults before we yield to the user and then it's up to them to change any of the values. Signed-off-by: Sam Phippen <firstname.lastname@example.org>
Did leave keyword params in for methods that took 5 or more params.
Ordering dependencies between files can result in a raised exception when rspec is loading files. If random ordering is turned on, then the exception will prevent the random seed from being displayed. This always outputs the random seed before file are loaded, so that the ordering that produced the exception can be repeated.
The old way "leaked" in the sense that it globally added an empty module that only a few tests need.
This fixes a bug where color is config'd but a formatter is given a file. - Fixes #675.