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.

Small named_scope_test.rb fix

#190

In the test that checks if named_scope is accessible through a has_many :through association was broken.

There is a typo in it and the appropriate fixtures aren't being loaded.

This patch fix it.

Reported by Rodrigo Kochenburger · May 14th, 2008 @ 02:31 AM

State: resolved
Milestone: 2.1.1
Assigned to: Rick Rick
Importance: none

Activity

  1. Rick
    Rick
    • Milestone set to 2.1.1
    • State changed from new to open
    • Assigned user set to Rick

    May 14th, 2008 @ 02:32 AM

  2. josh
    josh

    It seems like an obvious typo, but why are the tests passing without the fix?

    May 14th, 2008 @ 06:39 PM

  3. Rodrigo Kochenburger
    Rodrigo Kochenburger

    Hm? The tests failed for me. That is the reason why i created the patch in the first place.

    Can anybody else confirm this?

    May 14th, 2008 @ 07:32 PM

  4. josh
    josh

    Maybe, I should of clarified what db. I tested on sqlite 3.4 and mysql 5.0.51a.

    May 14th, 2008 @ 09:03 PM

  5. Rick
    Rick

    How'd it fail? It seems like an array of posts should never equal a scoped array of comments.

    May 16th, 2008 @ 01:54 AM

  6. Repository
    Repository
    • State changed from open to resolved

    (from [5a3fd46d42f05a241b3178af60d38fe5e0cebee6]) Fixing test for named_scope through a has_many :through

    The test that checks if a has_many :through association has access to named_scope had a typo and

    didn't have the proper fixtures loaded. [#190 state:resolved]

    http://github.com/rails/rails/co...

    May 16th, 2008 @ 01:57 AM

  7. Repository
    Repository
    • Tag set to activerecord, named_scope, patch, tests
    • Importance changed from to

    (from [b5efa8fc670a66b2251b74ffdf6224bee6aa2e84]) added tip about ruby-debug needing a different gem if using 1.9 [#190 state:resolved] http://github.com/rails/rails/commit/b5efa8fc670a66b2251b74ffdf6224...

    October 30th, 2010 @ 10:31 PM