solidus_product_assembly 1.0.0 → 1.2.0

data/spec/features/admin/orders_spec.rb

@@ -1,29 +1,127 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe "Orders", type: :feature, js: true do
   stub_authorization!
 
-  let(:order) { create(:order_with_line_items) }
-  let(:line_item) { order.line_items.first }
-  let(:bundle) { line_item.product }
-  let(:parts) { (1..3).map { create(:variant) } }
+  context 'when the order product is a bundle' do
+    let(:order) { create(:order_with_line_items) }
+    let(:line_item) { order.line_items.first }
+    let(:bundle) { line_item.product }
+    let(:parts) { (1..3).map { create(:variant) } }
+
+    before do
+      bundle.parts << [parts]
+      line_item.update!(quantity: 3)
+      order.reload.create_proposed_shipments
+      order.finalize!
+    end
+
+    if Spree.solidus_gem_version < Gem::Version.new('2.5')
+      context 'Adding tracking number' do
+        let(:tracking_number) { 'AA123' }
+
+        before { visit spree.edit_admin_order_path(order) }
+
+        it 'allows admin to edit tracking' do
+          expect(page).not_to have_content tracking_number
+          within '.show-tracking' do
+            find('.fa-edit').click
+          end
+          within '.edit-tracking' do
+            expect(page).to have_selector 'input[name="tracking"]'
+            fill_in :tracking, with: tracking_number
+            find('.fa-ok').click
+            expect(page).not_to have_selector 'input[name="tracking"]'
+          end
+          expect(page).to have_content tracking_number
+        end
+      end
+
+      context 'Changing carrier/shipping costs' do
+        before do
+          visit spree.edit_admin_order_path(order)
+        end
 
-  before do
-    bundle.parts << [parts]
-    line_item.update_attributes!(quantity: 3)
-    order.reload.create_proposed_shipments
-    order.finalize! 
+        it 'allows admin to edit shipping costs' do
+          expect(page).not_to have_content 'UPS Ground $100.00'
+          within '.show-method' do
+            find('.fa-edit').click
+          end
+          within '.edit-method' do
+            find('.fa-ok').click
+          end
+          expect(page).to have_content 'UPS Ground $100.00'
+        end
+      end
+    else
+      context 'Adding tracking number' do
+        let(:tracking_number) { 'AA123' }
+
+        before { visit spree.edit_admin_order_path(order) }
+
+        it 'allows admin to edit tracking' do
+          within '.edit-tracking' do
+            expect(page).not_to have_content tracking_number
+            find('.js-edit').click
+            expect(page).to have_selector 'input[name="tracking"]'
+            fill_in :tracking, with: tracking_number
+            find('.js-save').click
+            expect(page).not_to have_selector 'input[name="tracking"]'
+            expect(page).to have_content tracking_number
+          end
+        end
+      end
+
+      context 'Changing carrier/shipping costs' do
+        let(:carrier_name) { 'Fedex' }
+        let(:select_name) { 'selected_shipping_method_id' }
+
+        before do
+          create :shipping_method, name: carrier_name
+          visit spree.edit_admin_order_path(order)
+        end
+
+        it 'allows admin to edit shipping costs' do
+          within '.edit-shipping-method' do
+            expect(page).not_to have_content carrier_name
+            find('.js-edit').click
+            expect(page).to have_selector "select[name='#{select_name}']"
+            select "#{carrier_name} $10.00", from: select_name
+            find('.js-save').click
+            expect(page).not_to have_selector "select[name='#{select_name}']"
+            expect(page).to have_content carrier_name
+          end
+        end
+      end
+    end
   end
 
-  it "allows admin to edit product bundle" do
-    visit spree.edit_admin_order_path(order)
+  context 'when the order product is not a bundle' do
+    let(:order) { create(:order_ready_to_ship, state: 'complete', line_items_count: 1) }
+    let(:product) { order.products.first }
+    let(:location_name) { 'Foobar warehouse' }
+
+    before { create :stock_location, name: location_name }
 
-    within("table.product-bundles") do
-      find(".edit-line-item").click
-      fill_in "quantity", :with => "2"
-      find(".save-line-item").click
+    context 'Splitting items' do
+      before { visit spree.edit_admin_order_path(order) }
 
-      sleep(1) # avoid odd "cannot rollback - no transaction is active: rollback transaction"
+      it 'allows admin to split items to another stock location' do
+        find('.split-item').click
+        within '.stock-item-split' do
+          expect(page).to have_content "Move #{product.name} to"
+          find('.save-split').click
+          alert_text = page.driver.browser.switch_to.alert.text
+          expect(alert_text).to include 'Please select the split destination'
+          page.driver.browser.switch_to.alert.accept
+          find('.select2-container').click
+          find(:xpath, '//body').all('.select2-drop li.select2-result', text: "#{location_name} (0 on hand)")[0].click
+          find('.save-split').click
+        end
+        expect(page).to have_content /Pending package from '#{location_name}'/i
+      end
     end
   end
 end

data/spec/features/admin/parts_spec.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe "Parts", type: :feature, js: true do
   stub_authorization!
 
-  let!(:tshirt) { create(:product, :name => "T-Shirt") }
-  let!(:mug) { create(:product, :name => "Mug") }
+  let!(:tshirt) { create(:product, name: "T-Shirt") }
+  let!(:mug) { create(:product, name: "Mug") }
 
   before do
     visit spree.admin_product_path(mug)
@@ -19,7 +21,7 @@ describe "Parts", type: :feature, js: true do
     click_on "Search"
 
     within("#search_hits") { click_on "Select" }
-    page.should have_content(mug.sku)
+    expect(page).to have_content(mug.sku)
 
     within("#product_parts") do
       find(".remove_admin_product_part_link").click

data/spec/features/admin/return_items_spec.rb

@@ -0,0 +1,151 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Return Items", type: :feature, js: true do
+  stub_authorization!
+
+  let(:line_item) { order.line_items.first }
+
+  before do
+    allow_any_instance_of(Spree::Admin::ReimbursementsController).to receive(:try_spree_current_user) { Spree.user_class.new }
+  end
+
+  context 'when the order product is a bundle' do
+    let(:bundle) { line_item.product }
+    let(:parts) { (1..3).map { create(:variant) } }
+
+    before do
+      create :refund_reason, name: Spree::RefundReason::RETURN_PROCESSING_REASON, mutable: false
+      bundle.parts << [parts]
+      bundle.set_part_count(parts.first, 2)
+      parts.each { |p| p.stock_items.first.set_count_on_hand 4 }
+      3.times { order.next }
+      create :payment, order: order, amount: order.amount
+      order.next
+      order.complete
+      order.payments.each(&:capture!)
+      order.shipments.each { |s| s.update state: :ready }
+      order.shipments.each(&:ship!)
+    end
+
+    context 'with a single item cart' do
+      let(:order) { create(:order_with_line_items) }
+
+      it 'successfully generates a RMA, a reimbursement and a refund' do
+        visit spree.edit_admin_order_path(order)
+
+        expect(page).to have_selector '.line-item-total', text: '$10.00'
+
+        click_link 'RMA'
+        click_link 'New RMA'
+
+        expect(page).to have_selector '.return-item-charged', text: '$2.50', count: 4
+
+        find('#select-all').click
+        select Spree::StockLocation.first.name, from: 'Stock Location'
+        click_button 'Create'
+
+        expect(page).to have_content 'Return Authorization has been successfully created!'
+
+        click_link 'Customer Returns'
+        click_link 'New Customer Return'
+        find('#select-all').click
+        page.execute_script %{$('option[value="receive"]').attr('selected', 'selected')}
+        select Spree::StockLocation.first.name, from: 'Stock Location'
+        click_button 'Create'
+
+        expect(page).to have_content 'Customer Return has been successfully created!'
+
+        click_button 'Create reimbursement'
+        click_button 'Reimburse'
+
+        within 'table.reimbursement-refunds' do
+          expect(page).to have_content '$10.00'
+        end
+      end
+    end
+
+    context 'with a multiple items cart' do
+      let(:order) { create(:order_with_line_items, line_items_attributes: [{ quantity: 2 }]) }
+
+      it 'successfully generates a RMA, a reimbursement and a refund' do
+        visit spree.edit_admin_order_path(order)
+
+        expect(page).to have_selector '.line-item-total', text: '$20.00'
+
+        click_link 'RMA'
+        click_link 'New RMA'
+
+        expect(page).to have_selector '.return-item-charged', text: '$2.50', count: 8
+
+        find('#select-all').click
+        select Spree::StockLocation.first.name, from: 'Stock Location'
+        click_button 'Create'
+
+        expect(page).to have_content 'Return Authorization has been successfully created!'
+
+        click_link 'Customer Returns'
+        click_link 'New Customer Return'
+        find('#select-all').click
+        page.execute_script %{$('option[value="receive"]').attr('selected', 'selected')}
+        select Spree::StockLocation.first.name, from: 'Stock Location'
+        click_button 'Create'
+
+        expect(page).to have_content 'Customer Return has been successfully created!'
+
+        click_button 'Create reimbursement'
+        click_button 'Reimburse'
+
+        within 'table.reimbursement-refunds' do
+          expect(page).to have_content '$20.00'
+        end
+      end
+    end
+  end
+
+  context 'when the product is not a bundle' do
+    before do
+      order.reload.create_proposed_shipments
+      order.finalize!
+      order.update state: :complete
+      order.shipments.update_all state: :shipped
+    end
+
+    context 'with a single item cart' do
+      let(:order) { create(:order_with_line_items) }
+
+      it 'builds one return item for each product' do
+        visit spree.edit_admin_order_path(order)
+
+        expect(page).to have_selector '.item-total', text: '$10.00'
+
+        click_link 'RMA'
+        click_link 'New RMA'
+
+        within '.return-items-table tbody' do
+          expect(page).to have_selector 'tr', count: 1
+          expect(page).to have_selector '.return-item-pre-tax-refund-amount', text: '$10.00', count: 1
+        end
+      end
+    end
+
+    context 'with a multiple items cart' do
+      let(:order) { create(:order_with_line_items, line_items_attributes: [{ quantity: 2 }]) }
+
+      it 'builds one return item for each product' do
+        visit spree.edit_admin_order_path(order)
+
+        expect(page).to have_selector '.item-total', text: '$20.00'
+
+        click_link 'RMA'
+        click_link 'New RMA'
+
+        within '.return-items-table tbody' do
+          expect(page).to have_selector 'tr', count: 2
+          expect(page).to have_selector '.return-item-pre-tax-refund-amount', text: '$10.00', count: 2
+        end
+      end
+    end
+  end
+end

data/spec/features/checkout_spec.rb

@@ -1,48 +1,51 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe "Checkout", type: :feature do
   let!(:store) { create :store, default: true }
-  let!(:country) { create(:country, :name => "United States", :states_required => true) }
-  let!(:state) { create(:state, :name => "Ohio", :country => country) }
+  let!(:country) { create(:country, name: "United States", states_required: true) }
+  let!(:state) { create(:state, name: "Ohio", country: country) }
   let!(:shipping_method) { create(:shipping_method) }
   let!(:stock_location) { create(:stock_location) }
   let!(:payment_method) { create(:check_payment_method) }
   let!(:zone) { create(:zone) }
 
-  let(:product) { create(:product, :name => "RoR Mug") }
+  let(:product) { create(:product, name: "RoR Mug") }
   let(:variant) { create(:variant) }
+  let(:other_variant) { create(:variant) }
 
   stub_authorization!
 
-  before { product.parts << variant << create(:variant) }
+  before { product.parts << variant << other_variant }
 
   shared_context "purchases product with part included" do
     before do
+      visit spree.root_path
       add_product_to_cart
       click_button "Checkout"
 
-      fill_in "order_email", :with => "ryan@spreecommerce.com"
+      fill_in "order_email", with: "ryan@spreecommerce.com"
       click_button "Continue"
       fill_in_address
 
       click_button "Save and Continue"
       expect(current_path).to eql(spree.checkout_state_path("delivery"))
-      page.should have_content(variant.product.name)
+      expect(page).to have_content(variant.product.name)
 
       click_button "Save and Continue"
       expect(current_path).to eql(spree.checkout_state_path("payment"))
 
       click_button "Save and Continue"
-      expect(current_path).to eq spree.checkout_state_path('confirm')
-      page.should have_content(variant.product.name)
+      expect(page).to have_current_path spree.checkout_state_path('confirm'), ignore_query: true
+      expect(page).to have_content(variant.product.name)
 
       click_button "Place Order"
-      page.should have_content('Your order has been processed successfully')
+      expect(page).to have_content('Your order has been processed successfully')
     end
   end
 
-  context "backend order shipments UI", js: true do
-
+  shared_examples 'purchasable assembly' do
     context "ordering only the product assembly" do
       include_context "purchases product with part included"
 
@@ -50,7 +53,7 @@ describe "Checkout", type: :feature do
         visit spree.admin_orders_path
         click_on Spree::Order.last.number
 
-        page.should have_content(variant.product.name)
+        expect(page).to have_content(variant.product.name)
       end
     end
 
@@ -60,28 +63,107 @@ describe "Checkout", type: :feature do
         click_link variant.product.name
         click_button "add-to-cart-button"
       end
+
       include_context "purchases product with part included"
 
       it "views parts bundled and not" do
         visit spree.admin_orders_path
         click_on Spree::Order.last.number
 
-        page.should have_content(variant.product.name)
+        expect(page).to have_content(variant.product.name)
       end
     end
-
   end
 
+  context "backend order shipments UI", js: true do
+    context 'when the product and parts are backorderable' do
+      it_behaves_like 'purchasable assembly'
+    end
+
+    context 'when the product and parts are not backorderable' do
+      context 'when there is enough quantity available' do
+        before do
+          Spree::StockItem.update_all backorderable: false
+          variant.stock_items.each { |stock_item| stock_item.set_count_on_hand 2 }
+          other_variant.stock_items.each { |stock_item| stock_item.set_count_on_hand 1 }
+          product.stock_items.each { |stock_item| stock_item.set_count_on_hand 1 }
+        end
+
+        it_behaves_like 'purchasable assembly'
+
+        context 'when the order can be processed' do
+          include_context "purchases product with part included"
+
+          before do
+            visit spree.admin_orders_path
+            click_on Spree::Order.last.number
+          end
+
+          it 'marks the order as paid and shipped showing proper parts information' do
+            within '#order_tab_summary' do
+              expect(page).to have_selector '#shipment_status', text: 'Pending'
+              expect(page).to have_selector '#payment_status', text: 'Balance due'
+            end
+
+            click_link 'Payments'
+            find('.fa-capture').click
+
+            within '#order_tab_summary' do
+              expect(page).to have_selector '#shipment_status', text: 'Ready'
+              expect(page).to have_selector '#payment_status', text: 'Paid'
+            end
+
+            click_link 'Shipments'
+
+            within '.stock-contents' do
+              expect(page).to have_selector '.item-price', text: '---'
+              expect(page).to have_selector '.item-total', text: '---'
+              expect(page).to have_selector '.item-qty-show', count: 2, text: '1 x on hand'
+            end
+
+            expect(page).not_to have_selector '.stock-contents.carton'
+
+            click_button 'Ship'
+
+            within '.stock-contents' do
+              expect(page).to have_selector '.item-qty-show', count: 2, text: '1 x Shipped'
+            end
+
+            within '.stock-contents.carton' do
+              expect(page).to have_selector '.item-qty-show', count: 2, text: '1 x Shipped'
+              expect(page).to have_selector '.item-price', text: '---'
+              expect(page).to have_selector '.item-total', text: '---'
+            end
+
+            within '#order_tab_summary' do
+              expect(page).to have_selector '#shipment_status', text: 'Shipped'
+              expect(page).to have_selector '#payment_status', text: 'Paid'
+            end
+          end
+        end
+      end
+    end
+  end
 
   def fill_in_address
     address = "order_bill_address_attributes"
-    fill_in "#{address}_firstname", :with => "Ryan"
-    fill_in "#{address}_lastname", :with => "Bigg"
-    fill_in "#{address}_address1", :with => "143 Swan Street"
-    fill_in "#{address}_city", :with => "Richmond"
-    select "Ohio", :from => "#{address}_state_id"
-    fill_in "#{address}_zipcode", :with => "12345"
-    fill_in "#{address}_phone", :with => "(555) 555-5555"
+
+    if use_address_full_name?
+      fill_in "#{address}_name", with: "Ryan Bigg"
+    else
+      fill_in "#{address}_firstname", with: "Ryan"
+      fill_in "#{address}_lastname", with: "Bigg"
+    end
+
+    fill_in "#{address}_address1", with: "143 Swan Street"
+    fill_in "#{address}_city", with: "Richmond"
+    select "Ohio", from: "#{address}_state_id"
+    fill_in "#{address}_zipcode", with: "12345"
+    fill_in "#{address}_phone", with: "(555) 555-5555"
+  end
+
+  def use_address_full_name?
+    SolidusSupport.combined_first_and_last_name_in_address? 
   end
 
   def add_product_to_cart

data/spec/models/spree/assemblies_part_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Spree
@@ -11,7 +13,7 @@ module Spree
 
     context "get" do
       it "brings part by product and variant id" do
-        subject.class.get(product.id, variant.id).part.should == variant
+        expect(subject.class.get(product.id, variant.id).part).to eq variant
       end
     end
   end

data/spec/models/spree/inventory_unit_spec.rb

@@ -1,16 +1,25 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Spree
   describe InventoryUnit do
+    subject { described_class.create!(attributes) }
+
     let!(:order) { create(:order_with_line_items) }
     let(:line_item) { order.line_items.first }
     let(:product) { line_item.product }
+    let(:shipment) { create(:shipment, order: order) }
 
-    subject { InventoryUnit.create(line_item: line_item, variant: line_item.variant, order: order) }
+    if Spree.solidus_gem_version < Gem::Version.new('2.5.x')
+      let(:attributes) { { shipment: shipment, line_item: line_item, variant: line_item.variant, order: order } }
+    else
+      let(:attributes) { { shipment: shipment, line_item: line_item, variant: line_item.variant } }
+    end
 
-    context 'if the unit is not part of an assembly' do      
-      it 'it will return the percentage of a line item' do
-        expect(subject.percentage_of_line_item).to eql(BigDecimal.new(1))
+    context 'if the unit is not part of an assembly' do
+      it 'will return the percentage of a line item' do
+        expect(subject.percentage_of_line_item).to eql(BigDecimal(1))
       end
     end
 
@@ -23,9 +32,9 @@ module Spree
         order.finalize!
       end
 
-      it 'it will return the percentage of a line item' do
+      it 'will return the percentage of a line item' do
         subject.line_item = line_item
-      	expect(subject.percentage_of_line_item).to eql(BigDecimal.new(0.5, 2))
+        expect(subject.percentage_of_line_item).to eql(BigDecimal(0.5, 2))
       end
     end
   end

data/spec/models/spree/line_item_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Spree
@@ -53,8 +55,8 @@ module Spree
       end
 
       it "verifies inventory units via OrderInventoryAssembly" do
-        OrderInventoryAssembly.should_receive(:new).with(line_item).and_return(inventory)
-        inventory.should_receive(:verify).with(line_item.target_shipment)
+        expect(OrderInventoryAssembly).to receive(:new).with(line_item).and_return(inventory)
+        expect(inventory).to receive(:verify).with(line_item.target_shipment)
         line_item.quantity = 2
         line_item.save
       end
@@ -62,8 +64,8 @@ module Spree
 
     context "updates regular line item" do
       it "verifies inventory units via OrderInventory" do
-        OrderInventory.should_receive(:new).with(line_item.order, line_item).and_return(inventory)
-        inventory.should_receive(:verify).with(line_item.target_shipment)
+        expect(OrderInventory).to receive(:new).with(line_item.order, line_item).and_return(inventory)
+        expect(inventory).to receive(:verify).with(line_item.target_shipment)
         line_item.quantity = 2
         line_item.save
       end

data/spec/models/spree/order_contents_spec.rb

@@ -1,7 +1,11 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Spree
   describe OrderContents do
+    subject { described_class.new(order) }
+
     let!(:store) { create :store }
     let(:order) { Order.create }
 
@@ -10,8 +14,6 @@ module Spree
 
     let(:bundle) { create(:product) }
 
-    subject { OrderContents.new(order) }
-
     before { bundle.parts.push [guitar, bass] }
 
     context "same variant within bundle and as regular product" do
@@ -30,9 +32,8 @@ module Spree
         end
 
         it "destroys accurate number of inventory units" do
-          expect {
-            subject.remove(guitar, 3)
-          }.to change { InventoryUnit.count }.by(-3)
+          expect { subject.remove(guitar, 3) }.
+          to change(InventoryUnit, :count).by(-3)
         end
       end
     end

data/spec/models/spree/order_inventory_assembly_spec.rb

@@ -1,7 +1,11 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Spree
   describe OrderInventoryAssembly do
+    subject { described_class.new(line_item) }
+
     let(:order) { create(:order_with_line_items) }
     let(:line_item) { order.line_items.first }
     let(:bundle) { line_item.product }
@@ -11,13 +15,11 @@ module Spree
       bundle.parts << [parts]
       bundle.set_part_count(parts.first, 3)
 
-      line_item.update_attributes!(quantity: 3)
+      line_item.update!(quantity: 3)
       order.reload.create_proposed_shipments
-      order.finalize! 
+      order.finalize!
     end
 
-    subject { OrderInventoryAssembly.new(line_item) }
-
     context "inventory units count" do
       it "calculates the proper value for the bundle" do
         expected_units_count = line_item.quantity * bundle.assemblies_parts.to_a.sum(&:count)
@@ -34,7 +36,7 @@ module Spree
         it "inserts new inventory units for every bundle part" do
           expected_units_count = original_units_count + bundle.assemblies_parts.to_a.sum(&:count)
           subject.verify
-          expect(OrderInventoryAssembly.new(line_item.reload).inventory_units.count).to eql(expected_units_count)
+          expect(described_class.new(line_item.reload).inventory_units.count).to eql(expected_units_count)
         end
       end
 
@@ -46,7 +48,7 @@ module Spree
           subject.verify
 
           # needs to reload so that inventory units are fetched from updates order.shipments
-          updated_units_count = OrderInventoryAssembly.new(line_item.reload).inventory_units.count
+          updated_units_count = described_class.new(line_item.reload).inventory_units.count
           expect(updated_units_count).to eql(expected_units_count)
         end
       end