rconomic 0.3 → 0.4

data/spec/economic/creditor_contact_spec.rb

@@ -0,0 +1,106 @@
+require './spec/spec_helper'
+
+describe Economic::CreditorContact do
+  let(:session) { make_session }
+  subject { Economic::CreditorContact.new(:session => session) }
+
+  it "inherits from Economic::Entity" do
+    Economic::CreditorContact.ancestors.should include(Economic::Entity)
+  end
+
+  context "when saving" do
+    context "when creditor contact is new" do
+      subject { Economic::CreditorContact.new(:session => session) }
+
+      context "when creditor_handle is nil" do
+        before :each do
+          subject.creditor_handle = nil
+        end
+
+        it "should send request and let e-conomic return an error" do
+          session.expects(:request)
+          subject.save
+        end
+      end
+    end
+  end
+
+  describe ".creditor" do
+    context "when creditor_handle is not set" do
+      it "returns nil" do
+        subject.creditor.should be_nil
+      end
+    end
+
+    context "when creditor_handle is set" do
+      let(:handle) { Economic::CreditorContact::Handle.new({:number => 42}) }
+
+      before :each do
+        subject.creditor_handle = handle
+      end
+
+      it "returns a Creditor" do
+        session.creditors.expects(:find).with(42).returns(Economic::Creditor.new)
+        subject.creditor.should be_instance_of(Economic::Creditor)
+      end
+
+      it "only looks up the creditor the first time" do
+        session.creditors.expects(:find).with(42).returns(Economic::Creditor.new)
+        subject.creditor.should === subject.creditor
+      end
+    end
+  end
+
+  describe ".creditor=" do
+    let(:creditor) { make_creditor }
+    it "should set creditor_handle" do
+      subject.creditor = creditor
+      subject.creditor_handle.should == creditor.handle
+    end
+  end
+
+  describe ".creditor_handle=" do
+    let(:creditor) { make_creditor }
+    let(:handle) { creditor.handle }
+
+    it "should set creditor_handle" do
+      subject.creditor_handle = handle
+      subject.creditor_handle.should == handle
+    end
+
+    context "when creditor handle is for a different Creditor" do
+      before :each do
+        subject.creditor = creditor
+      end
+
+      it "should clear cached creditor and fetch the new creditor from API" do
+        savon.stubs('Creditor_GetData').returns(:success)
+        subject.creditor_handle = Economic::Creditor::Handle.new({:number => 1234})
+        subject.creditor.should be_instance_of(Economic::Creditor)
+      end
+    end
+
+    context "when creditor handle is for the current Creditor" do
+      before :each do
+        subject.creditor = creditor
+      end
+
+      it "should not clear cached creditor nor fetch the creditor from API" do
+        savon.stubs('Creditor_GetData').never
+        subject.creditor_handle = handle
+        subject.creditor.should be_instance_of(Economic::Creditor)
+      end
+    end
+  end
+
+  describe ".proxy" do
+    it "should return a CreditorContactProxy" do
+      subject.proxy.should be_instance_of(Economic::CreditorContactProxy)
+    end
+
+    it "should return a proxy owned by session" do
+      subject.proxy.session.should == session
+    end
+  end
+
+end

data/spec/economic/creditor_spec.rb

@@ -0,0 +1,52 @@
+require './spec/spec_helper'
+
+describe Economic::Creditor do
+  let(:session) { make_session }
+  subject { Economic::Creditor.new(:session => session) }
+
+  it "inherits from Economic::Entity" do
+    Economic::Creditor.ancestors.should include(Economic::Entity)
+  end
+
+  describe "class methods" do
+    subject { Economic::Creditor }
+
+    describe ".proxy" do
+      it "should return CreditorProxy" do
+        subject.proxy.should == Economic::CreditorProxy
+      end
+    end
+
+    describe ".key" do
+      it "should == :creditor" do
+        Economic::Creditor.key.should == :creditor
+      end
+    end
+  end
+
+  describe ".contacts" do
+    it "returns a CreditorContactProxy" do
+      subject.contacts.should be_instance_of(Economic::CreditorContactProxy)
+    end
+
+    it "memoizes the proxy" do
+      subject.contacts.should === subject.contacts
+    end
+
+    it "should store the session" do
+      subject.session.should_not be_nil
+      subject.contacts.session.should == subject.session
+    end
+  end
+
+  describe ".proxy" do
+    it "should return a CreditorProxy" do
+      subject.proxy.should be_instance_of(Economic::CreditorProxy)
+    end
+
+    it "should return a proxy owned by session" do
+      subject.proxy.session.should == session
+    end
+  end
+
+end

data/spec/economic/current_invoice_spec.rb

@@ -90,6 +90,20 @@ describe Economic::CurrentInvoice do
     end
   end
 
+  describe "#book_with_number" do
+    it 'should book the current invoice with the given number and return the created invoice object' do
+      savon.stubs('CurrentInvoice_BookWithNumber').returns(:success)
+      savon.expects("Invoice_GetData").with('entityHandle' => { 'Number' => 123 }).returns(:success)
+      subject.book_with_number(123).should be_instance_of(Economic::Invoice)
+    end
+
+    it 'should request with the right key for handle' do
+      savon.stubs('Invoice_GetData').returns(:success)
+      savon.expects("CurrentInvoice_BookWithNumber").with('currentInvoiceHandle' => { 'Id' => 512 }, 'number' => 123, :order! => ['currentInvoiceHandle', 'number']).returns(:success)
+      subject.book_with_number(123)
+    end
+  end
+
   describe "#attention" do
     let(:contact) { (c = Economic::DebtorContact.new).tap { c.session = session }}
 

data/spec/economic/debtor_entry_spec.rb

@@ -0,0 +1,20 @@
+require './spec/spec_helper'
+
+describe Economic::DebtorEntry do
+  let(:session) { make_session }
+  subject { Economic::DebtorEntry.new(:session => session) }
+
+  it "inherits from Economic::Entity" do
+    Economic::DebtorEntry.ancestors.should include(Economic::Entity)
+  end
+
+  describe ".proxy" do
+    it "should return a DebtorEntryProxy" do
+      subject.proxy.should be_instance_of(Economic::DebtorEntryProxy)
+    end
+
+    it "should return a proxy owned by session" do
+      subject.proxy.session.should == session
+    end
+  end
+end

data/spec/economic/debtor_spec.rb

@@ -81,4 +81,16 @@ describe Economic::Debtor do
     end
   end
 
+  describe "equality" do
+    context "when other handle is equal" do
+      context "when other is a different class" do
+        let(:other) { Economic::Invoice.new(:session => session, :handle => subject.handle) }
+
+        it "should return false" do
+          subject.should_not == other
+        end
+      end
+    end
+  end
+
 end

data/spec/economic/entity/handle_spec.rb

@@ -18,6 +18,27 @@ describe Economic::Entity::Handle do
     it "should not be equal when id or number is equal and the other isn't" do
       handle_a.should_not == handle_d
     end
+
+    describe "CashBookEntry handles" do
+      let(:handle_a) { Economic::Entity::Handle.new(:id1 => 1, :id2 => 2) }
+
+      context "when id1 and id2 values are equal" do
+        let(:handle_b) { Economic::Entity::Handle.new(:id1 => 1, :id2 => 2) }
+
+        it "should be equal" do
+          handle_a.should == handle_b
+        end
+      end
+
+      context "when id1 and id2 values are different" do
+        let(:handle_b) { Economic::Entity::Handle.new(:id1 => 11, :id2 => 12) }
+
+        it "should_not be equal" do
+          handle_a.should_not == handle_b
+        end
+      end
+    end
+
   end
 
   describe ".new" do

data/spec/economic/entity_spec.rb

@@ -213,4 +213,35 @@ describe Economic::Entity do
       subject.update_properties(:foo => 'bar', 'baz' => 'qux')
     end
   end
+
+  describe "equality" do
+    context "when other is nil do" do
+      let(:other) { nil }
+
+      it "should return true" do
+        subject.should_not == other
+      end
+    end
+
+    context "when other handle is equal" do
+      context "when other is same class" do
+        let(:other) { Economic::Entity.new(:session => session, :handle => subject.handle) }
+
+        it "should return true" do
+          subject.should == other
+        end
+      end
+
+      context "when other is child class" do
+        let(:other) { Economic::Debtor.new(:session => session, :handle => subject.handle) }
+
+        it "should return true" do
+          subject.should == other
+        end
+      end
+    end
+
+
+
+  end
 end

data/spec/economic/entry_spec.rb

@@ -0,0 +1,20 @@
+require './spec/spec_helper'
+
+describe Economic::Entry do
+  let(:session) { make_session }
+  subject { Economic::Entry.new(:session => session) }
+
+  it "inherits from Economic::Entity" do
+    Economic::Entry.ancestors.should include(Economic::Entity)
+  end
+
+  describe ".proxy" do
+    it "should return a EntryProxy" do
+      subject.proxy.should be_instance_of(Economic::EntryProxy)
+    end
+
+    it "should return a proxy owned by session" do
+      subject.proxy.session.should == session
+    end
+  end
+end

data/spec/economic/proxies/cash_book_entry_proxy_spec.rb

@@ -51,11 +51,44 @@ describe Economic::CashBookEntryProxy do
     end
   end
 
+  describe "#create_creditor_invoice" do
+    it 'should create a creditor invoice and then return the created cash book entry' do
+      savon.stubs('CashBookEntry_CreateCreditorInvoice').returns(:success)
+      savon.stubs('CashBookEntry_GetData').with(:id1 => 13, :id2 => 14).returns(:success)
+      cash_book_entry = subject.create_creditor_invoice(:creditor_handle => { :number => 2 }, :contra_account_handle => { :number => 3 })
+      cash_book_entry.should be_instance_of(Economic::CashBookEntry)
+    end
+
+    it 'should not send handles that were not given' do
+      savon.stubs('CashBookEntry_CreateCreditorInvoice').with(:cashBookHandle => { 'Number' => 13 }).returns(:success)
+      savon.stubs('CashBookEntry_GetData').with(:id1 => 13, :id2 => 14).returns(:success)
+      cash_book_entry = subject.create_creditor_invoice({})
+      cash_book_entry.should 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
+      savon.stubs('CashBookEntry_CreateCreditorPayment').returns(:success)
+      savon.stubs('CashBookEntry_GetData').with(:id1 => 13, :id2 => 14).returns(:success)
+      cash_book_entry = subject.create_creditor_payment(:creditor_handle => { :number => 2 }, :contra_account_handle => { :number => 3 })
+      cash_book_entry.should be_instance_of(Economic::CashBookEntry)
+    end
+  end
+
+  describe "#set_due_date" do
+    it 'should set due date' do
+      savon.expects('CashBookEntry_SetDueDate').with("cashBookEntryHandle" => { "Id1" => subject.owner.id, "Id2" => 234 }, :value => Date.new(2012, 12, 21)).returns(:success)
+      subject.set_due_date(234, Date.new(2012, 12, 21))
+    end
+  end
+
   describe "#all" do
     it 'should get the cash book entries' do
       savon.stubs('CashBook_GetEntries').returns(:success)
-      savon.stubs('CashBookEntry_GetData').returns(:success)
-      subject.all.size.should == 2
+      subject.expects(:find).in_sequence.with({:id1 => '1', :id2 => '2'})
+      subject.expects(:find).in_sequence.with({:id1 => '11', :id2 => '12'})
+      subject.all
     end
   end
 

data/spec/economic/proxies/creditor_contact_proxy_spec.rb

@@ -0,0 +1,55 @@
+require './spec/spec_helper'
+
+describe Economic::CreditorContactProxy do
+  let(:session) { make_session }
+  subject { Economic::CreditorContactProxy.new(session) }
+
+  describe ".new" do
+    it "stores session" do
+      subject.session.should === session
+    end
+  end
+
+  describe ".build" do
+    it "instantiates a new CreditorContact" do
+      subject.build.should be_instance_of(Economic::CreditorContact)
+    end
+
+    it "assigns the session to the CreditorContact" do
+      subject.build.session.should === session
+    end
+
+    it "should not build a partial CreditorContact" do
+      subject.build.should_not be_partial
+    end
+
+    context "when owner is a Creditor" do
+      let(:creditor) { make_creditor(:session => session) }
+      subject { creditor.contacts }
+
+      it "should use the Creditors session" do
+        subject.build.session.should == creditor.session
+      end
+
+      it "should initialize with values from Creditor" do
+        contact = subject.build
+        contact.creditor_handle.should == creditor.handle
+      end
+    end
+  end
+
+  describe ".find" do
+    before :each do
+      savon.stubs('CreditorContact_GetData').returns(:success)
+    end
+
+    it "gets contact data from API" do
+      savon.expects('CreditorContact_GetData').with('entityHandle' => {'Id' => 42}).returns(:success)
+      subject.find(42)
+    end
+
+    it "returns CreditorContact object" do
+      subject.find(42).should be_instance_of(Economic::CreditorContact)
+    end
+  end
+end

data/spec/economic/proxies/creditor_entry_proxy_spec.rb

@@ -0,0 +1,49 @@
+require './spec/spec_helper'
+
+describe Economic::CreditorEntryProxy do
+  let(:session) { make_session }
+  subject { Economic::CreditorEntryProxy.new(session) }
+
+  describe "new" do
+    it "stores session" do
+      subject.session.should === session
+    end
+  end
+
+  describe "#find_by_invoice_number" do
+    it 'should be able to find multiple creditor entries' do
+      savon.expects("CreditorEntry_FindByInvoiceNumber").with('invoiceNumber' => '123').returns(:many)
+      subject.find_by_invoice_number('123').should == [1, 2]
+    end
+
+    it 'should handle a single serial number in the response' do
+      savon.stubs("CreditorEntry_FindByInvoiceNumber").returns(:single)
+      subject.find_by_invoice_number('123').should == [1]
+    end
+  end
+
+  describe "#find" do
+    it 'should get a creditor entry by serial number' do
+      savon.expects("CreditorEntry_GetData").with('entityHandle' => { 'SerialNumber' => '123' }).returns(:success)
+      subject.find('123').should be_instance_of(Economic::CreditorEntry)
+    end
+  end
+
+  describe "#match" do
+    it 'should match two creditor entries by serial numbers' do
+      savon.stubs("CreditorEntry_MatchEntries").with(
+        :entries => [
+          { :creditor_entry_handle => { :serial_number => 1 } },
+          { :credior_entry_handle => { :serial_number => 2 } }
+        ]
+      ).returns(:success)
+      subject.match(1, 2)
+    end
+  end
+
+  describe "#entity_class" do
+    it "should return Economic::CreditorEntry" do
+      Economic::CreditorEntryProxy.entity_class.should == Economic::CreditorEntry
+    end
+  end
+end

data/spec/economic/proxies/creditor_proxy_spec.rb

@@ -0,0 +1,81 @@
+require './spec/spec_helper'
+
+describe Economic::CreditorProxy do
+  let(:session) { make_session }
+  subject { Economic::CreditorProxy.new(session) }
+
+  describe "new" do
+    it "stores session" do
+      subject.session.should === session
+    end
+  end
+
+  describe "find" do
+    before :each do
+      savon.stubs('Creditor_GetData').returns(:success)
+    end
+
+    it "gets creditor data from API" do
+      savon.expects('Creditor_GetData').with('entityHandle' => {'Number' => 42}).returns(:success)
+      subject.find(42)
+    end
+
+    it "returns Creditor object" do
+      subject.find(42).should be_instance_of(Economic::Creditor)
+    end
+  end
+
+  describe "find_by_number" do
+    it "can find a creditor" do
+      savon.expects('Creditor_FindByNumber').with('number' => '1').returns(:found)
+      result = subject.find_by_number('1')
+      result.should be_instance_of(Economic::Creditor)
+      result.number.should == 1
+      result.partial.should be_true
+      result.persisted.should be_true
+      result.handle.should == Economic::Entity::Handle.new({ :number => 1 })
+    end
+
+    it "returns nil when there is no creditor" do
+      savon.expects('Creditor_FindByNumber').with('number' => '1').returns(:not_found)
+      result = subject.find_by_number('1')
+      result.should be_nil
+    end
+  end
+
+  describe "build" do
+    subject { session.creditors.build }
+
+    it "instantiates a new Creditor" do
+      subject.should be_instance_of(Economic::Creditor)
+    end
+
+    it "assigns the session to the Creditor" do
+      subject.session.should === session
+    end
+  end
+
+  describe "#entity_class" do
+    it "should return Economic::Creditor" do
+      Economic::CreditorProxy.entity_class.should == Economic::Creditor
+    end
+  end
+
+  describe ".all" do
+    it "returns a single creditor" do
+      savon.expects('Creditor_GetAll').returns(:single)
+      savon.expects('Creditor_GetData').with('entityHandle' => {'Number' => 1}).returns(:success)
+      all = subject.all
+      all.size.should == 1
+      all.first.should be_instance_of(Economic::Creditor)
+    end
+
+    it "returns multiple creditors" do
+      savon.expects('Creditor_GetAll').returns(:multiple)
+      savon.expects('Creditor_GetDataArray').returns(:multiple)
+      all = subject.all
+      all.size.should == 2
+      all.first.should be_instance_of(Economic::Creditor)
+    end
+  end
+end

data/spec/economic/proxies/current_invoice_line_proxy_spec.rb

@@ -85,7 +85,7 @@ describe Economic::CurrentInvoiceLineProxy do
     it "can be iterated over" do
       line = Economic::CurrentInvoiceLine.new
       subject << line
-      subject.all? { |l| l == [line] }
+      subject.all? { |l| l.is_a?(Economic::CurrentInvoiceLine) }.should be_true
     end
   end
 end

data/spec/economic/proxies/current_invoice_proxy_spec.rb

@@ -68,7 +68,7 @@ describe Economic::CurrentInvoiceProxy do
     it "should be able to return a single current invoice" do
       from = Time.now - 60
       unto = Time.now
-      savon.expects('CurrentInvoice_FindByDateInterval').with('first' => from.iso8601, 'last' => unto.iso8601).returns(:single)
+      savon.expects('CurrentInvoice_FindByDateInterval').with('first' => from.iso8601, 'last' => unto.iso8601, :order! => ['first', 'last']).returns(:single)
       savon.expects('CurrentInvoice_GetData').returns(:success)
       results = subject.find_by_date_interval(from, unto)
       results.size.should == 1
@@ -78,7 +78,7 @@ describe Economic::CurrentInvoiceProxy do
     it "should be able to return multiple invoices" do
       from = Time.now - 60
       unto = Time.now
-      savon.expects('CurrentInvoice_FindByDateInterval').with('first' => from.iso8601, 'last' => unto.iso8601).returns(:many)
+      savon.expects('CurrentInvoice_FindByDateInterval').with('first' => from.iso8601, 'last' => unto.iso8601, :order! => ['first', 'last']).returns(:many)
       savon.expects('CurrentInvoice_GetData').returns(:success)
       savon.expects('CurrentInvoice_GetData').returns(:success)
       results = subject.find_by_date_interval(from, unto)
@@ -89,7 +89,7 @@ describe Economic::CurrentInvoiceProxy do
     it "should be able to return nothing" do
       from = Time.now - 60
       unto = Time.now
-      savon.expects('CurrentInvoice_FindByDateInterval').with('first' => from.iso8601, 'last' => unto.iso8601).returns(:none)
+      savon.expects('CurrentInvoice_FindByDateInterval').with('first' => from.iso8601, 'last' => unto.iso8601, :order! => ['first', 'last']).returns(:none)
       results = subject.find_by_date_interval(from, unto)
       results.size.should == 0
     end

data/spec/economic/proxies/debtor_entry_proxy_spec.rb

@@ -0,0 +1,47 @@
+require './spec/spec_helper'
+
+describe Economic::DebtorEntryProxy do
+  let(:session) { make_session }
+  subject { Economic::DebtorEntryProxy.new(session) }
+
+  describe "new" do
+    it "stores session" do
+      subject.session.should === session
+    end
+  end
+
+  describe "#find_by_invoice_number" do
+    it 'should be able to find multiple debtor entries' do
+      savon.expects("DebtorEntry_FindByInvoiceNumber").with('from' => '123', 'to' => '456', :order! => ['from', 'to']).returns(:many)
+      subject.find_by_invoice_number('123', '456').should == [1, 2]
+    end
+
+    it 'should be able to find debtor entries with one invoice id' do
+      savon.expects("DebtorEntry_FindByInvoiceNumber").with('from' => '123', 'to' => '123', :order! => ['from', 'to']).returns(:many)
+      subject.find_by_invoice_number('123').should == [1, 2]
+    end
+
+    it 'should handle a single serial number in the response' do
+      savon.stubs("DebtorEntry_FindByInvoiceNumber").returns(:single)
+      subject.find_by_invoice_number('123').should == [1]
+    end
+  end
+
+  describe "#match" do
+    it 'should match two debtor entries by serial numbers' do
+      savon.stubs("DebtorEntry_MatchEntries").with(
+        :entries => [
+          { :debtor_entry_handle => { :serial_number => 1 } },
+          { :debtor_entry_handle => { :serial_number => 2 } }
+        ]
+      ).returns(:success)
+      subject.match(1, 2)
+    end
+  end
+
+  describe "#entity_class" do
+    it "should return Economic::DebtorEntry" do
+      Economic::DebtorEntryProxy.entity_class.should == Economic::DebtorEntry
+    end
+  end
+end