solidus_core 2.4.2 → 2.5.0.beta1
Sign up to get free protection for your applications and to get access to all the features.
Potentially problematic release.
This version of solidus_core might be problematic. Click here for more details.
- checksums.yaml +5 -5
- data/README.md +1 -5
- data/Rakefile +23 -6
- data/app/helpers/spree/base_helper.rb +15 -10
- data/app/helpers/spree/checkout_helper.rb +1 -1
- data/app/helpers/spree/products_helper.rb +2 -2
- data/app/mailers/spree/carton_mailer.rb +2 -2
- data/app/mailers/spree/order_mailer.rb +4 -4
- data/app/mailers/spree/reimbursement_mailer.rb +2 -2
- data/app/mailers/spree/test_mailer.rb +1 -1
- data/app/models/concerns/spree/adjustment_source.rb +17 -11
- data/app/models/concerns/spree/calculated_adjustments.rb +2 -2
- data/app/models/spree/ability.rb +1 -1
- data/app/models/spree/address.rb +7 -0
- data/app/models/spree/adjustment.rb +6 -2
- data/app/models/spree/calculator.rb +1 -1
- data/app/models/spree/calculator/flexi_rate.rb +1 -0
- data/app/models/spree/calculator/free_shipping.rb +2 -0
- data/app/models/spree/calculator/percent_on_line_item.rb +6 -6
- data/app/models/spree/calculator/percent_per_item.rb +2 -0
- data/app/models/spree/calculator/returns/default_refund_amount.rb +4 -4
- data/app/models/spree/calculator/shipping/flat_percent_item_total.rb +1 -0
- data/app/models/spree/calculator/shipping/flat_rate.rb +1 -0
- data/app/models/spree/calculator/shipping/flexi_rate.rb +1 -0
- data/app/models/spree/calculator/shipping/per_item.rb +1 -0
- data/app/models/spree/calculator/shipping/price_sack.rb +1 -0
- data/app/models/spree/classification.rb +1 -1
- data/app/models/spree/customer_return.rb +1 -1
- data/app/models/spree/inventory_unit.rb +8 -2
- data/app/models/spree/log_entry.rb +0 -11
- data/app/models/spree/order.rb +31 -16
- data/app/models/spree/order_contents.rb +0 -14
- data/app/models/spree/order_shipping.rb +0 -5
- data/app/models/spree/order_updater.rb +0 -4
- data/app/models/spree/payment.rb +6 -1
- data/app/models/spree/payment/processing.rb +4 -4
- data/app/models/spree/payment_method.rb +8 -3
- data/app/models/spree/payment_method/credit_card.rb +1 -1
- data/app/models/spree/payment_method/store_credit.rb +4 -4
- data/app/models/spree/price.rb +7 -1
- data/app/models/spree/product.rb +34 -6
- data/app/models/spree/product/scopes.rb +2 -2
- data/app/models/spree/promotion.rb +1 -2
- data/app/models/spree/promotion/actions/create_adjustment.rb +3 -2
- data/app/models/spree/promotion/actions/create_item_adjustments.rb +7 -8
- data/app/models/spree/promotion/actions/free_shipping.rb +1 -1
- data/app/models/spree/promotion/rules/taxon.rb +25 -24
- data/app/models/spree/promotion_action.rb +7 -1
- data/app/models/spree/promotion_chooser.rb +1 -1
- data/app/models/spree/promotion_code.rb +3 -3
- data/app/models/spree/promotion_handler/coupon.rb +8 -29
- data/app/models/spree/promotion_rule.rb +1 -1
- data/app/models/spree/refund.rb +3 -3
- data/app/models/spree/reimbursement.rb +1 -1
- data/app/models/spree/return_authorization.rb +2 -2
- data/app/models/spree/return_item.rb +3 -3
- data/app/models/spree/return_item/eligibility_validator/base_validator.rb +2 -2
- data/app/models/spree/return_item/eligibility_validator/inventory_shipped.rb +1 -1
- data/app/models/spree/return_item/eligibility_validator/no_reimbursements.rb +1 -1
- data/app/models/spree/return_item/eligibility_validator/order_completed.rb +1 -1
- data/app/models/spree/return_item/eligibility_validator/rma_required.rb +1 -1
- data/app/models/spree/return_item/eligibility_validator/time_since_purchase.rb +1 -1
- data/app/models/spree/shipment.rb +16 -6
- data/app/models/spree/shipping_method.rb +7 -0
- data/app/models/spree/shipping_rate.rb +6 -6
- data/app/models/spree/shipping_rate_tax.rb +2 -2
- data/app/models/spree/stock/availability_validator.rb +2 -2
- data/app/models/spree/stock/estimator.rb +9 -1
- data/app/models/spree/stock/inventory_unit_builder.rb +1 -2
- data/app/models/spree/stock/inventory_validator.rb +2 -2
- data/app/models/spree/stock/package.rb +1 -1
- data/app/models/spree/stock/simple_coordinator.rb +2 -2
- data/app/models/spree/stock_item.rb +6 -0
- data/app/models/spree/stock_location.rb +2 -2
- data/app/models/spree/store_credit.rb +19 -12
- data/app/models/spree/store_credit_category.rb +1 -1
- data/app/models/spree/store_credit_event.rb +7 -1
- data/app/models/spree/tax/shipping_rate_taxer.rb +1 -1
- data/app/models/spree/tax_calculator/shipping_rate.rb +12 -3
- data/app/models/spree/tax_rate.rb +10 -3
- data/app/models/spree/taxon.rb +14 -4
- data/app/models/spree/taxonomy.rb +1 -1
- data/app/models/spree/unit_cancel.rb +1 -1
- data/app/models/spree/variant.rb +33 -1
- data/app/models/spree/variant/price_selector.rb +1 -1
- data/app/models/spree/variant/pricing_options.rb +1 -1
- data/app/models/spree/variant/scopes.rb +1 -1
- data/app/models/spree/variant/vat_price_generator.rb +1 -1
- data/app/views/spree/carton_mailer/shipped_email.html.erb +6 -6
- data/app/views/spree/carton_mailer/shipped_email.text.erb +6 -6
- data/app/views/spree/order_mailer/cancel_email.html.erb +5 -5
- data/app/views/spree/order_mailer/cancel_email.text.erb +5 -5
- data/app/views/spree/order_mailer/confirm_email.html.erb +9 -9
- data/app/views/spree/order_mailer/confirm_email.text.erb +9 -9
- data/app/views/spree/order_mailer/inventory_cancellation_email.html.erb +3 -3
- data/app/views/spree/order_mailer/inventory_cancellation_email.text.erb +3 -3
- data/app/views/spree/promotion_code_batch_mailer/promotion_code_batch_errored.text.erb +1 -1
- data/app/views/spree/promotion_code_batch_mailer/promotion_code_batch_finished.text.erb +1 -1
- data/app/views/spree/reimbursement_mailer/reimbursement_email.html.erb +5 -5
- data/app/views/spree/reimbursement_mailer/reimbursement_email.text.erb +5 -5
- data/app/views/spree/shared/_error_messages.html.erb +2 -2
- data/app/views/spree/test_mailer/test_email.html.erb +2 -2
- data/app/views/spree/test_mailer/test_email.text.erb +2 -2
- data/config/initializers/assets.rb +3 -1
- data/config/locales/en.yml +44 -48
- data/db/default/spree/store_credit.rb +1 -1
- data/db/default/spree/stores.rb +7 -8
- data/db/migrate/20160101010000_solidus_one_four.rb +4 -46
- data/db/migrate/20170317035819_add_lft_and_rgt_indexes_to_taxons.rb +6 -0
- data/db/migrate/20170319191942_remove_order_id_from_inventory_units.rb +28 -0
- data/db/migrate/20170608074534_rename_bogus_gateways.rb +4 -2
- data/db/migrate/20170831201542_remove_default_tax_from_spree_zones.rb +1 -1
- data/lib/generators/spree/custom_user/custom_user_generator.rb +11 -22
- data/lib/generators/spree/install/install_generator.rb +10 -5
- data/lib/spree/app_configuration.rb +103 -5
- data/lib/spree/core.rb +3 -2
- data/lib/spree/core/active_merchant_dependencies.rb +11 -0
- data/lib/spree/core/controller_helpers/auth.rb +3 -1
- data/lib/spree/core/controller_helpers/common.rb +3 -7
- data/lib/spree/core/engine.rb +18 -100
- data/lib/spree/core/environment.rb +3 -3
- data/lib/spree/core/importer/order.rb +0 -1
- data/lib/spree/core/permalinks.rb +9 -15
- data/lib/spree/core/product_filters.rb +5 -3
- data/lib/spree/core/role_configuration.rb +23 -7
- data/lib/spree/core/search/base.rb +16 -13
- data/lib/spree/core/version.rb +1 -1
- data/lib/spree/paranoia_deprecations.rb +19 -0
- data/lib/spree/permission_sets.rb +0 -4
- data/lib/spree/permitted_attributes.rb +0 -3
- data/lib/spree/preferences/preferable.rb +70 -34
- data/lib/spree/preferences/preferable_class_methods.rb +22 -0
- data/lib/spree/testing_support/authorization_helpers.rb +2 -0
- data/lib/spree/testing_support/common_rake.rb +0 -1
- data/lib/spree/testing_support/dummy_app.rb +100 -0
- data/lib/spree/testing_support/dummy_app/assets/javascripts/spree/backend/all.js +10 -0
- data/lib/spree/testing_support/dummy_app/assets/javascripts/spree/frontend/all.js +10 -0
- data/lib/spree/testing_support/dummy_app/assets/stylesheets/spree/backend/all.css +9 -0
- data/lib/spree/testing_support/dummy_app/assets/stylesheets/spree/frontend/all.css +9 -0
- data/lib/spree/testing_support/dummy_app/database.yml +31 -0
- data/lib/spree/testing_support/dummy_app/migrations.rb +35 -0
- data/lib/spree/testing_support/dummy_app/rake_tasks.rb +60 -0
- data/lib/spree/testing_support/dummy_app/views/layouts/application.html.erb +1 -0
- data/lib/spree/testing_support/factories.rb +1 -1
- data/lib/spree/testing_support/factories/address_factory.rb +2 -2
- data/lib/spree/testing_support/factories/adjustment_factory.rb +2 -2
- data/lib/spree/testing_support/factories/adjustment_reason_factory.rb +1 -1
- data/lib/spree/testing_support/factories/calculator_factory.rb +1 -1
- data/lib/spree/testing_support/factories/carton_factory.rb +1 -1
- data/lib/spree/testing_support/factories/country_factory.rb +1 -1
- data/lib/spree/testing_support/factories/credit_card_factory.rb +1 -1
- data/lib/spree/testing_support/factories/customer_return_factory.rb +2 -2
- data/lib/spree/testing_support/factories/image_factory.rb +1 -1
- data/lib/spree/testing_support/factories/inventory_unit_factory.rb +13 -4
- data/lib/spree/testing_support/factories/line_item_factory.rb +1 -1
- data/lib/spree/testing_support/factories/option_type_factory.rb +1 -1
- data/lib/spree/testing_support/factories/option_value_factory.rb +1 -1
- data/lib/spree/testing_support/factories/order_factory.rb +5 -1
- data/lib/spree/testing_support/factories/order_promotion_factory.rb +1 -1
- data/lib/spree/testing_support/factories/payment_factory.rb +1 -1
- data/lib/spree/testing_support/factories/payment_method_factory.rb +1 -1
- data/lib/spree/testing_support/factories/price_factory.rb +1 -1
- data/lib/spree/testing_support/factories/product_factory.rb +1 -1
- data/lib/spree/testing_support/factories/product_option_type_factory.rb +1 -1
- data/lib/spree/testing_support/factories/product_property_factory.rb +1 -1
- data/lib/spree/testing_support/factories/promotion_category_factory.rb +1 -1
- data/lib/spree/testing_support/factories/promotion_code_factory.rb +2 -2
- data/lib/spree/testing_support/factories/promotion_factory.rb +1 -1
- data/lib/spree/testing_support/factories/property_factory.rb +1 -1
- data/lib/spree/testing_support/factories/refund_factory.rb +1 -1
- data/lib/spree/testing_support/factories/refund_reason_factory.rb +1 -1
- data/lib/spree/testing_support/factories/reimbursement_factory.rb +1 -1
- data/lib/spree/testing_support/factories/reimbursement_type_factory.rb +1 -1
- data/lib/spree/testing_support/factories/return_authorization_factory.rb +1 -1
- data/lib/spree/testing_support/factories/return_item_factory.rb +1 -1
- data/lib/spree/testing_support/factories/return_reason_factory.rb +1 -1
- data/lib/spree/testing_support/factories/role_factory.rb +1 -1
- data/lib/spree/testing_support/factories/shipment_factory.rb +1 -2
- data/lib/spree/testing_support/factories/shipping_category_factory.rb +1 -1
- data/lib/spree/testing_support/factories/shipping_method_factory.rb +2 -2
- data/lib/spree/testing_support/factories/shipping_rate_factory.rb +1 -1
- data/lib/spree/testing_support/factories/state_factory.rb +1 -7
- data/lib/spree/testing_support/factories/stock_item_factory.rb +1 -1
- data/lib/spree/testing_support/factories/stock_location_factory.rb +1 -1
- data/lib/spree/testing_support/factories/stock_movement_factory.rb +1 -1
- data/lib/spree/testing_support/factories/stock_package_factory.rb +1 -1
- data/lib/spree/testing_support/factories/store_credit_category_factory.rb +1 -1
- data/lib/spree/testing_support/factories/store_credit_event_factory.rb +1 -1
- data/lib/spree/testing_support/factories/store_credit_factory.rb +1 -1
- data/lib/spree/testing_support/factories/store_credit_type_factory.rb +2 -2
- data/lib/spree/testing_support/factories/store_credit_update_reason_factory.rb +1 -1
- data/lib/spree/testing_support/factories/store_factory.rb +1 -1
- data/lib/spree/testing_support/factories/tax_category_factory.rb +1 -2
- data/lib/spree/testing_support/factories/tax_rate_factory.rb +1 -1
- data/lib/spree/testing_support/factories/taxon_factory.rb +1 -1
- data/lib/spree/testing_support/factories/taxonomy_factory.rb +1 -1
- data/lib/spree/testing_support/factories/user_factory.rb +4 -10
- data/lib/spree/testing_support/factories/variant_factory.rb +1 -1
- data/lib/spree/testing_support/factories/variant_property_rule_condition_factory.rb +1 -1
- data/lib/spree/testing_support/factories/variant_property_rule_factory.rb +1 -1
- data/lib/spree/testing_support/factories/variant_property_rule_value_factory.rb +1 -1
- data/lib/spree/testing_support/factories/zone_factory.rb +2 -4
- data/lib/spree/testing_support/flash.rb +2 -2
- data/lib/spree/testing_support/order_walkthrough.rb +9 -9
- data/lib/spree/testing_support/preferences.rb +4 -0
- data/lib/spree/testing_support/sequences.rb +3 -12
- data/solidus_core.gemspec +12 -8
- data/spec/helpers/base_helper_spec.rb +19 -3
- data/spec/helpers/products_helper_spec.rb +1 -1
- data/spec/lib/calculated_adjustments_spec.rb +20 -0
- data/spec/lib/i18n_spec.rb +4 -69
- data/spec/lib/search/base_spec.rb +1 -0
- data/spec/lib/search/variant_spec.rb +4 -4
- data/spec/lib/spree/core/controller_helpers/pricing_spec.rb +6 -6
- data/spec/lib/spree/core/importer/order_spec.rb +2 -2
- data/spec/lib/spree/core/role_configuration_spec.rb +6 -9
- data/spec/lib/spree/core/testing_support/factories/order_factory_spec.rb +12 -8
- data/spec/lib/spree/core/testing_support/factories/reimbursement_factory_spec.rb +1 -1
- data/spec/lib/spree/core/testing_support/factories/store_credit_category_factory_spec.rb +1 -1
- data/spec/mailers/carton_mailer_spec.rb +3 -7
- data/spec/mailers/order_mailer_spec.rb +7 -11
- data/spec/mailers/reimbursement_mailer_spec.rb +1 -5
- data/spec/mailers/test_mailer_spec.rb +0 -4
- data/spec/models/spree/ability_spec.rb +2 -1
- data/spec/models/spree/address_spec.rb +32 -19
- data/spec/models/spree/app_configuration_spec.rb +1 -1
- data/spec/models/spree/calculator/distributed_amount_spec.rb +1 -1
- data/spec/models/spree/calculator/percent_on_line_item_spec.rb +8 -10
- data/spec/models/spree/calculator/percent_per_item_spec.rb +4 -4
- data/spec/models/spree/calculator/refunds/default_refund_amount_spec.rb +10 -3
- data/spec/models/spree/classification_spec.rb +19 -0
- data/spec/models/spree/credit_card_spec.rb +1 -1
- data/spec/models/spree/customer_return_spec.rb +17 -6
- data/spec/models/spree/distributed_amounts_handler_spec.rb +1 -1
- data/spec/models/spree/exchange_spec.rb +1 -1
- data/spec/models/spree/inventory_unit_spec.rb +3 -7
- data/spec/models/spree/line_item_spec.rb +6 -6
- data/spec/models/spree/option_value_spec.rb +11 -6
- data/spec/models/spree/order/callbacks_spec.rb +2 -2
- data/spec/models/spree/order/checkout_spec.rb +31 -31
- data/spec/models/spree/order/risk_assessment_spec.rb +9 -9
- data/spec/models/spree/order_contents_spec.rb +1 -9
- data/spec/models/spree/order_inventory_spec.rb +2 -2
- data/spec/models/spree/order_mutex_spec.rb +3 -1
- data/spec/models/spree/order_shipping_spec.rb +7 -4
- data/spec/models/spree/order_spec.rb +36 -13
- data/spec/models/spree/order_taxation_spec.rb +7 -7
- data/spec/models/spree/order_updater_spec.rb +6 -4
- data/spec/models/spree/payment/cancellation_spec.rb +1 -1
- data/spec/models/spree/payment_create_spec.rb +2 -2
- data/spec/models/spree/payment_method/store_credit_spec.rb +13 -13
- data/spec/models/spree/payment_spec.rb +23 -4
- data/spec/models/spree/preferences/preferable_spec.rb +44 -0
- data/spec/models/spree/preferences/statically_configurable_spec.rb +1 -0
- data/spec/models/spree/price_spec.rb +6 -4
- data/spec/models/spree/product/scopes_spec.rb +1 -1
- data/spec/models/spree/product_spec.rb +49 -9
- data/spec/models/spree/promotion/actions/create_adjustment_spec.rb +13 -3
- data/spec/models/spree/promotion/actions/create_item_adjustments_spec.rb +146 -134
- data/spec/models/spree/promotion/actions/create_quantity_adjustments_spec.rb +6 -6
- data/spec/models/spree/promotion/rules/taxon_spec.rb +14 -14
- data/spec/models/spree/promotion_code_spec.rb +31 -19
- data/spec/models/spree/promotion_handler/coupon_spec.rb +42 -17
- data/spec/models/spree/promotion_handler/shipping_spec.rb +1 -6
- data/spec/models/spree/promotion_spec.rb +14 -14
- data/spec/models/spree/refund_spec.rb +1 -1
- data/spec/models/spree/return_item/eligibility_validator/inventory_shipped_spec.rb +1 -1
- data/spec/models/spree/return_item/eligibility_validator/no_reimbursements_spec.rb +1 -1
- data/spec/models/spree/return_item/eligibility_validator/order_completed_spec.rb +3 -2
- data/spec/models/spree/return_item/eligibility_validator/rma_required_spec.rb +1 -1
- data/spec/models/spree/return_item/eligibility_validator/time_since_purchase_spec.rb +2 -2
- data/spec/models/spree/return_item_spec.rb +6 -6
- data/spec/models/spree/shipment_spec.rb +14 -10
- data/spec/models/spree/shipping_manifest_spec.rb +22 -9
- data/spec/models/spree/shipping_method_spec.rb +9 -5
- data/spec/models/spree/stock/availability_spec.rb +16 -2
- data/spec/models/spree/stock/estimator_spec.rb +11 -12
- data/spec/models/spree/stock/inventory_unit_builder_spec.rb +0 -4
- data/spec/models/spree/stock/package_spec.rb +3 -2
- data/spec/models/spree/stock/simple_coordinator_spec.rb +1 -24
- data/spec/models/spree/stock_item_spec.rb +6 -6
- data/spec/models/spree/stock_location_spec.rb +16 -3
- data/spec/models/spree/store_credit_spec.rb +12 -6
- data/spec/models/spree/tax/taxation_integration_spec.rb +6 -6
- data/spec/models/spree/tax_calculator/default_spec.rb +7 -7
- data/spec/models/spree/taxon_spec.rb +42 -19
- data/spec/models/spree/variant/vat_price_generator_spec.rb +1 -1
- data/spec/models/spree/variant_spec.rb +69 -11
- data/spec/rails_helper.rb +7 -13
- data/spec/spec_helper.rb +3 -12
- data/spec/support/concerns/default_price.rb +1 -1
- data/spec/support/dummy_ability.rb +2 -0
- metadata +144 -73
- data/Gemfile +0 -3
- data/app/models/spree/order_stock_location.rb +0 -15
- data/app/models/spree/stock_transfer.rb +0 -110
- data/app/models/spree/transfer_item.rb +0 -54
- data/lib/generators/spree/custom_user/templates/initializer.rb.tt +0 -1
- data/lib/spree/permission_sets/restricted_stock_transfer_display.rb +0 -17
- data/lib/spree/permission_sets/restricted_stock_transfer_management.rb +0 -52
- data/lib/spree/permission_sets/stock_transfer_display.rb +0 -10
- data/lib/spree/permission_sets/stock_transfer_management.rb +0 -11
- data/lib/spree/testing_support/factories/stock_transfer_factory.rb +0 -28
- data/lib/spree/testing_support/i18n.rb +0 -97
- data/spec/lib/spree/core/testing_support/factories/stock_transfer_factory_spec.rb +0 -12
- data/spec/models/spree/order_stock_location_spec.rb +0 -18
- data/spec/models/spree/permission_sets/restricted_stock_transfer_display_spec.rb +0 -49
- data/spec/models/spree/permission_sets/restricted_stock_transfer_management_spec.rb +0 -218
- data/spec/models/spree/permission_sets/stock_transfer_display_spec.rb +0 -23
- data/spec/models/spree/permission_sets/stock_transfer_management_spec.rb +0 -23
- data/spec/models/spree/stock_transfer_spec.rb +0 -308
- data/spec/models/spree/transfer_item_spec.rb +0 -275
- data/spec/support/test_gateway.rb +0 -2
@@ -1,275 +0,0 @@
|
|
1
|
-
require 'rails_helper'
|
2
|
-
|
3
|
-
RSpec.describe Spree::TransferItem do
|
4
|
-
let(:stock_location) { create(:stock_location, name: "Warehouse") }
|
5
|
-
let(:stock_transfer) { create(:stock_transfer_with_items, source_location: stock_location) }
|
6
|
-
let(:transfer_item) { stock_transfer.transfer_items.first }
|
7
|
-
|
8
|
-
subject { transfer_item }
|
9
|
-
|
10
|
-
describe "validation" do
|
11
|
-
before do
|
12
|
-
transfer_item.assign_attributes(expected_quantity: expected_quantity, received_quantity: received_quantity)
|
13
|
-
end
|
14
|
-
|
15
|
-
describe "expected vs received quantity" do
|
16
|
-
context "expected quantity is the same as the received quantity" do
|
17
|
-
let(:expected_quantity) { 1 }
|
18
|
-
let(:received_quantity) { 1 }
|
19
|
-
it { is_expected.to be_valid }
|
20
|
-
end
|
21
|
-
|
22
|
-
context "expected quantity is larger than the received quantity" do
|
23
|
-
let(:expected_quantity) { 3 }
|
24
|
-
let(:received_quantity) { 1 }
|
25
|
-
it { is_expected.to be_valid }
|
26
|
-
end
|
27
|
-
|
28
|
-
context "expected quantity is lower than the received quantity" do
|
29
|
-
let(:expected_quantity) { 1 }
|
30
|
-
let(:received_quantity) { 3 }
|
31
|
-
it { is_expected.to_not be_valid }
|
32
|
-
end
|
33
|
-
end
|
34
|
-
|
35
|
-
describe "numericality" do
|
36
|
-
context "expected_quantity is less than 0" do
|
37
|
-
let(:expected_quantity) { -1 }
|
38
|
-
let(:received_quantity) { 3 }
|
39
|
-
it { is_expected.to_not be_valid }
|
40
|
-
end
|
41
|
-
|
42
|
-
context "received_quantity is less than 0" do
|
43
|
-
let(:expected_quantity) { 1 }
|
44
|
-
let(:received_quantity) { -3 }
|
45
|
-
it { is_expected.to_not be_valid }
|
46
|
-
end
|
47
|
-
end
|
48
|
-
|
49
|
-
describe "availability" do
|
50
|
-
let(:stock_item) do
|
51
|
-
transfer_item.variant.stock_items.find_by(stock_location: stock_transfer.source_location)
|
52
|
-
end
|
53
|
-
let(:expected_quantity) { 1 }
|
54
|
-
let(:received_quantity) { 1 }
|
55
|
-
|
56
|
-
subject { transfer_item.valid? }
|
57
|
-
|
58
|
-
shared_examples_for 'availability check fails' do
|
59
|
-
it "validates the availability" do
|
60
|
-
subject
|
61
|
-
expect(transfer_item.errors.full_messages).to include Spree.t('errors.messages.transfer_item_insufficient_stock')
|
62
|
-
end
|
63
|
-
end
|
64
|
-
|
65
|
-
shared_examples_for 'availability check passes' do
|
66
|
-
it "doesn't validate the availability" do
|
67
|
-
subject
|
68
|
-
expect(transfer_item.errors.full_messages).to_not include Spree.t('errors.messages.transfer_item_insufficient_stock')
|
69
|
-
end
|
70
|
-
end
|
71
|
-
|
72
|
-
context "transfer order is shipped" do
|
73
|
-
before do
|
74
|
-
stock_transfer.update_attributes!(shipped_at: Time.current)
|
75
|
-
end
|
76
|
-
|
77
|
-
context "variant is not available" do
|
78
|
-
before do
|
79
|
-
stock_item.set_count_on_hand(0)
|
80
|
-
end
|
81
|
-
include_examples 'availability check passes'
|
82
|
-
|
83
|
-
context "stock location doesn't check stock" do
|
84
|
-
before do
|
85
|
-
stock_location.update_attributes!(check_stock_on_transfer: false)
|
86
|
-
end
|
87
|
-
include_examples 'availability check passes'
|
88
|
-
end
|
89
|
-
end
|
90
|
-
|
91
|
-
context "variant available" do
|
92
|
-
before do
|
93
|
-
stock_item.set_count_on_hand(transfer_item.expected_quantity)
|
94
|
-
end
|
95
|
-
include_examples 'availability check passes'
|
96
|
-
end
|
97
|
-
|
98
|
-
context "variant does not exist in stock location" do
|
99
|
-
before do
|
100
|
-
stock_item.destroy
|
101
|
-
end
|
102
|
-
include_examples 'availability check passes'
|
103
|
-
end
|
104
|
-
end
|
105
|
-
|
106
|
-
context "transfer order isn't closed" do
|
107
|
-
before do
|
108
|
-
stock_transfer.update_attributes!(closed_at: nil)
|
109
|
-
end
|
110
|
-
|
111
|
-
context "variant is not available" do
|
112
|
-
before do
|
113
|
-
stock_item.set_count_on_hand(0)
|
114
|
-
end
|
115
|
-
include_examples 'availability check fails'
|
116
|
-
|
117
|
-
context "stock location doesn't check stock" do
|
118
|
-
before do
|
119
|
-
stock_location.update_attributes!(check_stock_on_transfer: false)
|
120
|
-
end
|
121
|
-
include_examples 'availability check passes'
|
122
|
-
end
|
123
|
-
end
|
124
|
-
|
125
|
-
context "variant available" do
|
126
|
-
before do
|
127
|
-
stock_item.set_count_on_hand(transfer_item.expected_quantity)
|
128
|
-
end
|
129
|
-
include_examples 'availability check passes'
|
130
|
-
end
|
131
|
-
|
132
|
-
context "variant does not exist in stock location" do
|
133
|
-
before do
|
134
|
-
stock_item.destroy
|
135
|
-
end
|
136
|
-
include_examples 'availability check fails'
|
137
|
-
end
|
138
|
-
end
|
139
|
-
end
|
140
|
-
end
|
141
|
-
|
142
|
-
describe "received stock transfer guard" do
|
143
|
-
subject { transfer_item.reload.update_attributes(received_quantity: 2) }
|
144
|
-
|
145
|
-
describe "closed stock transfer" do
|
146
|
-
context "stock_transfer is not closed" do
|
147
|
-
before do
|
148
|
-
stock_transfer.update_attributes!(closed_at: nil)
|
149
|
-
end
|
150
|
-
|
151
|
-
it { is_expected.to eq true }
|
152
|
-
end
|
153
|
-
|
154
|
-
context "stock_transfer is closed" do
|
155
|
-
before do
|
156
|
-
stock_transfer.update_attributes!(closed_at: Time.current)
|
157
|
-
end
|
158
|
-
|
159
|
-
it { is_expected.to eq false }
|
160
|
-
|
161
|
-
it "adds an error message" do
|
162
|
-
subject
|
163
|
-
expect(transfer_item.errors.full_messages).to include Spree.t('errors.messages.cannot_modify_transfer_item_closed_stock_transfer')
|
164
|
-
end
|
165
|
-
end
|
166
|
-
end
|
167
|
-
end
|
168
|
-
|
169
|
-
describe "expected quantity update guard" do
|
170
|
-
let(:attrs) { { expected_quantity: 1 } }
|
171
|
-
|
172
|
-
subject { transfer_item.update_attributes(attrs) }
|
173
|
-
|
174
|
-
context "stock transfer is finalized" do
|
175
|
-
before do
|
176
|
-
stock_transfer.update_attributes(finalized_at: Time.current)
|
177
|
-
end
|
178
|
-
|
179
|
-
it "adds an error message" do
|
180
|
-
subject
|
181
|
-
expect(transfer_item.errors.full_messages).to include Spree.t('errors.messages.cannot_update_expected_transfer_item_with_finalized_stock_transfer')
|
182
|
-
end
|
183
|
-
|
184
|
-
context "updating received_quantity" do
|
185
|
-
let(:attrs) { { received_quantity: 1 } }
|
186
|
-
|
187
|
-
it "updates the received quantity successfully" do
|
188
|
-
expect { subject }.to change { transfer_item.received_quantity }.to(1)
|
189
|
-
end
|
190
|
-
end
|
191
|
-
end
|
192
|
-
|
193
|
-
context "stock transfer is not finalized" do
|
194
|
-
before do
|
195
|
-
stock_transfer.update_attributes(finalized_at: nil, shipped_at: nil)
|
196
|
-
end
|
197
|
-
|
198
|
-
it "updates the expected quantity successfully" do
|
199
|
-
expect { subject }.to change { transfer_item.expected_quantity }.to(1)
|
200
|
-
end
|
201
|
-
end
|
202
|
-
end
|
203
|
-
|
204
|
-
describe "destroy finalized stock transfer guard" do
|
205
|
-
subject { transfer_item.destroy }
|
206
|
-
|
207
|
-
context "stock transfer is finalized" do
|
208
|
-
before do
|
209
|
-
stock_transfer.update_attributes(finalized_at: Time.current)
|
210
|
-
end
|
211
|
-
|
212
|
-
it "does not destroy the transfer item" do
|
213
|
-
expect { subject }.to_not change { Spree::TransferItem.count }
|
214
|
-
end
|
215
|
-
|
216
|
-
it "adds an error message" do
|
217
|
-
subject
|
218
|
-
expect(transfer_item.errors.full_messages).to include Spree.t('errors.messages.cannot_delete_transfer_item_with_finalized_stock_transfer')
|
219
|
-
end
|
220
|
-
end
|
221
|
-
|
222
|
-
context "stock transfer is not finalized" do
|
223
|
-
before do
|
224
|
-
stock_transfer.update_attributes(finalized_at: nil, shipped_at: nil)
|
225
|
-
end
|
226
|
-
|
227
|
-
it "destroys the transfer item" do
|
228
|
-
expect { subject }.to change { Spree::TransferItem.count }.by(-1)
|
229
|
-
end
|
230
|
-
end
|
231
|
-
|
232
|
-
context "scopes" do
|
233
|
-
let(:partially_received) { stock_transfer.transfer_items.first }
|
234
|
-
let(:fully_received) { stock_transfer.transfer_items.last }
|
235
|
-
let(:variant) { create(:variant) }
|
236
|
-
|
237
|
-
before do
|
238
|
-
fully_received.update_attributes(expected_quantity: 1, received_quantity: 1)
|
239
|
-
partially_received.update_attributes(expected_quantity: 2, received_quantity: 1)
|
240
|
-
|
241
|
-
stock_transfer.source_location.stock_item(variant).set_count_on_hand(5)
|
242
|
-
stock_transfer.transfer_items.create!(variant: variant, expected_quantity: 1, received_quantity: 0)
|
243
|
-
end
|
244
|
-
|
245
|
-
context '.received' do
|
246
|
-
it 'only returns items that have received quantity greater than 0' do
|
247
|
-
expect(Spree::TransferItem.received).to match_array [fully_received, partially_received]
|
248
|
-
end
|
249
|
-
end
|
250
|
-
|
251
|
-
context '.fully_received' do
|
252
|
-
it 'returns only items that have not been fully received' do
|
253
|
-
expect(Spree::TransferItem.fully_received).to eq [fully_received]
|
254
|
-
end
|
255
|
-
end
|
256
|
-
|
257
|
-
context '.partially_received' do
|
258
|
-
it 'returns only items where received quantity is less that expected' do
|
259
|
-
expect(Spree::TransferItem.partially_received).to eq [partially_received]
|
260
|
-
end
|
261
|
-
end
|
262
|
-
end
|
263
|
-
end
|
264
|
-
|
265
|
-
describe "variant association" do
|
266
|
-
context "variant has been soft-deleted" do
|
267
|
-
before do
|
268
|
-
subject.variant.destroy
|
269
|
-
end
|
270
|
-
it "still returns the variant" do
|
271
|
-
expect(subject.variant).not_to be_nil
|
272
|
-
end
|
273
|
-
end
|
274
|
-
end
|
275
|
-
end
|