rconomic 0.5.1 → 0.5.2

data/spec/economic/current_invoice_line_spec.rb

@@ -45,4 +45,25 @@ describe Economic::CurrentInvoiceLine do
     end
   end
 
+  describe "#save" do
+    context "when successful" do
+      it "builds and sends data to API" do
+        mock_request(
+          "CurrentInvoiceLine_CreateFromData", {
+            "data" => {
+              "Number" => 0,
+              "DeliveryDate" => nil,
+              "DiscountAsPercent" => 0,
+              "UnitCostPrice" => 0,
+              "TotalNetAmount" => nil,
+              "TotalMargin" => 0,
+              "MarginAsPercent" => 0
+            }
+          },
+          :success
+        )
+        subject.save
+      end
+    end
+  end
 end

data/spec/economic/debtor_spec.rb

@@ -56,18 +56,43 @@ describe Economic::Debtor do
     end
   end
 
-  describe ".contacts" do
-    it "returns a DebtorContactProxy" do
-      expect(subject.contacts).to be_instance_of(Economic::DebtorContactProxy)
+  describe ".invoices" do
+    it "return nothing if no handle" do
+      expect(subject.invoices).to be_empty
+    end
+    it "returns invoices if there is a handle" do
+      mock_request('Debtor_GetInvoices', {"debtorHandle"=>{"Number"=>1}}, :success)
+      subject.handle = Economic::Entity::Handle.new({:number => "1"})
+      subject.invoices.each do |i|
+        expect(i).to be_instance_of(Economic::Invoice)
+      end
     end
+  end
 
-    it "memoizes the proxy" do
-      expect(subject.contacts).to equal(subject.contacts)
+  describe ".orders" do
+    it "return nothing if no handle" do
+      expect(subject.orders).to be_empty
+    end
+    it "returns invoices if there is a handle" do
+      mock_request('Debtor_GetOrders', {"debtorHandle"=>{"Number"=>1}}, :success)
+      subject.handle = Economic::Entity::Handle.new({:number => "1"})
+      subject.orders.each do |i|
+        expect(i).to be_instance_of(Economic::Order)
+      end
     end
+  end
 
-    it "should store the session" do
-      expect(subject.session).to_not be_nil
-      expect(subject.contacts.session).to eq(subject.session)
+  describe ".contacts" do
+    it "returns nothing if no handle" do
+      expect(subject.contacts).to be_empty
+    end
+
+    it "returns debtor contacts if there is a handle" do
+      mock_request('Debtor_GetDebtorContacts', {"debtorHandle"=>{"Number"=>1}}, :multiple)
+      subject.handle = Economic::Entity::Handle.new({:number => "1"})
+      subject.contacts.each do |contact|
+        expect(contact).to be_instance_of(Economic::DebtorContact)
+      end
     end
   end
 

data/spec/economic/endpoint_spec.rb

@@ -68,4 +68,22 @@ describe Economic::Endpoint do
       expect(subject.soap_action_name("FooBar", "Stuff")).to eq(:foo_bar_stuff)
     end
   end
+
+  describe "savon globals configuration" do
+    it "sets the log_level option of the endpoint" do
+      subject.client.globals.should_receive(:log_level).with(:fatal)
+      subject.log_level = :fatal
+    end
+
+    it "sets the log option of the endpoint" do
+      subject.client.globals.should_receive(:log).with(true)
+      subject.log = true
+    end
+
+    it "sets the logger option of the boolean" do
+      logger = double("MyLogger")
+      subject.client.globals.should_receive(:logger).with(logger)
+      subject.logger = logger
+    end
+  end
 end

data/spec/economic/entity/handle_spec.rb

@@ -162,10 +162,10 @@ describe Economic::Entity::Handle do
   end
 
   describe ".to_hash" do
-    subject { Economic::Entity::Handle.new({:id => 42, :number => 37, :serial_number => 7, :code => "USD", :vat_code => 1}) }
+    subject { Economic::Entity::Handle.new({:id => 42, :number => 37, :serial_number => 7, :code => "USD", :name => "Bob", :vat_code => 1}) }
 
     it "should return a handle for putting into the body of a SOAP request" do
-      expect(subject.to_hash).to eq({'Id' => 42, 'Number' => 37, 'SerialNumber' => 7, 'Code' => 'USD', 'VatCode' => 1})
+      expect(subject.to_hash).to eq({'Id' => 42, 'Number' => 37, 'SerialNumber' => 7, 'Code' => 'USD', 'Name' => 'Bob', 'VatCode' => 1})
     end
 
     it "includes only the named value in the hash" do

data/spec/economic/order_spec.rb

@@ -0,0 +1,80 @@
+require './spec/spec_helper'
+
+describe Economic::Order do
+  let(:session) { make_session }
+  subject { Economic::Order.new(:session => session, :number => 512) }
+
+  it "inherits from Economic::Entity" do
+    expect(Economic::Order.ancestors).to include(Economic::Entity)
+  end
+
+  describe ".proxy" do
+    it "should return a OrderProxy" do
+      expect(subject.proxy).to be_instance_of(Economic::OrderProxy)
+    end
+
+    it "should return a proxy owned by session" do
+      expect(subject.proxy.session).to eq(session)
+    end
+  end
+
+  describe ".key" do
+    it "should == :invoice" do
+      expect(Economic::Order.key).to eq(:order)
+    end
+  end
+
+  describe "#pdf" do
+    it "gets PDF data from API" do
+      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')
+    end
+  end
+
+  describe "#attention" do
+    let(:contact) {
+      Economic::DebtorContact.new.tap do |c|
+        c.session = session
+        c.id = 5
+      end
+    }
+
+    it "should be set- and gettable" do
+      subject.attention = contact
+      expect(subject.attention).to eq(contact)
+    end
+
+    it "updates the handle" do
+      handle = Economic::Entity::Handle.new(:number => 42)
+      contact.handle = handle
+      subject.attention = contact
+      expect(subject.attention_handle).to eq(handle)
+    end
+  end
+
+  describe "#debtor" do
+    let(:debtor) {
+      Economic::Debtor.new.tap do |c|
+        c.session = session
+        c.number = 5
+      end
+    }
+
+    it "should be set- and gettable" do
+      subject.debtor = debtor
+      expect(subject.debtor).to eq(debtor)
+    end
+
+    it "updates the handle" do
+      handle = Economic::Entity::Handle.new(:number => 42)
+      debtor.handle = handle
+      subject.debtor = debtor
+      expect(subject.debtor_handle).to eq(handle)
+    end
+  end
+end

data/spec/economic/proxies/debtor_contact_proxy_spec.rb

@@ -25,7 +25,7 @@ describe Economic::DebtorContactProxy do
 
     context "when owner is a Debtor" do
       let(:debtor) { make_debtor(:session => session) }
-      subject { debtor.contacts }
+      subject { Economic::DebtorContactProxy.new(debtor) }
 
       it "should use the Debtors session" do
         expect(subject.build.session).to eq(debtor.session)

data/spec/economic/proxies/debtor_proxy_spec.rb

@@ -68,6 +68,23 @@ describe Economic::DebtorProxy do
     end
   end
 
+  describe "find_by_telephone_and_fax_number" do
+    it "can find a debtor" do
+      mock_request('Debtor_FindByTelephoneAndFaxNumber', {'telephoneAndFaxNumber' => '22334455'}, :found)
+      result = subject.find_by_telephone_and_fax_number('22334455')
+      expect(result).to be_instance_of(Economic::Debtor)
+      expect(result.number).to eq(1)
+      expect(result.partial).to be_true
+      expect(result.persisted).to be_true
+    end
+
+    it "returns nil when there is no debtor" do
+      mock_request('Debtor_FindByTelephoneAndFaxNumber', {'telephoneAndFaxNumber' => '22334455'}, :not_found)
+      result = subject.find_by_telephone_and_fax_number('22334455')
+      expect(result).to be_nil
+    end
+  end
+
   describe "next_available_number" do
     it "gets the next available debtor number from API" do
       mock_request('Debtor_GetNextAvailableNumber', nil, :success)
@@ -112,4 +129,56 @@ describe Economic::DebtorProxy do
       expect(all.first).to be_instance_of(Economic::Debtor)
     end
   end
+
+  describe ".get_debtor_contacts" do
+    let(:handle) { Economic::Entity::Handle.new({:number => 1}) }
+    it "gets debtor contact data from API" do
+      mock_request('Debtor_GetDebtorContacts', {"debtorHandle"=>{"Number"=>1}}, :multiple)
+      subject.get_debtor_contacts(handle)
+    end
+
+    it "returns DebtorContact objects" do
+      stub_request('Debtor_GetDebtorContacts', nil, :multiple)
+      subject.get_debtor_contacts(handle).each do |d|
+        expect(d).to be_instance_of(Economic::DebtorContact)
+      end
+    end
+  end
+
+  describe ".get_invoices" do
+    let(:handle) { Economic::Entity::Handle.new({:number => 1}) }
+    it "gets invoice data from API" do
+      mock_request('Debtor_GetInvoices', {"debtorHandle"=>{"Number"=>1}}, :success)
+      subject.get_invoices(handle)
+    end
+
+    it "returns Invoice object" do
+      stub_request('Debtor_GetInvoices', nil, :success)
+      subject.get_invoices(handle).each do |i|
+        expect(i).to be_instance_of(Economic::Invoice)
+      end
+    end
+  end
+
+  describe ".get_order" do
+    let(:handle) { Economic::Entity::Handle.new({:number => 1}) }
+    it "gets invoice data from API" do
+      mock_request('Debtor_GetOrders', {"debtorHandle"=>{"Number"=>1}}, :success)
+      subject.get_orders(handle)
+    end
+
+    it "returns Order object" do
+      stub_request('Debtor_GetOrders', nil, :success)
+      subject.get_orders(handle).each do |i|
+        expect(i).to be_instance_of(Economic::Order)
+      end
+    end
+
+    it "sets the number" do
+      stub_request('Debtor_GetOrders', nil, :success)
+      subject.get_orders(handle).each do |i|
+        expect(i.number).to eq(1)
+      end
+    end
+  end
 end

data/spec/economic/proxies/order_proxy_spec.rb

@@ -0,0 +1,67 @@
+require './spec/spec_helper'
+
+describe Economic::OrderProxy do
+
+  let(:session) { make_session }
+  subject { Economic::OrderProxy.new(session) }
+
+  describe ".new" do
+    it "stores session" do
+      expect(subject.session).to equal(session)
+    end
+  end
+
+  describe ".build" do
+    it "instantiates a new Order" do
+      expect(subject.build).to be_instance_of(Economic::Order)
+    end
+
+    it "assigns the session to the Order" do
+      expect(subject.build.session).to equal(session)
+    end
+
+    it "should not build a partial Order" do
+      expect(subject.build).to_not be_partial
+    end
+  end
+
+  describe ".find" do
+    it "gets order data from API" do
+      mock_request('Order_GetData', {'entityHandle' => {'Id' => 42}}, :success)
+      subject.find(42)
+    end
+
+    it "returns Order object" do
+      stub_request('Order_GetData', nil, :success)
+      expect(subject.find(42)).to be_instance_of(Economic::Order)
+    end
+  end
+
+  describe ".find_by_date_interval" do
+    let(:from) { Time.now - 60 }
+    let(:unto) { Time.now }
+
+    it "should be able to return a single current order" do
+      mock_request('Order_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :single)
+      mock_request('Order_GetDataArray', :any, :single)
+      results = subject.find_by_date_interval(from, unto)
+      expect(results.size).to eq(1)
+      expect(results.first).to be_instance_of(Economic::Order)
+    end
+
+    it "should be able to return multiple orders" do
+      mock_request('Order_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :many)
+      mock_request('Order_GetDataArray', :any, :multiple)
+      results = subject.find_by_date_interval(from, unto)
+      expect(results.size).to eq(2)
+      expect(results.first).to be_instance_of(Economic::Order)
+    end
+
+    it "should be able to return nothing" do
+      mock_request('Order_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :none)
+      results = subject.find_by_date_interval(from, unto)
+      expect(results.size).to eq(0)
+    end
+
+  end
+end

data/spec/economic/session_spec.rb

@@ -1,24 +1,42 @@
 require './spec/spec_helper'
 
 describe Economic::Session do
-  subject { Economic::Session.new(123456, 'api', 'passw0rd') }
+  let(:credentials) { [123456, 'api', 'passw0rd'] }
+  subject { Economic::Session.new }
 
   let(:endpoint) { subject.endpoint }
 
   describe "new" do
-    it "should store authentication details" do
-      expect(subject.agreement_number).to eq(123456)
-      expect(subject.user_name).to eq('api')
-      expect(subject.password).to eq('passw0rd')
+    describe "legacy connect" do
+      subject { Economic::Session.new *credentials }
+      it "should store authentication details" do
+        expect(subject.agreement_number).to eq(123456)
+        expect(subject.user_name).to eq('api')
+        expect(subject.password).to eq('passw0rd')
+      end
+    end
+
+    it "yields the endpoint if a block is given" do
+      endpoint_mock = double("Endpoint")
+      Economic::Endpoint.stub(:new).and_return(endpoint_mock)
+      expect{|b|
+        Economic::Session.new(123456, 'api', 'passw0rd', &b)
+      }.to yield_with_args(endpoint_mock)
     end
   end
 
   describe "connect" do
     let(:authentication_details) { {:agreementNumber => 123456, :userName => 'api', :password => 'passw0rd'} }
 
+    it "can connect old-style" do
+      mock_request(:connect, authentication_details, :success)
+      e = Economic::Session.new(*credentials)
+      e.connect
+    end
+
     it "connects to e-conomic with authentication details" do
       mock_request(:connect, authentication_details, :success)
-      subject.connect
+      subject.connect_with_credentials(*credentials)
     end
 
     it "stores the authentication token for later requests" do
@@ -28,7 +46,7 @@ describe Economic::Session do
       }
       stub_request('Connect', authentication_details, response)
 
-      subject.connect
+      subject.connect_with_credentials(*credentials)
 
       expect(subject.authentication_token.collect { |cookie|
         cookie.name_and_value.split("=").last
@@ -37,11 +55,11 @@ describe Economic::Session do
 
     it "updates the authentication token for new sessions" do
       stub_request("Connect", nil, {:headers => {"Set-Cookie" => "authentication token"}})
-      subject.connect
+      subject.connect_with_credentials(*credentials)
 
       stub_request('Connect', nil, {:headers => {"Set-Cookie" => "another token"}})
-      other_session = Economic::Session.new(123456, 'api', 'passw0rd')
-      other_session.connect
+      other_session = Economic::Session.new
+      other_session.connect_with_credentials(123456, 'api', 'passw0rd')
 
       expect(subject.authentication_token.collect { |cookie|
         cookie.name_and_value.split("=").last
@@ -53,7 +71,54 @@ describe Economic::Session do
 
     it "doesn't use existing authentication details when connecting" do
       expect(endpoint).to receive(:call).with(:connect, instance_of(Hash))
-      subject.connect
+      subject.connect_with_credentials(*credentials)
+    end
+  end
+
+  describe "connecting with access ID" do
+    # As per http://www.e-conomic.com/developer/tutorials
+    let(:authentication_details) { {:appToken => 'the_private_app_id', :token => 'the_access_id_you_got_from_the_grant'} }
+    let(:private_app_id) { authentication_details[:appToken] }
+    let(:access_id) { authentication_details[:token] }
+    subject { Economic::Session.new }
+    it "connects to e-conomic with tokens details" do
+      mock_request(:connect_with_token, authentication_details, :success)
+      subject.connect_with_token private_app_id, access_id
+    end
+
+    it "stores the authentication token for later requests" do
+      response = {
+        :headers => {'Set-Cookie' => 'cookie value from e-conomic'},
+        :body => fixture(:connect_with_token, :success)
+      }
+      stub_request('ConnectWithToken', authentication_details, response)
+
+      subject.connect_with_token private_app_id, access_id
+
+      expect(subject.authentication_token.collect { |cookie|
+        cookie.name_and_value.split("=").last
+      }).to eq(["cookie value from e-conomic"])
+    end
+
+    it "updates the authentication token for new sessions" do
+      stub_request("ConnectWithToken", nil, {:headers => {"Set-Cookie" => "authentication token"}})
+      subject.connect_with_token private_app_id, access_id
+
+      stub_request('Connect', nil, {:headers => {"Set-Cookie" => "another token"}})
+      other_session = Economic::Session.new
+      other_session.connect_with_credentials(123456, 'api', 'passw0rd')
+
+      expect(subject.authentication_token.collect { |cookie|
+        cookie.name_and_value.split("=").last
+      }).to eq(["authentication token"])
+      expect(other_session.authentication_token.collect { |cookie|
+        cookie.name_and_value.split("=").last
+      }).to eq(["another token"])
+    end
+
+    it "doesn't use existing authentication details when connecting" do
+      expect(endpoint).to receive(:call).with(:connect_with_token, instance_of(Hash))
+      subject.connect_with_token private_app_id, access_id
     end
   end
 
@@ -135,4 +200,27 @@ describe Economic::Session do
     end
   end
 
+  describe "savon configuration" do
+    let(:endpoint) { double("Endpoint") }
+
+    before :each do
+      Economic::Endpoint.stub(:new).and_return(endpoint)
+    end
+
+    it "sets the log_level option of the endpoint" do
+      endpoint.should_receive(:log_level=).with(:info)
+      subject.log_level = :info
+    end
+
+    it "sets the log option of the endpoint" do
+      endpoint.should_receive(:log=).with(true)
+      subject.log = true
+    end
+
+    it "sets the logger option of the boolean" do
+      logger = double("MyLogger")
+      endpoint.should_receive(:logger=).with(logger)
+      subject.logger = logger
+    end
+  end
 end