solidus_importer 0.1.0 → 0.2.0

data/spec/lib/solidus_importer/processors/bill_address_spec.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe SolidusImporter::Processors::BillAddress do
+  describe '#call' do
+    subject(:described_method) { described_class.call(context) }
+
+    let(:context) do
+      { data: data }
+    end
+    let(:data) do
+      {
+        'Billing First Name' => 'John',
+        'Billing Last Name' => 'Doe',
+        'Billing Address1' => 'An address',
+        'Billing Address2' => '',
+        'Billing City' => 'A Beautyful city',
+        'Billing Company' => 'A Company',
+        'Billing Zip' => '00000',
+        'Billing Phone' => '555-123123123',
+        'Billing Country Code' => 'US',
+        'Billing Province Code' => 'NM'
+      }
+    end
+
+    it 'put bill_address_attributes into order data' do
+      described_method
+      expect(context).to have_key(:order)
+      expect(context[:order][:bill_address_attributes]).not_to be_empty
+    end
+  end
+end

data/spec/lib/solidus_importer/processors/customer_address_spec.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe SolidusImporter::Processors::CustomerAddress do
+  describe '#call' do
+    subject(:described_method) { described_class.call(context) }
+
+    let(:context) { { data: data, user: user } }
+    let(:user) { create(:user) }
+    let!(:state) { create(:state, state_code: 'WA', country_iso: 'US') }
+    let(:country) { state.country }
+    let(:data) do
+      {
+        'First Name' => 'John',
+        'Last Name' => 'Doe',
+        'Address1' => 'My Cool Address, n.1',
+        'Address2' => '',
+        'City' => 'My beautiful City',
+        'Zip' => '12345',
+        'Phone' => '(555)-123123123',
+        'Country Code' => 'US',
+        'Province Code' => 'WA'
+      }
+    end
+    let(:address) { Spree::Address.last }
+
+    context 'when there is a state with the same code attached to another country' do
+      let!(:wrong_state) { create(:state, state_code: 'WA', country_iso: 'IT') }
+      let!(:state) { create(:state, state_code: 'XX', country_iso: 'US') }
+
+      it 'tries to fetch it from the current country' do
+        expect { described_method }.to change(Spree::Address, :count).by(1)
+        expect(Spree::Address.last.state).to be_nil
+      end
+
+      context 'when the country requires a states' do
+        before { country.update states_required: true }
+
+        it 'fails creating the address' do
+          expect { described_method }.not_to change(Spree::Address, :count)
+        end
+      end
+    end
+
+    it 'create an address' do
+      expect { described_method }.to change(Spree::Address, :count).by(1)
+
+      aggregate_failures do
+        expect(address.full_name).to eq("John Doe")
+        expect(address.address1).to eq('My Cool Address, n.1')
+        expect(address.address2).to eq('')
+        expect(address.city).to eq('My beautiful City')
+        expect(address.zipcode).to eq('12345')
+        expect(address.phone).to eq('(555)-123123123')
+        expect(address.country).to eq(country)
+        expect(address.state).to eq(state)
+      end
+    end
+
+    it 'adds the address in the user addressbook and set it as ship/bill address' do
+      expect { described_method }.to change(user.addresses, :count).by(1)
+      expect(user.bill_address).to eq address
+      expect(user.ship_address).to eq address
+    end
+  end
+end

data/spec/lib/solidus_importer/processors/customer_spec.rb

@@ -30,7 +30,20 @@ RSpec.describe SolidusImporter::Processors::Customer do
 
     context 'with a customer row with a file entity' do
       let(:context) do
-        { data: build(:solidus_importer_row_customer, :with_import).data }
+        { data: build(:solidus_importer_row_customer, :with_import).data.merge(customer_address_data) }
+      end
+      let(:customer_address_data) do
+        {
+          'First Name' => 'John',
+          'Last Name' => 'Doe',
+          'Address1' => 'My Cool Address, n.1',
+          'Address2' => '',
+          'City' => 'My beautiful City',
+          'Zip' => '12345',
+          'Phone' => '(555)-123123123',
+          'Country Code' => 'US',
+          'Province Code' => 'WA'
+        }
       end
       let(:result) { context.merge(user: Spree::User.last) }
 

data/spec/lib/solidus_importer/processors/line_item_spec.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe SolidusImporter::Processors::LineItem do
+  describe '#call' do
+    subject(:described_method) { described_class.call(context) }
+
+    let(:context) do
+      { data: data }
+    end
+    let(:data) do
+      { 'Lineitem sku' => 'a-sku' }
+    end
+
+    it 'put line_items_attributes into order data' do
+      described_method
+      expect(context).to have_key(:order)
+      expect(context[:order][:line_items_attributes]).not_to be_empty
+    end
+  end
+end

data/spec/lib/solidus_importer/processors/order_spec.rb

@@ -23,33 +23,12 @@ RSpec.describe SolidusImporter::Processors::Order do
         { data: data }
       end
       let(:data) { build(:solidus_importer_row_order, :with_import).data }
-      let(:result) { context.merge(order: Spree::Order.last) }
 
       before { allow(Spree::Store).to receive(:default).and_return(build_stubbed(:store)) }
 
       it 'creates a new order' do
-        expect { described_method }.to change { Spree::Order.count }.by(1)
-        expect(described_method).to eq(result)
-      end
-
-      context 'with an existing valid order' do
-        let!(:order) { create(:order, number: data['Name'], email: data['Email']) }
-
-        it 'updates the order' do
-          expect { described_method }.not_to(change{ Spree::Order.count })
-          expect(described_method).to eq(result)
-          expect(order.reload.email).to eq('an_email@example.com')
-        end
-      end
-
-      context 'with an existing invalid order' do
-        let!(:order) { create(:order, number: data['Name'], email: data['Email']) }
-
-        before { order.update_column(:state, 'an invalid state') }
-
-        it 'raises an exception' do
-          expect { described_method }.to raise_error(ActiveRecord::RecordInvalid, /State is invalid/)
-        end
+        described_method
+        expect(context[:order]).to match(hash_including(number: 'R123456789'))
       end
     end
   end

data/spec/lib/solidus_importer/processors/payment_spec.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe SolidusImporter::Processors::Payment do
+  describe '#call' do
+    subject(:described_method) { described_class.call(context) }
+
+    let(:context) do
+      { data: data }
+    end
+    let(:data) do
+      {
+        'Financial Status' => 'paid',
+        'Lineitem price' => '10.5',
+        'Lineitem quantity' => '2'
+      }
+    end
+    let(:expected_params) do
+      [{ payment_method: payment_method_name, amount: amount }]
+    end
+    let(:payment_method_name) { 'SolidusImporter PaymentMethod' }
+    let(:amount) { 21.0 }
+
+    it 'put payments_attributes into order data' do
+      described_method
+      expect(context).to have_key(:order)
+      expect(context[:order][:payments_attributes]).to eq expected_params
+    end
+  end
+end

data/spec/lib/solidus_importer/processors/ship_address_spec.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe SolidusImporter::Processors::ShipAddress do
+  describe '#call' do
+    subject(:described_method) { described_class.call(context) }
+
+    let(:context) do
+      { data: data }
+    end
+    let(:data) do
+      {
+        'Shipping First Name' => 'John',
+        'Shipping Last Name' => 'Doe',
+        'Shipping Address1' => 'An address',
+        'Shipping Address2' => '',
+        'Shipping City' => 'A Beautyful city',
+        'Shipping Company' => 'A Company',
+        'Shipping Zip' => '00000',
+        'Shipping Phone' => '555-123123123',
+        'Shipping Country Code' => 'US',
+        'Shipping Province Code' => 'NM'
+      }
+    end
+
+    it 'put ship_address_attributes into order data' do
+      described_method
+      expect(context).to have_key(:order)
+      expect(context[:order][:ship_address_attributes]).not_to be_empty
+    end
+  end
+end

data/spec/lib/solidus_importer/processors/shipment_spec.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe SolidusImporter::Processors::Shipment do
+  describe '#call' do
+    subject(:described_method) { described_class.call(context) }
+
+    let(:context) do
+      { data: data }
+    end
+    let(:data) do
+      { 'Shipping Line Title' => 'ACME Shipping' }
+    end
+
+    it 'put shipments_attributes into order data' do
+      described_method
+      expect(context).to have_key(:order)
+      expect(context[:order][:shipments_attributes]).not_to be_empty
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_importer
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Mattia Roccoberton
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-06-26 00:00:00.000000000 Z
+date: 2021-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: solidus_core
@@ -69,6 +69,8 @@ files:
 - ".github/stale.yml"
 - ".gitignore"
 - ".hound.yml"
+- ".readme/import-products-1.gif"
+- ".readme/import-products-2.gif"
 - ".rspec"
 - ".rubocop.yml"
 - Gemfile
@@ -83,7 +85,9 @@ files:
 - app/controllers/spree/admin/solidus_importer/imports_controller.rb
 - app/jobs/solidus_importer/import_job.rb
 - app/models/solidus_importer/import.rb
+- app/models/solidus_importer/order_updater.rb
 - app/models/solidus_importer/row.rb
+- app/models/solidus_importer/spree_core_importer_order.rb
 - app/views/spree/admin/solidus_importer/import_rows/show.html.erb
 - app/views/spree/admin/solidus_importer/imports/_form.html.erb
 - app/views/spree/admin/solidus_importer/imports/index.html.erb
@@ -110,17 +114,23 @@ files:
 - lib/solidus_importer/engine.rb
 - lib/solidus_importer/exception.rb
 - lib/solidus_importer/factories.rb
+- lib/solidus_importer/order_importer.rb
 - lib/solidus_importer/process_import.rb
 - lib/solidus_importer/process_row.rb
-- lib/solidus_importer/processors/address.rb
 - lib/solidus_importer/processors/base.rb
+- lib/solidus_importer/processors/bill_address.rb
 - lib/solidus_importer/processors/customer.rb
+- lib/solidus_importer/processors/customer_address.rb
+- lib/solidus_importer/processors/line_item.rb
 - lib/solidus_importer/processors/log.rb
 - lib/solidus_importer/processors/option_types.rb
 - lib/solidus_importer/processors/option_values.rb
 - lib/solidus_importer/processors/order.rb
+- lib/solidus_importer/processors/payment.rb
 - lib/solidus_importer/processors/product.rb
 - lib/solidus_importer/processors/product_images.rb
+- lib/solidus_importer/processors/ship_address.rb
+- lib/solidus_importer/processors/shipment.rb
 - lib/solidus_importer/processors/taxon.rb
 - lib/solidus_importer/processors/variant.rb
 - lib/solidus_importer/processors/variant_images.rb
@@ -143,17 +153,23 @@ files:
 - spec/fixtures/solidus_importer/thinking-cat.jpg
 - spec/jobs/solidus_importer/import_job_spec.rb
 - spec/lib/solidus_importer/base_importer_spec.rb
+- spec/lib/solidus_importer/order_importer_spec.rb
 - spec/lib/solidus_importer/process_import_spec.rb
 - spec/lib/solidus_importer/process_row_spec.rb
-- spec/lib/solidus_importer/processors/address_spec.rb
 - spec/lib/solidus_importer/processors/base_spec.rb
+- spec/lib/solidus_importer/processors/bill_address_spec.rb
+- spec/lib/solidus_importer/processors/customer_address_spec.rb
 - spec/lib/solidus_importer/processors/customer_spec.rb
+- spec/lib/solidus_importer/processors/line_item_spec.rb
 - spec/lib/solidus_importer/processors/log_spec.rb
 - spec/lib/solidus_importer/processors/option_types_spec.rb
 - spec/lib/solidus_importer/processors/option_values_spec.rb
 - spec/lib/solidus_importer/processors/order_spec.rb
+- spec/lib/solidus_importer/processors/payment_spec.rb
 - spec/lib/solidus_importer/processors/product_images_spec.rb
 - spec/lib/solidus_importer/processors/product_spec.rb
+- spec/lib/solidus_importer/processors/ship_address_spec.rb
+- spec/lib/solidus_importer/processors/shipment_spec.rb
 - spec/lib/solidus_importer/processors/taxon_spec.rb
 - spec/lib/solidus_importer/processors/variant_images_spec.rb
 - spec/lib/solidus_importer/processors/variant_spec.rb
@@ -169,7 +185,7 @@ metadata:
   homepage_uri: https://github.com/nebulab/solidus_importer#readme
   source_code_uri: https://github.com/nebulab/solidus_importer#readme
   changelog_uri: https://github.com/nebulab/solidus_importer/releases
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -184,8 +200,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Solidus Importer extension
 test_files:
@@ -206,17 +222,23 @@ test_files:
 - spec/fixtures/solidus_importer/thinking-cat.jpg
 - spec/jobs/solidus_importer/import_job_spec.rb
 - spec/lib/solidus_importer/base_importer_spec.rb
+- spec/lib/solidus_importer/order_importer_spec.rb
 - spec/lib/solidus_importer/process_import_spec.rb
 - spec/lib/solidus_importer/process_row_spec.rb
-- spec/lib/solidus_importer/processors/address_spec.rb
 - spec/lib/solidus_importer/processors/base_spec.rb
+- spec/lib/solidus_importer/processors/bill_address_spec.rb
+- spec/lib/solidus_importer/processors/customer_address_spec.rb
 - spec/lib/solidus_importer/processors/customer_spec.rb
+- spec/lib/solidus_importer/processors/line_item_spec.rb
 - spec/lib/solidus_importer/processors/log_spec.rb
 - spec/lib/solidus_importer/processors/option_types_spec.rb
 - spec/lib/solidus_importer/processors/option_values_spec.rb
 - spec/lib/solidus_importer/processors/order_spec.rb
+- spec/lib/solidus_importer/processors/payment_spec.rb
 - spec/lib/solidus_importer/processors/product_images_spec.rb
 - spec/lib/solidus_importer/processors/product_spec.rb
+- spec/lib/solidus_importer/processors/ship_address_spec.rb
+- spec/lib/solidus_importer/processors/shipment_spec.rb
 - spec/lib/solidus_importer/processors/taxon_spec.rb
 - spec/lib/solidus_importer/processors/variant_images_spec.rb
 - spec/lib/solidus_importer/processors/variant_spec.rb

data/lib/solidus_importer/processors/address.rb

@@ -1,47 +0,0 @@
-# frozen_string_literal: true
-
-module SolidusImporter
-  module Processors
-    class Address < Base
-      def call(context)
-        @data = context.fetch(:data)
-
-        return unless address?
-
-        context.merge!(address: process_address)
-      end
-
-      private
-
-      def process_address
-        prepare_address.tap(&:save)
-      end
-
-      def prepare_address
-        Spree::Address.new(
-          firstname: @data['First Name'],
-          lastname: @data['Last Name'],
-          address1: @data['Address1'],
-          address2: @data['Address2'],
-          city: @data['City'],
-          zipcode: @data['Zip'],
-          phone: @data['Phone'],
-          country: extract_country(@data['Country Code']),
-          state: extract_state(@data['Province Code'])
-        )
-      end
-
-      def extract_country(iso)
-        Spree::Country.find_by(iso: iso)
-      end
-
-      def extract_state(iso)
-        Spree::State.find_by(abbr: iso) || Spree::State.find_by(name: iso)
-      end
-
-      def address?
-        @data['Address1'].present?
-      end
-    end
-  end
-end