RubyGems - spree_core - Versions diffs - 3.1.5 → 3.1.6 - Mend

spree_core 3.1.5 → 3.1.6

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (215) hide show

checksums.yaml +4 -4
data/app/models/concerns/spree/ransackable_attributes.rb +6 -1
data/app/models/concerns/spree/user_methods.rb +10 -0
data/app/models/spree/legacy_user.rb +0 -8
data/app/models/spree/order.rb +1 -1
data/app/models/spree/product.rb +8 -0
data/app/models/spree/product/scopes.rb +7 -3
data/app/models/spree/stock_transfer.rb +1 -1
data/config/locales/en.yml +2 -0
data/lib/spree/core/version.rb +1 -1
data/spree_core.gemspec +2 -2
metadata +11 -208
data/spec/fixtures/microdata.html +0 -22
data/spec/fixtures/microdata_itemref.html +0 -15
data/spec/fixtures/microdata_no_itemscope.html +0 -20
data/spec/fixtures/thinking-cat.jpg +0 -0
data/spec/helpers/base_helper_spec.rb +0 -200
data/spec/helpers/products_helper_spec.rb +0 -296
data/spec/lib/calculated_adjustments_spec.rb +0 -7
data/spec/lib/i18n_spec.rb +0 -123
data/spec/lib/search/base_spec.rb +0 -86
data/spec/lib/spree/core/controller_helpers/auth_spec.rb +0 -103
data/spec/lib/spree/core/controller_helpers/order_spec.rb +0 -110
data/spec/lib/spree/core/controller_helpers/search_spec.rb +0 -17
data/spec/lib/spree/core/controller_helpers/store_spec.rb +0 -72
data/spec/lib/spree/core/controller_helpers/strong_parameters_spec.rb +0 -39
data/spec/lib/spree/core/delegate_belongs_to_spec.rb +0 -22
data/spec/lib/spree/core/importer/order_spec.rb +0 -605
data/spec/lib/spree/core/number_generator_spec.rb +0 -175
data/spec/lib/spree/core/token_generator_spec.rb +0 -24
data/spec/lib/spree/core/validators/email_spec.rb +0 -53
data/spec/lib/spree/core_spec.rb +0 -23
data/spec/lib/spree/localized_number_spec.rb +0 -48
data/spec/lib/spree/migrations_spec.rb +0 -36
data/spec/lib/spree/money_spec.rb +0 -122
data/spec/lib/tasks/exchanges_spec.rb +0 -136
data/spec/mailers/order_mailer_spec.rb +0 -122
data/spec/mailers/reimbursement_mailer_spec.rb +0 -47
data/spec/mailers/shipment_mailer_spec.rb +0 -81
data/spec/mailers/test_mailer_spec.rb +0 -38
data/spec/models/option_type_prototype_spec.rb +0 -9
data/spec/models/spree/ability_spec.rb +0 -251
data/spec/models/spree/address_spec.rb +0 -291
data/spec/models/spree/adjustable/adjuster/base_spec.rb +0 -10
data/spec/models/spree/adjustable/adjuster/promotion_spec.rb +0 -211
data/spec/models/spree/adjustable/adjuster/tax_spec.rb +0 -86
data/spec/models/spree/adjustable/adjustments_updater_spec.rb +0 -26
data/spec/models/spree/adjustment_spec.rb +0 -189
data/spec/models/spree/app_configuration_spec.rb +0 -26
data/spec/models/spree/asset_spec.rb +0 -28
data/spec/models/spree/calculator/default_tax_spec.rb +0 -152
data/spec/models/spree/calculator/flat_percent_item_total_spec.rb +0 -25
data/spec/models/spree/calculator/flat_rate_spec.rb +0 -47
data/spec/models/spree/calculator/flexi_rate_spec.rb +0 -41
data/spec/models/spree/calculator/percent_on_line_item_spec.rb +0 -15
data/spec/models/spree/calculator/price_sack_spec.rb +0 -30
data/spec/models/spree/calculator/refunds/default_refund_amount_spec.rb +0 -51
data/spec/models/spree/calculator/shipping.rb +0 -8
data/spec/models/spree/calculator/shipping/flat_percent_item_total_spec.rb +0 -23
data/spec/models/spree/calculator/shipping/flat_rate_spec.rb +0 -13
data/spec/models/spree/calculator/shipping/flexi_rate_spec.rb +0 -52
data/spec/models/spree/calculator/shipping/per_item_spec.rb +0 -20
data/spec/models/spree/calculator/shipping/price_sack_spec.rb +0 -29
data/spec/models/spree/calculator/tiered_flat_rate_spec.rb +0 -40
data/spec/models/spree/calculator/tiered_percent_spec.rb +0 -51
data/spec/models/spree/calculator_spec.rb +0 -69
data/spec/models/spree/classification_spec.rb +0 -93
data/spec/models/spree/concerns/display_money_spec.rb +0 -43
data/spec/models/spree/concerns/user_methods_spec.rb +0 -55
data/spec/models/spree/concerns/vat_price_calculation_spec.rb +0 -66
data/spec/models/spree/country_spec.rb +0 -55
data/spec/models/spree/credit_card_spec.rb +0 -328
data/spec/models/spree/customer_return_spec.rb +0 -240
data/spec/models/spree/exchange_spec.rb +0 -75
data/spec/models/spree/gateway/bogus_simple.rb +0 -20
data/spec/models/spree/gateway/bogus_spec.rb +0 -13
data/spec/models/spree/gateway_spec.rb +0 -61
data/spec/models/spree/image_spec.rb +0 -8
data/spec/models/spree/inventory_unit_spec.rb +0 -225
data/spec/models/spree/line_item_spec.rb +0 -327
data/spec/models/spree/option_type_spec.rb +0 -14
data/spec/models/spree/option_value_spec.rb +0 -18
data/spec/models/spree/order/address_spec.rb +0 -50
data/spec/models/spree/order/adjustments_spec.rb +0 -29
data/spec/models/spree/order/callbacks_spec.rb +0 -42
data/spec/models/spree/order/checkout_spec.rb +0 -782
data/spec/models/spree/order/currency_updater_spec.rb +0 -32
data/spec/models/spree/order/finalizing_spec.rb +0 -114
data/spec/models/spree/order/helpers_spec.rb +0 -5
data/spec/models/spree/order/payment_spec.rb +0 -214
data/spec/models/spree/order/risk_assessment_spec.rb +0 -84
data/spec/models/spree/order/shipments_spec.rb +0 -43
data/spec/models/spree/order/state_machine_spec.rb +0 -211
data/spec/models/spree/order/store_credit_spec.rb +0 -426
data/spec/models/spree/order/tax_spec.rb +0 -84
data/spec/models/spree/order/totals_spec.rb +0 -24
data/spec/models/spree/order/updating_spec.rb +0 -18
data/spec/models/spree/order/validations_spec.rb +0 -15
data/spec/models/spree/order_contents_spec.rb +0 -297
data/spec/models/spree/order_inventory_spec.rb +0 -238
data/spec/models/spree/order_merger_spec.rb +0 -135
data/spec/models/spree/order_spec.rb +0 -967
data/spec/models/spree/order_updater_spec.rb +0 -305
data/spec/models/spree/payment/gateway_options_spec.rb +0 -127
data/spec/models/spree/payment/store_credit_spec.rb +0 -60
data/spec/models/spree/payment_method/store_credit_spec.rb +0 -291
data/spec/models/spree/payment_method_spec.rb +0 -103
data/spec/models/spree/payment_spec.rb +0 -919
data/spec/models/spree/preference_spec.rb +0 -80
data/spec/models/spree/preferences/configuration_spec.rb +0 -30
data/spec/models/spree/preferences/preferable_spec.rb +0 -348
data/spec/models/spree/preferences/scoped_store_spec.rb +0 -58
data/spec/models/spree/preferences/store_spec.rb +0 -46
data/spec/models/spree/price_spec.rb +0 -128
data/spec/models/spree/product/scopes_spec.rb +0 -183
data/spec/models/spree/product_duplicator_spec.rb +0 -103
data/spec/models/spree/product_filter_spec.rb +0 -26
data/spec/models/spree/product_option_type_spec.rb +0 -9
data/spec/models/spree/product_promotion_rule_spec.rb +0 -9
data/spec/models/spree/product_property_spec.rb +0 -22
data/spec/models/spree/product_spec.rb +0 -540
data/spec/models/spree/promotion/actions/create_adjustment_spec.rb +0 -50
data/spec/models/spree/promotion/actions/create_item_adjustments_spec.rb +0 -148
data/spec/models/spree/promotion/actions/create_line_items_spec.rb +0 -86
data/spec/models/spree/promotion/actions/free_shipping_spec.rb +0 -36
data/spec/models/spree/promotion/rules/first_order_spec.rb +0 -75
data/spec/models/spree/promotion/rules/item_total_spec.rb +0 -282
data/spec/models/spree/promotion/rules/one_use_per_user_spec.rb +0 -42
data/spec/models/spree/promotion/rules/option_value_spec.rb +0 -90
data/spec/models/spree/promotion/rules/product_spec.rb +0 -143
data/spec/models/spree/promotion/rules/taxon_spec.rb +0 -102
data/spec/models/spree/promotion/rules/user_logged_in_spec.rb +0 -27
data/spec/models/spree/promotion/rules/user_spec.rb +0 -45
data/spec/models/spree/promotion_action_spec.rb +0 -10
data/spec/models/spree/promotion_category_spec.rb +0 -17
data/spec/models/spree/promotion_handler/cart_spec.rb +0 -102
data/spec/models/spree/promotion_handler/coupon_spec.rb +0 -323
data/spec/models/spree/promotion_handler/free_shipping_spec.rb +0 -48
data/spec/models/spree/promotion_handler/page_spec.rb +0 -44
data/spec/models/spree/promotion_rule_spec.rb +0 -29
data/spec/models/spree/promotion_rule_taxon_spec.rb +0 -9
data/spec/models/spree/promotion_rule_user_spec.rb +0 -9
data/spec/models/spree/promotion_spec.rb +0 -644
data/spec/models/spree/property_prototype_spec.rb +0 -9
data/spec/models/spree/property_spec.rb +0 -5
data/spec/models/spree/prototype_spec.rb +0 -5
data/spec/models/spree/prototype_taxon_spec.rb +0 -9
data/spec/models/spree/refund_reason_spec.rb +0 -7
data/spec/models/spree/refund_spec.rb +0 -195
data/spec/models/spree/reimbursement/credit_spec.rb +0 -36
data/spec/models/spree/reimbursement/reimbursement_type_engine_spec.rb +0 -140
data/spec/models/spree/reimbursement/reimbursement_type_validator_spec.rb +0 -83
data/spec/models/spree/reimbursement_performer_spec.rb +0 -30
data/spec/models/spree/reimbursement_spec.rb +0 -188
data/spec/models/spree/reimbursement_tax_calculator_spec.rb +0 -63
data/spec/models/spree/reimbursement_type/credit_spec.rb +0 -53
data/spec/models/spree/reimbursement_type/exchange_spec.rb +0 -46
data/spec/models/spree/reimbursement_type/original_payment_spec.rb +0 -55
data/spec/models/spree/reimbursement_type/store_credit_spec.rb +0 -101
data/spec/models/spree/return_authorization_reason_spec.rb +0 -7
data/spec/models/spree/return_authorization_spec.rb +0 -230
data/spec/models/spree/return_item/eligibility_validator/default_spec.rb +0 -77
data/spec/models/spree/return_item/eligibility_validator/inventory_shipped_spec.rb +0 -58
data/spec/models/spree/return_item/eligibility_validator/no_reimbursements_spec.rb +0 -61
data/spec/models/spree/return_item/eligibility_validator/order_completed_spec.rb +0 -32
data/spec/models/spree/return_item/eligibility_validator/rma_required_spec.rb +0 -29
data/spec/models/spree/return_item/eligibility_validator/time_since_purchase_spec.rb +0 -35
data/spec/models/spree/return_item/exchange_variant_eligibility/same_option_value_spec.rb +0 -65
data/spec/models/spree/return_item/exchange_variant_eligibility/same_product_spec.rb +0 -43
data/spec/models/spree/return_item_spec.rb +0 -731
data/spec/models/spree/returns_calculator_spec.rb +0 -14
data/spec/models/spree/role_spec.rb +0 -7
data/spec/models/spree/shipment_spec.rb +0 -740
data/spec/models/spree/shipping_calculator_spec.rb +0 -45
data/spec/models/spree/shipping_category_spec.rb +0 -19
data/spec/models/spree/shipping_method_spec.rb +0 -95
data/spec/models/spree/shipping_rate_spec.rb +0 -140
data/spec/models/spree/state_spec.rb +0 -29
data/spec/models/spree/stock/availability_validator_spec.rb +0 -36
data/spec/models/spree/stock/content_item_spec.rb +0 -31
data/spec/models/spree/stock/coordinator_spec.rb +0 -51
data/spec/models/spree/stock/differentiator_spec.rb +0 -39
data/spec/models/spree/stock/estimator_spec.rb +0 -202
data/spec/models/spree/stock/inventory_unit_builder_spec.rb +0 -38
data/spec/models/spree/stock/package_spec.rb +0 -194
data/spec/models/spree/stock/packer_spec.rb +0 -70
data/spec/models/spree/stock/prioritizer_spec.rb +0 -125
data/spec/models/spree/stock/quantifier_spec.rb +0 -126
data/spec/models/spree/stock/splitter/backordered_spec.rb +0 -29
data/spec/models/spree/stock/splitter/base_spec.rb +0 -21
data/spec/models/spree/stock/splitter/shipping_category_spec.rb +0 -47
data/spec/models/spree/stock/splitter/weight_spec.rb +0 -32
data/spec/models/spree/stock_item_spec.rb +0 -428
data/spec/models/spree/stock_location_spec.rb +0 -243
data/spec/models/spree/stock_movement_spec.rb +0 -56
data/spec/models/spree/stock_transfer_spec.rb +0 -50
data/spec/models/spree/store_credit_event_spec.rb +0 -101
data/spec/models/spree/store_credit_spec.rb +0 -786
data/spec/models/spree/store_spec.rb +0 -78
data/spec/models/spree/tax_category_spec.rb +0 -32
data/spec/models/spree/tax_rate_spec.rb +0 -542
data/spec/models/spree/taxon_spec.rb +0 -74
data/spec/models/spree/taxonomy_spec.rb +0 -18
data/spec/models/spree/tracker_spec.rb +0 -21
data/spec/models/spree/user_spec.rb +0 -203
data/spec/models/spree/variant_spec.rb +0 -795
data/spec/models/spree/zone_member_spec.rb +0 -38
data/spec/models/spree/zone_spec.rb +0 -470
data/spec/spec_helper.rb +0 -77
data/spec/support/big_decimal.rb +0 -5
data/spec/support/concerns/adjustment_source.rb +0 -23
data/spec/support/concerns/default_price.rb +0 -37
data/spec/support/rake.rb +0 -13
data/spec/support/test_gateway.rb +0 -2
data/spec/validators/db_maximum_length_validator_spec.rb +0 -22

data/spec/fixtures/microdata_itemref.html DELETED Viewed

@@ -1,15 +0,0 @@
-<!DOCTYPE html>
-<html>
-  <head>
-      <title>itemref example from Microdata spec</title>
-  </head>
-  <body>
-    <div itemscope id="amanda" itemref="a b"></div>
-    <p id="a">Name: <span itemprop="name">Amanda</span></p>
-    <div id="b" itemprop="band" itemscope itemref="c"></div>
-    <div id="c">
-      <p>Band: <span itemprop="name">Jazz Band</span></p>
-      <p>Size: <span itemprop="size">12</span> players</p>
-    </div>
-  </body>
-</html>

data/spec/fixtures/microdata_no_itemscope.html DELETED Viewed

@@ -1,20 +0,0 @@
-<!doctype html>
-<html>
-    <head>
-        <title>Jason Ronallo</title>
-    </head>
-    <body>
-      <span itemtype="http://schema.org/Person"
-        itemid="http://ronallo.com#me">
-        <a itemprop="url" href="http://twitter.com/ronallo">
-          <span itemprop="name">Jason Ronallo</span>
-        </a> is the
-        <span itemprop="jobTitle">Associate Head of Digital Library Initiatives</span> at
-        <span itemprop="affiliation" itemtype="http://schema.org/Library" itemid="http://lib.ncsu.edu">
-          <span itemprop="name">
-            <a itemprop="url" href="http://www.lib.ncsu.edu">NCSU Libraries</a>
-          </span>
-        </span>
-      </span>
-    </body>
-</html>

data/spec/fixtures/thinking-cat.jpg DELETED Viewed

Binary file

data/spec/helpers/base_helper_spec.rb DELETED Viewed

@@ -1,200 +0,0 @@
-require 'spec_helper'
-describe Spree::BaseHelper, type: :helper do
-  include Spree::BaseHelper
-  let(:current_store){ create :store }
-  context "available_countries" do
-    let(:country) { create(:country) }
-    before do
-      3.times { create(:country) }
-    end
-    context "with no checkout zone defined" do
-      before do
-        Spree::Config[:checkout_zone] = nil
-      end
-      it "return complete list of countries" do
-        expect(available_countries.count).to eq(Spree::Country.count)
-      end
-    end
-    context "with a checkout zone defined" do
-      context "checkout zone is of type country" do
-        before do
-          @country_zone = create(:zone, :name => "CountryZone")
-          @country_zone.members.create(:zoneable => country)
-          Spree::Config[:checkout_zone] = @country_zone.name
-        end
-        it "return only the countries defined by the checkout zone" do
-          expect(available_countries).to eq([country])
-        end
-      end
-      context "checkout zone is of type state" do
-        before do
-          state_zone = create(:zone, :name => "StateZone")
-          state = create(:state, :country => country)
-          state_zone.members.create(:zoneable => state)
-          Spree::Config[:checkout_zone] = state_zone.name
-        end
-        it "return complete list of countries" do
-          expect(available_countries.count).to eq(Spree::Country.count)
-        end
-      end
-    end
-  end
-  # Regression test for #1436
-  context "defining custom image helpers" do
-    let(:product) { mock_model(Spree::Product, :images => [], :variant_images => []) }
-    before do
-      Spree::Image.class_eval do
-        attachment_definitions[:attachment][:styles].merge!({:very_strange => '1x1'})
-      end
-    end
-    it "should not raise errors when style exists" do
-      expect { very_strange_image(product) }.not_to raise_error
-    end
-    it "should raise NoMethodError when style is not exists" do
-      expect { another_strange_image(product) }.to raise_error(NoMethodError)
-    end
-  end
-  context "link_to_tracking" do
-    it "returns tracking link if available" do
-      a = link_to_tracking_html(shipping_method: true, tracking: '123', tracking_url: 'http://g.c/?t=123').css('a')
-      expect(a.text).to eq '123'
-      expect(a.attr('href').value).to eq 'http://g.c/?t=123'
-    end
-    it "returns tracking without link if link unavailable" do
-      html = link_to_tracking_html(shipping_method: true, tracking: '123', tracking_url: nil)
-      expect(html.css('span').text).to eq '123'
-    end
-    it "returns nothing when no shipping method" do
-      html = link_to_tracking_html(shipping_method: nil, tracking: '123')
-      expect(html.css('span').text).to eq ''
-    end
-    it "returns nothing when no tracking" do
-      html = link_to_tracking_html(tracking: nil)
-      expect(html.css('span').text).to eq ''
-    end
-    def link_to_tracking_html(options = {})
-      node = link_to_tracking(double(:shipment, options))
-      Nokogiri::HTML(node.to_s)
-    end
-  end
-  # Regression test for #2396
-  context "meta_data_tags" do
-    it "truncates a product description to 160 characters" do
-      # Because the controller_name method returns "test"
-      # controller_name is used by this method to infer what it is supposed
-      # to be generating meta_data_tags for
-      text = FFaker::Lorem.paragraphs(2).join(" ")
-      @test = Spree::Product.new(:description => text)
-      tags = Nokogiri::HTML.parse(meta_data_tags)
-      content = tags.css("meta[name=description]").first["content"]
-      assert content.length <= 160, "content length is not truncated to 160 characters"
-    end
-  end
-  # Regression test for #5384
-  context "custom image helpers conflict with inproper statements" do
-    let(:product) { mock_model(Spree::Product, :images => [], :variant_images => []) }
-    before do
-      Spree::Image.class_eval do
-        attachment_definitions[:attachment][:styles].merge!({:foobar => '1x1'})
-      end
-    end
-    it "should not raise errors when helper method called" do
-      expect { foobar_image(product) }.not_to raise_error
-    end
-    it "should raise NoMethodError when statement with name equal to style name called" do
-      expect { foobar(product) }.to raise_error(NoMethodError)
-    end
-  end
-  context "pretty_time" do
-    it "prints in a format" do
-      expect(pretty_time(DateTime.new(2012, 5, 6, 13, 33))).to eq "May 06, 2012  1:33 PM"
-    end
-  end
-  describe "#display_price" do
-    let!(:product) { create(:product) }
-    let(:current_currency) { "USD" }
-    let(:current_price_options) { { tax_zone: current_tax_zone } }
-    context "when there is no current order" do
-      let (:current_tax_zone) { nil }
-      it "returns the price including default vat" do
-        expect(display_price(product)).to eq("$19.99")
-      end
-      context "with a default VAT" do
-        let(:current_tax_zone) { create(:zone_with_country, default_tax: true) }
-        let!(:tax_rate) do
-          create :tax_rate,
-                 included_in_price: true,
-                 zone: current_tax_zone,
-                 tax_category: product.tax_category,
-                 amount: 0.2
-        end
-        it "returns the price adding the VAT" do
-          expect(display_price(product)).to eq("$19.99")
-        end
-      end
-    end
-    context "with an order that has a tax zone" do
-      let(:current_tax_zone) { create(:zone_with_country) }
-      let(:current_order) { Spree::Order.new }
-      let(:default_zone) { create(:zone_with_country, default_tax: true) }
-      let!(:default_vat) do
-        create :tax_rate,
-               included_in_price: true,
-               zone: default_zone,
-               tax_category: product.tax_category,
-               amount: 0.2
-      end
-      context "that matches no VAT" do
-        it "returns the price excluding VAT" do
-          expect(display_price(product)).to eq("$16.66")
-        end
-      end
-      context "that matches a VAT" do
-        let!(:other_vat) do
-          create :tax_rate,
-                 included_in_price: true,
-                 zone: current_tax_zone,
-                 tax_category: product.tax_category,
-                 amount: 0.4
-        end
-        it "returns the price adding the VAT" do
-          expect(display_price(product)).to eq("$23.32")
-        end
-      end
-    end
-  end
-end

data/spec/helpers/products_helper_spec.rb DELETED Viewed

@@ -1,296 +0,0 @@
-# encoding: utf-8
-require "spec_helper"
-module Spree
-  describe ProductsHelper, :type => :helper do
-    include ProductsHelper
-    let(:product) { create(:product) }
-    let(:currency) { 'USD' }
-    before do
-      allow(helper).to receive(:current_currency) { currency }
-    end
-    context "#variant_price_diff" do
-      let(:product_price) { 10 }
-      let(:variant_price) { 10 }
-      before do
-        @variant = create(:variant, :product => product)
-        product.price = 15
-        @variant.price = 10
-        allow(product).to receive(:amount_in) { product_price }
-        allow(@variant).to receive(:amount_in) { variant_price }
-      end
-      subject { helper.variant_price(@variant) }
-      context "when variant is same as master" do
-        it { is_expected.to be_nil }
-      end
-      context "when the master has no price" do
-        let(:product_price) { nil }
-        it { is_expected.to be_nil }
-      end
-      context "when currency is default" do
-        context "when variant is more than master" do
-          let(:variant_price) { 15 }
-          it { is_expected.to eq("(Add: $5.00)") }
-          # Regression test for #2737
-          it { is_expected.to be_html_safe }
-        end
-        context "when variant is less than master" do
-          let(:product_price) { 15 }
-          it { is_expected.to eq("(Subtract: $5.00)") }
-        end
-      end
-      context "when currency is JPY" do
-        let(:variant_price) { 100 }
-        let(:product_price) { 100 }
-        let(:currency) { 'JPY' }
-        context "when variant is more than master" do
-          let(:variant_price) { 150 }
-          it { is_expected.to eq("(Add: &#x00A5;50)") }
-        end
-        context "when variant is less than master" do
-          let(:product_price) { 150 }
-          it { is_expected.to eq("(Subtract: &#x00A5;50)") }
-        end
-      end
-    end
-    context "#variant_price_full" do
-      before do
-        Spree::Config[:show_variant_full_price] = true
-        @variant1 = create(:variant, :product => product)
-        @variant2 = create(:variant, :product => product)
-      end
-      context "when currency is default" do
-        it "should return the variant price if the price is different than master" do
-          product.price = 10
-          @variant1.price = 15
-          @variant2.price = 20
-          expect(helper.variant_price(@variant1)).to eq("$15.00")
-          expect(helper.variant_price(@variant2)).to eq("$20.00")
-        end
-      end
-      context "when currency is JPY" do
-        let(:currency) { 'JPY' }
-        before do
-          product.variants.active.each do |variant|
-            variant.prices.each do |price|
-              price.currency = currency
-              price.save!
-            end
-          end
-        end
-        it "should return the variant price if the price is different than master" do
-          product.price = 100
-          @variant1.price = 150
-          expect(helper.variant_price(@variant1)).to eq("&#x00A5;150")
-        end
-      end
-      it "should be nil when all variant prices are equal" do
-        product.price = 10
-        @variant1.default_price.update_column(:amount, 10)
-        @variant2.default_price.update_column(:amount, 10)
-        expect(helper.variant_price(@variant1)).to be_nil
-        expect(helper.variant_price(@variant2)).to be_nil
-      end
-    end
-    context "#product_description" do
-      # Regression test for #1607
-      it "renders a product description without excessive paragraph breaks" do
-        product.description = %Q{
-<p>Lorem ipsum dolor sit amet, consectetur adipiscing elit. Vivamus a ligula leo. Proin eu arcu at ipsum dapibus ullamcorper. Pellentesque egestas orci nec magna condimentum luctus. Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus. Ut ac ante et mauris bibendum ultricies non sed massa. Fusce facilisis dui eget lacus scelerisque eget aliquam urna ultricies. Duis et rhoncus quam. Praesent tellus nisi, ultrices sed iaculis quis, euismod interdum ipsum.</p>
-<ul>
-<li>Lorem ipsum dolor sit amet</li>
-<li>Lorem ipsum dolor sit amet</li>
-</ul>
-        }
-        description = product_description(product)
-        expect(description.strip).to eq(product.description.strip)
-      end
-      it "renders a product description with automatic paragraph breaks" do
-        product.description = %Q{
-THIS IS THE BEST PRODUCT EVER!
-"IT CHANGED MY LIFE" - Sue, MD}
-        description = product_description(product)
-        expect(description.strip).to eq(%Q{<p>\nTHIS IS THE BEST PRODUCT EVER!</p>"IT CHANGED MY LIFE" - Sue, MD})
-      end
-      it "renders a product description without any formatting based on configuration" do
-        initialDescription = %Q{
-            <p>hello world</p>
-            <p>tihs is completely awesome and it works</p>
-            <p>why so many spaces in the code. and why some more formatting afterwards?</p>
-        }
-        product.description = initialDescription
-        Spree::Config[:show_raw_product_description] = true
-        description = product_description(product)
-        expect(description).to eq(initialDescription)
-      end
-      context "renders a product description default description incase description is blank" do
-        before { product.description = '' }
-        it { expect(product_description(product)).to eq(Spree.t(:product_has_no_description)) }
-      end
-    end
-    shared_examples_for "line item descriptions" do
-      context 'variant has a blank description' do
-        let(:description) { nil }
-        it { is_expected.to eq(Spree.t(:product_has_no_description)) }
-      end
-      context 'variant has a description' do
-        let(:description) { 'test_desc' }
-        it { is_expected.to eq(description) }
-      end
-      context 'description has nonbreaking spaces' do
-        let(:description) { 'test&nbsp;desc' }
-        it { is_expected.to eq('test desc') }
-      end
-      context 'description has line endings' do
-        let(:description) { "test\n\r\ndesc" }
-        it { is_expected.to eq('test desc') }
-      end
-    end
-    context "#line_item_description" do
-      let(:variant) { create(:variant, :product => product, description: description) }
-      subject { line_item_description_text(variant.product.description) }
-      it_should_behave_like "line item descriptions"
-    end
-    context '#line_item_description_text' do
-      subject { line_item_description_text description }
-      it_should_behave_like "line item descriptions"
-    end
-    context '#cache_key_for_products' do
-      let(:zone) { Spree::Zone.new }
-      let(:price_options) { { tax_zone: zone } }
-      subject { helper.cache_key_for_products }
-      before(:each) do
-        @products = double('products collection')
-        allow(helper).to receive(:params) { {:page => 10} }
-        allow(helper).to receive(:current_price_options) { price_options }
-      end
-      context 'when there is a maximum updated date' do
-        let(:updated_at) { Date.new(2011, 12, 13) }
-        before :each do
-          allow(@products).to receive(:count) { 5 }
-          allow(@products).to receive(:maximum).with(:updated_at) { updated_at }
-        end
-        it { is_expected.to eq("en/USD/spree/zones/new/spree/products/all-10-20111213-5") }
-      end
-      context 'when there is no considered maximum updated date' do
-        let(:today) { Date.new(2013, 12, 11) }
-        before :each do
-          allow(@products).to receive(:count) { 1234567 }
-          allow(@products).to receive(:maximum).with(:updated_at) { nil }
-          allow(Date).to receive(:today) { today }
-        end
-        it { is_expected.to eq("en/USD/spree/zones/new/spree/products/all-10-20131211-1234567") }
-      end
-    end
-    context "#cache_key_for_product" do
-      let(:product) { Spree::Product.new }
-      let(:price_options) { { tax_zone: zone } }
-      subject(:cache_key) { helper.cache_key_for_product(product) }
-      before do
-        allow(helper).to receive(:current_price_options) { price_options }
-      end
-      context "when there is a current tax zone" do
-        let(:zone) { Spree::Zone.new }
-        it "includes the current_tax_zone" do
-          is_expected.to eq("en/USD/spree/zones/new/spree/products/new/")
-        end
-      end
-      context "when there is no current tax zone" do
-        let(:zone) { nil }
-        it { is_expected.to eq("en/USD/spree/products/new/") }
-      end
-      context "when current_price_options includes nil values" do
-        let(:price_options) do
-          {
-            a: nil,
-            b: Spree::Zone.new
-          }
-        end
-        it "does not include nil values" do
-          expect(cache_key).to eq("en/USD/spree/zones/new/spree/products/new/")
-        end
-      end
-      context "when current_price_options includes values that do not implement cache_key" do
-        let(:price_options) do
-          {
-            a: true,
-            b: Spree::Zone.new
-          }
-        end
-        it "includes string representations of these values" do
-          expect(cache_key).to eq("en/USD/true/spree/zones/new/spree/products/new/")
-        end
-      end
-      context "when keys in the options hash are inserted in non-alphabetical order" do
-        let(:price_options) do
-          {
-            b: Spree::Zone.new,
-            a: true
-          }
-        end
-        it "the values are nevertheless returned in alphabetical order of their keys" do
-          expect(cache_key).to eq("en/USD/true/spree/zones/new/spree/products/new/")
-        end
-      end
-    end
-  end
-end