From 87297f1c4be22c10de138709ba2d6c3ace4b0117 Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@loudthinking.com>
Date: Thu, 16 Dec 2004 13:42:21 +0000
Subject: Added assumption that a Symbol-based scope should end in _id unless
 it does so already and that you can pass vanilla string-based scopes as a
 parameter

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@183 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
---
 activerecord/test/fixtures/mixin.rb | 10 +++++++++-
 activerecord/test/mixin_test.rb     | 11 ++++++++---
 2 files changed, 17 insertions(+), 4 deletions(-)

(limited to 'activerecord/test')

diff --git a/activerecord/test/fixtures/mixin.rb b/activerecord/test/fixtures/mixin.rb
index 6da85e55e7..0c3bf5000a 100644
--- a/activerecord/test/fixtures/mixin.rb
+++ b/activerecord/test/fixtures/mixin.rb
@@ -9,5 +9,13 @@ class ListMixin < ActiveRecord::Base
   def self.table_name
     "mixins"
   end
-    
+end
+
+
+class ListWithStringScopeMixin < ActiveRecord::Base
+  acts_as_list :column => "pos", :scope => 'parent_id = #{parent_id}'
+
+  def self.table_name
+    "mixins"
+  end
 end
\ No newline at end of file
diff --git a/activerecord/test/mixin_test.rb b/activerecord/test/mixin_test.rb
index 107d6d2f0a..58e7ff1807 100644
--- a/activerecord/test/mixin_test.rb
+++ b/activerecord/test/mixin_test.rb
@@ -28,7 +28,7 @@ class ListTest < Test::Unit::TestCase
     
   end
 
-  def test_insert          
+  def test_insert
     new = ListMixin.create("parent_id"=>5)
     assert_equal 1, new.pos
     assert new.first?
@@ -48,8 +48,7 @@ class ListTest < Test::Unit::TestCase
     assert_equal 1, new.pos
     assert new.first?
     assert new.last?
-  end  
-  
+  end    
   
   def test_delete_middle
     first = ListMixin.create("parent_id"=>5)
@@ -68,6 +67,12 @@ class ListTest < Test::Unit::TestCase
     
   end  
 
+  def test_with_string_based_scope
+    new = ListWithStringScopeMixin.create("parent_id"=>5)
+    assert_equal 1, new.pos
+    assert new.first?
+    assert new.last?
+  end 
 end
 
 class TreeTest < Test::Unit::TestCase
-- 
cgit v1.2.3