diff options
| author | wycats <wycats@gmail.com> | 2010-04-03 02:30:06 -0700 |
|---|---|---|
| committer | wycats <wycats@gmail.com> | 2010-04-03 02:44:02 -0700 |
| commit | 3eb97531b8650db5cc7b9558cc3828c56a526b6a (patch) | |
| tree | 70f28e707c552b3be295fff52ab387e901698825 /actionpack/test/template/active_model_helper_test.rb | |
| parent | 13004d4f849682772060371273fda3312dd3b884 (diff) | |
| download | rails-3eb97531b8650db5cc7b9558cc3828c56a526b6a.tar.gz rails-3eb97531b8650db5cc7b9558cc3828c56a526b6a.tar.bz2 rails-3eb97531b8650db5cc7b9558cc3828c56a526b6a.zip | |
Refactored url_for in AV to have its own instances of the helpers instead of proxying back to the controller. This potentially allows for more standalone usage of AV. It also kicked up a lot of dust in the tests, which were mocking out controllers to get this behavior. By moving it to the view, it made a lot of the tests more standalone (a win)
Diffstat (limited to 'actionpack/test/template/active_model_helper_test.rb')
| -rw-r--r-- | actionpack/test/template/active_model_helper_test.rb | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/actionpack/test/template/active_model_helper_test.rb b/actionpack/test/template/active_model_helper_test.rb index 1a5316a689..47eb620f7a 100644 --- a/actionpack/test/template/active_model_helper_test.rb +++ b/actionpack/test/template/active_model_helper_test.rb @@ -118,13 +118,12 @@ class ActiveModelHelperTest < ActionView::TestCase setup_user @response = ActionController::TestResponse.new + end - @controller = Object.new - def @controller.url_for(options) - options = options.symbolize_keys + def url_for(options) + options = options.symbolize_keys - [options[:action], options[:id].to_param].compact.join('/') - end + [options[:action], options[:id].to_param].compact.join('/') end def test_generic_input_tag |