This patch removes deprecated catch-all routes from AM

It also removes a monkey patch from AM::Base
上级 0a829f7d
......@@ -8,11 +8,16 @@ class WelcomeController < ActionController::Base
AppRoutes = ActionDispatch::Routing::RouteSet.new
class ActionMailer::Base
include AppRoutes.url_helpers
AppRoutes.draw do
get "/welcome" => "foo#bar", as: "welcome"
get "/dummy_model" => "foo#baz", as: "dummy_model"
get "/welcome/greeting", to: "welcome#greeting"
get "/a/b(/:id)", to: "a#b"
end
class UrlTestMailer < ActionMailer::Base
include AppRoutes.url_helpers
default_url_options[:host] = "www.basecamphq.com"
configure do |c|
......@@ -80,14 +85,6 @@ def setup
def test_url_for
UrlTestMailer.delivery_method = :test
AppRoutes.draw do
ActiveSupport::Deprecation.silence do
get ":controller(/:action(/:id))"
get "/welcome" => "foo#bar", as: "welcome"
get "/dummy_model" => "foo#baz", as: "dummy_model"
end
end
# string
assert_url_for "http://foo/", "http://foo/"
......@@ -111,13 +108,6 @@ def test_url_for
def test_signed_up_with_url
UrlTestMailer.delivery_method = :test
AppRoutes.draw do
ActiveSupport::Deprecation.silence do
get ":controller(/:action(/:id))"
get "/welcome" => "foo#bar", as: "welcome"
end
end
expected = new_mail
expected.to = @recipient
expected.subject = "[Signed up] Welcome #{@recipient}"
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册