Tests for HasMany#find

lib/conductor/associations/has_many.rb

@@ -71,17 +71,6 @@ def deleted_records
     original_records.reject { |record| records.include?(record) }
   end
 
-  # Called from Conductor::Base#save! This is necessary because when the base_record is a new record
-  # the foreign key won't be automatically assigned in update_item.
-  def set_foreign_keys
-    records.each do |record|
-      if record.send(primary_key_name).nil?
-        record.send("#{primary_key_name}=", base_record.id)
-      end
-    end
-  end
-
-  # TODO: Individual test
   def find(id)
     records.find { |record| record.id == id }
   end
@@ -96,6 +85,16 @@ def has_key_requirement?
     !required_key.nil?
   end
 
+  # Called from Conductor::Base#save! This is necessary because when the base_record is a new record
+  # the foreign key won't be automatically assigned in update_item.
+  def set_foreign_keys
+    records.each do |record|
+      if record.send(primary_key_name).nil?
+        record.send("#{primary_key_name}=", base_record.id)
+      end
+    end
+  end
+
   private
 
     def build_record(parameters)

spec/associations/has_many_spec.rb

@@ -129,4 +129,19 @@ module Conductor::Associations
       end
     end
   end
+
+  describe HasMany, "with some records" do
+    before do
+      @association = Conductor::Associations::HasMany.new(stub_everything, :foo)
+      @association.stubs(:records).returns([stub(:id => 4), stub(:id => 9), stub(:id => 1)])
+    end
+
+    it "should find a record when given an id that exists" do
+      @association.find(1).should == @association.records[2]
+    end
+
+    it "should not find a record when given an id that does not exist" do
+      @association.find(3).should == nil
+    end
+  end
 end