solidus_avatax 0.2.0

data/spec/models/spree_avatax/sales_shared_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe SpreeAvatax::SalesShared do
+
+  describe '.reset_tax_attributes' do
+    subject do
+      SpreeAvatax::SalesShared.reset_tax_attributes(order)
+    end
+
+    let(:order) { create(:order_with_line_items, additional_tax_total: 1, adjustment_total: 1, included_tax_total: 1, line_items_count: 1) }
+    let(:line_item) { order.line_items.first }
+
+    before do
+      line_item.adjustments.eligible.tax.additional.create!({
+        adjustable: line_item,
+        amount: 1.23,
+        order: order,
+        label: 'Previous Tax',
+        included: false,
+        finalized: true,
+      })
+
+      line_item.update_attributes!({
+        additional_tax_total: 1,
+        adjustment_total: 1,
+        pre_tax_amount: 1,
+        included_tax_total: 1,
+      })
+    end
+
+    context 'when order is completed' do
+      before do
+        order.update_attributes!(completed_at: Time.now)
+      end
+
+      it 'should leave adjustments in place' do
+        subject
+        expect(line_item.adjustments.tax.count).to eq 1
+      end
+    end
+
+    it 'should remove all eligible tax adjustments' do
+      subject
+      expect(line_item.adjustments.tax.count).to eq 0
+    end
+  end
+end

data/spec/models/spree_avatax/shared_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+
+describe SpreeAvatax::Shared do
+  describe '.logger' do
+    subject { SpreeAvatax::Shared.logger }
+    it { is_expected.to be_a Logger }
+  end
+
+  describe '.taxable_order?' do
+    subject { SpreeAvatax::Shared.taxable_order?(order) }
+
+    context 'when the order is taxable' do
+      let(:order) { create(:shipped_order, line_items_count: 1) }
+      it { is_expected.to be true }
+    end
+
+    context 'when the order has no line items' do
+      let(:order) { create(:order) }
+      it { is_expected.to be false }
+    end
+
+    context 'when the order has no ship address' do
+      let(:order) { create(:order_with_totals, ship_address: nil) }
+      it { is_expected.to be false }
+    end
+  end
+
+  describe '.tax_svc' do
+    subject { SpreeAvatax::Shared.tax_svc }
+    it { is_expected.to be_a AvaTax::TaxService }
+  end
+
+  describe '.require_success!' do
+    subject do
+      SpreeAvatax::Shared.require_success!(response)
+    end
+
+    context 'when the response is a success' do
+      let(:response) { {transaction_id: "4315047353885220", result_code: 'Success'} }
+
+      it 'does not raise' do
+        expect { subject }.not_to raise_error
+      end
+    end
+
+    context 'when the response if a failure' do
+      # See notes in shared.rb about the two formats
+      context 'format 1' do
+        let(:response) do
+          {
+            transaction_id: "4315046676197187",
+            result_code: 'Error',
+            messages: {
+              message: {summary: 'Something awful happened.'},
+            },
+          }
+        end
+
+        it 'raises a FailedApiResponse' do
+          expect { subject }.to(raise_error { |error|
+            expect(error).to be_a(SpreeAvatax::Shared::FailedApiResponse)
+            expect(error.message).to eq('["Something awful happened."]')
+            expect(error.messages).to eq([{summary: 'Something awful happened.'}])
+          })
+        end
+      end
+
+      context 'format 2' do
+        let(:response) do
+          {
+            transaction_id: "4315046676197187",
+            result_code: 'Error',
+            messages: [
+              {summary: 'Something awful happened.'},
+            ],
+          }
+        end
+
+        it 'raises a FailedApiResponse' do
+          expect { subject }.to(raise_error { |error|
+            expect(error).to be_a(SpreeAvatax::Shared::FailedApiResponse)
+            expect(error.message).to eq('["Something awful happened."]')
+            expect(error.messages).to eq([{summary: 'Something awful happened.'}])
+          })
+        end
+      end
+    end
+  end
+end

data/spec/models/spree_avatax/short_ship_return_invoice_spec.rb

@@ -0,0 +1,181 @@
+require 'spec_helper'
+
+describe SpreeAvatax::ShortShipReturnInvoice do
+  include ShortShipReturnInvoiceSoapResponses
+
+  describe '.generate' do
+    before do
+      # disable avatax tax calculation. we're not testing calculating tax.
+      allow(SpreeAvatax::Shared).to receive(:taxable_order?).and_return(false)
+      allow(SpreeAvatax::SalesShared).to receive(:reset_tax_attributes)
+    end
+
+    let(:order) do
+      create(
+        :order_with_line_items,
+        line_items_price: 10,
+        line_items_count: 2,
+      )
+    end
+
+    def line_item_1
+      order.line_items.first
+    end
+
+    def line_item_2
+      order.line_items.last
+    end
+
+    def inventory_unit_1
+      line_item_1.inventory_units.first
+    end
+
+    def inventory_unit_2
+      line_item_2.inventory_units.first
+    end
+
+    # add some tax and complete the order
+    before do
+      line_item_1.adjustments.create!(
+        amount: 1,
+        label: 'fake tax',
+        order: order,
+        source: Spree::TaxRate.avatax_the_one_rate,
+        finalized: true,
+      )
+      order.contents.advance
+      create(:payment, amount: order.total, order: order)
+      order.contents.advance
+      order.complete!
+      order.reload
+    end
+
+    describe 'gettax params' do
+      let(:now) { Time.now }
+
+      around do |example|
+        Timecop.freeze(now) do
+          example.run
+        end
+      end
+
+      def expected_gettax_params
+        {
+          doccode:       "#{order.number}-short-#{now.to_f}",
+          referencecode: order.number,
+          customercode:  order.user_id,
+          companycode:   SpreeAvatax::Config.company_code,
+
+          doctype: SpreeAvatax::ShortShipReturnInvoice::DOC_TYPE,
+          docdate: now.to_date,
+
+          commit: true,
+
+          addresses: [
+            {
+              addresscode: SpreeAvatax::ShortShipReturnInvoice::DESTINATION_CODE,
+              line1:       REXML::Text.normalize(order.ship_address.address1),
+              line2:       REXML::Text.normalize(order.ship_address.address2),
+              city:        REXML::Text.normalize(order.ship_address.city),
+              postalcode:  REXML::Text.normalize(order.ship_address.zipcode),
+            },
+          ],
+
+          lines: [
+            {
+              no:                  inventory_unit_1.id,
+              itemcode:            inventory_unit_1.line_item.variant.sku,
+              taxcode:             inventory_unit_1.line_item.tax_category.tax_code,
+              qty:                 1,
+              amount:              -10.to_d,
+              origincodeline:      SpreeAvatax::ShortShipReturnInvoice::DESTINATION_CODE,
+              destinationcodeline: SpreeAvatax::ShortShipReturnInvoice::DESTINATION_CODE,
+
+              taxoverridetypeline: SpreeAvatax::ShortShipReturnInvoice::TAX_OVERRIDE_TYPE,
+              reasonline:          SpreeAvatax::ShortShipReturnInvoice::TAX_OVERRIDE_REASON,
+              taxamountline:       -1.to_d,
+              taxdateline:         now.to_date,
+
+              description: REXML::Text.normalize(inventory_unit_1.line_item.variant.product.description[0...100]),
+            },
+            {
+              no:                  inventory_unit_2.id,
+              itemcode:            inventory_unit_2.line_item.variant.sku,
+              taxcode:             inventory_unit_2.line_item.tax_category.tax_code,
+              qty:                 1,
+              amount:              -10.to_d,
+              origincodeline:      SpreeAvatax::ShortShipReturnInvoice::DESTINATION_CODE,
+              destinationcodeline: SpreeAvatax::ShortShipReturnInvoice::DESTINATION_CODE,
+
+              taxoverridetypeline: SpreeAvatax::ShortShipReturnInvoice::TAX_OVERRIDE_TYPE,
+              reasonline:          SpreeAvatax::ShortShipReturnInvoice::TAX_OVERRIDE_REASON,
+              taxamountline:       0.to_d,
+              taxdateline:         now.to_date,
+
+              description: REXML::Text.normalize(inventory_unit_2.line_item.variant.product.description[0...100]),
+            },
+          ],
+        }
+      end
+
+      let(:unit_cancels) do
+        Spree::OrderCancellations.new(order).short_ship(order.inventory_units)
+      end
+
+      it 'generates the expected params for avatax' do
+        expect(SpreeAvatax::Shared.tax_svc).to(
+          receive(:gettax).
+          with(expected_gettax_params).
+          and_return(short_ship_return_invoice_gettax_response)
+        )
+
+        SpreeAvatax::ShortShipReturnInvoice.generate(unit_cancels: unit_cancels)
+      end
+    end
+
+    context 'with a successful response' do
+      let(:unit_cancels) do
+        Spree::OrderCancellations.new(order).short_ship(order.inventory_units)
+      end
+
+      before do
+        expect(SpreeAvatax::Shared.tax_svc).to(
+          receive(:gettax).
+          and_return(short_ship_return_invoice_gettax_response)
+        )
+      end
+
+      it 'creates a return invoice with the correct inventory units' do
+        expect {
+          SpreeAvatax::ShortShipReturnInvoice.generate(unit_cancels: unit_cancels)
+        }.to change { SpreeAvatax::ShortShipReturnInvoice.count }.by(1)
+
+        short_ship_return_invoice = SpreeAvatax::ShortShipReturnInvoice.last
+
+        expect(short_ship_return_invoice.inventory_units).to eq(
+          [inventory_unit_1, inventory_unit_2]
+        )
+      end
+    end
+
+    context 'with inventory units from multiple orders' do
+      let(:order2) { create(:order_ready_to_ship) }
+
+      let(:unit_cancels1) do
+        Spree::OrderCancellations.new(order).short_ship([inventory_unit_1])
+      end
+      let(:unit_cancels2) do
+        Spree::OrderCancellations.new(order2).short_ship(order2.inventory_units)
+      end
+
+      it 'fails' do
+        expect {
+          SpreeAvatax::ShortShipReturnInvoice.generate(
+            unit_cancels: unit_cancels1 + unit_cancels2
+          )
+        }.to raise_error(/more than one order/)
+      end
+    end
+
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,85 @@
+# Run Coverage report
+require "simplecov"
+SimpleCov.start do
+  add_filter "spec/dummy"
+  add_group "Controllers", "app/controllers"
+  add_group "Helpers", "app/helpers"
+  add_group "Mailers", "app/mailers"
+  add_group "Models", "app/models"
+  add_group "Views", "app/views"
+  add_group "Libraries", "lib"
+end
+
+# Configure Rails Environment
+ENV["RAILS_ENV"] = "test"
+
+require File.expand_path("../dummy/config/environment.rb",  __FILE__)
+
+require "rspec/rails"
+
+require "database_cleaner"
+require "ffaker"
+
+require "spree/testing_support/factories"
+require "spree/testing_support/controller_requests"
+require "spree/testing_support/authorization_helpers"
+require "spree/testing_support/url_helpers"
+
+require 'spree_avatax/factories'
+
+require 'avatax_taxservice'
+require 'hashie'
+require 'vcr'
+
+Dir[File.join(File.dirname(__FILE__), "support/**/*.rb")].each { |f| require f }
+
+VCR.configure do |c|
+  c.cassette_library_dir = 'spec/fixtures/vcr_cassettes'
+  c.hook_into :webmock
+  c.filter_sensitive_data("[Username]") { ENV["AVATAX_USERNAME"] }
+  c.filter_sensitive_data("[Password]") { ENV["AVATAX_PASSWORD"] }
+  c.filter_sensitive_data("[Company]") { ENV["AVATAX_COMPANY_CODE"] }
+end
+
+RSpec.configure do |config|
+  config.infer_spec_type_from_file_location!
+  config.mock_with :rspec
+
+  config.color = true
+  config.order = "random"
+
+  config.expose_current_running_example_as :example
+  config.fail_fast = ENV["FAIL_FAST"] || false
+
+  config.filter_run focus: true
+  config.run_all_when_everything_filtered = true
+  config.use_transactional_fixtures = false
+
+  config.include FactoryGirl::Syntax::Methods
+  config.include Spree::TestingSupport::ControllerRequests, type: :controller
+  config.include Spree::TestingSupport::UrlHelpers, type: :controller
+
+  config.include ActionView::Helpers::TagHelper
+  config.include ActionView::Context
+  config.include SalesInvoiceSoapResponses
+  config.include ReturnInvoiceSoapResponses
+
+  # Ensure Suite is set to use transactions for speed.
+  config.before :suite do
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with :truncation
+  end
+
+  # Before each spec check if it is a Javascript test and switch between using database transactions or not where necessary.
+  config.before :each do
+    DatabaseCleaner.strategy = example.metadata[:js] ? :truncation : :transaction
+    DatabaseCleaner.start
+
+    FactoryGirl.create(:tax_rate, calculator: create(:avatax_tax_calculator))
+  end
+
+  # After each spec clean the database.
+  config.after :each do
+    DatabaseCleaner.clean
+  end
+end

data/spec/support/return_invoice_soap_responses.rb

@@ -0,0 +1,117 @@
+module ReturnInvoiceSoapResponses
+  def return_invoice_gettax_response(doc_code, return_item_id, time = Time.now)
+    {
+      transaction_id: "4289094359310439",
+      result_code: "Success",
+      doc_id: '55527402',
+      doc_type: "ReturnInvoice",
+      doc_code: doc_code,
+      doc_date: time.to_date,
+      doc_status: "Saved",
+      reconciled: false,
+      timestamp: time,
+      total_amount: "-10",
+      total_discount: "0",
+      total_exemption: "-10",
+      total_taxable: "0",
+      total_tax: "0",
+      total_tax_calculated: "0",
+      hash_code: "0",
+      tax_lines: {
+        tax_line: {
+          no: return_item_id.to_s,
+          tax_code: "P0000000",
+          taxability: true,
+          boundary_level: "Zip5",
+          exemption: "-10",
+          discount: "0",
+          taxable: "0",
+          rate: "0",
+          tax: "-0.01",
+          tax_calculated: "-0.01",
+          tax_included: false,
+          tax_details: {
+            tax_detail: {
+              country: "US",
+              region: "AL",
+              juris_type: "State",
+              juris_code: "01",
+              tax_type: "Sales",
+              base: "0",
+              taxable: "0",
+              rate: "0",
+              tax: "-0.01",
+              tax_calculated: "-0.01",
+              non_taxable: "-10",
+              exemption: "0",
+              juris_name: "ALABAMA",
+              tax_name: "AL STATE TAX",
+              tax_authority_type: "45",
+              tax_group: nil,
+              rate_type: "G",
+              state_assigned_no: nil,
+            },
+          },
+          exempt_cert_id: "0",
+          tax_date: time.to_date,
+          reporting_date: time.to_date,
+          accounting_method: "Accrual",
+        },
+      },
+      tax_addresses: {
+        tax_address: {
+          address: "Northwest",
+          address_code: "1",
+          boundary_level: "2",
+          city: "Herndon",
+          country: "US",
+          postal_code: "35005",
+          region: "AL",
+          tax_region_id: "2052474",
+          juris_code: "0107300460",
+          latitude: nil,
+          longitude: nil,
+          geocode_type: "ZIP5Centroid",
+          validate_status: "HouseNotOnStreet",
+          distance_to_boundary: "0",
+        },
+      },
+      locked: false,
+      adjustment_reason: "0",
+      adjustment_description: nil,
+      version: "1",
+      tax_date: time.to_date,
+      tax_summary: nil,
+      volatile_tax_rates: false,
+      messages: [
+        {
+          summary: nil,
+          details: nil,
+          helplink: nil,
+          refersto: nil,
+          severity: nil,
+          source: nil,
+        },
+      ],
+    }
+  end
+
+  def return_invoice_posttax_response
+    {
+      transaction_id: "4289138364000017",
+      result_code:    "Success",
+      doc_id:         "55529724",
+
+      messages: [
+        {
+          summary:  nil,
+          details:  nil,
+          helplink: nil,
+          refersto: nil,
+          severity: nil,
+          source:   nil,
+        },
+      ],
+    }
+  end
+end