Adding legacy factories. Another high level test for string primary k…

…ey quoting for delegated group association.

test/factories.rb

@@ -16,7 +16,24 @@
 Factory.define :employee_with_reports, :class => 'Employee' do |e|
   e.name      { "Factory Employee ##{Factory.next(:id)}" }
   e.email     { Factory.next(:email) }
-  e.reports { |employee| [employee.association(:report),employee.association(:report)] }
+  e.reports   { |employee| [employee.association(:report),employee.association(:report)] }
+end
+
+
+Factory.define :legacy_employee do |e|
+  e.name      { "Legacy Factory Employee ##{Factory.next(:id)}" }
+  e.email     { Factory.next(:email) }
+end
+
+Factory.define :legacy_report do |r|
+  r.title     { Factory.next(:title) }
+  r.body      'Legacy bla bla. Legacy. Legacy bla.'
+end
+
+Factory.define :legacy_employee_with_reports, :class => 'LegacyEmployee' do |e|
+  e.name      { "Legacy Factory Employee ##{Factory.next(:id)}" }
+  e.email     { Factory.next(:email) }
+  e.reports   { |employee| [employee.association(:legacy_report),employee.association(:legacy_report)] }
 end
 
 

test/grouped_scope/has_many_association_test.rb

@@ -10,10 +10,8 @@ def setup
 
     context 'for an Employee' do
 
-      setup { @employee = Factory(:employee) }
-
-      should 'simply work' do
-        assert_instance_of Array, @employee.reports
+      setup do 
+        @employee = Factory(:employee)
       end
 
       should 'scope existing association to owner' do
@@ -30,6 +28,27 @@ def setup
 
     end
 
+    context 'for a LegacyEmployee' do
+
+      setup do
+        @employee = Factory(:legacy_employee)
+      end
+
+      should 'scope existing association to owner' do
+        assert_sql(/"legacy_reports".email = '#{@employee.id}'/) do
+          @employee.reports(true)
+        end
+      end
+
+      should 'scope group association to group' do
+        assert_sql(/"legacy_reports".email IN \('#{@employee.id}'\)/) do
+          @employee.group.reports(true)
+        end
+      end
+
+    end
+
+
   end