Skip to content

Commit e7aeaf7

Browse files
authored
Add note about request specs and Rack Test to ADR
1 parent dc373d7 commit e7aeaf7

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

Diff for: decisions/0003-switching-to-request-specs-for-controllers.md

+5
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ Additionally, writing controller specs is a discouraged pattern:
2323
2424
From [RSpec — Controller or Request Specs?][]
2525

26+
**Warning**: Request specs in the `cloud_controller_ng` codebase are not actually Rspec/Rails request specs (which would be [declared with `type: :request`](https://relishapp.com/rspec/rspec-rails/docs/request-specs/request-spec)), but use [Rack Test](https://github.com/rack-test/rack-test). Rack Test has similar looking helper methods for making requests (e.g. `get`, `post`, etc), but these take different arguments than the equivalent Rspec methods.
27+
2628
Decision
2729
--------
2830

@@ -37,3 +39,6 @@ Consequences
3739

3840

3941
[RSpec — Controller or Request Specs?]: https://medium.com/just-tech/rspec-controller-or-request-specs-d93ef563ef11
42+
43+
44+

0 commit comments

Comments
 (0)