spree_backend 3.1.5 → 3.1.6

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

@@ -1,37 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  module Admin
-    describe ReturnIndexController, type: :controller do
-      stub_authorization!
-
-      describe "#return_authorizations" do
-        let(:return_authorization) { create(:return_authorization) }
-
-        subject do
-          spree_get :return_authorizations
-        end
-
-        before { subject }
-
-        it "loads return authorizations" do
-          expect(assigns(:collection)).to include(return_authorization)
-        end
-      end
-
-      describe "#customer_returns" do
-        let(:customer_return) { create(:customer_return) }
-
-        subject do
-          spree_get :customer_returns
-        end
-
-        before { subject }
-
-        it "loads customer returns" do
-          expect(assigns(:collection)).to include(customer_return)
-        end
-      end
-    end
-  end
-end

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

@@ -1,27 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Admin::ReturnItemsController, :type => :controller do
-  stub_authorization!
-
-  describe '#update' do
-    let(:customer_return) { create(:customer_return) }
-    let(:return_item) { customer_return.return_items.first }
-    let(:old_acceptance_status) { 'accepted' }
-    let(:new_acceptance_status) { 'rejected' }
-
-    subject do
-      spree_put :update, id: return_item.to_param, return_item: {acceptance_status: new_acceptance_status}
-    end
-
-    it 'updates the return item' do
-      expect {
-        subject
-      }.to change { return_item.reload.acceptance_status }.from(old_acceptance_status).to(new_acceptance_status)
-    end
-
-    it 'redirects to the custome return' do
-      subject
-      expect(response).to redirect_to spree.edit_admin_order_customer_return_path(customer_return.order, customer_return)
-    end
-  end
-end

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

@@ -1,35 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Admin::RootController do
-
-  context "unauthorized request" do
-
-    before :each do
-      allow(controller).to receive(:spree_current_user).and_return(nil)
-    end
-
-    it "redirects to orders path by default" do
-      get :index
-
-      expect(response).to redirect_to '/admin/orders'
-    end
-  end
-
-  context "authorized request" do
-    stub_authorization!
-
-    it "redirects to orders path by default" do
-      get :index
-
-      expect(response).to redirect_to '/admin/orders'
-    end
-
-    it "redirects to wherever admin_root_redirects_path tells it to" do
-      expect(controller).to receive(:admin_root_redirect_path).and_return('/grooot')
-
-      get :index
-
-      expect(response).to redirect_to '/grooot'
-    end
-  end
-end

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

@@ -1,56 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Admin::SearchController, :type => :controller do
-  stub_authorization!
-  # Regression test for ernie/ransack#176
-
-  describe "users" do
-    let(:user) { create(:user, :email => "spree_commerce@example.com") }
-
-    before do
-      user.ship_address = create(:address)
-      user.bill_address = create(:address)
-      user.save
-    end
-
-    it "can find a user by their email "do
-      spree_xhr_get :users, :q => user.email
-      expect(assigns[:users]).to include(user)
-    end
-
-    it "can find a user by their ship address's first name" do
-      spree_xhr_get :users, :q => user.ship_address.firstname
-      expect(assigns[:users]).to include(user)
-    end
-
-    it "can find a user by their ship address's last name" do
-      spree_xhr_get :users, :q => user.ship_address.lastname
-      expect(assigns[:users]).to include(user)
-    end
-
-    it "can find a user by their bill address's first name" do
-      spree_xhr_get :users, :q => user.bill_address.firstname
-      expect(assigns[:users]).to include(user)
-    end
-
-    it "can find a user by their bill address's last name" do
-      spree_xhr_get :users, :q => user.bill_address.lastname
-      expect(assigns[:users]).to include(user)
-    end
-  end
-
-  describe "products" do
-    let(:product) { create(:product, :name => "Example Product") }
-
-    it "can find a product by its name "do
-      spree_xhr_get :products, :q => product.name
-      expect(assigns[:products]).to include(product)
-    end
-
-    it "can find a product by its slug "do
-      spree_xhr_get :products, :q => product.slug
-      expect(assigns[:products]).to include(product)
-    end
-  end
-
-end

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

@@ -1,14 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Admin::ShippingMethodsController, :type => :controller do
-  stub_authorization!
-
-  # Regression test for #1240
-  it "should not hard-delete shipping methods" do
-    shipping_method = stub_model(Spree::ShippingMethod)
-    allow(Spree::ShippingMethod).to receive_messages :find => shipping_method
-    expect(shipping_method.deleted_at).to be_nil
-    spree_delete :destroy, :id => 1
-    expect(shipping_method.reload.deleted_at).not_to be_nil
-  end
-end

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

@@ -1,19 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  module Admin
-    describe StockItemsController, :type => :controller do
-      stub_authorization!
-
-      context "formats" do
-        let!(:stock_item) { create(:variant).stock_items.first }
-
-        it "destroy stock item via js" do
-          expect {
-            spree_delete :destroy, format: :js, id: stock_item
-          }.to change{ StockItem.count }.by(-1)
-        end
-      end
-    end
-  end
-end

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

@@ -1,41 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  module Admin
-    describe StockLocationsController, :type => :controller do
-      stub_authorization!
-      
-      # Regression for #4272
-      context "with no countries present" do
-        it "cannot create a new stock location" do
-          spree_get :new
-          expect(flash[:error]).to eq(Spree.t(:stock_locations_need_a_default_country))
-          expect(response).to redirect_to(spree.admin_stock_locations_path)
-        end
-      end
-
-      context "with a default country present" do
-        before do
-          country = FactoryGirl.create(:country)
-          Spree::Config[:default_country_id] = country.id
-        end
-
-        it "can create a new stock location" do
-          spree_get :new
-          expect(response).to be_success
-        end
-      end
-
-      context "with a country with the ISO code of 'US' existing" do
-        before do
-          FactoryGirl.create(:country, iso: 'US')
-        end
-
-        it "can create a new stock location" do
-          spree_get :new
-          expect(response).to be_success
-        end 
-      end
-    end
-  end
-end

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

@@ -1,41 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  describe Admin::StockTransfersController, :type => :controller do
-    stub_authorization!
-
-    let!(:stock_transfer1) {
-      StockTransfer.create do |transfer|
-        transfer.source_location_id = 1
-        transfer.destination_location_id = 2
-        transfer.reference = 'PO 666'
-      end }
-
-    let!(:stock_transfer2) {
-      StockTransfer.create do |transfer|
-        transfer.source_location_id = 3
-        transfer.destination_location_id = 4
-        transfer.reference = 'PO 666'
-      end }
-
-
-    context "#index" do
-      it "gets all transfers without search criteria" do
-        spree_get :index
-        expect(assigns[:stock_transfers].count).to eq 2
-      end
-
-      it "searches by source location" do
-        spree_get :index, :q => { :source_location_id_eq => 1 }
-        expect(assigns[:stock_transfers].count).to eq 1
-        expect(assigns[:stock_transfers]).to include(stock_transfer1)
-      end
-
-      it "searches by destination location" do
-        spree_get :index, :q => { :destination_location_id_eq => 4 }
-        expect(assigns[:stock_transfers].count).to eq 1
-        expect(assigns[:stock_transfers]).to include(stock_transfer2)
-      end
-    end
-  end
-end

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

@@ -1,34 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  module Admin
-    describe TaxCategoriesController, :type => :controller do
-      stub_authorization!
-
-      describe 'GET #index' do
-        subject { spree_get :index }
-
-        it 'should be successful' do
-          expect(subject).to be_success
-        end
-      end
-
-      describe 'PUT #update' do
-        let(:tax_category) { create :tax_category }
-    
-        subject { spree_put :update, {id: tax_category.id, tax_category: { name: 'Foo', tax_code: 'Bar' }}}
-
-        it 'should redirect' do
-          expect(subject).to be_redirect
-        end
-
-        it 'should update' do
-          subject
-          tax_category.reload
-          expect(tax_category.name).to eq('Foo')
-          expect(tax_category.tax_code).to eq('Bar')
-        end
-      end
-    end
-  end
-end

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

@@ -1,161 +0,0 @@
-require 'spec_helper'
-require 'spree/testing_support/bar_ability'
-
-describe Spree::Admin::UsersController, :type => :controller do
-  let(:user) { create(:user) }
-  let(:mock_user) { mock_model Spree.user_class }
-
-  before do
-    allow(controller).to receive_messages :spree_current_user => user
-    user.spree_roles.clear
-    stub_const('Spree::User', user.class)
-  end
-
-  context "#show" do
-    before do
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
-    end
-
-    it "redirects to edit" do
-      spree_get :show, id: user.id
-      expect(response).to redirect_to spree.edit_admin_user_path(user)
-    end
-  end
-
-  context '#authorize_admin' do
-    before { use_mock_user }
-
-    it 'grant access to users with an admin role' do
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
-      spree_post :index
-      expect(response).to render_template :index
-    end
-
-    it "allows admins to update a user's API key" do
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
-      expect(mock_user).to receive(:generate_spree_api_key!).and_return(true)
-      spree_put :generate_api_key, id: mock_user.id
-      expect(response).to redirect_to(spree.edit_admin_user_path(mock_user))
-    end
-
-    it "allows admins to clear a user's API key" do
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
-      expect(mock_user).to receive(:clear_spree_api_key!).and_return(true)
-      spree_put :clear_api_key, id: mock_user.id
-      expect(response).to redirect_to(spree.edit_admin_user_path(mock_user))
-    end
-
-    it 'deny access to users with an bar role' do
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'bar')
-      Spree::Ability.register_ability(BarAbility)
-      spree_post :index
-      expect(response).to redirect_to(spree.forbidden_path)
-    end
-
-    it 'deny access to users with an bar role' do
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'bar')
-      Spree::Ability.register_ability(BarAbility)
-      spree_post :update, { id: '9' }
-      expect(response).to redirect_to(spree.forbidden_path)
-    end
-
-    it 'deny access to users without an admin role' do
-      allow(user).to receive_messages :has_spree_role? => false
-      spree_post :index
-      expect(response).to redirect_to(spree.forbidden_path)
-    end
-  end
-
-  describe "#create" do
-    before do
-      use_mock_user
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
-    end
-
-    it "can create a shipping_address" do
-      expect(Spree.user_class).to receive(:new).with(hash_including(
-        "ship_address_attributes" => { "city" => "New York" }
-      ))
-      spree_post :create, { :user => { :ship_address_attributes => { :city => "New York" } } }
-    end
-
-    it "can create a billing_address" do
-      expect(Spree.user_class).to receive(:new).with(hash_including(
-        "bill_address_attributes" => { "city" => "New York" }
-      ))
-      spree_post :create, { :user => { :bill_address_attributes => { :city => "New York" } } }
-    end
-  end
-
-  describe "#update" do
-    before do
-      use_mock_user
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
-    end
-
-    it "allows shipping address attributes through" do
-      expect(mock_user).to receive(:update_attributes).with(hash_including(
-        "ship_address_attributes" => { "city" => "New York" }
-      ))
-      spree_put :update, { :id => mock_user.id, :user => { :ship_address_attributes => { :city => "New York" } } }
-    end
-
-    it "allows billing address attributes through" do
-      expect(mock_user).to receive(:update_attributes).with(hash_including(
-        "bill_address_attributes" => { "city" => "New York" }
-      ))
-      spree_put :update, { :id => mock_user.id, :user => { :bill_address_attributes => { :city => "New York" } } }
-    end
-
-    it "allows updating without password resetting" do
-      expect(mock_user).to receive(:update_attributes).with(hash_not_including(password: '', password_confirmation: ''))
-      spree_put :update, id: mock_user.id, user: { password: '', password_confirmation: '', email: 'spree@example.com' }
-    end
-  end
-
-  describe "#orders" do
-    let(:order) { create(:order) }
-    before do
-      user.orders << order
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
-    end
-
-    it "assigns a list of the users orders" do
-      spree_get :orders, { :id => user.id }
-      expect(assigns[:orders].count).to eq 1
-      expect(assigns[:orders].first).to eq order
-    end
-
-    it "assigns a ransack search for Spree::Order" do
-      spree_get :orders, { :id => user.id }
-      expect(assigns[:search]).to be_a Ransack::Search
-      expect(assigns[:search].klass).to eq Spree::Order
-    end
-  end
-
-  describe "#items" do
-    let(:order) { create(:order) }
-    before do
-      user.orders << order
-      user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
-    end
-
-    it "assigns a list of the users orders" do
-      spree_get :items, { :id => user.id }
-      expect(assigns[:orders].count).to eq 1
-      expect(assigns[:orders].first).to eq order
-    end
-
-    it "assigns a ransack search for Spree::Order" do
-      spree_get :items, { :id => user.id }
-      expect(assigns[:search]).to be_a Ransack::Search
-      expect(assigns[:search].klass).to eq Spree::Order
-    end
-  end
-end
-
-def use_mock_user
-  allow(mock_user).to receive(:save).and_return(true)
-  allow(Spree.user_class).to receive(:find).with(mock_user.id.to_s).and_return(mock_user)
-  allow(Spree.user_class).to receive(:new).and_return(mock_user)
-end