From 314cbea9a30679f3ebfcca0cd17756ca8cad81ea Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Fri, 9 May 2014 12:05:24 -0700 Subject: [PATCH] just call the method and assert the return value do not test internals --- actionpack/test/controller/routing_test.rb | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/actionpack/test/controller/routing_test.rb b/actionpack/test/controller/routing_test.rb index df453a0251..b22bc2dc25 100644 --- a/actionpack/test/controller/routing_test.rb +++ b/actionpack/test/controller/routing_test.rb @@ -419,14 +419,7 @@ def test_optimised_named_route_with_host get 'page' => 'content#show_page', :as => 'pages', :host => 'foo.com' end routes = setup_for_named_route - routes.expects(:url_for).with({ - :host => 'foo.com', - :only_path => false, - :controller => 'content', - :action => 'show_page', - :use_route => 'pages' - }).once - routes.send(:pages_url) + assert_equal "http://foo.com/page", routes.pages_url end def setup_for_named_route(options = {}) -- GitLab