fixes issue #11 by prioritizing #attributes over #to_hash

lib/active_remote/bulk.rb

@@ -104,7 +104,12 @@ def destroy_all(*records)
       #
       def parse_records(*records)
         records.flatten!
-        records.collect!(&:to_hash)
+
+        if records.first.respond_to?(:attributes)
+          records.collect!(&:attributes)
+        else
+          records.collect!(&:to_hash)
+        end
 
         return records.first if records.first.has_key?(:records)
 

spec/lib/active_remote/bulk_spec.rb

@@ -42,12 +42,24 @@
 
   describe ".parse_records" do
     let(:records) { [ Hash.new ] }
+    let(:attribute_record) {
+      record = double(Hash)
+      record.stub(:attributes) { {} }
+      record
+    }
+    let(:records) { [ Hash.new ] }
 
     it "preps records to be built into a bulk request" do
       parsed_records = { :records => records }
       Tag.parse_records(records).should eq parsed_records
     end
 
+    it "preps records to be built into a bulk request (prioritizing :attributes over :to_hash)" do
+      attribute_record.should_receive(:attributes)
+      parsed_records = { :records => [ {} ] }
+      Tag.parse_records([ attribute_record ]).should eq parsed_records
+    end
+
     context "when given a bulk message" do
       let(:records) { [ tag.to_hash ] }
       let(:tag) { Generic::Remote::Tag.new }
@@ -71,4 +83,4 @@
       Tag.update_all(records)
     end
   end
-end
+end