Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

remove MockResponse #518

Merged
merged 1 commit into from

3 participants

@nashby
Collaborator

do we really need to mock response? :suspect:

@rafaelfranca

Seems not. But I don't know. @carlosantoniodasilva?

@carlosantoniodasilva
Collaborator

I think that was required on earlier Rails versions, it seems it's not anymore :D

Thanks @nashby :)

@carlosantoniodasilva carlosantoniodasilva merged commit 51f680a into plataformatec:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 16, 2012
  1. @nashby

    remove MockResponse

    nashby authored
This page is out of date. Refresh to see the latest.
Showing with 0 additions and 19 deletions.
  1. +0 −14 test/support/mock_response.rb
  2. +0 −5 test/test_helper.rb
View
14 test/support/mock_response.rb
@@ -1,14 +0,0 @@
-class MockResponse
-
- def initialize(test_case)
- @test_case = test_case
- end
-
- def content_type
- 'text/html'
- end
-
- def body
- @test_case.send :output_buffer
- end
-end
View
5 test/test_helper.rb
@@ -39,17 +39,12 @@ class ActionView::TestCase
include SimpleForm::ActionViewExtensions::FormHelper
setup :set_controller
- setup :set_response
setup :setup_new_user
def set_controller
@controller = MockController.new
end
- def set_response
- @response = MockResponse.new(self)
- end
-
def setup_new_user(options={})
@user = User.new({
:id => 1,
Something went wrong with that request. Please try again.