xeroizer 2.17.1 → 3.0.0

data/test/unit/record/connection_test.rb

@@ -0,0 +1,60 @@
+require 'unit_test_helper'
+
+class ConnectionTest < UnitTestCase
+  include TestHelper
+
+  setup do
+    @client = Xeroizer::OAuth2Application.new("client id", "client secret", access_token: "access token")
+  end
+
+  context "when the request succeeds" do
+    setup do
+      stub_request(:get, "https://api.xero.com/connections").to_return(
+        body: [
+          {
+            id: "fe4cd81c-624a-4506-ab18-11c656742436",
+            tenantId: "c84e0419-ade8-4296-9794-289876d4bc22",
+            tenantType: "ORGANISATION",
+            createdDateUtc: "2019-12-13T18:10:43.3063640",
+            updatedDateUtc: "2019-12-13T18:10:43.3084790"
+          },
+          {
+            id: "fe4cd81c-624a-4506-ab18-11c656742436",
+            tenantId: "c84e0419-ade8-4296-9794-289876d4bc22",
+            tenantType: "ORGANISATION",
+            createdDateUtc: "2019-12-13T18:10:43.3063640",
+            updatedDateUtc: "2019-12-13T18:10:43.3084790"
+          }
+        ].to_json
+      )
+    end
+
+    it 'returns connections using the passed client' do
+      result = Xeroizer::Connection.current_connections(@client.client)
+      assert_equal 2, result.count
+      assert_equal "c84e0419-ade8-4296-9794-289876d4bc22", result.first.tenant_id
+    end
+  end
+
+  context 'when the request fails' do
+    setup do
+      @body = {
+        title: "Unauthorized",
+        status: 401,
+        detail: "AuthenticationUnsuccessful",
+        instance: "e1d2cb91-82bf-41a5-84dd-fe6a5c6f070c"
+      }.to_json
+
+      stub_request(:get, "https://api.xero.com/connections").to_return(
+        status: 401,
+        body: @body
+      )
+    end
+
+    it 'raises token_invalid error' do
+      assert_raises(Xeroizer::OAuth::TokenInvalid, @body) {
+        Xeroizer::Connection.current_connections(@client.client)
+      }
+    end
+  end
+end

data/test/unit/record/model_definition_test.rb

@@ -1,12 +1,12 @@
-require 'test_helper'
+require 'unit_test_helper'
 
 class ModelDefinitionsTest < Test::Unit::TestCase
   include TestHelper
-  
+
   class FirstRecord < Xeroizer::Record::Base
-    
+
     set_primary_key :primary_key_id
-    
+
     guid      :primary_key_id
     string    :string1
     boolean   :boolean1
@@ -14,14 +14,14 @@ class ModelDefinitionsTest < Test::Unit::TestCase
     decimal   :decimal1
     date      :date1
     datetime  :datetime1
-    
+
   end
   class Xeroizer::Record::FirstRecordModel < Xeroizer::Record::BaseModel; end
-  
+
   class SecondRecord < Xeroizer::Record::Base
 
     set_primary_key :primary_key_id
-    
+
     guid      :primary_key_id
     string    :string2
     boolean   :boolean2
@@ -32,63 +32,63 @@ class ModelDefinitionsTest < Test::Unit::TestCase
 
   end
   class Xeroizer::Record::SecondRecordModel < Xeroizer::Record::BaseModel; end
-    
+
   class TestRecord < Xeroizer::Record::Base
-    
+
     string    :xml_name, :api_name => 'ApiNameHere', :internal_name => :internal_name_here
     string    :name
-    
+
   end
   class Xeroizer::Record::TestRecordModel < Xeroizer::Record::BaseModel; end
-  
+
   class SummaryOnlyRecord < Xeroizer::Record::Base
     class Xeroizer::Record::SummaryOnlyRecordModel < Xeroizer::Record::BaseModel; end
-    
+
     list_contains_summary_only true
     set_possible_primary_keys :primary_key_id
     set_primary_key :primary_key_id
-    
+
     string :primary_key_id
     string :name
   end
-  
+
   class SummaryOnlyOffRecord < Xeroizer::Record::Base
     class Xeroizer::Record::SummaryOnlyOffRecordModel < Xeroizer::Record::BaseModel; end
-    
+
     set_possible_primary_keys :primary_key_id
     set_primary_key :primary_key_id
-    
+
     string :primary_key_id
     string :name
   end
-  
+
   def setup
-    @client = Xeroizer::PublicApplication.new(CONSUMER_KEY, CONSUMER_SECRET)
+    @client = Xeroizer::OAuth2Application.new(CLIENT_ID, CLIENT_SECRET)
     parent = stub(:application => @client, :mark_dirty => nil)
     @first = FirstRecord.new(parent)
     @second = SecondRecord.new(parent)
     @record = TestRecord.build({}, @client.Contact)
     @contact = @client.Contact.build
   end
-  
+
   context "list contains summary only test" do
-    
+
     should "show download complete if not summary record and id set" do
       record = SummaryOnlyRecord.build({}, @client.Contact)
       record.id = "NOTBLANK"
       assert_equal(false, record.new_record?)
       assert_equal(false, record.complete_record_downloaded?)
-      
+
       record = SummaryOnlyOffRecord.build({}, @client.Contact)
       record.id = "NOTBLANK"
       assert_equal(false, record.new_record?)
       assert_equal(true, record.complete_record_downloaded?)
     end
-    
+
   end
-  
+
   context "record field definition" do
-    
+
     should "define primary key with real name" do
       assert_nil(@first.id)
       value = "PRIMARY KEY VALUE"
@@ -96,7 +96,7 @@ class ModelDefinitionsTest < Test::Unit::TestCase
       assert_equal(value, @first.primary_key_id)
       assert_equal(value, @first.id)
     end
-    
+
     should "define primary key with shortcut #id method" do
       assert_nil(@first.id)
       value = "PRIMARY KEY VALUE"
@@ -104,7 +104,7 @@ class ModelDefinitionsTest < Test::Unit::TestCase
       assert_equal(value, @first.id)
       assert_equal(value, @first.primary_key_id)
     end
-    
+
     should "only have proper fields" do
       fieldset = [:primary_key_id, :string1, :boolean1, :integer1, :decimal1, :date1, :datetime1]
       fieldset.each do | field |
@@ -116,16 +116,16 @@ class ModelDefinitionsTest < Test::Unit::TestCase
       fieldset.each do | field |
         assert(@second.class.fields.keys.include?(field), "#{field} not in SecondRecord.fields")
       end
-      assert_equal(fieldset.size, @second.class.fields.size)        
+      assert_equal(fieldset.size, @second.class.fields.size)
     end
-    
+
     should "have correct names" do
       assert(@record.respond_to?(:internal_name_here), "Internal name should be internal_name_here")
       assert(@record.class.fields.keys.include?(:xml_name), "Field key name should be xml_name")
       assert_equal('ApiNameHere', @record.class.fields[:xml_name][:api_name])
     end
-    
-    should "have shortcut reader/writer" do 
+
+    should "have shortcut reader/writer" do
       assert_nil(@first.string1)
       value = 'TEST VALUE'
       @first.string1 = value
@@ -136,30 +136,30 @@ class ModelDefinitionsTest < Test::Unit::TestCase
       @first[:string1] = value
       assert_equal(value, @first.attributes[:string1])
     end
-        
+
     should "define reader/writer methods" do
       assert(@record.respond_to?(:name), "FirstRecord#name should exist.")
       assert(@record.respond_to?(:name=), "FirstRecord#name= should exist.")
-      
+
       value = "TEST NAME"
       @record.attributes[:name] = value
       assert_equal(value, @record.attributes[:name])
       assert_equal(value, @record[:name])
       assert_equal(value, @record.name)
-      
+
       value = "TEST DIFFERENT"
       @record.name = value
       assert_equal(value, @record.attributes[:name])
       assert_equal(value, @record[:name])
       assert_equal(value, @record.name)
-      
+
       value = "TEST DIFFERENT AGAIN"
       @record[:name] = value
       assert_equal(value, @record.attributes[:name])
       assert_equal(value, @record[:name])
       assert_equal(value, @record.name)
     end
-    
+
   end
-  
+
 end

data/test/unit/record/parse_params_test.rb

@@ -0,0 +1,59 @@
+require 'unit_test_helper'
+
+module Xeroizer
+  module Record
+
+    class ParseParamTestModel < BaseModel
+    end
+
+    class ParseParamTest < Base
+
+      set_primary_key :primary_key_id
+
+      guid      :primary_key_id
+      string    :string1
+      boolean   :boolean1
+      integer   :integer1
+      decimal   :decimal1
+      date      :date1
+      datetime  :datetime1
+
+    end
+
+  end
+end
+
+class ParseParamsTest < Test::Unit::TestCase
+  include TestHelper
+
+  def setup
+    @client = Xeroizer::OAuth2Application.new(CLIENT_ID, CLIENT_SECRET)
+    @model = Xeroizer::Record::ParseParamTestModel.new(@client, "ParseParamTest")
+  end
+
+  context "should return valid and filtered params" do
+    should "filter unsupported keys" do
+      params = @model.send(:parse_params, {
+        :should_be_filtered_out => 'should be filtered',
+        :modified_since => Date.parse("2010-01-05"),
+        :include_archived => true,
+        :order => :order,
+        :where => 'where string',
+        :IDs => ['29ed7958-0466-486d-bf57-3fd966ea37d7', 'd561892a-9023-498c-a28d-c626ed3940d8'],
+        :InvoiceNumbers => 'INV-0034,INV-0035,INV-0036,INV-0037',
+        :ContactIDs => 'b919a496-1a1c-4fc6-b6ef-8c561e0dd8c2,8289cca4-90a9-466b-95f3-f0adf351b2ac',
+        :Statuses => ['DRAFT', nil, 'SUBMITTED'],
+        :offset => 100,
+        :page => 2
+      })
+
+      params.assert_valid_keys(:ModifiedAfter, :includeArchived, :order, :where,
+                               :IDs, :InvoiceNumbers, :ContactIDs, :Statuses,
+                               :offset, :page)
+      assert_equal(params[:IDs], '29ed7958-0466-486d-bf57-3fd966ea37d7,d561892a-9023-498c-a28d-c626ed3940d8')
+      assert_equal(params[:InvoiceNumbers], 'INV-0034,INV-0035,INV-0036,INV-0037')
+      assert_equal(params[:ContactIDs], 'b919a496-1a1c-4fc6-b6ef-8c561e0dd8c2,8289cca4-90a9-466b-95f3-f0adf351b2ac')
+      assert_equal(params[:Statuses], 'DRAFT,,SUBMITTED')
+    end
+  end
+end

data/test/unit/record/parse_where_hash_test.rb

@@ -1,11 +1,11 @@
-require 'test_helper'
+require 'unit_test_helper'
 
 module Xeroizer
   module Record
-    
+
     class WhereHashTestModel < BaseModel
     end
-    
+
     class WhereHashTest < Base
 
       set_primary_key :primary_key_id
@@ -19,32 +19,32 @@ module Xeroizer
       datetime  :datetime1
 
     end
-  
+
   end
-end  
+end
 
 class ParseWhereHashTest < Test::Unit::TestCase
   include TestHelper
-    
+
   def setup
-    @client = Xeroizer::PublicApplication.new(CONSUMER_KEY, CONSUMER_SECRET)
+    @client = Xeroizer::OAuth2Application.new(CLIENT_ID, CLIENT_SECRET)
     @model = Xeroizer::Record::WhereHashTestModel.new(@client, "WhereHashTest")
   end
-  
+
   context "where hash" do
-    
+
     should "parse valid hash" do
       @model.send(:parse_where_hash, {
-        :primary_key_id => 'f7eca431-5c97-4d24-93fd-004bb8a6c644', 
-        :string1 => 'test', 
-        :boolean1 => true, 
+        :primary_key_id => 'f7eca431-5c97-4d24-93fd-004bb8a6c644',
+        :string1 => 'test',
+        :boolean1 => true,
         :integer1 => 10,
         :decimal1 => 123.45,
         :date1 => Date.parse("2010-01-05"),
         :datetime1 => Time.parse("2010-02-30 09:10:20")
       })
     end
-    
+
     should 'have valid expression components' do
       assert_equal('String1=="abc"', CGI.unescape(@model.send(:parse_where_hash, {:string1 => 'abc'})))
       assert_equal('String1<>"abc"', CGI.unescape(@model.send(:parse_where_hash, {:string1_is_not => 'abc'})))

data/test/unit/record/record_association_test.rb

@@ -1,16 +1,16 @@
-require 'test_helper'
+require 'unit_test_helper'
 
 class RecordAssociationTest < Test::Unit::TestCase
   include TestHelper
 
   def setup
-    @client = Xeroizer::PublicApplication.new(CONSUMER_KEY, CONSUMER_SECRET)
+    @client = Xeroizer::OAuth2Application.new(CLIENT_ID, CLIENT_SECRET)
     mock_api('Invoices')
     @client.stubs(:http_put).returns(get_record_xml(:invoice, "762aa45d-4632-45b5-8087-b4f47690665e"))
   end
-  
+
   context "belongs_to association" do
-    
+
     should "auto-load complete record if summary" do
       invoice = @client.Invoice.first
       assert_nil(invoice.attributes[:contact].contact_status)
@@ -18,11 +18,11 @@ class RecordAssociationTest < Test::Unit::TestCase
       assert_not_nil(invoice.contact.contact_status)
       assert_equal(true, invoice.complete_record_downloaded?)
     end
-    
+
   end
-  
+
   context "has_many association" do
-    
+
     should "auto-load complete records if summary" do
       invoice = @client.Invoice.first
       assert_nil(invoice.attributes[:line_items])
@@ -30,7 +30,7 @@ class RecordAssociationTest < Test::Unit::TestCase
       assert(invoice.line_items.size > 0, "There should be one or more line items.")
       assert_equal(true, invoice.complete_record_downloaded?)
     end
-    
+
     should "auto-build belongs_to item when passed hash" do
       invoice = @client.Invoice.build
       assert_nil(invoice.contact)
@@ -38,7 +38,7 @@ class RecordAssociationTest < Test::Unit::TestCase
       assert_kind_of(Xeroizer::Record::Contact, invoice.contact)
       assert_equal("Test Contact", invoice.contact.name)
     end
-    
+
     should "auto-build has_many items when passed hash" do
       contact = @client.Contact.build
       assert_equal([], contact.phones)
@@ -46,11 +46,11 @@ class RecordAssociationTest < Test::Unit::TestCase
       assert_kind_of(Xeroizer::Record::Phone, contact.phones.first)
       assert_equal("1234", contact.phones.first.number)
     end
-    
+
     should "auto-build has_many items when passed array" do
       contact = @client.Contact.build
       assert_equal([], contact.phones)
-      
+
       data = [
         {:type => "DEFAULT", :number => "1111"},
         {:type => "FAX", :number => "2222"}
@@ -58,12 +58,12 @@ class RecordAssociationTest < Test::Unit::TestCase
       contact.phones = data.dup
 
       assert_equal(2, contact.phones.size)
-      contact.phones.each_with_index do | phone, index | 
+      contact.phones.each_with_index do | phone, index |
         assert_kind_of(Xeroizer::Record::Phone, phone)
         assert_equal(data[index][:number], phone.number)
       end
     end
-    
+
     should "retain unsaved items after create" do
       invoice = @client.Invoice.build :type => "ACCREC", :contact => { :name => "A" }
       invoice.save
@@ -89,5 +89,5 @@ class RecordAssociationTest < Test::Unit::TestCase
       assert_equal(1, invoice.line_items.size, "There should be one line item.")
     end
   end
-  
+
 end