spree_backend 3.1.5 → 3.1.6

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

@@ -1,252 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  module Admin
-    class WidgetsController < Spree::Admin::ResourceController
-      prepend_view_path('spec/test_views')
-
-      def model_class
-        Widget
-      end
-    end
-  end
-end
-
-describe Spree::Admin::WidgetsController, :type => :controller do
-  stub_authorization!
-
-  after(:all) do
-    # Spree::Core::Engine.routes.reload_routes!
-    Rails.application.reload_routes!
-  end
-
-  with_model 'Widget', scope: :all do
-    table do |t|
-      t.string :name
-      t.integer :position
-      t.timestamps null: false
-    end
-
-    model do
-      acts_as_list
-      validates :name, presence: true
-    end
-  end
-
-  before do
-    Spree::Core::Engine.routes.draw do
-      namespace :admin do
-        resources :widgets do
-          post :update_positions, on: :member
-        end
-      end
-    end
-    stub_const('Spree::Widget', Widget)
-  end
-
-  describe '#new' do
-    subject do
-      spree_get :new
-    end
-
-    it 'succeeds' do
-      subject
-      expect(response).to be_success
-    end
-  end
-
-  describe '#edit' do
-    let(:widget) { Widget.create!(name: 'a widget') }
-
-    subject do
-      spree_get :edit, id: widget.to_param
-    end
-
-    it 'succeeds' do
-      subject
-      expect(response).to be_success
-    end
-  end
-
-  describe '#create' do
-    let(:params) do
-      {widget: {name: 'a widget'}}
-    end
-
-    subject { spree_post :create, params }
-
-    it 'creates the resource' do
-      expect { subject }.to change { Widget.count }.by(1)
-    end
-
-    context 'without any parameters' do
-      let(:params) { {} }
-
-      before do
-        allow_any_instance_of(Widget).to receive(:name).and_return('some name')
-      end
-
-      it 'creates the resource' do
-        expect { subject }.to change { Widget.count }.by(1)
-      end
-    end
-  end
-
-  describe '#update' do
-    let(:widget) { Widget.create!(name: 'a widget') }
-
-    let(:params) do
-      {
-        id: widget.to_param,
-        widget: {name: 'widget renamed'},
-      }
-    end
-
-    subject { spree_put :update, params }
-
-    it 'updates the resource' do
-      expect { subject }.to change { widget.reload.name }.from('a widget').to('widget renamed')
-    end
-  end
-
-  describe '#destroy' do
-    let!(:widget) { Widget.create!(name: 'a widget') }
-    let(:params) { {id: widget.id} }
-
-    subject {
-      spree_delete :destroy, params
-    }
-
-    it 'destroys the resource' do
-      expect { subject }.to change { Widget.count }.from(1).to(0)
-    end
-  end
-
-  describe '#update_positions' do
-    let(:widget_1) { Widget.create!(name: 'widget 1', position: 1) }
-    let(:widget_2) { Widget.create!(name: 'widget 2', position: 2) }
-
-    subject do
-      spree_post :update_positions, id: widget_1.to_param,
-        positions: { widget_1.id => '2', widget_2.id => '1' }, format: 'js'
-    end
-
-    it 'updates the position of widget 1' do
-      expect { subject }.to change { widget_1.reload.position }.from(1).to(2)
-    end
-
-    it 'updates the position of widget 2' do
-      expect { subject }.to change { widget_2.reload.position }.from(2).to(1)
-    end
-
-    it 'touches updated_at' do
-      Timecop.scale(3600) do
-        expect { subject }.to change { widget_1.reload.updated_at }
-      end
-    end
-  end
-end
-
-module Spree
-  module Submodule
-    class Post < Spree::Base
-    end
-  end
-  module Admin
-    module Submodule
-      class PostsController < Spree::Admin::ResourceController
-        prepend_view_path('spec/test_views')
-
-        def model_class
-          Spree::Submodule::Post
-        end
-      end
-    end
-  end
-end
-
-describe Spree::Admin::Submodule::PostsController, type: :controller do
-  stub_authorization!
-
-  after(:all) do
-    # Spree::Core::Engine.routes.reload_routes!
-    Rails.application.reload_routes!
-  end
-
-  with_table 'spree_posts', scope: :all do |t|
-    t.string :name
-    t.integer :position
-    t.timestamps null: false
-  end
-
-  before do
-    Spree::Core::Engine.routes.draw do
-      namespace :admin do
-        namespace :submodule do
-          resources :posts
-        end
-      end
-    end
-  end
-
-  describe '#new' do
-    subject do
-      spree_get :new
-    end
-
-    it 'succeeds' do
-      subject
-      expect(response).to be_success
-    end
-  end
-
-  describe '#edit' do
-    let(:submodule_post) { Spree::Submodule::Post.create!(name: 'a post') }
-
-    subject do
-      spree_get :edit, id: submodule_post.to_param
-    end
-
-    it 'succeeds' do
-      subject
-      expect(response).to be_success
-    end
-  end
-
-  describe '#create' do
-    let(:params) do
-      { submodule_post: { name: 'a post' } }
-    end
-
-    subject { spree_post :create, params }
-
-    it 'creates the resource' do
-      expect { subject }.to change { Spree::Submodule::Post.count }.by(1)
-    end
-  end
-
-  describe '#update' do
-    let(:post) { Spree::Submodule::Post.create!(name: 'a post') }
-
-    let(:params) do
-      { id: post.to_param, submodule_post: { name: 'post renamed' } }
-    end
-
-    subject { spree_put :update, params }
-
-    it 'updates the resource' do
-      expect { subject }.to change { post.reload.name }.from('a post').to('post renamed')
-    end
-  end
-
-  describe '#destroy' do
-    let!(:post) { Spree::Submodule::Post.create!(name: 'a post') }
-    let(:params) { { id: post.id } }
-
-    subject { spree_delete :destroy, params }
-
-    it 'destroys the resource' do
-      expect { subject }.to change { Spree::Submodule::Post.count }.from(1).to(0)
-    end
-  end
-end

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

@@ -1,225 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Admin::ReturnAuthorizationsController, :type => :controller do
-  stub_authorization!
-
-  # Regression test for #1370 #3
-  let!(:order) { create(:shipped_order, line_items_count: 3) }
-  let!(:return_authorization_reason) { create(:return_authorization_reason) }
-  let(:inventory_unit_1) { order.inventory_units.order('id asc')[0] }
-  let(:inventory_unit_2) { order.inventory_units.order('id asc')[1] }
-  let(:inventory_unit_3) { order.inventory_units.order('id asc')[2] }
-
-  describe "#load_return_authorization_reasons" do
-    let!(:inactive_rma_reason) { create(:return_authorization_reason, active: false) }
-
-    context "return authorization has an associated inactive reason" do
-      let!(:other_inactive_rma_reason) { create(:return_authorization_reason, active: false) }
-      let(:return_authorization) { create(:return_authorization, reason: inactive_rma_reason) }
-
-      it "loads all the active rma reasons" do
-        spree_get :edit, id: return_authorization.to_param, order_id: return_authorization.order.to_param
-        expect(assigns(:reasons)).to include(return_authorization_reason)
-        expect(assigns(:reasons)).to include(inactive_rma_reason)
-        expect(assigns(:reasons)).not_to include(other_inactive_rma_reason)
-      end
-    end
-
-    context "return authorization has an associated active reason" do
-      let(:return_authorization) { create(:return_authorization, reason: return_authorization_reason) }
-
-      it "loads all the active rma reasons" do
-        spree_get :edit, id: return_authorization.to_param, order_id: return_authorization.order.to_param
-        expect(assigns(:reasons)).to eq [return_authorization_reason]
-      end
-    end
-
-    context "return authorization doesn't have an associated reason" do
-      it "loads all the active rma reasons" do
-        spree_get :new, order_id: order.to_param
-        expect(assigns(:reasons)).to eq [return_authorization_reason]
-      end
-    end
-  end
-
-  describe "#load_return_items" do
-    shared_context 'without existing return items' do
-      context 'without existing return items' do
-        it 'has 3 new @form_return_items' do
-          subject
-          expect(assigns(:form_return_items).size).to eq 3
-          expect(assigns(:form_return_items).select(&:new_record?).size).to eq 3
-        end
-      end
-    end
-
-    shared_context 'with existing return items' do
-      context 'with existing return items' do
-        let!(:return_item_1) { create(:return_item, inventory_unit: inventory_unit_1, return_authorization: return_authorization) }
-
-        it 'has 1 existing return item and 2 new return items' do
-          subject
-          expect(assigns(:form_return_items).size).to eq 3
-          expect(assigns(:form_return_items).select(&:persisted?)).to eq [return_item_1]
-          expect(assigns(:form_return_items).select(&:new_record?).size).to eq 2
-        end
-      end
-    end
-
-    context '#new' do
-      subject { spree_get :new, order_id: order.to_param }
-
-      include_context 'without existing return items'
-    end
-
-    context '#edit' do
-      subject do
-        spree_get :edit, {
-          id: return_authorization.to_param,
-          order_id: order.to_param,
-        }
-      end
-
-      let(:return_authorization) { create(:return_authorization, order: order) }
-
-      include_context 'without existing return items'
-      include_context 'with existing return items'
-    end
-
-    context '#create failed' do
-      subject do
-        spree_post :create, {
-          return_authorization: {return_authorization_reason_id: -1}, # invalid reason_id
-          order_id: order.to_param,
-        }
-      end
-
-      include_context 'without existing return items'
-    end
-
-    context '#update failed' do
-      subject do
-        spree_put :update, {
-          return_authorization: {return_authorization_reason_id: -1}, # invalid reason_id
-          id: return_authorization.to_param,
-          order_id: order.to_param,
-        }
-      end
-
-      let(:return_authorization) { create(:return_authorization, order: order) }
-
-      include_context 'without existing return items'
-      include_context 'with existing return items'
-    end
-  end
-
-  describe "#load_reimbursement_types" do
-    let(:order)                             { create(:order) }
-    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) }
-
-    before do
-      spree_get :new, order_id: order.to_param
-    end
-
-    it "loads all the active reimbursement types" do
-      expect(assigns(:reimbursement_types)).to include(first_active_reimbursement_type)
-      expect(assigns(:reimbursement_types)).to include(second_active_reimbursement_type)
-      expect(assigns(:reimbursement_types)).not_to include(inactive_reimbursement_type)
-    end
-  end
-
-  context '#create' do
-    let(:stock_location) { create(:stock_location) }
-
-    subject { spree_post :create, params }
-
-    let(:params) do
-      {
-        order_id: order.to_param,
-        return_authorization: return_authorization_params,
-      }
-    end
-
-    let(:return_authorization_params) do
-      {
-        memo: "",
-        stock_location_id: stock_location.id,
-        return_authorization_reason_id: return_authorization_reason.id,
-      }
-    end
-
-    it "can create a return authorization" do
-      subject
-      expect(response).to redirect_to spree.admin_order_return_authorizations_path(order)
-    end
-  end
-
-  context '#update' do
-    let(:return_authorization) { create(:return_authorization, order: order) }
-
-    let(:params) do
-      {
-        id: return_authorization.to_param,
-        order_id: order.to_param,
-        return_authorization: return_authorization_params,
-      }
-    end
-    let(:return_authorization_params) do
-      {
-        memo: "",
-        return_items_attributes: return_items_params,
-      }
-    end
-
-    subject { spree_put :update, params }
-
-    context "adding an item" do
-      let(:return_items_params) do
-        {
-          '0' => {inventory_unit_id: inventory_unit_1.to_param}
-        }
-      end
-
-      context 'without existing items' do
-        it 'creates a new item' do
-          expect { subject }.to change { Spree::ReturnItem.count }.by(1)
-        end
-      end
-
-      context 'with existing completed items' do
-        let!(:completed_return_item) do
-          create(:return_item, {
-            return_authorization: return_authorization,
-            inventory_unit: inventory_unit_1,
-            reception_status: 'received',
-          })
-        end
-
-        it 'does not create new items' do
-          expect { subject }.to_not change { Spree::ReturnItem.count }
-          expect(assigns[:return_authorization].errors['return_items.inventory_unit']).to eq ["#{inventory_unit_1.id} has already been taken by return item #{completed_return_item.id}"]
-        end
-      end
-    end
-
-    context "removing an item" do
-      let!(:return_item) do
-        create(:return_item, return_authorization: return_authorization, inventory_unit: inventory_unit_1)
-      end
-
-      let(:return_items_params) do
-        {
-          '0' => {id: return_item.to_param, _destroy: '1'}
-        }
-      end
-
-      context 'with existing items' do
-        it 'removes the item' do
-          expect { subject }.to change { Spree::ReturnItem.count }.by(-1)
-        end
-      end
-    end
-  end
-end