Permalink
Browse files

Ensure that url_helpers included after application's ones have higher…

… priority
  • Loading branch information...
1 parent 6c95e0f commit 1e6612ef80b019e602231e6833c934817cccb858 @drogus drogus committed Jul 19, 2010
Showing with 15 additions and 0 deletions.
  1. +15 −0 actionpack/test/dispatch/prefix_generation_test.rb
@@ -14,6 +14,7 @@ def self.routes
match "/posts", :to => "inside_engine_generating#index", :as => :posts
match "/url_to_application", :to => "inside_engine_generating#url_to_application"
match "/polymorphic_path_for_engine", :to => "inside_engine_generating#polymorphic_path_for_engine"
+ match "/conflicting_url", :to => "inside_engine_generating#conflicting"
end
routes
@@ -37,6 +38,7 @@ def self.routes
match "/generate", :to => "outside_engine_generating#index"
match "/polymorphic_path_for_engine", :to => "outside_engine_generating#polymorphic_path_for_engine"
match "/polymorphic_with_url_for", :to => "outside_engine_generating#polymorphic_with_url_for"
+ match "/conflicting_url", :to => "outside_engine_generating#conflicting"
root :to => "outside_engine_generating#index"
end
@@ -90,6 +92,10 @@ def url_to_application
def polymorphic_path_for_engine
render :text => polymorphic_path(Post.new)
end
+
+ def conflicting
+ render :text => "engine"
+ end
end
class ::OutsideEngineGeneratingController < ActionController::Base
@@ -106,6 +112,10 @@ def polymorphic_path_for_engine
def polymorphic_with_url_for
render :text => blog_engine.url_for(Post.new)
end
+
+ def conflicting
+ render :text => "application"
+ end
end
class EngineObject
@@ -162,6 +172,11 @@ def setup
assert_equal "/pure-awesomeness/blog/posts/1", last_response.body
end
+ test "[ENGINE] url_helpers from engine have higher priotity than application's url_helpers" do
+ get "/awesome/blog/conflicting_url"
+ assert_equal "engine", last_response.body
+ end
+
# Inside Application
test "[APP] generating engine's route includes prefix" do
get "/generate"

0 comments on commit 1e6612e

Please sign in to comment.