This project is archived and is in readonly mode.
Minor refactorings and order-dependent test failure fix
Reported by Adam Milligan | December 27th, 2008 @ 08:24 AM | in 2.x
The has_many_through_associations_test.rb file has a test that fails only when running this file individually, rather than as part of the ActiveRecord test suite. This patch adds the necessary require statements to fix this. Also, one or two minor refactorings: removed a few lines of dead code, fixed some spelling and grammar errors in comments.
Comments and changes to this ticket
-
Repository March 7th, 2009 @ 12:37 PM
- State changed from “new” to “resolved”
(from [f36513653fceb01691288dfdebfdc56d17dc85c9]) Fix RDoc grammar and ensure hm:t tests can run in isolation. [#1644 state:resolved]
Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...
-
Pratik March 7th, 2009 @ 12:38 PM
- State changed from “resolved” to “new”
I didn't commit the refactoring as it looks like they might be being used by plugins.
Thanks.
-
Pratik March 7th, 2009 @ 12:38 PM
- State changed from “new” to “resolved”
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>
People watching this ticket
Adam Milligan
David Stevenson
Pratik
Attachments
Referenced by
-
1644
Minor refactorings and order-dependent test failure fix
(from [f36513653fceb01691288dfdebfdc56d17dc85c9]) Fix RDo...