Add Status to Tax Rates

Updated the fixture to use the example provided in https://developer.xero.com/documentation/api/tax-rates
xero-gateway · Jul 12, 2017 · a690f76 · a690f76
1 parent fc55955
commit a690f76
Show file tree

Hide file tree

Showing 4 changed files with 54 additions and 39 deletions.
diff --git a/lib/xero_gateway/tax_rate.rb b/lib/xero_gateway/tax_rate.rb
@@ -3,6 +3,7 @@ class TaxRate < BaseRecord
     attributes({
       "Name"                  => :string,
       "TaxType"               => :string,
+      "Status"                => :string,
       "CanApplyToAssets"      => :boolean,
       "CanApplyToEquity"      => :boolean,
       "CanApplyToExpenses"    => :boolean,

diff --git a/test/integration/get_tax_rates_test.rb b/test/integration/get_tax_rates_test.rb
@@ -2,24 +2,24 @@
 
 class GetTaxRatesTest < Test::Unit::TestCase
   include TestHelper
-  
+
   def setup
     @gateway = XeroGateway::Gateway.new(CONSUMER_KEY, CONSUMER_SECRET)
-    
+
     if STUB_XERO_CALLS
       @gateway.xero_url = "DUMMY_URL"
-      
-      @gateway.stubs(:http_get).with {|client, url, params| url =~ /TaxRates$/ }.returns(get_file_as_string("tax_rates.xml"))          
+
+      @gateway.stubs(:http_get).with {|client, url, params| url =~ /TaxRates$/ }.returns(get_file_as_string("tax_rates.xml"))
     end
   end
-  
+
   def test_get_tax_rates
     result = @gateway.get_tax_rates
     assert result.success?
     assert !result.response_xml.nil?
-    
+
     assert result.tax_rates.size > 0
     assert_equal XeroGateway::TaxRate, result.tax_rates.first.class
-    assert_equal "GST on Expenses", result.tax_rates.first.name
+    assert_equal "15% GST on Expenses", result.tax_rates.first.name
   end
-end
+end
diff --git a/test/stub_responses/tax_rates.xml b/test/stub_responses/tax_rates.xml
@@ -5,48 +5,61 @@
   <DateTimeUTC>2009-10-04T03:29:54.6788414Z</DateTimeUTC>
   <TaxRates>
     <TaxRate>
-      <Name>GST on Expenses</Name>
-      <TaxType>INPUT</TaxType>
+      <Name>15% GST on Expenses</Name>
+      <TaxType>INPUT2</TaxType>
       <CanApplyToAssets>true</CanApplyToAssets>
       <CanApplyToEquity>true</CanApplyToEquity>
       <CanApplyToExpenses>true</CanApplyToExpenses>
       <CanApplyToLiabilities>true</CanApplyToLiabilities>
       <CanApplyToRevenue>false</CanApplyToRevenue>
-      <DisplayTaxRate>12.5000</DisplayTaxRate>
-      <EffectiveRate>12.5000</EffectiveRate>
+      <DisplayTaxRate>15.0000</DisplayTaxRate>
+      <EffectiveRate>15.0000</EffectiveRate>
+      <Status>ACTIVE</Status>
+      <TaxComponents>
+        <TaxComponent>
+          <Name>GST</Name>
+          <Rate>15.0000</Rate>
+          <IsCompound>false</IsCompound>
+        </TaxComponent>
+      </TaxComponents>
     </TaxRate>
     <TaxRate>
-      <Name>GST on Income</Name>
-      <TaxType>OUTPUT</TaxType>
+      <Name>15% GST on Income</Name>
+      <TaxType>OUTPUT2</TaxType>
       <CanApplyToAssets>true</CanApplyToAssets>
       <CanApplyToEquity>true</CanApplyToEquity>
       <CanApplyToExpenses>false</CanApplyToExpenses>
       <CanApplyToLiabilities>true</CanApplyToLiabilities>
       <CanApplyToRevenue>true</CanApplyToRevenue>
-      <DisplayTaxRate>12.5000</DisplayTaxRate>
-      <EffectiveRate>12.5000</EffectiveRate>
+      <DisplayTaxRate>15.0000</DisplayTaxRate>
+      <EffectiveRate>15.0000</EffectiveRate>
+      <Status>ACTIVE</Status>
+      <TaxComponents>
+        <TaxComponent>
+          <Name>GST</Name>
+          <Rate>15.0000</Rate>
+          <IsCompound>false</IsCompound>
+        </TaxComponent>
+      </TaxComponents>
     </TaxRate>
     <TaxRate>
-      <Name>No GST</Name>
-      <TaxType>NONE</TaxType>
-      <CanApplyToAssets>true</CanApplyToAssets>
-      <CanApplyToEquity>true</CanApplyToEquity>
-      <CanApplyToExpenses>true</CanApplyToExpenses>
-      <CanApplyToLiabilities>true</CanApplyToLiabilities>
-      <CanApplyToRevenue>true</CanApplyToRevenue>
-      <DisplayTaxRate>0.0000</DisplayTaxRate>
-      <EffectiveRate>0.0000</EffectiveRate>
-    </TaxRate>
-    <TaxRate>
-      <Name>Zero Rated</Name>
-      <TaxType>ZERORATED</TaxType>
+      <Name>GST on Imports</Name>
+      <TaxType>GSTONIMPORTS</TaxType>
       <CanApplyToAssets>false</CanApplyToAssets>
       <CanApplyToEquity>false</CanApplyToEquity>
       <CanApplyToExpenses>false</CanApplyToExpenses>
-      <CanApplyToLiabilities>false</CanApplyToLiabilities>
-      <CanApplyToRevenue>true</CanApplyToRevenue>
+      <CanApplyToLiabilities>true</CanApplyToLiabilities>
+      <CanApplyToRevenue>false</CanApplyToRevenue>
       <DisplayTaxRate>0.0000</DisplayTaxRate>
       <EffectiveRate>0.0000</EffectiveRate>
+      <Status>ACTIVE</Status>
+      <TaxComponents>
+        <TaxComponent>
+          <Name>GST</Name>
+          <Rate>0.0000</Rate>
+          <IsCompound>false</IsCompound>
+        </TaxComponent>
+      </TaxComponents>
     </TaxRate>
   </TaxRates>
-</Response>
+</Response>
diff --git a/test/unit/tax_rate_test.rb b/test/unit/tax_rate_test.rb
@@ -1,11 +1,11 @@
 require File.join(File.dirname(__FILE__), '../test_helper.rb')
 
 class TaxRateTest < Test::Unit::TestCase
-  
+
   # Tests that a tax rate can be converted into XML that Xero can understand, and then converted back to a tax rate
   def test_build_and_parse_xml
     tax_rate = create_test_tax_rate
-    
+
     # Generate the XML message
     tax_rate_as_xml = tax_rate.to_xml
 
@@ -14,18 +14,19 @@ def test_build_and_parse_xml
 
     # Build a new account from the XML
     result_tax_rate = XeroGateway::TaxRate.from_xml(tax_rate_element)
-    
+
     # Check the account details
     assert_equal tax_rate, result_tax_rate
   end
-  
-  
+
+
   private
-  
+
   def create_test_tax_rate
     XeroGateway::TaxRate.new.tap do |tax_rate|
        tax_rate.name = "GST on Expenses"
        tax_rate.tax_type = "INPUT"
+       tax_rate.status = "ACTIVE"
        tax_rate.can_apply_to_assets      = true
        tax_rate.can_apply_to_equity      = true
        tax_rate.can_apply_to_expenses    = true
@@ -35,4 +36,4 @@ def create_test_tax_rate
        tax_rate.effective_rate           = 12.500
     end
   end
-end
+end