spree_avatax_official 1.7.1

data/spec/lib/spree_avatax_official/seeder_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe SpreeAvataxOfficial::Seeder do
+  describe '#seed!' do
+    subject { described_class.new.seed! }
+
+    let!(:usa) { create(:country, name: 'United States', iso3: 'USA') }
+    let!(:pennslyvania) { create(:state, name: 'Pennsylvania', abbr: 'PA') }
+    let!(:shipping_method) { create(:shipping_method, tax_category: nil) }
+    let(:clothing_tax_category) { Spree::TaxCategory.find_by(name: 'Clothing') }
+    let(:shipping_tax_category) { Spree::TaxCategory.find_by(name: 'Shipping') }
+
+    it 'creates Clothing and Shipping tax categories and assigns Shipping tax category' do
+      subject
+
+      expect(clothing_tax_category).to be_present
+      expect(clothing_tax_category.tax_code).to eq 'P0000000'
+      expect(shipping_tax_category).to be_present
+      expect(shipping_tax_category.tax_code).to eq 'FR'
+      expect(shipping_method.reload.tax_category).to eq shipping_tax_category
+    end
+  end
+end

data/spec/models/spree/line_item_spec.rb

@@ -0,0 +1,76 @@
+require 'spec_helper'
+
+describe Spree::LineItem do
+  describe '#avatax_number' do
+    let(:line_item) { create(:line_item) }
+
+    it 'returns line item id with avatax code' do
+      expect(line_item.avatax_number).to eq "LI-#{line_item.avatax_uuid}"
+    end
+  end
+
+  describe '#update_tax_charge' do
+    context 'with AvaTax tax calculcation', :avatax_enabled do
+      let(:order) { create(:avatax_order, ship_address: create(:usa_address)) }
+      let(:line_item) { order.line_items.first }
+
+      before do
+        VCR.use_cassette('spree_order/update_tax_charge/create_line_item') do
+          create(:line_item, price: 10.0, quantity: 2, order: order)
+
+          order.reload
+          order.updater.update
+        end
+      end
+
+      context 'when line item is created' do
+        it 'updates line item and order tax' do
+          expect(order.total).to eq 21.6
+          expect(order.additional_tax_total).to eq 1.6
+          expect(line_item.reload.additional_tax_total).to eq 1.6
+        end
+      end
+
+      context 'when line item quantity is increased' do
+        it 'updates line item and order tax' do
+          VCR.use_cassette('spree_line_item/update_tax_charge/increase_quantity') do
+            line_item.update(quantity: 3)
+
+            order.reload
+            order.updater.update
+          end
+
+          expect(order.total).to eq 32.4
+          expect(order.additional_tax_total).to eq 2.4
+          expect(line_item.reload.additional_tax_total).to eq 2.4
+        end
+      end
+
+      context 'when line item quantity is decreased' do
+        it 'updates line item and order tax' do
+          VCR.use_cassette('spree_line_item/update_tax_charge/decrease_quantity') do
+            line_item.update(quantity: 1)
+
+            order.reload
+            order.updater.update
+          end
+
+          expect(order.total).to eq 10.8
+          expect(order.additional_tax_total).to eq 0.8
+          expect(line_item.reload.additional_tax_total).to eq 0.8
+        end
+      end
+
+      context 'when line item is destroyed' do
+        it 'updates line item and order tax' do
+          line_item.destroy
+
+          order.reload.updater.update
+
+          expect(order.total).to eq 0
+          expect(Spree::Adjustment.tax.count).to eq 0
+        end
+      end
+    end
+  end
+end

data/spec/models/spree/order_spec.rb

@@ -0,0 +1,210 @@
+require 'spec_helper'
+
+describe Spree::Order do
+  describe 'associations' do
+    let(:order) { create(:order) }
+
+    describe 'avatax_sales_invoice_transaction' do
+      let!(:return_invoice_transaction) { create(:spree_avatax_official_transaction, :return_invoice, order: order) }
+      let!(:sales_invoice_transaction)  { create(:spree_avatax_official_transaction, order: order) }
+
+      it 'returns SpreeAvataxOfficial::Transaction object with transaction_type: SalesInvoice' do
+        expect(order.avatax_sales_invoice_transaction).to eq sales_invoice_transaction
+      end
+    end
+  end
+
+  describe '#cancel', :avatax_enabled do
+    let!(:avatax_tax_rate) { create(:avatax_tax_rate) }
+    let(:order) { create(:order, ship_address: create(:usa_address)) }
+
+    before do
+      VCR.use_cassette('spree_order/simple_order_with_single_line_item') do
+        create(:line_item, price: 10.0, quantity: 1, order: order)
+        order.update(state: :complete, completed_at: Time.current)
+      end
+
+      allow(Spree::OrderMailer).to receive_message_chain(:cancel_email, :deliver_later)
+    end
+
+    it 'calls void service' do
+      expect(SpreeAvataxOfficial::Transactions::VoidService).to receive(:call)
+
+      order.cancel
+    end
+  end
+
+  describe '#taxable_items' do
+    let(:order) { create(:shipped_order, line_items_count: 2) }
+
+    it 'returns array of shipments and line items' do
+      expect(order.taxable_items).to eq [order.line_items.first, order.line_items.last, order.shipments.first]
+    end
+  end
+
+  describe '#complete', :avatax_enabled do
+    let(:order) do
+      VCR.use_cassette('spree_order/order_transition_to_completed') do
+        create(:avatax_order, line_items_count: 1, ship_address: create(:usa_address)).tap do |order|
+          order.payments << create(:payment)
+
+          order.next!
+          # Unfortunetly state machine does not allow me to stub create_proposed_shipments method
+          # Stubbing results with `Wrong number of arguments. Expected 0, got 1.` without stacktrace
+          create(:avatax_shipment, order: order)
+          order.update(state: 'delivery')
+          order.reload
+          2.times { order.next! }
+        end
+      end
+    end
+
+    before do
+      allow(Spree::OrderMailer).to receive_message_chain(:confirm_email, :deliver_later)
+    end
+
+    context 'commit transaction enabled' do
+      before { SpreeAvataxOfficial::Config.commit_transaction_enabled = true }
+
+      it 'creates a commited SalesInvoice transaction' do
+        expect(order.state).to eq 'confirm'
+
+        VCR.use_cassette('spree_order/complete_order') do
+          expect { order.next! }.to change { order.avatax_transactions.count }.by(1)
+        end
+      end
+    end
+
+    context 'commit transaction disabled' do
+      around do |example|
+        SpreeAvataxOfficial::Config.commit_transaction_enabled = false
+        example.run
+        SpreeAvataxOfficial::Config.commit_transaction_enabled = true
+      end
+
+      it 'doesnt create a commited SalesInvoice transaction' do
+        expect(order.state).to eq 'confirm'
+
+        VCR.use_cassette('spree_order/complete_order_no_transaction') do
+          expect { order.next! }.to_not change { order.avatax_transactions.count }
+        end
+      end
+    end
+  end
+
+  describe 'tax estimation triggering', :avatax_enabled do
+    let(:order) { create(:avatax_order, with_shipment: true, ship_address: create(:usa_address)) }
+    let(:line_item) { order.line_items.first }
+    let(:shipment) { order.shipments.first }
+    let(:tax_adjustment) { line_item.adjustments.tax.first }
+
+    before do
+      VCR.use_cassette('spree_order/simple_order_with_single_line_item_and_shipment') do
+        create(:line_item, price: 10.0, quantity: 1, order: order)
+
+        order.reload
+        order.updater.update
+      end
+    end
+
+    context 'when line item discount promotion is applied' do
+      let(:promotion) { create(:promotion, :with_line_item_adjustment, adjustment_rate: 5, code: 'promotion_code') }
+
+      it 'triggers tax estimation' do
+        expect(order.total).to eq 16.2
+
+        VCR.use_cassette('spree_order/order_with_line_item_adjustment') do
+          order.coupon_code = promotion.code
+          Spree::PromotionHandler::Coupon.new(order).apply
+
+          order.updater.update
+        end
+
+        expect(order.total).to eq 10.8
+        expect(line_item.reload.additional_tax_total).to eq 0.4
+        expect(shipment.reload.additional_tax_total).to eq 0.4
+        expect(tax_adjustment.amount).to eq 0.4
+      end
+    end
+
+    context 'when order discount promotion is applied' do
+      let(:promotion) { create(:promotion, :avatax_with_order_adjustment, weighted_order_adjustment_amount: 5.0, code: 'promotion_code') }
+
+      it 'triggers tax estimation' do
+        expect(order.total).to eq 16.2
+
+        VCR.use_cassette('spree_order/order_with_order_adjustment') do
+          order.coupon_code = promotion.code
+          Spree::PromotionHandler::Coupon.new(order).apply
+
+          order.updater.update
+        end
+
+        expect(order.total).to eq 10.8
+        expect(shipment.reload.additional_tax_total).to eq 0.4
+        expect(tax_adjustment.amount).to eq 0.4
+      end
+    end
+
+    context 'when shipping address is changed', if: (::Gem::Version.new(::Spree.version) >= ::Gem::Version.new('3.0.0')) do
+      let(:california_address) { create(:usa_address, :from_california) }
+
+      it 'triggers tax estimation' do
+        expect(order.total).to eq 16.2
+
+        VCR.use_cassette('spree_order/california_order') do
+          order.tax_address.update(
+              address1: california_address.address1,
+              address2: california_address.address2,
+              city:     california_address.city,
+              zipcode:  california_address.zipcode,
+              state_id: california_address.state_id
+          )
+          california_address.run_callbacks(:save)
+        end
+
+        expect(order.reload.total).to eq 15.73
+        expect(line_item.reload.additional_tax_total).to eq 0.73
+        # California does not charge shipping tax
+        # https://www.avalara.com/us/en/blog/2016/01/do-i-charge-tax-on-shipping-costs-in-california.html
+        expect(shipment.reload.additional_tax_total).to eq 0
+
+        expect(tax_adjustment.amount).to eq 0.73
+      end
+    end
+  end
+
+  describe '#validate_tax_address' do
+    let(:order) { create(:order_with_line_items, ship_address: ship_address, state: :address) }
+
+    around do |example|
+      SpreeAvataxOfficial::Config.address_validation_enabled = true
+
+      example.run
+
+      SpreeAvataxOfficial::Config.address_validation_enabled = false
+    end
+
+    context 'when address is invalid' do
+      let(:ship_address) { create(:invalid_usa_address) }
+
+      it 'does not change order state to delivery and adds an error' do
+        VCR.use_cassette('spree_avatax_official/address/validate_failure') do
+          expect { order.next! }.to raise_error StateMachines::InvalidTransition
+          expect(order.errors.count).to eq 1
+          expect(order.state).to eq 'address'
+        end
+      end
+    end
+
+    context 'when address is valid' do
+      let(:ship_address) { create(:usa_address) }
+
+      it 'changes state from address to delivery' do
+        VCR.use_cassette('spree_avatax_official/address/validate_success') do
+          expect { order.next! }.to change(order, :state).to 'delivery'
+        end
+      end
+    end
+  end
+end

data/spec/models/spree/shipment_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe Spree::Shipment do
+  let(:shipment) { create(:shipment) }
+
+  describe '#avatax_number' do
+    it 'returns shipment id with avatax code' do
+      expect(shipment.avatax_number).to eq "FR-#{shipment.avatax_uuid}"
+    end
+  end
+end

data/spec/models/spree_avatax_official/avatax_log_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+
+describe SpreeAvataxOfficial::AvataxLog, type: :model do
+  let(:file_name)       { 'test_file.log' }
+  let(:logger)          { described_class.new }
+  let(:example_payload) { { test: 'Test!' } }
+  let!(:order)          { create(:order) }
+  let(:payload)         { File.new(Rails.root.join('log', file_name)).read }
+  let(:log_data)        { "[AVATAX] #{order.class} #{order.id} : #{order.number}" }
+
+  before do
+    SpreeAvataxOfficial::Config.log_file_name = file_name
+    SpreeAvataxOfficial::Config.log_to_stdout = false
+    SpreeAvataxOfficial::Config.log           = true
+  end
+
+  describe '#enabled?' do
+    it 'returns a boolean value' do
+      SpreeAvataxOfficial::Config.log = true
+
+      expect(logger).to be_enabled
+    end
+  end
+
+  describe '#info' do
+    it 'logs info with given message' do
+      logger.info(example_payload)
+
+      expect(payload).to include "[AVATAX] #{example_payload.to_json}"
+    end
+
+    it 'returns nil if logger is not enabled' do
+      SpreeAvataxOfficial::Config.log = false
+
+      expect(logger.info('this_wont_change')).to be_nil
+    end
+
+    context 'STDOUT' do
+      before do
+        SpreeAvataxOfficial::Config.log_to_stdout = true
+      end
+
+      it 'writes logs to STDOUT' do
+        expect { logger.info('Test!') }.to output.to_stdout_from_any_process
+      end
+    end
+  end
+
+  describe '#debug' do
+    it 'receives debug with message' do
+      logger.debug(order, example_payload)
+
+      expect(payload).to include "#{log_data} #{example_payload.to_json}"
+    end
+
+    it 'returns nil if logger is not enabled' do
+      SpreeAvataxOfficial::Config.log = false
+
+      expect(logger.debug(['Test!'])).to be_nil
+    end
+  end
+
+  describe '#error' do
+    it 'logs error with given message' do
+      logger.error(order, example_payload)
+
+      expect(payload).to include "#{log_data} #{example_payload.to_json}"
+    end
+
+    it 'returns nil if logger is not enabled' do
+      SpreeAvataxOfficial::Config.log = false
+
+      expect(logger.error('this_wont_change')).to be_nil
+    end
+
+    context 'message with status and body' do
+      it 'logs error with given message' do
+        message         = Struct.new(:message, :status, :body)
+        example_payload = message.new(400, test: 'Test!')
+        log_data        = "#{log_data} #{example_payload.status} #{example_payload.body.to_json}"
+
+        logger.error(order, example_payload)
+        expect(payload).to include log_data
+      end
+    end
+  end
+end

data/spec/models/spree_avatax_official/entity_use_code_spec.rb

@@ -0,0 +1,9 @@
+require 'spec_helper'
+
+describe SpreeAvataxOfficial::EntityUseCode do
+  describe '.table_name' do
+    subject { described_class.table_name }
+
+    it      { is_expected.to eq 'spree_avatax_official_entity_use_codes' }
+  end
+end

data/spec/models/spree_avatax_official/has_uuid_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe SpreeAvataxOfficial::HasUuid do
+  class Spree::LineItem
+    include ::SpreeAvataxOfficial::HasUuid
+  end
+
+  let(:line_item) { create(:line_item) }
+
+  describe '#avatax_uuid' do
+    it 'generates uuid for new instance' do
+      expect(line_item.avatax_uuid).not_to be_nil
+    end
+  end
+
+  describe '#avatax_number' do
+    it 'return uuid with avatax code' do
+      expect(line_item.avatax_number).to eq "LI-#{line_item.avatax_uuid}"
+    end
+  end
+end

data/spec/models/spree_avatax_official/spree/refund_decorator_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe SpreeAvataxOfficial::Spree::RefundDecorator do
+  describe '#create', :avatax_enabled do
+    let(:refund) { create(:refund, amount: 10) }
+
+    context 'commit transaction enabled' do
+      before { SpreeAvataxOfficial::Config.commit_transaction_enabled = true }
+
+      it 'calls refund service' do
+        expect(SpreeAvataxOfficial::Transactions::RefundService).to receive(:call)
+
+        refund
+      end
+    end
+
+    context 'commit transaction disabled' do
+      around do |example|
+        SpreeAvataxOfficial::Config.commit_transaction_enabled = false
+        example.run
+        SpreeAvataxOfficial::Config.commit_transaction_enabled = true
+      end
+
+      it 'doesnt call refund service' do
+        expect(SpreeAvataxOfficial::Transactions::RefundService).to_not receive(:call)
+
+        refund
+      end
+    end
+  end
+end