Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

Minor refactorings and order-dependent test failure fix

#1644

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.

Reported by Adam Milligan · December 27th, 2008 @ 08:24 AM

State: resolved
Milestone: 2.x
Assigned to: nobody
Importance: none

Activity

  1. Repository
    Repository
    • State changed from new to resolved

    (from [f36513653fceb01691288dfdebfdc56d17dc85c9]) Fix RDoc grammar and ensure hm:t tests can run in isolation. [#1644 Minor refactorings and order-dependent test failure fix state:resolved]

    Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...

    March 7th, 2009 @ 12:37 PM

  2. Pratik
    Pratik
    • State changed from resolved to new

    I didn't commit the refactoring as it looks like they might be being used by plugins.

    Thanks.

    March 7th, 2009 @ 12:38 PM

  3. Pratik
    Pratik
    • State changed from new to resolved

    March 7th, 2009 @ 12:38 PM