rconomic 0.5.2 → 0.6.0

data/spec/economic/invoice_spec.rb

@@ -1,4 +1,4 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::Invoice do
   let(:session) { make_session }
@@ -9,9 +9,9 @@ describe Economic::Invoice do
   end
 
   describe '#remainder' do
-    it 'should get the remainder' do
-      mock_request('Invoice_GetRemainder', {"invoiceHandle" => { "Number" => 512 }}, :success)
-      expect(subject.remainder).to eq("512.32")
+    it "should get the remainder" do
+      mock_request("Invoice_GetRemainder", {"invoiceHandle" => {"Number" => 512}}, :success)
+      expect(subject.remainder).to eq(512.32)
     end
   end
 
@@ -33,13 +33,13 @@ describe Economic::Invoice do
 
   describe "#pdf" do
     it "gets PDF data from API" do
-      mock_request('Invoice_GetPdf', {'invoiceHandle' => {'Number' => 512}}, :success)
+      mock_request("Invoice_GetPdf", {"invoiceHandle" => {"Number" => 512}}, :success)
       subject.pdf
     end
 
     it "decodes the base64Binary encoded data" do
-      stub_request('Invoice_GetPdf', nil, :success)
-      expect(subject.pdf).to eq('This is not really PDF data')
+      stub_request("Invoice_GetPdf", nil, :success)
+      expect(subject.pdf).to eq("This is not really PDF data")
     end
   end
 
@@ -84,4 +84,46 @@ describe Economic::Invoice do
       expect(subject.debtor_handle).to eq(handle)
     end
   end
+
+  describe "#days_past_due" do
+    it "after date has passed" do
+      subject.due_date = (Date.today - 3).to_datetime
+      expect(subject.days_past_due).to eq(3)
+    end
+
+    it "before date has passed" do
+      subject.due_date = (Date.today + 3).to_datetime
+      expect(subject.days_past_due).to eq(0)
+    end
+  end
+
+  describe "#past_due?" do
+    it "when due date has passed and the invoiced hasn't been paid" do
+      subject.due_date = (Date.today - 3).to_datetime
+      subject.instance_variable_set "@remainder", 1
+
+      expect(subject.past_due?).to eq(true)
+    end
+
+    it "when due date has passed and the invoice has been paid" do
+      subject.due_date = (Date.today - 3).to_datetime
+      subject.instance_variable_set "@remainder", 0
+
+      expect(subject.past_due?).to eq(false)
+    end
+
+    it "before due date has passed and the invoiced hasn't been paid" do
+      subject.due_date = (Date.today + 3).to_datetime
+      subject.instance_variable_set "@remainder", 1
+
+      expect(subject.past_due?).to eq(false)
+    end
+
+    it "before due date has passed and the invoice has been paid" do
+      subject.due_date = (Date.today + 3).to_datetime
+      subject.instance_variable_set "@remainder", 0
+
+      expect(subject.past_due?).to eq(false)
+    end
+  end
 end

data/spec/economic/order_spec.rb

@@ -1,4 +1,4 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::Order do
   let(:session) { make_session }
@@ -26,13 +26,13 @@ describe Economic::Order do
 
   describe "#pdf" do
     it "gets PDF data from API" do
-      mock_request('Order_GetPdf', {'orderHandle' => {'Number' => 512}}, :success)
+      mock_request("Order_GetPdf", {"orderHandle" => {"Number" => 512}}, :success)
       subject.pdf
     end
 
     it "decodes the base64Binary encoded data" do
-      stub_request('Order_GetPdf', nil, :success)
-      expect(subject.pdf).to eq('This is not really PDF data')
+      stub_request("Order_GetPdf", nil, :success)
+      expect(subject.pdf).to eq("This is not really PDF data")
     end
   end
 

data/spec/economic/product_spec.rb

@@ -0,0 +1,72 @@
+require "./spec/spec_helper"
+
+describe Economic::Product do
+  let(:session) { make_session }
+  subject { Economic::Product.new(:session => session) }
+
+  it "inherits from Economic::Entity" do
+    expect(Economic::Product.ancestors).to include(Economic::Entity)
+  end
+
+  describe "class methods" do
+    subject { Economic::Product }
+
+    describe ".proxy" do
+      it "should return ProductProxy" do
+        expect(subject.proxy).to eq(Economic::ProductProxy)
+      end
+    end
+
+    describe ".key" do
+      it "should == :product" do
+        expect(Economic::Product.key).to eq(:product)
+      end
+    end
+  end
+
+  describe ".proxy" do
+    it "should return a CreditorProxy" do
+      expect(subject.proxy).to be_instance_of(Economic::ProductProxy)
+    end
+
+    it "should return a proxy owned by session" do
+      expect(subject.proxy.session).to eq(session)
+    end
+  end
+
+  describe "#save" do
+    it "should save it" do
+      stub_request("Product_CreateFromData", nil, :success)
+      subject.save
+    end
+
+    it "builds and sends data to API" do
+      mock_request(
+        :product_create_from_data, {
+          "data" => {
+            "Handle" => {},
+            "Number" => nil,
+            "ProductGroupHandle" => {"Number" => 1},
+            "Name" => nil,
+            "SalesPrice" => nil,
+            "CostPrice" => nil,
+            "RecommendedPrice" => nil,
+            "UnitHandle" => {"Number" => 2},
+            "IsAccessible" => nil,
+            "Volume" => nil,
+            "DepartmentHandle" => {"Number" => 1},
+            "DistributionKeyHandle" => {"Number" => 314}
+          }
+        },
+        :success
+      )
+
+      subject.product_group_handle = Economic::Entity::Handle.new(:number => 1)
+      subject.unit_handle = Economic::Entity::Handle.new(:number => 2)
+      subject.department_handle = Economic::Entity::Handle.new(:number => 1)
+      subject.distribution_key_handle = Economic::Entity::Handle.new(:number => 314)
+
+      subject.save
+    end
+  end
+end

data/spec/economic/proxies/account_proxy_spec.rb

@@ -0,0 +1,74 @@
+require "./spec/spec_helper"
+
+describe Economic::AccountProxy do
+  let(:session) { make_session }
+  subject { Economic::AccountProxy.new(session) }
+
+  describe ".new" do
+    it "stores session" do
+      expect(subject.session).to equal(session)
+    end
+  end
+
+  describe ".build" do
+    it "instantiates a new Account" do
+      expect(subject.build).to be_instance_of(Economic::Account)
+    end
+
+    it "assigns the session to the Account" do
+      expect(subject.build.session).to equal(session)
+    end
+
+    it "should not build a partial Account" do
+      expect(subject.build).to_not be_partial
+    end
+  end
+
+  describe ".find" do
+    it "gets account data from API" do
+      mock_request(
+        "Account_GetData",
+        {"entityHandle" => {"Id" => 42}},
+        :success
+      )
+      subject.find(42)
+    end
+
+    it "returns Account object" do
+      stub_request("Account_GetData", nil, :success)
+      expect(subject.find(42)).to be_instance_of(Economic::Account)
+    end
+  end
+
+  describe ".all" do
+    it "returns an empty array when there are no accounts" do
+      stub_request("Account_GetAll", nil, :none)
+      expect(subject.all.size).to eq(0)
+    end
+
+    it "finds and adds a single account" do
+      stub_request("Account_GetAll", nil, :single)
+      mock_request(
+        "Account_GetData",
+        {"entityHandle" => {"Number" => "1"}},
+        :success
+      )
+
+      accounts = subject.all
+      expect(accounts).to be_instance_of(Economic::AccountProxy)
+
+      expect(accounts.size).to eq(1)
+      expect(accounts.first).to be_instance_of(Economic::Account)
+    end
+
+    it "adds multiple accounts" do
+      stub_request("Account_GetAll", nil, :multiple)
+      stub_request("Account_GetDataArray", nil, :multiple)
+
+      accounts = subject.all
+      expect(accounts.size).to eq(2)
+      expect(accounts.first).to be_instance_of(Economic::Account)
+      expect(accounts.last).to be_instance_of(Economic::Account)
+    end
+  end
+end

data/spec/economic/proxies/actions/debtor_contact/all_spec.rb

@@ -0,0 +1,26 @@
+require "./spec/spec_helper"
+
+describe Economic::Proxies::Actions::DebtorContact::All do
+  let(:session) { make_session }
+  let(:proxy) { Economic::DebtorContactProxy.new(session) }
+
+  subject {
+    Economic::Proxies::Actions::DebtorContact::All.new(proxy)
+  }
+
+  before :each do
+    allow(session).to receive(:number) { 123 }
+  end
+
+  describe "#call" do
+    it "returns debtor contacts" do
+      stub_request("Debtor_GetDebtorContacts", nil, :multiple)
+      expect(subject.call.first).to be_instance_of(Economic::DebtorContact)
+    end
+
+    it "returns empty array when nothing is found" do
+      stub_request("Debtor_GetDebtorContacts", nil, :none)
+      expect(subject.call).to be_empty
+    end
+  end
+end

data/spec/economic/proxies/actions/find_by_name_spec.rb

@@ -1,4 +1,4 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::Proxies::Actions::FindByName do
   let(:session) { make_session }
@@ -10,23 +10,23 @@ describe Economic::Proxies::Actions::FindByName do
 
   describe "#call" do
     it "gets contact data from the API" do
-      mock_request('CreditorContact_FindByName', {'name' => 'Bob'}, :multiple)
+      mock_request("CreditorContact_FindByName", {"name" => "Bob"}, :multiple)
       subject.call
     end
 
     it "returns creditor contacts" do
-      stub_request('CreditorContact_FindByName', nil, :multiple)
+      stub_request("CreditorContact_FindByName", nil, :multiple)
       expect(subject.call.first).to be_instance_of(Economic::CreditorContact)
     end
 
     it "returns empty when nothing is found" do
-      stub_request('CreditorContact_FindByName', nil, :none)
+      stub_request("CreditorContact_FindByName", nil, :none)
       expect(subject.call).to be_empty
     end
 
     context "when calling proxy is owned by session" do
       it "returns all creditor contacts" do
-        stub_request('CreditorContact_FindByName', nil, :multiple)
+        stub_request("CreditorContact_FindByName", nil, :multiple)
         expect(subject.call.size).to eq(2)
       end
     end
@@ -35,7 +35,7 @@ describe Economic::Proxies::Actions::FindByName do
       it "returns only contacts for creditor" do
         # Note the order of these stubs actually matters. They need to match
         # the order they are called in in the implementation
-        stub_request('CreditorContact_FindByName', nil, :multiple)
+        stub_request("CreditorContact_FindByName", nil, :multiple)
         stub_request("CreditorContact_GetData", nil, :success)
         stub_request("Creditor_GetData", nil, :success)
         stub_request("CreditorContact_GetData", nil, :success)
@@ -47,5 +47,4 @@ describe Economic::Proxies::Actions::FindByName do
       end
     end
   end
-
 end

data/spec/economic/proxies/cash_book_entry_proxy_spec.rb

@@ -1,7 +1,6 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::CashBookEntryProxy do
-
   let(:session) { make_session }
   let(:cash_book) { Economic::CashBook.new(:session => session) }
   subject { Economic::CashBookEntryProxy.new(cash_book) }
@@ -28,63 +27,62 @@ describe Economic::CashBookEntryProxy do
   end
 
   describe "#create_finance_voucher" do
-    it 'should create a finance voucher and return the created cash book entry' do
-      stub_request('CashBookEntry_CreateFinanceVoucher', nil, :success)
-      stub_request('CashBookEntry_GetData', {"entityHandle" => {"Id1" => 15, "Id2" => 16}}, :success)
-      cash_book_entry = subject.create_finance_voucher(:account_handle => { :number => 2 }, :contra_account_handle => { :number => 3 })
+    it "should create a finance voucher and return the created cash book entry" do
+      stub_request("CashBookEntry_CreateFinanceVoucher", nil, :success)
+      stub_request("CashBookEntry_GetData", {"entityHandle" => {"Id1" => 15, "Id2" => 16}}, :success)
+      cash_book_entry = subject.create_finance_voucher(:account_handle => {:number => 2}, :contra_account_handle => {:number => 3})
       expect(cash_book_entry).to be_instance_of(Economic::CashBookEntry)
     end
   end
 
   describe "#create_debtor_payment" do
-    it 'should create a debtor payment and then return the created cash book entry' do
-      stub_request('CashBookEntry_CreateDebtorPayment', nil, :success)
-      stub_request('CashBookEntry_GetData', {"entityHandle" => {"Id1" => 13, "Id2" => 14}}, :success)
-      cash_book_entry = subject.create_debtor_payment(:debtor_handle => { :number => 2 }, :contra_account_handle => { :number => 3 })
+    it "should create a debtor payment and then return the created cash book entry" do
+      stub_request("CashBookEntry_CreateDebtorPayment", nil, :success)
+      stub_request("CashBookEntry_GetData", {"entityHandle" => {"Id1" => 13, "Id2" => 14}}, :success)
+      cash_book_entry = subject.create_debtor_payment(:debtor_handle => {:number => 2}, :contra_account_handle => {:number => 3})
       expect(cash_book_entry).to be_instance_of(Economic::CashBookEntry)
     end
   end
 
   describe "#create_creditor_invoice" do
-    it 'should create a creditor invoice and then return the created cash book entry' do
-      stub_request('CashBookEntry_CreateCreditorInvoice', nil, :success)
-      stub_request('CashBookEntry_GetData', {"entityHandle" => {"Id1" => 13, "Id2" => 14}}, :success)
-      cash_book_entry = subject.create_creditor_invoice(:creditor_handle => { :number => 2 }, :contra_account_handle => { :number => 3 })
+    it "should create a creditor invoice and then return the created cash book entry" do
+      stub_request("CashBookEntry_CreateCreditorInvoice", nil, :success)
+      stub_request("CashBookEntry_GetData", {"entityHandle" => {"Id1" => 13, "Id2" => 14}}, :success)
+      cash_book_entry = subject.create_creditor_invoice(:creditor_handle => {:number => 2}, :contra_account_handle => {:number => 3})
       expect(cash_book_entry).to be_instance_of(Economic::CashBookEntry)
     end
 
-    it 'should not send handles that were not given' do
-      stub_request('CashBookEntry_CreateCreditorInvoice', {"cashBookHandle" => { 'Number' => 42 }}, :success)
-      stub_request('CashBookEntry_GetData', {"entityHandle" => {"Id1" => 13, "Id2" => 14}}, :success)
+    it "should not send handles that were not given" do
+      stub_request("CashBookEntry_CreateCreditorInvoice", {"cashBookHandle" => {"Number" => 42}}, :success)
+      stub_request("CashBookEntry_GetData", {"entityHandle" => {"Id1" => 13, "Id2" => 14}}, :success)
       cash_book.number = 42
-      cash_book_entry = subject.create_creditor_invoice({:number => 13})
+      cash_book_entry = subject.create_creditor_invoice(:number => 13)
       expect(cash_book_entry).to be_instance_of(Economic::CashBookEntry)
     end
   end
 
   describe "#create_creditor_payment" do
-    it 'should create a creditor payment and then return the created cash book entry' do
-      stub_request('CashBookEntry_CreateCreditorPayment', nil, :success)
-      stub_request('CashBookEntry_GetData', {"entityHandle" => {"Id1" => 13, "Id2" => 14}}, :success)
-      cash_book_entry = subject.create_creditor_payment(:creditor_handle => { :number => 2 }, :contra_account_handle => { :number => 3 })
+    it "should create a creditor payment and then return the created cash book entry" do
+      stub_request("CashBookEntry_CreateCreditorPayment", nil, :success)
+      stub_request("CashBookEntry_GetData", {"entityHandle" => {"Id1" => 13, "Id2" => 14}}, :success)
+      cash_book_entry = subject.create_creditor_payment(:creditor_handle => {:number => 2}, :contra_account_handle => {:number => 3})
       expect(cash_book_entry).to be_instance_of(Economic::CashBookEntry)
     end
   end
 
   describe "#set_due_date" do
-    it 'should set due date' do
-      mock_request('CashBookEntry_SetDueDate', {"cashBookEntryHandle" => { "Id1" => subject.owner.id, "Id2" => 234 }, :value => Date.new(2012, 12, 21)}, :success)
+    it "should set due date" do
+      mock_request("CashBookEntry_SetDueDate", {"cashBookEntryHandle" => {"Id1" => subject.owner.id, "Id2" => 234}, :value => Date.new(2012, 12, 21)}, :success)
       subject.set_due_date(234, Date.new(2012, 12, 21))
     end
   end
 
   describe "#all" do
-    it 'should get the cash book entries' do
-      stub_request('CashBook_GetEntries', nil, :success)
-      expect(subject).to receive(:find).with({:id1 => '1', :id2 => '2'})
-      expect(subject).to receive(:find).with({:id1 => '11', :id2 => '12'})
+    it "should get the cash book entries" do
+      stub_request("CashBook_GetEntries", nil, :success)
+      expect(subject).to receive(:find).with(:id1 => "1", :id2 => "2")
+      expect(subject).to receive(:find).with(:id1 => "11", :id2 => "12")
       subject.all
     end
   end
-
 end

data/spec/economic/proxies/cash_book_proxy_spec.rb

@@ -1,20 +1,16 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::CashBookProxy do
-
   let(:session) { make_session }
   subject { Economic::CashBookProxy.new(session) }
 
   describe ".new" do
-
     it "stores session" do
       expect(subject.session).to equal(session)
     end
-
   end
 
   describe ".build" do
-
     it "instantiates a new CashBook" do
       expect(subject.build).to be_instance_of(Economic::CashBook)
     end
@@ -22,14 +18,12 @@ describe Economic::CashBookProxy do
     it "assigns the session to the CashBook" do
       expect(subject.build.session).to equal(session)
     end
-
   end
 
   describe ".all" do
-
     it "returns multiple cashbooks" do
-      stub_request('CashBook_GetAll', nil, :multiple)
-      stub_request('CashBook_GetDataArray', nil, :multiple)
+      stub_request("CashBook_GetAll", nil, :multiple)
+      stub_request("CashBook_GetDataArray", nil, :multiple)
 
       all = subject.all
       expect(all.size).to eq(2)
@@ -38,11 +32,11 @@ describe Economic::CashBookProxy do
 
     it "properly fills out handles of cash books" do
       # Issue #12
-      stub_request('CashBook_GetAll', nil, :multiple)
-      stub_request('CashBook_GetDataArray', nil, :multiple)
-      stub_request('CashBook_GetData', nil, :success)
-      stub_request('CashBook_GetAll', nil, :multiple)
-      stub_request('CashBook_GetDataArray', nil, :multiple)
+      stub_request("CashBook_GetAll", nil, :multiple)
+      stub_request("CashBook_GetDataArray", nil, :multiple)
+      stub_request("CashBook_GetData", nil, :success)
+      stub_request("CashBook_GetAll", nil, :multiple)
+      stub_request("CashBook_GetDataArray", nil, :multiple)
 
       cash_book = subject.find(subject.all.first.handle)
       expect(subject.all.first.handle).to eq(cash_book.handle)
@@ -50,21 +44,19 @@ describe Economic::CashBookProxy do
   end
 
   describe ".get_name" do
-
-    it 'returns a cash book with a name' do
-      mock_request('CashBook_GetName', {"cashBookHandle" => { "Number" => "52" }}, :success)
+    it "returns a cash book with a name" do
+      mock_request("CashBook_GetName", {"cashBookHandle" => {"Number" => "52"}}, :success)
       result = subject.get_name("52")
       expect(result).to be_instance_of(Economic::CashBook)
       expect(result.number).to eq("52")
       expect(result.name).to be_a(String)
     end
-
   end
 
   describe "#last" do
     it "returns the last cash book" do
-      stub_request('CashBook_GetAll', nil, :multiple)
-      stub_request('CashBook_GetDataArray', nil, :multiple)
+      stub_request("CashBook_GetAll", nil, :multiple)
+      stub_request("CashBook_GetDataArray", nil, :multiple)
 
       expect(subject.all.last.name).to eq("Another cash book")
     end
@@ -72,8 +64,8 @@ describe Economic::CashBookProxy do
 
   describe "#[]" do
     it "returns the specific cash book" do
-      stub_request('CashBook_GetAll', nil, :multiple)
-      stub_request('CashBook_GetDataArray', nil, :multiple)
+      stub_request("CashBook_GetAll", nil, :multiple)
+      stub_request("CashBook_GetDataArray", nil, :multiple)
 
       expect(subject.all[1].name).to eq("Another cash book")
     end

data/spec/economic/proxies/company_proxy_spec.rb

@@ -0,0 +1,47 @@
+require "./spec/spec_helper"
+
+describe Economic::CompanyProxy do
+  let(:session) { make_session }
+  subject { Economic::CompanyProxy.new(session) }
+
+  describe ".get" do
+    let!(:company) { Economic::Company.new.tap { |e| e.session = session } }
+
+    it "Calls CompanyGet endpoint" do
+      mock_request("Company_Get", nil, :success)
+      stub_request("Company_GetData",  {"entityHandle" => {"Number" => "string"}}, :success)
+      company.get
+    end
+
+    it "Calls CopmanyGetData endpoint with valid data" do
+      stub_request("Company_Get", nil, :success)
+      mock_request("Company_GetData",  {"entityHandle" => {"Number" => "string"}}, :success)
+      company.get
+    end
+
+    it "Returns a valid company" do
+      stub_request("Company_Get", nil, :success)
+      stub_request("Company_GetData",  {"entityHandle" => {"Number" => "string"}}, :success)
+      expectation = {:handle => {:number => "string"},
+                     :number                => "string",
+                     :base_currency_handle  => {:code => "string"},
+                     :name                  => "string",
+                     :address1              => "string",
+                     :address2              => "string",
+                     :postal_code           => "string",
+                     :city                  => "string",
+                     :county                => "string",
+                     :country               => "string",
+                     :telephone_number      => "string",
+                     :fax_number            => "string",
+                     :mobile_number         => "string",
+                     :contact               => "string",
+                     :web_site              => "string",
+                     :email                 => "string",
+                     :ci_number             => "string",
+                     :vat_number            => "string",
+                     :sign_up_date          => "dateTime"}
+      expect(company.get).to eq expectation
+    end
+  end
+end

data/spec/economic/proxies/creditor_contact_proxy_spec.rb

@@ -1,4 +1,4 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::CreditorContactProxy do
   let(:session) { make_session }
@@ -40,13 +40,19 @@ describe Economic::CreditorContactProxy do
 
   describe ".find" do
     it "gets contact data from API" do
-      mock_request('CreditorContact_GetData', {'entityHandle' => {'Id' => 42}}, :success)
+      mock_request(
+        "CreditorContact_GetData",
+        {"entityHandle" => {"Id" => 42}},
+        :success
+      )
       subject.find(42)
     end
 
     it "returns CreditorContact object" do
-      stub_request('CreditorContact_GetData', nil, :success)
-      expect(subject.find(42)).to be_instance_of(Economic::CreditorContact)
+      stub_request("CreditorContact_GetData", nil, :success)
+      expect(
+        subject.find(42)
+      ).to be_instance_of(Economic::CreditorContact)
     end
   end
 
@@ -54,7 +60,7 @@ describe Economic::CreditorContactProxy do
     it "uses the FindByName command" do
       expect(Economic::Proxies::Actions::FindByName).to receive(:new).
         with(subject, "Bob").
-        and_return(lambda { "Result" })
+        and_return(-> { "Result" })
       expect(subject.find_by_name("Bob")).to eq("Result")
     end
   end

data/spec/economic/proxies/creditor_entry_proxy_spec.rb

@@ -1,4 +1,4 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::CreditorEntryProxy do
   let(:session) { make_session }
@@ -11,32 +11,32 @@ describe Economic::CreditorEntryProxy do
   end
 
   describe "#find_by_invoice_number" do
-    it 'should be able to find multiple creditor entries' do
-      mock_request("CreditorEntry_FindByInvoiceNumber", {'invoiceNumber' => '123'}, :many)
-      expect(subject.find_by_invoice_number('123')).to eq([1, 2])
+    it "should be able to find multiple creditor entries" do
+      mock_request("CreditorEntry_FindByInvoiceNumber", {"invoiceNumber" => "123"}, :many)
+      expect(subject.find_by_invoice_number("123")).to eq([1, 2])
     end
 
-    it 'should handle a single serial number in the response' do
+    it "should handle a single serial number in the response" do
       stub_request("CreditorEntry_FindByInvoiceNumber", nil, :single)
-      expect(subject.find_by_invoice_number('123')).to eq([1])
+      expect(subject.find_by_invoice_number("123")).to eq([1])
     end
   end
 
   describe "#find" do
-    it 'should get a creditor entry by serial number' do
-      mock_request("CreditorEntry_GetData", {'entityHandle' => { 'SerialNumber' => '123' }}, :success)
-      expect(subject.find('123')).to be_instance_of(Economic::CreditorEntry)
+    it "should get a creditor entry by serial number" do
+      mock_request("CreditorEntry_GetData", {"entityHandle" => {"SerialNumber" => "123"}}, :success)
+      expect(subject.find("123")).to be_instance_of(Economic::CreditorEntry)
     end
   end
 
   describe "#match" do
-    it 'should match two creditor entries by serial numbers' do
+    it "should match two creditor entries by serial numbers" do
       stub_request(
         "CreditorEntry_MatchEntries",
         {:entries => {
           "CreditorEntryHandle" => [
-            {"SerialNumber"=>1},
-            {"SerialNumber"=>2}
+            {"SerialNumber" => 1},
+            {"SerialNumber" => 2}
           ]
         }},
         :success