This project is archived and is in readonly mode.
test/railties/engine_test.rb:91 fails
Reported by codemanki | October 8th, 2010 @ 11:19 AM
Test railties/test/railties/engine_test.rb fails:
Failure
test_engine_is_a_rack_app_and_can_have_his_own_middleware_stack'
test_engine_is_a_rack_app_and_can_have_his_own_middleware_stack(RailtiesTest::EngineTest)
[test/railties/engine_test.rb:91:in
/Users/codemanki/projects/misc/rails/activesupport/lib/active_support/testing/isolation.rb:65:in `run'
/Users/codemanki/projects/misc/rails/activesupport/lib/active_support/testing/isolation.rb:100:in `run_in_isolation'
/Users/codemanki/projects/misc/rails/activesupport/lib/active_support/testing/isolation.rb:97:in `fork'
/Users/codemanki/projects/misc/rails/activesupport/lib/active_support/testing/isolation.rb:97:in `run_in_isolation'
/Users/codemanki/projects/misc/rails/activesupport/lib/active_support/testing/isolation.rb:63:in `run']:
<["HELLO WORLD"]> expected but was
<["< html xmlns="http://www.w3.org/1999/xhtml">\n\n
.... "]>
Problem
line 91: assert_equal ["HELLO WORLD"], response[2]
response[2] itself is an array.
Solution
Line 65
response[2][0].upcase!
Instead of
response[2].upcase!
Comments and changes to this ticket
-
David Trasbo October 8th, 2010 @ 11:24 AM
- State changed from new to open
- Importance changed from to Low
+1
Patch applies cleanly to
master
and fixes the failing test. -
Rohit Arondekar October 8th, 2010 @ 11:50 AM
+1 nice catch, this is only on master and fixes the failing test.
-
Santiago Pastorino October 8th, 2010 @ 06:32 PM
- State changed from open to resolved
Sorry i was going to apply it and was already fixed here e68c87bb
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>