Skip to content

Testcase for GH #5114. #5119

Merged
merged 1 commit into from Feb 22, 2012

3 participants

@kennyj
kennyj commented Feb 21, 2012

I think that the original issue is journey's one.
This PR is only rails side testcase and corresponds to rails/journey#19.

I sent this PR to clarify specification as rails,although it seems that this issue is journey side issue.

Please see #5114.

@tenderlove tenderlove merged commit ef5670e into rails:master Feb 22, 2012
@exviva exviva commented on the diff Feb 22, 2012
actionpack/test/dispatch/routing_test.rb
@@ -2402,3 +2402,29 @@ def app; Routes end
end
+class TestTildeAndMinusPaths < ActionDispatch::IntegrationTest
+ Routes = ActionDispatch::Routing::RouteSet.new.tap do |app|
+ app.draw do
+ match "/~user" => lambda { |env|
+ [200, { 'Content-Type' => 'text/plain' }, []]
+ }, :as => :tilde_path
+ match "/young-and-fine" => lambda { |env|
+ [200, { 'Content-Type' => 'text/plain' }, []]
+ }, :as => :tilde_path
@exviva
exviva added a note Feb 22, 2012

Didn't you mean :as => :minus_path? Actually, you don't need the :as option at all for that test.

@kennyj
kennyj added a note Feb 25, 2012

Thanks @exviva ! I'll fix it and send a PR ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.