maint: make rspec exit with status 1 when there are failures

Hudson wasn't notifying us when there was a test failure because
fail_on_error was set to false. This appears to be because of a
misconception that setting this to true caused the specs to run slower.
That is not the case.

Paired-with: Nick Lewis