Skip to content

Commit 4d4d764

Browse files
committed
Rack implements redirect? so we don't need it
Rack [already implements `redirect?` on the response object](https://github.com/rack/rack/blob/1569a985e17d9caaf94d0e97d95ef642c4ab14ba/lib/rack/response.rb#L141) so we don't need to implement our own.
1 parent 9dc06bc commit 4d4d764

File tree

2 files changed

+0
-6
lines changed

2 files changed

+0
-6
lines changed

actionpack/lib/action_controller/test_case.rb

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -139,9 +139,6 @@ class LiveTestResponse < Live::Response
139139
# Was the URL not found?
140140
alias_method :missing?, :not_found?
141141

142-
# Were we redirected?
143-
alias_method :redirect?, :redirection?
144-
145142
# Was there a server-side error?
146143
alias_method :error?, :server_error?
147144
end

actionpack/lib/action_dispatch/testing/test_response.rb

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,6 @@ def self.from_response(response)
1616
# Was the URL not found?
1717
alias_method :missing?, :not_found?
1818

19-
# Were we redirected?
20-
alias_method :redirect?, :redirection?
21-
2219
# Was there a server-side error?
2320
alias_method :error?, :server_error?
2421
end

0 commit comments

Comments
 (0)