solidus_pagy 1.0.0

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

@@ -0,0 +1,90 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::PromotionsController, type: :controller do
+  stub_authorization!
+
+  let!(:promotion1) do
+    create(
+      :promotion,
+      :with_action,
+      name: 'name1',
+      code: 'code1',
+      path: 'path1'
+    )
+  end
+  let!(:promotion2) do
+    create(
+      :promotion,
+      :with_action,
+      name: 'name2',
+      code: 'code2',
+      path: 'path2'
+    )
+  end
+  let!(:promotion3) do
+    create(
+      :promotion,
+      :with_action,
+      name: 'name2',
+      code: 'code3',
+      path: 'path3',
+      expires_at: Date.yesterday
+    )
+  end
+  let!(:category) { create :promotion_category }
+
+  context 'pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { page: 1, per_page: 10 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(10)
+    end
+  end
+
+  describe '#index' do
+    it 'succeeds' do
+      get :index
+      expect(assigns[:promotions])
+        .to match_array [promotion3, promotion2, promotion1]
+    end
+
+    it 'assigns promotion categories' do
+      get :index
+      expect(assigns[:promotion_categories]).to match_array [category]
+    end
+
+    context 'search' do
+      it 'pages results' do
+        get :index, params: { per_page: '1' }
+        expect(assigns[:promotions]).to eq [promotion3]
+      end
+
+      it 'filters by name' do
+        get :index, params: { q: { name_cont: promotion1.name } }
+        expect(assigns[:promotions]).to eq [promotion1]
+      end
+
+      it 'filters by code' do
+        get :index,
+          params: {
+            q: {
+              codes_value_cont: promotion1.codes.first.value
+            }
+          }
+        expect(assigns[:promotions]).to eq [promotion1]
+      end
+
+      it 'filters by path' do
+        get :index, params: { q: { path_cont: promotion1.path } }
+        expect(assigns[:promotions]).to eq [promotion1]
+      end
+
+      it 'filters by active' do
+        get :index, params: { q: { active: true } }
+        expect(assigns[:promotions]).to match_array [promotion2, promotion1]
+      end
+    end
+  end
+end

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

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::PropertiesController, type: :controller do
+  stub_authorization!
+
+  context 'pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { page: 1 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(15)
+    end
+  end
+end

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

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::StockItemsController, type: :controller do
+  stub_authorization!
+
+  let(:stock_location) { create(:stock_location) }
+
+  context 'pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { page: 1 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(15)
+    end
+  end
+
+  describe '#index' do
+    context 'without stock location' do
+      it 'returns 200' do
+        get :index
+
+        expect(response.status).to be(200)
+      end
+    end
+
+    context 'with stock location' do
+      it 'returns 200' do
+        get :index, params: { stock_location_id: stock_location.id }
+
+        expect(response.status).to be(200)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::StockLocationsController, type: :controller do
+  stub_authorization!
+
+  let!(:stock_location) { create(:stock_location) }
+
+  context 'with pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { page: 1 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(10)
+    end
+  end
+
+  describe '#index' do
+    it 'returns 200' do
+      get :index
+
+      expect(assigns[:collection]).not_to be_empty
+      expect(assigns[:collection]).to include(stock_location)
+    end
+  end
+end

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

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::StockMovementsController, type: :controller do
+  stub_authorization!
+
+  let!(:stock_location) do
+    create(
+      :stock_location_with_items
+    )
+  end
+  let!(:stock_movement1) do
+    create(
+      :stock_movement,
+      stock_item: stock_location.stock_items.first
+    )
+  end
+  let!(:stock_movement2) do
+    create(
+      :stock_movement,
+      stock_item: stock_location.stock_items.last
+    )
+  end
+
+  context 'pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { stock_location_id: stock_location.id, page: 1 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(20)
+    end
+  end
+
+  describe '#index' do
+    subject { get :index, params: params }
+
+    context 'with no params' do
+      let(:params) { { stock_location_id: stock_location.id } }
+
+      it 'responds with a successful status code' do
+        subject
+
+        expect(response).to be_successful
+      end
+
+      it 'responds with all the stock locations stock movements' do
+        subject
+
+        expect(assigns[:stock_movements]).to contain_exactly(
+          stock_movement1,
+          stock_movement2
+        )
+      end
+    end
+
+    context 'with search parameters' do
+      let(:params) do
+        {
+          stock_location_id: stock_location.id,
+          q: {
+            variant_sku_eq: stock_movement1.stock_item.variant.sku
+          }
+        }
+      end
+
+      it 'responds with a successful status code' do
+        subject
+
+        expect(response).to be_successful
+      end
+
+      it 'responds with the stock movements that match the search criteria' do
+        subject
+
+        expect(assigns[:stock_movements]).to contain_exactly(
+          stock_movement1,
+        )
+      end
+    end
+  end
+end

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

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::TaxRatesController, type: :controller do
+  stub_authorization!
+
+  context 'with pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { page: 1 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(10)
+    end
+  end
+end

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

@@ -0,0 +1,117 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::UsersController, type: :controller do
+  stub_authorization!
+
+  let(:user) { create(:user) }
+  let(:state) { create(:state, state_code: 'NY') }
+  let(:valid_address_attributes) do
+    {
+      name: 'Foo Bar',
+      city: 'New York',
+      country_id: state.country.id,
+      state_id: state.id,
+      phone: '555-555-5555',
+      address1: '123 Fake St.',
+      zipcode: '10001',
+    }
+  end
+
+  context 'with pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { id: user.id, page: 1 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(20)
+    end
+  end
+
+  describe '#index' do
+    stub_authorization! do |_user|
+      can :manage, Spree.user_class
+    end
+
+    context 'when the user can manage all users' do
+      it 'assigns a list of all users as @collection' do
+        get :index, params: { id: user.id }
+        expect(assigns(:collection)).to eq [user]
+      end
+
+      it 'assigns a ransack search for Spree.user_class' do
+        get :index, params: { id: user.id }
+        expect(assigns[:search]).to be_a Ransack::Search
+        expect(assigns[:search].klass).to eq Spree.user_class
+      end
+    end
+
+    context 'when user cannot manage some users' do
+      stub_authorization! do |_user|
+        can :manage, Spree.user_class
+        cannot :manage, Spree.user_class, email: 'not_accessible_user@example.com'
+      end
+
+      it 'assigns a list of accessible users as @collection' do
+        create(:user, email: 'not_accessible_user@example.com')
+
+        get :index, params: { id: user.id }
+        expect(assigns(:collection)).to eq [user]
+      end
+    end
+
+    context 'when Spree.user_class does not respond to #accessible_by' do
+      it 'assigns a list of all users as @collection' do
+        allow(Spree.user_class).to receive(:respond_to?).and_call_original
+        allow(Spree.user_class).to receive(:respond_to?)
+          .with(:accessible_by).and_return(false)
+
+        get :index, params: { id: user.id }
+        expect(assigns(:collection)).to eq [user]
+      end
+    end
+  end
+
+  describe '#orders' do
+    stub_authorization! do |_user|
+      can :manage, Spree.user_class
+    end
+
+    let(:order) { create(:order) }
+
+    before { user.orders << order }
+
+    it 'assigns a list of the users orders' do
+      get :orders, params: { 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
+      get :orders, params: { id: user.id }
+      expect(assigns[:search]).to be_a(Ransack::Search)
+      expect(assigns[:search].klass).to eq(Spree::Order)
+    end
+  end
+
+  describe '#items' do
+    stub_authorization! do |_user|
+      can :manage, Spree.user_class
+    end
+
+    let(:order) { create(:order) }
+
+    before { user.orders << order }
+
+    it 'assigns a list of the users orders' do
+      get :items, params: { 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
+      get :items, params: { id: user.id }
+      expect(assigns[:search]).to be_a(Ransack::Search)
+      expect(assigns[:search].klass).to eq(Spree::Order)
+    end
+  end
+end

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

@@ -0,0 +1,76 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::VariantsController, type: :controller do
+  stub_authorization!
+
+  let!(:product) { create(:product) }
+
+  context 'with pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { product_id: product.slug, page: 1 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(20)
+    end
+  end
+
+  describe '#index' do
+    let(:params) { { product_id: product.slug } }
+
+    subject { get :index, params: params }
+
+    context 'when the value of @parent' do
+      it 'is the product' do
+        subject
+        expect(assigns(:parent)).to eq product
+      end
+
+      context 'with a deleted product' do
+        before { product.discard }
+
+        it 'is the product' do
+          subject
+          expect(assigns(:parent)).to eq product
+        end
+      end
+    end
+
+    context 'when the value of @collection' do
+      let!(:variant) { create(:variant, product: product) }
+      let!(:deleted_variant) { create(:variant, product: product) }
+
+      context 'with soft-deleted variants' do
+        before { deleted_variant.discard }
+
+        context 'when deleted is not requested' do
+          it 'excludes deleted variants' do
+            subject
+            expect(assigns(:collection)).to include variant
+            expect(assigns(:collection)).not_to include deleted_variant
+          end
+        end
+
+        context 'when deleted is requested' do
+          let(:params) { { product_id: product.slug, deleted: 'on' } }
+
+          it 'includes deleted variants' do
+            subject
+            expect(assigns(:collection)).to include variant
+            expect(assigns(:collection)).to include deleted_variant
+          end
+        end
+
+        context 'when existent product id not given' do
+          let(:params) { { product_id: 'non-existent-product' } }
+
+          it 'cannot find non-existent product' do
+            subject
+            expect(response).to redirect_to(spree.admin_products_path)
+            expect(flash[:error]).to eql('Product is not found')
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::ZonesController, type: :controller do
+  stub_authorization!
+
+  context 'with pagination' do
+    it 'can page through the promotions' do
+      get :index, params: { page: 1 }
+      expect(assigns[:pagy].page).to eq(1)
+      expect(assigns[:pagy].items).to eq(20)
+    end
+  end
+end

data/spec/solidus_pagy/configuration_spec.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe SolidusPagy::Configuration do
+  before do
+    SolidusPagy.configure do |config|
+      config.nav_helper = 'pagy_nav'
+      config.page = 1
+      config.items = 20
+      config.outset = 0
+      config.orders_per_page = 15
+      config.properties_per_page = 15
+      config.promotions_per_page = 15
+      config.admin_products_per_page = 10
+      config.admin_variants_per_page = 20
+    end
+  end
+
+  it 'read nav_helper value' do
+    expect(SolidusPagy.config.nav_helper).to be('pagy_nav')
+  end
+
+  it 'read page value' do
+    expect(SolidusPagy.config.page).to be(1)
+  end
+
+  it 'read items value' do
+    expect(SolidusPagy.config.items).to be(20)
+  end
+
+  it 'read outset value' do
+    expect(SolidusPagy.config.outset).to be(0)
+  end
+
+  it 'read orders_per_page value' do
+    expect(SolidusPagy.config.orders_per_page).to be(15)
+  end
+
+  it 'read properties_per_page value' do
+    expect(SolidusPagy.config.properties_per_page).to be(15)
+  end
+
+  it 'read promotions_per_page value' do
+    expect(SolidusPagy.config.promotions_per_page).to be(15)
+  end
+
+  it 'read admin_products_per_page value' do
+    expect(SolidusPagy.config.admin_products_per_page).to be(10)
+  end
+
+  it 'read admin_variants_per_page value' do
+    expect(SolidusPagy.config.admin_variants_per_page).to be(20)
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+# Configure Rails Environment
+ENV['RAILS_ENV'] = 'test'
+
+# Run Coverage report
+require 'solidus_dev_support/rspec/coverage'
+
+# Create the dummy app if it's still missing.
+dummy_env = "#{__dir__}/dummy/config/environment.rb"
+system 'bin/rake extension:test_app' unless File.exist? dummy_env
+require dummy_env
+
+# Requires factories and other useful helpers defined in spree_core.
+require 'solidus_dev_support/rspec/feature_helper'
+
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir["#{__dir__}/support/**/*.rb"].sort.each { |f| require f }
+
+# Requires factories defined in lib/solidus_pagy/testing_support/factories.rb
+SolidusDevSupport::TestingSupport::Factories.load_for(SolidusPagy::Engine)
+
+RSpec.configure do |config|
+  config.infer_spec_type_from_file_location!
+  config.use_transactional_fixtures = false
+
+  if Spree.solidus_gem_version < Gem::Version.new('2.11')
+    config.extend Spree::TestingSupport::AuthorizationHelpers::Request, type: :system
+  end
+end

data/spec/support/devise.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+require 'devise'
+
+RSpec.configure do |config|
+  config.include Devise::Test::ControllerHelpers, type: :controller
+  config.include Devise::Test::IntegrationHelpers, type: :request
+end

data/spec/support/pagy.rb.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+require 'pagy'
+
+RSpec.configure do |config|
+  config.include Pagy::Backend
+  config.include Pagy::Frontend
+end

data/spec/support/solidus.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require 'spree/testing_support/flash'
+require 'spree/testing_support/url_helpers'
+require 'spree/testing_support/controller_requests'
+require 'spree/testing_support/authorization_helpers'
+require 'spree/testing_support/translations'
+require 'spree/testing_support/order_walkthrough'
+
+RSpec.configure do |config|
+  config.include Spree::TestingSupport::Flash
+  config.include Spree::TestingSupport::UrlHelpers
+  config.include Spree::TestingSupport::ControllerRequests, type: :controller
+  config.include Spree::TestingSupport::Translations
+  config.include Rails.application.routes.url_helpers
+end