xeroizer 0.3.5 → 0.4.0

data/test/unit/oauth_config_test.rb

@@ -0,0 +1,20 @@
+require "test_helper"
+
+class OAuthConfigTest < Test::Unit::TestCase
+  include Xeroizer
+
+  must "load oauth configuration from yaml" do
+    the_yaml = "
+    consumer:
+        key: key
+        secret: secret
+        key_file: key_file
+    "
+
+    result = OAuthConfig.load the_yaml
+
+    assert_equal "key",      result.consumer_key,    "Unexpected consumer_key value"
+    assert_equal "secret",   result.consumer_secret, "Unexpected consumer_secret value"
+    assert_equal "key_file", result.key_file,        "Unexpected key_file value"
+  end
+end

data/test/unit/oauth_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../test_helper.rb')
+require 'test_helper'
 
 class OAuthTest < Test::Unit::TestCase
   include TestHelper

data/test/unit/private_application_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../test_helper.rb')
+require 'test_helper'
 
 class PrivateApplicationTest < Test::Unit::TestCase
   include TestHelper
@@ -17,4 +17,4 @@ class PrivateApplicationTest < Test::Unit::TestCase
     
   end
     
-end
+end

data/test/unit/record/base_model_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../../test_helper.rb')
+require 'test_helper'
 
 class RecordBaseModelTest < Test::Unit::TestCase
   include TestHelper
@@ -47,4 +47,4 @@ class RecordBaseModelTest < Test::Unit::TestCase
     
   end
 
-end
+end

data/test/unit/record/base_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../../test_helper.rb')
+require 'test_helper'
 
 class RecordBaseTest < Test::Unit::TestCase
   include TestHelper
@@ -54,4 +54,41 @@ class RecordBaseTest < Test::Unit::TestCase
     
   end
   
+  context "about logging" do
+    setup do
+      @example_class = Class.new(Xeroizer::Record::Base) do
+        def valid?; true; end
+        def to_xml(b = nil); "<FakeRequest />" end
+        string :id
+      end
+    end
+
+    must "log the request and response xml when saving a new record" do
+      Xeroizer::Logging::Log.expects(:info).once.with {|arg| arg =~ /\[CREATE SENT\]/}
+      Xeroizer::Logging::Log.expects(:info).once.with {|arg| arg =~ /\[CREATE RECEIVED\]/}
+
+      a_fake_parent = mock "Mock parent",
+        :http_put => "<FakeResponse />",
+        :parse_response => stub("Stub response", :response_items => [])
+
+      an_example_instance = @example_class.new(a_fake_parent)
+
+      an_example_instance.id = nil
+      an_example_instance.save
+    end
+
+    must "log the request and response xml when updating an existing record" do
+      Xeroizer::Logging::Log.expects(:info).once.with {|arg| arg =~ /\[UPDATE SENT\]/}
+      Xeroizer::Logging::Log.expects(:info).once.with {|arg| arg =~ /\[UPDATE RECEIVED\]/}
+
+      a_fake_parent = mock "Mock parent",
+        :http_post => "<FakeResponse />",
+        :parse_response => stub("Stub response", :response_items => [])
+
+      an_example_instance = @example_class.new(a_fake_parent)
+
+      an_example_instance.id = "phil's lunch box"
+      an_example_instance.save
+    end
+  end
 end

data/test/unit/record/block_validator_test.rb

@@ -0,0 +1,125 @@
+require "test_helper"
+
+class BlockValidatorTest < Test::Unit::TestCase
+  def setup
+    @fake_record_class = Class.new do
+      attr_accessor :errors
+      def initialize; @errors = []; end
+    end
+  end
+
+  it "returns valid when block returns true" do
+    record = @fake_record_class.new
+
+    the_block_returning_true = Proc.new{ true }
+
+    block_validator = Xeroizer::Record::Validator::BlockValidator.new(
+      :name,
+      :block => the_block_returning_true
+    )
+
+    block_validator.valid?(record)
+
+    assert_empty record.errors, "Expected validation to pass. #{record.errors.inspect}"
+  end
+
+  it "returns invalid when block returns a non-null object" do
+    record = @fake_record_class.new
+
+    the_block_returning_true = Proc.new{ Object }
+
+    block_validator = Xeroizer::Record::Validator::BlockValidator.new(
+      :name,
+      :block => the_block_returning_true
+    )
+
+    block_validator.valid?(record)
+
+    assert_equal 1, record.errors.size, "Expected validation to fail with one error. #{record.errors.inspect}"
+  end
+
+  it "returns invalid when block returns false" do
+    record = @fake_record_class.new
+
+    the_block_returning_false = Proc.new{ false }
+
+    block_validator = Xeroizer::Record::Validator::BlockValidator.new(
+      :name,
+      {
+        :block => the_block_returning_false
+      }
+    )
+
+    block_validator.valid?(record)
+
+    assert_equal 1, record.errors.size, "Expected validation to fail with one error. #{record.errors.inspect}"
+  end
+
+  it "uses the message supplied when validation fails" do
+    record = @fake_record_class.new
+
+    the_block_returning_false = Proc.new{ false }
+
+    expected_error = "Cornrows are usually a headwear mistake"
+
+    block_validator = Xeroizer::Record::Validator::BlockValidator.new(
+      :name,
+      {
+        :block => the_block_returning_false,
+        :message => expected_error
+      }
+    )
+
+    block_validator.valid?(record)
+
+    assert_equal 1, record.errors.size,
+      "Expected validation to fail with one error. #{record.errors.inspect}"
+    assert_equal expected_error, record.errors.first[1],
+      "There is an error, but it doesn't match"
+  end
+
+  it "uses a default message when validation fails and no message has been supplied" do
+    record = @fake_record_class.new
+
+    the_block_returning_false = Proc.new{ false }
+
+    expected_error = "block condition failed"
+
+    block_validator = Xeroizer::Record::Validator::BlockValidator.new(
+      :name,
+      {
+        :block => the_block_returning_false
+      }
+    )
+
+    block_validator.valid?(record)
+
+    assert_equal 1, record.errors.size,
+      "Expected validation to fail with one error. #{record.errors.inspect}"
+    assert_equal expected_error, record.errors.first[1],
+      "There is an error, but it doesn't match"
+  end
+
+  it "uses a default message when validation fails and message has been supplied as empty" do
+    record = @fake_record_class.new
+
+    the_block_returning_false = Proc.new{ false }
+
+    expected_error = "block condition failed"
+
+    block_validator = Xeroizer::Record::Validator::BlockValidator.new(
+      :name,
+      {
+        :block => the_block_returning_false,
+        :message => ""
+      }
+    )
+
+    block_validator.valid?(record)
+
+    assert_equal 1, record.errors.size,
+      "Expected validation to fail with one error. #{record.errors.inspect}"
+    assert_equal expected_error, record.errors.first[1],
+      "There is an error, but it doesn't match"
+  end
+end

data/test/unit/record/model_definition_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../../test_helper.rb')
+require 'test_helper'
 
 class ModelDefinitionsTest < Test::Unit::TestCase
   include TestHelper
@@ -156,4 +156,4 @@ class ModelDefinitionsTest < Test::Unit::TestCase
     
   end
   
-end
+end

data/test/unit/record/parse_where_hash_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../../test_helper.rb')
+require 'test_helper'
 
 module Xeroizer
   module Record
@@ -60,4 +60,4 @@ class ParseWhereHashTest < Test::Unit::TestCase
     end
   end
 
-end
+end

data/test/unit/record/record_association_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../../test_helper.rb')
+require 'test_helper'
 
 class RecordAssociationTest < Test::Unit::TestCase
   include TestHelper

data/test/unit/record/validators_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../../test_helper.rb')
+require 'test_helper'
 
 class ValidatorsTest < Test::Unit::TestCase
   include TestHelper
@@ -11,6 +11,8 @@ class ValidatorsTest < Test::Unit::TestCase
     string  :name
     string  :name_conditional_if
     string  :name_conditional_unless
+    string  :name_conditional_method_if
+    string  :name_conditional_method_unless
     string  :type
     string  :type_blank
     integer :value
@@ -21,10 +23,20 @@ class ValidatorsTest < Test::Unit::TestCase
     validates_presence_of :name, :message => "blank"
     validates_presence_of :name_conditional_if, :if => Proc.new { | record | record.value == 10 }, :message => "blank_if_10"
     validates_presence_of :name_conditional_unless, :unless => Proc.new { | record | record.value == 20 }, :message => "blank_unless_20"
+    validates_presence_of :name_conditional_method_if, :if => :value_equals_ten?, :message => "blank_if_10"
+    validates_presence_of :name_conditional_method_unless, :unless => :value_equals_twenty?, :message => "blank_unless_20"
     validates_inclusion_of :type, :in => %w(phone fax mobile), :message => "not_included"
     validates_inclusion_of :type_blank, :in => %w(phone fax mobile), :message => "not_included_blank", :allow_blanks => true
     validates_associated :contact, :message => "association_invalid"
     validates_associated :addresses, :message => "association_invalid_blank", :allow_blanks => true
+    
+    def value_equals_ten?
+      value == 10
+    end
+    
+    def value_equals_twenty?
+      value == 20
+    end
   end
   
   def setup
@@ -157,6 +169,24 @@ class ValidatorsTest < Test::Unit::TestCase
       assert_nil(error)
     end
     
+    should "have name if value_equals_ten?" do
+      @record.value = 10
+      assert_equal(false, @record.valid?)
+      error = @record.errors_for(:name_conditional_method_if).first
+      assert_equal('blank_if_10', error)
+      
+      @record.name_conditional_method_if = "NOT BLANK"
+      @record.valid?
+      error = @record.errors_for(:name_conditional_method_if).first
+      assert_nil(error)
+            
+      @record.name_conditional_method_if = nil
+      @record.value = 50
+      @record.valid?
+      error = @record.errors_for(:name_conditional_method_if).first
+      assert_nil(error)
+    end
+    
     should "have name unless value is 20" do
       @record.value = 50
       assert_equal(false, @record.valid?)
@@ -174,7 +204,25 @@ class ValidatorsTest < Test::Unit::TestCase
       error = @record.errors_for(:name_conditional_unless).first
       assert_nil(error)
     end
-        
+    
+    should "have name unless value_equals_twenty?" do
+      @record.value = 50
+      assert_equal(false, @record.valid?)
+      error = @record.errors_for(:name_conditional_method_unless).first
+      assert_equal('blank_unless_20', error)
+      
+      @record.name_conditional_method_unless = "NOT BLANK"
+      @record.valid?
+      error = @record.errors_for(:name_conditional_method_unless).first
+      assert_nil(error)
+            
+      @record.name_conditional_method_unless = nil
+      @record.value = 20
+      @record.valid?
+      error = @record.errors_for(:name_conditional_method_unless).first
+      assert_nil(error)
+    end
+    
   end
   
-end
+end

data/test/unit/record_definition_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../test_helper.rb')
+require 'test_helper'
 
 class RecordDefinitionTest < Test::Unit::TestCase
   include TestHelper
@@ -24,4 +24,4 @@ class RecordDefinitionTest < Test::Unit::TestCase
     
   end
 
-end
+end

data/test/unit/report_definition_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../test_helper.rb')
+require 'test_helper'
 
 class ReportDefinitionTest < Test::Unit::TestCase
   include TestHelper
@@ -23,4 +23,4 @@ class ReportDefinitionTest < Test::Unit::TestCase
     
   end
 
-end
+end

data/test/unit/report_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../test_helper.rb')
+require 'test_helper'
 
 class FactoryTest < Test::Unit::TestCase
   include TestHelper

data/xeroizer.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{xeroizer}
-  s.version = "0.3.5"
+  s.version = "0.4.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Wayne Robinson"]
-  s.date = %q{2011-06-27}
+  s.date = %q{2012-02-10}
   s.description = %q{Ruby library for the Xero accounting system API.}
   s.email = %q{wayne.robinson@gmail.com}
   s.extra_rdoc_files = [
@@ -23,18 +23,21 @@ Gem::Specification.new do |s|
     "README.md",
     "Rakefile",
     "VERSION",
-    "lib/.DS_Store",
     "lib/big_decimal_to_s.rb",
     "lib/class_level_inheritable_attributes.rb",
     "lib/xeroizer.rb",
     "lib/xeroizer/application_http_proxy.rb",
     "lib/xeroizer/ca-certificates.crt",
+    "lib/xeroizer/configuration.rb",
     "lib/xeroizer/exceptions.rb",
     "lib/xeroizer/generic_application.rb",
     "lib/xeroizer/http.rb",
     "lib/xeroizer/http_encoding_helper.rb",
+    "lib/xeroizer/logging.rb",
     "lib/xeroizer/models/account.rb",
     "lib/xeroizer/models/address.rb",
+    "lib/xeroizer/models/bank_account.rb",
+    "lib/xeroizer/models/bank_transaction.rb",
     "lib/xeroizer/models/branding_theme.rb",
     "lib/xeroizer/models/contact.rb",
     "lib/xeroizer/models/contact_group.rb",
@@ -44,10 +47,13 @@ Gem::Specification.new do |s|
     "lib/xeroizer/models/external_link.rb",
     "lib/xeroizer/models/invoice.rb",
     "lib/xeroizer/models/item.rb",
-    "lib/xeroizer/models/item_purchase_sale_details.rb",
+    "lib/xeroizer/models/item_purchase_details.rb",
+    "lib/xeroizer/models/item_sales_details.rb",
     "lib/xeroizer/models/journal.rb",
     "lib/xeroizer/models/journal_line.rb",
+    "lib/xeroizer/models/line_amount_type.rb",
     "lib/xeroizer/models/line_item.rb",
+    "lib/xeroizer/models/line_item_sum.rb",
     "lib/xeroizer/models/manual_journal.rb",
     "lib/xeroizer/models/manual_journal_line.rb",
     "lib/xeroizer/models/option.rb",
@@ -69,6 +75,7 @@ Gem::Specification.new do |s|
     "lib/xeroizer/record/record_association_helper.rb",
     "lib/xeroizer/record/validation_helper.rb",
     "lib/xeroizer/record/validators/associated_validator.rb",
+    "lib/xeroizer/record/validators/block_validator.rb",
     "lib/xeroizer/record/validators/inclusion_of_validator.rb",
     "lib/xeroizer/record/validators/presence_of_validator.rb",
     "lib/xeroizer/record/validators/validator.rb",
@@ -85,6 +92,10 @@ Gem::Specification.new do |s|
     "lib/xeroizer/report/row/xml_helper.rb",
     "lib/xeroizer/report/xml_helper.rb",
     "lib/xeroizer/response.rb",
+    "test/acceptance/about_creating_bank_transactions_test.rb",
+    "test/acceptance/about_fetching_bank_transactions_test.rb",
+    "test/acceptance/acceptance_test.rb",
+    "test/acceptance/bank_transaction_reference_data.rb",
     "test/stub_responses/accounts.xml",
     "test/stub_responses/api_exception.xml",
     "test/stub_responses/bogus_oauth_error",
@@ -316,13 +327,20 @@ Gem::Specification.new do |s|
     "test/stub_responses/tracking_categories.xml",
     "test/stub_responses/unknown_error.xml",
     "test/test_helper.rb",
+    "test/unit/models/bank_transaction_model_parsing_test.rb",
+    "test/unit/models/bank_transaction_test.rb",
+    "test/unit/models/bank_transaction_validation_test.rb",
     "test/unit/models/contact_test.rb",
     "test/unit/models/credit_note_test.rb",
     "test/unit/models/invoice_test.rb",
+    "test/unit/models/line_item_sum_test.rb",
+    "test/unit/models/line_item_test.rb",
+    "test/unit/oauth_config_test.rb",
     "test/unit/oauth_test.rb",
     "test/unit/private_application_test.rb",
     "test/unit/record/base_model_test.rb",
     "test/unit/record/base_test.rb",
+    "test/unit/record/block_validator_test.rb",
     "test/unit/record/model_definition_test.rb",
     "test/unit/record/parse_where_hash_test.rb",
     "test/unit/record/record_association_test.rb",
@@ -335,17 +353,49 @@ Gem::Specification.new do |s|
   s.homepage = %q{http://github.com/waynerobinson/xeroizer}
   s.licenses = ["MIT"]
   s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.5.0}
+  s.rubygems_version = %q{1.3.6}
   s.summary = %q{Xero library}
+  s.test_files = [
+    "test/acceptance/about_creating_bank_transactions_test.rb",
+    "test/acceptance/about_fetching_bank_transactions_test.rb",
+    "test/acceptance/acceptance_test.rb",
+    "test/acceptance/bank_transaction_reference_data.rb",
+    "test/stub_responses/refresh_responses.rb",
+    "test/test_helper.rb",
+    "test/unit/models/bank_transaction_model_parsing_test.rb",
+    "test/unit/models/bank_transaction_test.rb",
+    "test/unit/models/bank_transaction_validation_test.rb",
+    "test/unit/models/contact_test.rb",
+    "test/unit/models/credit_note_test.rb",
+    "test/unit/models/invoice_test.rb",
+    "test/unit/models/line_item_sum_test.rb",
+    "test/unit/models/line_item_test.rb",
+    "test/unit/oauth_config_test.rb",
+    "test/unit/oauth_test.rb",
+    "test/unit/private_application_test.rb",
+    "test/unit/record/base_model_test.rb",
+    "test/unit/record/base_test.rb",
+    "test/unit/record/block_validator_test.rb",
+    "test/unit/record/model_definition_test.rb",
+    "test/unit/record/parse_where_hash_test.rb",
+    "test/unit/record/record_association_test.rb",
+    "test/unit/record/validators_test.rb",
+    "test/unit/record_definition_test.rb",
+    "test/unit/report_definition_test.rb",
+    "test/unit/report_test.rb"
+  ]
 
   if s.respond_to? :specification_version then
+    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
     s.specification_version = 3
 
-    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
+    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<builder>, [">= 2.1.2"])
       s.add_runtime_dependency(%q<oauth>, [">= 0.3.6"])
       s.add_runtime_dependency(%q<activesupport>, [">= 0"])
       s.add_runtime_dependency(%q<nokogiri>, [">= 0"])
+      s.add_runtime_dependency(%q<i18n>, [">= 0"])
+      s.add_runtime_dependency(%q<yard>, [">= 0"])
       s.add_runtime_dependency(%q<builder>, [">= 2.1.2"])
       s.add_runtime_dependency(%q<oauth>, [">= 0.3.6"])
       s.add_runtime_dependency(%q<activesupport>, [">= 0"])
@@ -357,6 +407,8 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<oauth>, [">= 0.3.6"])
       s.add_dependency(%q<activesupport>, [">= 0"])
       s.add_dependency(%q<nokogiri>, [">= 0"])
+      s.add_dependency(%q<i18n>, [">= 0"])
+      s.add_dependency(%q<yard>, [">= 0"])
       s.add_dependency(%q<builder>, [">= 2.1.2"])
       s.add_dependency(%q<oauth>, [">= 0.3.6"])
       s.add_dependency(%q<activesupport>, [">= 0"])
@@ -369,6 +421,8 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<oauth>, [">= 0.3.6"])
     s.add_dependency(%q<activesupport>, [">= 0"])
     s.add_dependency(%q<nokogiri>, [">= 0"])
+    s.add_dependency(%q<i18n>, [">= 0"])
+    s.add_dependency(%q<yard>, [">= 0"])
     s.add_dependency(%q<builder>, [">= 2.1.2"])
     s.add_dependency(%q<oauth>, [">= 0.3.6"])
     s.add_dependency(%q<activesupport>, [">= 0"])