spec/lib/aba/transaction_spec.rb in aba-0.5.0 vs spec/lib/aba/transaction_spec.rb in aba-1.0.0

@@ -1,27 +1,40 @@
 # encoding: UTF-8
 
 require "spec_helper"
 
 describe Aba::Transaction do
-  let(:transaction_params) { {
-    :account_number => 23432342,
-    :transaction_code => 53,
-    :amount => 50050,
-    :account_name => "John Doe",
-    :bsb => "345-453",
-    :witholding_amount => 87,
-    :indicator => "W",
-    :lodgement_reference => "R45343",
-    :trace_bsb => "123-234",
-    :trace_account_number => "4647642",
-    :name_of_remitter => "Remitter"
-  } }
   subject(:transaction) { Aba::Transaction.new(transaction_params) }
 
+  let(:transaction_attributes) { {amount: 50050, transaction_code: 53} }
+
+  let(:transaction_params) do
+    {
+      :account_number => 23432342,
+      :transaction_code => transaction_attributes[:transaction_code],
+      :amount => transaction_attributes[:amount],
+      :account_name => "John Doe",
+      :bsb => "345-453",
+      :witholding_amount => 87,
+      :indicator => "W",
+      :lodgement_reference => "R45343",
+      :trace_bsb => "123-234",
+      :trace_account_number => "4647642",
+      :name_of_remitter => "Remitter",
+    }
+  end
+
   describe "#to_s" do
     it "should create a transaction row" do
       expect(subject.to_s).to include("1345-453 23432342W530000050050John Doe                        R45343            123-234  4647642Remitter        00000087")
+    end
+
+    context 'when supplied amount is negative' do
+      let(:transaction_attributes) { {amount: -50050, transaction_code: 53} }
+
+      it "should create a transaction row where the amount does not have a sign" do
+        expect(subject.to_s).to include("1345-453 23432342W530000050050John Doe                        R45343            123-234  4647642Remitter        00000087")
+      end
     end
   end
 
   describe "#valid?" do
     it "should be valid" do