Merge pull request spree#47 from bonobos/send_retro_refunds

add order_id to refund serializer

app/serializers/spree/wombat/refund_serializer.rb

@@ -3,12 +3,24 @@
 module Spree
   module Wombat
     class RefundSerializer < ActiveModel::Serializer
-      attributes :reason, :amount, :description
+      attributes :reason, :amount, :description, :order_id, :store, :code
       has_one :payment, serializer: Spree::Wombat::PaymentSerializer
 
       def reason
         object.reason.try(:name)
       end
+
+      def code
+        object.reason.try(:code)
+      end
+
+      def order_id
+        object.payment.order.try(:number)
+      end
+
+      def store
+        { code: object.payment.order.store.code }
+      end
     end
   end
 end

spec/serializers/spree/hub/refund_serializer_spec.rb

@@ -5,15 +5,28 @@ module Wombat
     describe RefundSerializer do
       let(:refund) { build(:refund) }
       let(:serialized_refund) { JSON.parse(RefundSerializer.new(refund, root: false).to_json, symbolize_names: true) }
+      before { refund.reason.update_attributes!(code: '12345') }
 
       context "format" do
         it "sets the reason" do
           expect(serialized_refund[:reason]).to match /Refund/
         end
 
+        it 'sets the code' do
+          expect(serialized_refund[:code]).to eq '12345'
+        end
+
         it "includes the payment" do
           expect(serialized_refund[:payment]).to be_present
         end
+
+        it "includes the order_id" do
+          expect(serialized_refund[:order_id]).to eq refund.payment.order.number
+        end
+
+        it "includes the store" do
+          expect(serialized_refund[:store][:code]).to eq refund.payment.order.store.code
+        end
       end
     end
   end