Skip to content

Commit

Permalink
third parameter for rack must respond to each
Browse files Browse the repository at this point in the history
  • Loading branch information
tenderlove committed Sep 22, 2010
1 parent 0a515dc commit 92cb0b8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions railties/test/application/routing_test.rb
Expand Up @@ -177,7 +177,7 @@ def index
assert_equal 'admin::foo', last_response.body
end

test "reloads appended route blocks" do
def test_reloads_appended_route_blocks
app_file 'config/routes.rb', <<-RUBY
AppTemplate::Application.routes.draw do
match ':controller#:action'
Expand All @@ -186,7 +186,7 @@ def index

add_to_config <<-R
routes.append do
match '/win' => lambda { |e| [200, {'Content-Type'=>'text/plain'}, 'WIN'] }
match '/win' => lambda { |e| [200, {'Content-Type'=>'text/plain'}, ['WIN']] }
end
R

Expand Down

4 comments on commit 92cb0b8

@exviva
Copy link
Contributor

@exviva exviva commented on 92cb0b8 Sep 23, 2010

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but String responds to each. Isn't that good enough?

@dmathieu
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does responds to each in 1.8. But not in 1.9.
So no it's not enough. Thanks Aaron.

@krekoten
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not in ruby 1.9

@exviva
Copy link
Contributor

@exviva exviva commented on 92cb0b8 Sep 23, 2010

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, guys!

Please sign in to comment.