carbonfive/raygun — pull request #36 by jonah-carbonfive (2012-11-01 01:02:15+00:00)
Title: Use the Carbon Five branch of rspec-rails.

One way we might resolve issue #4.

Forked rspec-rails to define unit/integration/acceptance specs. Unfortunately this is a breaking change to rspec-rails since `spec/integration` was a synonym for `spec/requests` so I couldn't (and did not try to) maintain backwards compatibility with the upstream spec structure. That's going to make it difficult to get this sort of change adopted upsteam.

Most tests have just been moved from `spec/` into `spec/integration`. I'd like to see them pulled out into `spec/unit` where possible.

TODO:
- Enforce unit test constraints (no db access). One option is to raise in a `before :suite` in `unit_spec_helper.rb` if `Rails` is a defined constant. Hopefully that wouldn't be too restrictive to allow request, view, and maybe helper specs to default to unit specs.
- Make sure integration and acceptance specs are correctly wrapped in transactions to rollback any changes.
- I haven't tested acceptance specs enough to be confident that they will be scaffolded correctly.