This project is archived and is in readonly mode.
Small named_scope_test.rb fix
Reported by Rodrigo Kochenburger | May 14th, 2008 @ 02:31 AM | in 2.1.1
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.
Comments and changes to this ticket
-
Rick May 14th, 2008 @ 02:32 AM
- Milestone set to 2.1.1
- State changed from new to open
- Assigned user set to Rick
-
josh May 14th, 2008 @ 06:39 PM
It seems like an obvious typo, but why are the tests passing without the fix?
-
Rodrigo Kochenburger May 14th, 2008 @ 07:32 PM
Hm? The tests failed for me. That is the reason why i created the patch in the first place.
Can anybody else confirm this?
-
josh May 14th, 2008 @ 09:03 PM
Maybe, I should of clarified what db. I tested on sqlite 3.4 and mysql 5.0.51a.
-
Rick May 16th, 2008 @ 01:54 AM
How'd it fail? It seems like an array of posts should never equal a scoped array of comments.
-
Repository May 16th, 2008 @ 01:57 AM
- 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]
-
Repository October 30th, 2010 @ 10:31 PM
- 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...
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
Attachments
Referenced by
- 190 Small named_scope_test.rb fix (from [b5efa8fc670a66b2251b74ffdf6224bee6aa2e84]) added t...