xeroizer 2.18.1 → 3.0.1

data/test/unit/record/base_test.rb

@@ -1,10 +1,11 @@
-require 'test_helper'
+require 'unit_test_helper'
 
 class RecordBaseTest < Test::Unit::TestCase
   include TestHelper
 
   def setup
-    @client = Xeroizer::PublicApplication.new(CONSUMER_KEY, CONSUMER_SECRET)
+    @client = Xeroizer::OAuth2Application.new(CLIENT_ID, CLIENT_SECRET, tenant_id: TENANT_ID)
+    @client.authorize_from_access(ACCESS_TOKEN)
     @contact = @client.Contact.build(:name => 'Test Contact Name ABC')
   end
 
@@ -26,14 +27,14 @@ class RecordBaseTest < Test::Unit::TestCase
   context "new_record? states" do
 
     should "new_record? should be false when loading data" do
-      Xeroizer::OAuth.any_instance.stubs(:get).returns(stub(:plain_body => get_record_xml(:contact), :code => '200'))
+      Xeroizer::OAuth2.any_instance.stubs(:get).returns(stub(:plain_body => get_record_xml(:contact), :code => '200'))
       contact = @client.Contact.find('TESTID')
       assert_kind_of(Xeroizer::Record::Contact, contact)
       assert_equal(false, contact.new_record?)
     end
 
     should "new_record? should be false after successfully creating a record" do
-      Xeroizer::OAuth.any_instance.stubs(:put).returns(stub(:plain_body => get_record_xml(:contact), :code => '200'))
+      Xeroizer::OAuth2.any_instance.stubs(:put).returns(stub(:plain_body => get_record_xml(:contact), :code => '200'))
       assert_equal(true, @contact.new_record?)
       assert_nil(@contact.contact_id)
       assert_equal(true, @contact.save, "Error saving contact: #{@contact.errors.inspect}")
@@ -104,6 +105,16 @@ class RecordBaseTest < Test::Unit::TestCase
     end
   end
 
+  context 'build' do
+
+    should "raise an undefined method error with useful message" do
+      assert_raise_message("undefined method `this_method_does_not_exist=' for #<Xeroizer::Record::Contact >") do
+        @client.Contact.build(:this_method_does_not_exist =>  true)
+      end
+    end
+
+  end
+
   context 'saving' do
     context 'invalid record' do
       setup do

data/test/unit/record/block_validator_test.rb

@@ -1,4 +1,4 @@
-require "test_helper"
+require 'unit_test_helper'
 
 class BlockValidatorTest < Test::Unit::TestCase
   def setup

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

@@ -1,4 +1,4 @@
-require 'test_helper'
+require 'unit_test_helper'
 
 module Xeroizer
   module Record
@@ -27,7 +27,7 @@ class ParseParamsTest < 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::ParseParamTestModel.new(@client, "ParseParamTest")
   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