solidus_core 1.1.0 → 1.1.1

data/spec/models/spree/permission_sets/restricted_stock_display_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::RestrictedStockDisplay do
+  let(:ability) { Spree::Ability.new(user) }
+  let(:user) { create :user }
+
+  subject { ability }
+
+  let!(:variant) { create :variant }
+
+  let(:sl1) { create :stock_location, active: false }
+  let(:sl2) { create :stock_location, active: false }
+
+  let(:item1) { variant.stock_items.where(stock_location_id: sl1.id).first }
+  let(:item2) { variant.stock_items.where(stock_location_id: sl2.id).first }
+
+  before do
+    user.stock_locations << sl1
+  end
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:display, sl1) }
+    it { is_expected.to_not be_able_to(:display, sl2) }
+
+    it { is_expected.to be_able_to(:display, item1) }
+    it { is_expected.to_not be_able_to(:display, item2) }
+  end
+
+  context "when not activated" do
+    it { is_expected.to_not be_able_to(:display, sl1) }
+    it { is_expected.to_not be_able_to(:display, sl2) }
+
+    it { is_expected.to_not be_able_to(:display, item1) }
+    it { is_expected.to_not be_able_to(:display, item2) }
+  end
+end
+

data/spec/models/spree/permission_sets/restricted_stock_management_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::RestrictedStockManagement do
+  let(:ability) { Spree::Ability.new(user) }
+  let(:user) { create :user }
+
+  subject { ability }
+
+  let!(:variant) { create :variant }
+
+  let(:sl1) { create :stock_location, active: false }
+  let(:sl2) { create :stock_location, active: false }
+
+  let(:item1) { variant.stock_items.where(stock_location_id: sl1.id).first }
+  let(:item2) { variant.stock_items.where(stock_location_id: sl2.id).first }
+
+  before do
+    user.stock_locations << sl1
+  end
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:display, sl1) }
+    it { is_expected.to_not be_able_to(:display, sl2) }
+
+    it { is_expected.to be_able_to(:manage, item1) }
+    it { is_expected.to_not be_able_to(:manage, item2) }
+  end
+
+  context "when not activated" do
+    it { is_expected.to_not be_able_to(:display, sl1) }
+    it { is_expected.to_not be_able_to(:display, sl2) }
+
+    it { is_expected.to_not be_able_to(:manage, item1) }
+    it { is_expected.to_not be_able_to(:manage, item2) }
+  end
+end
+

data/spec/models/spree/permission_sets/restricted_stock_transfer_display_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::RestrictedStockTransferDisplay do
+  let(:ability) { Spree::Ability.new(user) }
+  let(:user) { create :user }
+
+  subject { ability }
+
+  let!(:sl1) { create :stock_location, active: false }
+  let!(:sl2) { create :stock_location, active: false }
+
+  let!(:source_transfer) { create :stock_transfer, source_location: sl1 }
+  let!(:other_source_transfer) { create :stock_transfer, source_location: sl2 }
+  let!(:dest_transfer) { create :stock_transfer, source_location: sl2, destination_location: sl1 }
+
+
+  before do
+    user.stock_locations << sl1
+  end
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:display, sl1) }
+    it { is_expected.to_not be_able_to(:display, sl2) }
+
+    it { is_expected.to be_able_to(:display, source_transfer) }
+    it { is_expected.to_not be_able_to(:display, other_source_transfer) }
+    it { is_expected.to be_able_to(:display, dest_transfer) }
+
+    it { is_expected.to be_able_to(:admin, source_transfer) }
+    it { is_expected.to_not be_able_to(:admin, other_source_transfer) }
+    it { is_expected.to be_able_to(:admin, dest_transfer) }
+  end
+
+  context "when not activated" do
+    it { is_expected.to_not be_able_to(:display, sl1) }
+    it { is_expected.to_not be_able_to(:display, sl2) }
+
+    it { is_expected.to_not be_able_to(:display, source_transfer) }
+    it { is_expected.to_not be_able_to(:display, other_source_transfer) }
+    it { is_expected.to_not be_able_to(:display, dest_transfer) }
+
+    it { is_expected.to_not be_able_to(:admin, source_transfer) }
+    it { is_expected.to_not be_able_to(:admin, other_source_transfer) }
+    it { is_expected.to_not be_able_to(:admin, dest_transfer) }
+  end
+end

data/spec/models/spree/permission_sets/restricted_stock_transfer_management_spec.rb

@@ -0,0 +1,160 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::RestrictedStockTransferManagement do
+  let(:ability) { Spree::Ability.new(user) }
+
+  subject { ability }
+
+  # Inactive stock locations will default to not being visible
+  # for users without explicit permissions.
+  let!(:source_location) { create :stock_location, active: false }
+  let!(:destination_location) { create :stock_location, active: false }
+
+  # This has the side effect of creating a stock item for each stock location above,
+  # which is what we actually want.
+  let!(:variant) { create :variant }
+
+  let(:transfer_with_source) { create :stock_transfer, source_location: source_location }
+  let(:transfer_with_destination) { create :stock_transfer, source_location: destination_location  }
+  let(:transfer_with_source_and_destination) do
+    create :stock_transfer, source_location: source_location, destination_location: destination_location
+  end
+
+  let(:transfer_amount) { 1 }
+  let(:source_transfer_item) do
+    transfer_with_source.transfer_items.create(variant: variant, expected_quantity: transfer_amount)
+  end
+  let(:destination_transfer_item) do
+    transfer_with_destination.transfer_items.create(variant: variant, expected_quantity: transfer_amount)
+  end
+  let(:source_and_destination_transfer_item) do
+    transfer_with_source_and_destination.transfer_items.create(variant: variant, expected_quantity: transfer_amount)
+  end
+
+  context "when activated" do
+    let(:user) { create :user, stock_locations: stock_locations }
+    let(:stock_locations) {[]}
+
+    before do
+      user.stock_locations = stock_locations
+      # When creating transfer_items for a stock transfer, stock items must have a count on hand
+      # with an amount that would allow a transfer item to pass validations (meaning the count on hand has to be equal
+      # to the expected_quantity for the transfer)
+      variant.stock_items.update_all count_on_hand: transfer_amount
+
+      described_class.new(ability).activate!
+    end
+
+    context "when the user is only associated with the source location" do
+      let(:stock_locations) {[source_location]}
+
+      it { is_expected.to be_able_to(:display, source_location) }
+      it { is_expected.to_not be_able_to(:display, destination_location) }
+
+      it { is_expected.to be_able_to(:display, Spree::StockTransfer) }
+      it { is_expected.to be_able_to(:admin, Spree::StockTransfer) }
+      it { is_expected.to be_able_to(:create, Spree::StockTransfer) }
+
+      it { is_expected.to be_able_to(:transfer_from, source_location) }
+      it { is_expected.to be_able_to(:transfer_to, source_location) }
+
+      it { is_expected.not_to be_able_to(:transfer_from, destination_location) }
+      it { is_expected.not_to be_able_to(:transfer_to, destination_location) }
+
+      it { is_expected.to be_able_to(:display, transfer_with_source) }
+      it { is_expected.to be_able_to(:display, transfer_with_source_and_destination) }
+      it { is_expected.not_to be_able_to(:display, transfer_with_destination) }
+
+      it { is_expected.to be_able_to(:manage, transfer_with_source) }
+      it { is_expected.not_to be_able_to(:manage, transfer_with_destination) }
+      it { is_expected.not_to be_able_to(:manage, transfer_with_source_and_destination) }
+
+      it { is_expected.to be_able_to(:manage, source_transfer_item) }
+      it { is_expected.not_to be_able_to(:manage, destination_transfer_item) }
+      it { is_expected.not_to be_able_to(:manage, source_and_destination_transfer_item) }
+    end
+
+
+    context "when the user is associated with both locations" do
+      let(:stock_locations) {[source_location, destination_location]}
+
+      it { is_expected.to be_able_to(:display, source_location) }
+      it { is_expected.to be_able_to(:display, destination_location) }
+
+      it { is_expected.to be_able_to(:display, Spree::StockTransfer) }
+      it { is_expected.to be_able_to(:admin, Spree::StockTransfer) }
+      it { is_expected.to be_able_to(:create, Spree::StockTransfer) }
+
+      it { is_expected.to be_able_to(:transfer_from, source_location) }
+      it { is_expected.to be_able_to(:transfer_to, source_location) }
+
+      it { is_expected.to be_able_to(:transfer_from, destination_location) }
+      it { is_expected.to be_able_to(:transfer_to, destination_location) }
+
+      it { is_expected.to be_able_to(:display, transfer_with_source) }
+      it { is_expected.to be_able_to(:display, transfer_with_source_and_destination) }
+      it { is_expected.to be_able_to(:display, transfer_with_destination) }
+
+      it { is_expected.to be_able_to(:manage, transfer_with_source) }
+      it { is_expected.to be_able_to(:manage, transfer_with_destination) }
+      it { is_expected.to be_able_to(:manage, transfer_with_source_and_destination) }
+
+      it { is_expected.to be_able_to(:manage, source_transfer_item) }
+      it { is_expected.to be_able_to(:manage, destination_transfer_item) }
+      it { is_expected.to be_able_to(:manage, source_and_destination_transfer_item) }
+    end
+
+    context "when the user is associated with neither location" do
+      let(:stock_locations) {[]}
+
+      it { is_expected.to_not be_able_to(:display, source_location) }
+      it { is_expected.to_not be_able_to(:display, destination_location) }
+
+      it { is_expected.not_to be_able_to(:display, Spree::StockTransfer) }
+      it { is_expected.not_to be_able_to(:admin, Spree::StockTransfer) }
+      it { is_expected.not_to be_able_to(:create, Spree::StockTransfer) }
+
+      it { is_expected.not_to be_able_to(:transfer_from, source_location) }
+      it { is_expected.not_to be_able_to(:transfer_to, source_location) }
+
+      it { is_expected.not_to be_able_to(:transfer_from, destination_location) }
+      it { is_expected.not_to be_able_to(:transfer_to, destination_location) }
+
+      it { is_expected.not_to be_able_to(:manage, transfer_with_source) }
+      it { is_expected.not_to be_able_to(:manage, transfer_with_destination) }
+      it { is_expected.not_to be_able_to(:manage, transfer_with_source_and_destination) }
+
+      it { is_expected.not_to be_able_to(:manage, source_transfer_item) }
+      it { is_expected.not_to be_able_to(:manage, destination_transfer_item) }
+      it { is_expected.not_to be_able_to(:manage, source_and_destination_transfer_item) }
+    end
+  end
+
+  context "when not activated" do
+    let(:user) { create :user }
+
+    it { is_expected.not_to be_able_to(:display, Spree::StockTransfer) }
+    it { is_expected.not_to be_able_to(:admin, Spree::StockTransfer) }
+    it { is_expected.not_to be_able_to(:create, Spree::StockTransfer) }
+
+    it { is_expected.to_not be_able_to(:display, source_location) }
+    it { is_expected.to_not be_able_to(:display, destination_location) }
+
+    it { is_expected.not_to be_able_to(:transfer_from, source_location) }
+    it { is_expected.not_to be_able_to(:transfer_to, source_location) }
+
+    it { is_expected.not_to be_able_to(:transfer_from, destination_location) }
+    it { is_expected.not_to be_able_to(:transfer_to, destination_location) }
+
+    it { is_expected.to_not be_able_to(:display, source_location) }
+    it { is_expected.to_not be_able_to(:display, destination_location) }
+
+    it { is_expected.not_to be_able_to(:manage, transfer_with_source) }
+    it { is_expected.not_to be_able_to(:manage, transfer_with_destination) }
+    it { is_expected.not_to be_able_to(:manage, transfer_with_source_and_destination) }
+
+    it { is_expected.not_to be_able_to(:manage, source_transfer_item) }
+    it { is_expected.not_to be_able_to(:manage, destination_transfer_item) }
+    it { is_expected.not_to be_able_to(:manage, source_and_destination_transfer_item) }
+  end
+end

data/spec/models/spree/permission_sets/stock_display_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::StockDisplay do
+  let(:ability) { DummyAbility.new }
+
+  subject { ability }
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:display, Spree::StockItem) }
+    it { is_expected.to be_able_to(:admin, Spree::StockItem) }
+    it { is_expected.to be_able_to(:display, Spree::StockLocation) }
+  end
+
+  context "when not activated" do
+    it { is_expected.not_to be_able_to(:display, Spree::StockItem) }
+    it { is_expected.not_to be_able_to(:admin, Spree::StockItem) }
+    it { is_expected.not_to be_able_to(:display, Spree::StockLocation) }
+  end
+end
+

data/spec/models/spree/permission_sets/stock_management_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::StockManagement do
+  let(:ability) { DummyAbility.new }
+
+  subject { ability }
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:manage, Spree::StockItem) }
+    it { is_expected.to be_able_to(:display, Spree::StockLocation) }
+  end
+
+  context "when not activated" do
+    it { is_expected.not_to be_able_to(:manage, Spree::StockItem) }
+    it { is_expected.not_to be_able_to(:display, Spree::StockLocation) }
+  end
+end
+

data/spec/models/spree/permission_sets/stock_transfer_display_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::StockTransferDisplay do
+  let(:ability) { DummyAbility.new }
+
+  subject { ability }
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:display, Spree::StockTransfer) }
+    it { is_expected.to be_able_to(:admin, Spree::StockTransfer) }
+    it { is_expected.to be_able_to(:display, Spree::StockLocation) }
+  end
+
+  context "when not activated" do
+    it { is_expected.not_to be_able_to(:display, Spree::StockTransfer) }
+    it { is_expected.not_to be_able_to(:admin, Spree::StockTransfer) }
+    it { is_expected.not_to be_able_to(:display, Spree::StockLocation) }
+  end
+end
+

data/spec/models/spree/permission_sets/stock_transfer_management_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::StockTransferManagement do
+  let(:ability) { DummyAbility.new }
+
+  subject { ability }
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:manage, Spree::StockTransfer) }
+    it { is_expected.to be_able_to(:manage, Spree::TransferItem) }
+    it { is_expected.to be_able_to(:display, Spree::StockLocation) }
+  end
+
+  context "when not activated" do
+    it { is_expected.to_not be_able_to(:manage, Spree::StockTransfer) }
+    it { is_expected.to_not be_able_to(:manage, Spree::TransferItem) }
+    it { is_expected.not_to be_able_to(:display, Spree::StockLocation) }
+  end
+end
+
+

data/spec/models/spree/permission_sets/user_display_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::UserDisplay do
+  let(:ability) { DummyAbility.new }
+
+  subject { ability }
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:display, Spree.user_class) }
+    it { is_expected.to be_able_to(:admin, Spree.user_class) }
+    it { is_expected.to be_able_to(:edit, Spree.user_class) }
+    it { is_expected.to be_able_to(:addresses, Spree.user_class) }
+    it { is_expected.to be_able_to(:orders, Spree.user_class) }
+    it { is_expected.to be_able_to(:items, Spree.user_class) }
+    it { is_expected.to be_able_to(:display, Spree::StoreCredit) }
+    it { is_expected.to be_able_to(:admin, Spree::StoreCredit) }
+    it { is_expected.to be_able_to(:display, Spree::Role) }
+    it { is_expected.not_to be_able_to(:delete, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:destroy, Spree.user_class) }
+  end
+
+  context "when not activated" do
+    it { is_expected.not_to be_able_to(:display, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:admin, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:edit, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:addresses, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:orders, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:items, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:display, Spree::StoreCredit) }
+    it { is_expected.not_to be_able_to(:admin, Spree::StoreCredit) }
+    it { is_expected.not_to be_able_to(:display, Spree::Role) }
+  end
+end
+

data/spec/models/spree/permission_sets/user_management_spec.rb

@@ -0,0 +1,48 @@
+require 'spec_helper'
+
+describe Spree::PermissionSets::UserManagement do
+  let(:ability) { DummyAbility.new }
+
+  subject { ability }
+
+  context "when activated" do
+    before do
+      described_class.new(ability).activate!
+    end
+
+    it { is_expected.to be_able_to(:admin, Spree.user_class) }
+    it { is_expected.to be_able_to(:display, Spree.user_class) }
+    it { is_expected.to be_able_to(:create, Spree.user_class) }
+    it { is_expected.to be_able_to(:update, Spree.user_class) }
+    it { is_expected.to be_able_to(:save_in_address_book, Spree.user_class) }
+    it { is_expected.to be_able_to(:remove_from_address_book, Spree.user_class) }
+    it { is_expected.to be_able_to(:addresses, Spree.user_class) }
+    it { is_expected.to be_able_to(:orders, Spree.user_class) }
+    it { is_expected.to be_able_to(:items, Spree.user_class) }
+
+    it { is_expected.to be_able_to(:update_email, Spree.user_class.new(spree_roles: [])) }
+    it { is_expected.not_to be_able_to(:update_email, Spree.user_class.new(spree_roles: [create(:role)])) }
+
+    it { is_expected.not_to be_able_to(:delete, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:destroy, Spree.user_class) }
+
+    it { is_expected.to be_able_to(:manage, Spree::StoreCredit) }
+    it { is_expected.to be_able_to(:display, Spree::Role) }
+  end
+
+  context "when not activated" do
+    it { is_expected.not_to be_able_to(:manage, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:delete, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:destroy, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:update, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:save_in_address_book, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:remove_from_address_book, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:addresses, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:orders, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:items, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:destroy, Spree.user_class) }
+    it { is_expected.not_to be_able_to(:manage, Spree::StoreCredit) }
+    it { is_expected.not_to be_able_to(:display, Spree::Role) }
+  end
+end
+