solidus_backend 2.9.6 → 2.11.0

data/app/assets/javascripts/spree/backend/returns/return_item_selection.js

@@ -1,35 +0,0 @@
-Spree.ready(function() {
-  function checkAddItemBox() {
-    $(this).closest('tr').find('input.add-item').attr('checked', 'checked');
-    updateSuggestedAmount();
-  }
-
-  function updateSuggestedAmount() {
-    var totalPretaxRefund = 0;
-    var checkedItems = formFields.find('input.add-item:checked');
-    $.each(checkedItems, function(i, checkbox) {
-      totalPretaxRefund += parseFloat($(checkbox).parents('tr').find('.refund-amount-input').val());
-    });
-
-    var displayTotal = isNaN(totalPretaxRefund) ? '' : totalPretaxRefund.toFixed(2);
-    formFields.find('span#total_pre_tax_refund').html(displayTotal);
-  }
-
-  var formFields = $("[data-hook='admin_customer_return_form_fields'], \
-                     [data-hook='admin_return_authorization_form_fields']");
-
-  if(formFields.length > 0) {
-    updateSuggestedAmount();
-
-    formFields.find('input#select-all').on('change', function(ev) {
-      var checkBoxes = $(ev.currentTarget).parents('table:first').find('input.add-item');
-      checkBoxes.prop('checked', this.checked);
-      updateSuggestedAmount();
-    });
-
-    formFields.find('input.add-item').on('change', updateSuggestedAmount);
-    formFields.find('.refund-amount-input').on('keyup', updateSuggestedAmount);
-
-    formFields.find('input, select').not('.add-item').on('change', checkAddItemBox);
-  }
-});

data/app/assets/javascripts/spree/backend/templates/variants/autocomplete_stock.hbs

@@ -1,56 +0,0 @@
-<fieldset class="no-border-bottom">
-  <legend align="center">{{ t "select_stock" }}</legend>
-    <table class="stock-levels" data-hook="stock-levels">
-      <colgroup>
-        <col style="width: 30%;" />
-        <col style="width: 40%;" />
-        <col style="width: 20%;" />
-        <col style="width: 10%;" />
-      </colgroup>
-      <thead>
-        <th>{{ t "location" }}</th>
-        <th>{{ t "count_on_hand" }}</th>
-        <th>{{ t "quantity" }}</th>
-        <th class="actions"></th>
-      </thead>
-      <tbody>
-        {{#each variant.stock_items}}
-          <tr class="vertical-middle">
-            <td>{{this.stock_location_name}}</td>
-            {{#unless ../variant.track_inventory}}
-              <td>
-                It doesn't track inventory
-              </td>
-              <td>
-                <form>
-                  <input class="quantity" id="stock_item_quantity" data-stock-location-id="{{this.stock_location_id}}" type="number" min="1" value="1">
-                </form>
-              </td>
-              <td class="actions">
-                <button class="add_variant no-text fa fa-plus icon_link with-tip" data-stock-location-id="{{this.stock_location_id}}" title="{{ t "add" }}" data-action="add"></button>
-              </td>
-            {{else}}
-              {{#if this.available}}
-                <td>
-                  {{this.count_on_hand}}
-                  {{#if this.backorderable}} ({{ t "backorders_allowed" }}) {{/if}}
-                </td>
-                <td>
-                  <form>
-                    <input class="quantity" id="stock_item_quantity" data-stock-location-id="{{this.stock_location_id}}" type="number" min="1" value="1">
-                  </form>
-                </td>
-                <td class="actions">
-                  <button class="add_variant no-text fa fa-plus icon_link with-tip" data-stock-location-id="{{this.stock_location_id}}" title="{{ t "add" }}" data-action="add"></button>
-                </td>
-              {{else}}
-                <td>{{ t "out_of_stock" }}</td>
-                <td>0</td>
-                <td></td>
-              {{/if}}
-            {{/unless}}
-          </tr>
-        {{/each}}
-      </tbody>
-    </table>
-</fieldset>

data/app/views/spree/admin/orders/_add_product.html.erb

@@ -1,12 +0,0 @@
-<div id="add-line-item" class="js-shipment-add-variant" data-hook>
-  <fieldset class="no-border-bottom">
-    <legend align="center"><%= t('spree.add_product') %></legend>
-
-    <div data-hook="add_product_name" class="field">
-      <%= label_tag :add_variant_id, t('spree.name_or_sku') %>
-      <%= text_field_tag :add_variant_id, "", class: "variant_autocomplete fullwidth" %>
-    </div>
-  </fieldset>
-
-  <div id="stock_details"></div>
-</div>

data/script/rails

@@ -1,10 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
-
-ENGINE_ROOT = File.expand_path('..', __dir__)
-ENGINE_PATH = File.expand_path('../lib/spree/backend/engine', __dir__)
-
-require 'rails/all'
-require 'rails/engine/commands'

data/spec/controllers/spree/admin/base_controller_spec.rb

@@ -1,26 +0,0 @@
-# frozen_string_literal: true
-
-# Spree's rpsec controller tests get the Spree::ControllerHacks
-# we don't need those for the anonymous controller here, so
-# we call process directly instead of get
-require 'spec_helper'
-
-describe Spree::Admin::BaseController, type: :controller do
-  controller(Spree::Admin::BaseController) do
-    def index
-      authorize! :update, Spree::Order
-      render plain: 'test'
-    end
-  end
-
-  context "unauthorized request" do
-    before do
-      allow_any_instance_of(Spree::Admin::BaseController).to receive(:try_spree_current_user).and_return(nil)
-    end
-
-    it "redirects to unauthorized" do
-      get :index
-      expect(response).to redirect_to '/unauthorized'
-    end
-  end
-end

data/spec/controllers/spree/admin/cancellations_controller_spec.rb

@@ -1,83 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe Spree::Admin::CancellationsController do
-  stub_authorization!
-
-  describe "#index" do
-    subject { get :index, params: { order_id: order.number } }
-
-    let(:order) { create(:order_ready_to_ship, line_items_count: 1) }
-
-    context "for an order with inventory" do
-      render_views
-
-      it "succeeds" do
-        expect(response).to be_ok
-      end
-    end
-  end
-
-  describe "#cancel" do
-    subject { post :short_ship, params: { order_id: order.number, inventory_unit_ids: inventory_units.map(&:id) } }
-
-    let(:order) { create(:order_with_line_items, line_items_attributes: [{ quantity: 4 }]) }
-    let(:referer) { "order_admin_page" }
-
-    context "no inventory unit ids are provided" do
-      subject { post :short_ship, params: { order_id: order.number } }
-
-      it "redirects back" do
-        subject
-        expect(response).to redirect_to(spree.admin_order_cancellations_path(order))
-      end
-
-      it "sets an error message" do
-        subject
-        expect(flash[:error]).to eq I18n.t('spree.no_inventory_selected')
-      end
-    end
-
-    context "unable to find all the inventory" do
-      let(:inventory_units) { [Spree::InventoryUnit.new(id: -1)] }
-
-      it "redirects back" do
-        subject
-        expect(response).to redirect_to(spree.admin_order_cancellations_path(order))
-      end
-
-      it "sets an error message" do
-        subject
-        expect(flash[:error]).to eq I18n.t('spree.unable_to_find_all_inventory_units')
-      end
-    end
-
-    context "successfully cancels inventory" do
-      let(:inventory_units) { order.inventory_units.not_canceled }
-
-      it "redirects to admin order edit" do
-        subject
-        expect(response).to redirect_to(spree.edit_admin_order_path(order))
-      end
-
-      it "sets an success message" do
-        subject
-        expect(flash[:success]).to eq I18n.t('spree.inventory_canceled')
-      end
-
-      it "creates a unit cancel" do
-        expect { subject }.to change { Spree::UnitCancel.count }.by(4)
-      end
-
-      it "cancels the inventory" do
-        subject
-        expect(order.reload.inventory_units.map(&:state).uniq).to match_array(['canceled'])
-      end
-
-      it "adjusts the order" do
-        expect { subject }.to change { order.reload.total }.by(-40.0)
-      end
-    end
-  end
-end

data/spec/controllers/spree/admin/customer_returns_controller_spec.rb

@@ -1,235 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-module Spree
-  module Admin
-    describe CustomerReturnsController, type: :controller do
-      stub_authorization!
-
-      describe "#index" do
-        let(:order)           { customer_return.order }
-        let(:customer_return) { create(:customer_return) }
-
-        subject do
-          get :index, params: { order_id: customer_return.order.to_param }
-        end
-
-        before { subject }
-
-        it "loads the order" do
-          expect(assigns(:order)).to eq order
-        end
-
-        it "loads the customer return" do
-          expect(assigns(:customer_returns)).to include(customer_return)
-        end
-      end
-
-      describe "#new" do
-        let(:order) { create(:shipped_order, line_items_count: 1) }
-        let!(:inactive_reimbursement_type)      { create(:reimbursement_type, active: false) }
-        let!(:first_active_reimbursement_type)  { create(:reimbursement_type) }
-        let!(:second_active_reimbursement_type) { create(:reimbursement_type) }
-
-        subject do
-          get :new, params: { order_id: order.to_param }
-        end
-
-        it "loads the order" do
-          subject
-          expect(assigns(:order)).to eq order
-        end
-
-        it "builds a new customer return" do
-          subject
-          expect(assigns(:customer_return)).to_not be_persisted
-        end
-
-        context "with previous customer return" do
-          let(:return_reason_1) { create(:return_reason) }
-          let(:return_reason_2) { create(:return_reason) }
-          let(:return_reason_3) { create(:return_reason) }
-          let!(:inactive_rma_reason) { create(:return_reason, active: false) }
-
-          let(:order) { create(:shipped_order, line_items_count: 4) }
-          let(:rma) { create(:return_authorization, order: order, reason: return_reason_1) }
-
-          let!(:rma_return_item) { create(:return_item, return_authorization: rma, inventory_unit: order.inventory_units.first, return_reason: return_reason_2) }
-          let!(:customer_return_return_item) { create(:return_item, return_authorization: nil, inventory_unit: order.inventory_units.last, return_reason: return_reason_3) }
-
-          context "all return items are associated with a customer return" do
-            let!(:previous_customer_return) { create(:customer_return_without_return_items, return_items: [rma_return_item, customer_return_return_item]) }
-
-            before { subject }
-
-            it "loads the possible return items" do
-              total_inventory_count = 4
-              rma_return_items_count = 1
-              customer_return_return_items_count = 1
-              expect(assigns(:new_return_items).length).to eq(total_inventory_count - rma_return_items_count - customer_return_return_items_count)
-            end
-
-            it "builds new return items" do
-              expect(assigns(:new_return_items).all? { |return_item| !return_item.persisted? }).to eq true
-            end
-
-            it "does not have any rma return items" do
-              expect(assigns(:rma_return_items)).to eq []
-            end
-
-            it "loads the correct return authorization reasons" do
-              expect(assigns(:reasons)).to match_array([return_reason_1, return_reason_2, return_reason_3])
-            end
-          end
-
-          context "there is a return item associated with an rma but not a customer return" do
-            let!(:previous_customer_return) { create(:customer_return_without_return_items, return_items: [customer_return_return_item]) }
-
-            before do
-              subject
-            end
-
-            it "loads the possible return items" do
-              rma_return_item_count = rma.return_items.count
-              total_unit_count = order.inventory_units.count
-              customer_returned_count = previous_customer_return.return_items.count
-              expected_total = total_unit_count - customer_returned_count - rma_return_item_count
-              expect(assigns(:new_return_items).length).to eq expected_total
-            end
-
-            it "builds new return items" do
-              expect(assigns(:new_return_items).all? { |return_item| !return_item.persisted? }).to eq true
-            end
-
-            it "loads the persisted rma return items" do
-              expect(assigns(:rma_return_items).all?(&:persisted?)).to eq true
-            end
-
-            it "has one rma return item" do
-              expect(assigns(:rma_return_items)).to include(rma_return_item)
-            end
-
-            it "loads the correct return authorization reasons" do
-              expect(assigns(:reasons)).to match_array([return_reason_1, return_reason_2, return_reason_3])
-            end
-          end
-        end
-      end
-
-      describe "#edit" do
-        let(:order)           { customer_return.order }
-        let(:customer_return) { create(:customer_return, line_items_count: 3) }
-
-        let!(:inactive_rma_reason) { create(:return_reason, active: false) }
-
-        let!(:accepted_return_item)            { customer_return.return_items.order('id').first.tap(&:accept!) }
-        let!(:rejected_return_item)            { customer_return.return_items.order('id').second.tap(&:reject!) }
-        let!(:manual_intervention_return_item) { customer_return.return_items.order('id').third.tap(&:require_manual_intervention!) }
-
-        subject do
-          get :edit, params: { order_id: order.to_param, id: customer_return.to_param }
-        end
-
-        it "loads the order" do
-          subject
-          expect(assigns(:order)).to eq order
-        end
-
-        it "loads the customer return" do
-          subject
-          expect(assigns(:customer_return)).to eq customer_return
-        end
-
-        it "loads the accepted return items" do
-          subject
-          expect(assigns(:accepted_return_items)).to eq [accepted_return_item]
-        end
-
-        it "loads the rejected return items" do
-          subject
-          expect(assigns(:rejected_return_items)).to eq [rejected_return_item]
-        end
-
-        it "loads the return items that require manual intervention" do
-          subject
-          expect(assigns(:manual_intervention_return_items)).to eq [manual_intervention_return_item]
-        end
-
-        it "loads the return items that are still pending" do
-          subject
-          expect(assigns(:pending_return_items)).to eq []
-        end
-
-        it "loads the reimbursements that are still pending" do
-          subject
-          expect(assigns(:pending_reimbursements)).to eq []
-        end
-
-        it "loads the correct return authorization reasons" do
-          subject
-          expect(assigns(:reasons)).to match(Spree::ReturnReason.active)
-        end
-
-        context "a return item has an inactive return authorization reason" do
-          before(:each) do
-            accepted_return_item.update_attributes(return_reason_id: inactive_rma_reason.id)
-          end
-
-          it "includes the inactive return authorization reason" do
-            subject
-            expect(assigns(:reasons)).to match(Spree::ReturnReason.active + [inactive_rma_reason])
-          end
-        end
-      end
-
-      describe "#create" do
-        let(:order) { create(:shipped_order, line_items_count: 1) }
-        let(:reception_status_event) { 'receive' }
-        let(:stock_location_id) { order.shipments.last.stock_location.id }
-        let(:customer_return_params) do
-          {
-            order_id: order.to_param,
-            customer_return: {
-              stock_location_id: stock_location_id,
-              return_items_attributes: {
-                "0" => {
-                  returned: "1",
-                  amount: "15.99",
-                  inventory_unit_id: order.inventory_units.shipped.last.id,
-                  reception_status_event: reception_status_event
-                }
-              }
-            }
-          }
-        end
-
-        subject { post :create, params: customer_return_params }
-
-        it { expect { subject }.to change { Spree::CustomerReturn.count }.by(1) }
-        it do
-          subject
-          expect(response).to redirect_to spree.edit_admin_order_customer_return_path(order, id: Spree::CustomerReturn.last.id)
-        end
-
-        it 'executes the reception status event on the return items' do
-          subject
-          customer_return = Spree::CustomerReturn.last
-          expect(customer_return.return_items.map(&:reception_status).uniq).to eq ['received']
-        end
-
-        context "missing stock location" do
-          let(:stock_location_id) { '' }
-          it { expect{ subject }.to_not change { Spree::CustomerReturn.count } }
-          it { subject; expect(response).to render_template(:new) }
-        end
-
-        context "missing reception status event" do
-          let(:reception_status_event) { '' }
-          it { expect{ subject }.to_not change { Spree::CustomerReturn.count } }
-          it { subject; expect(response).to redirect_to spree.new_admin_order_customer_return_path(order) }
-        end
-      end
-    end
-  end
-end