solidus_frontend 1.0.2 → 1.0.3

data/spec/features/checkout_unshippable_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe "checkout with unshippable items", type: :feature, inaccessible: true do
+  let!(:stock_location) { create(:stock_location) }
+  let(:order) { OrderWalkthrough.up_to(:address) }
+
+  before do
+    OrderWalkthrough.add_line_item!(order)
+    line_item = order.line_items.last
+    stock_item = stock_location.stock_item(line_item.variant)
+    stock_item.adjust_count_on_hand -999
+    stock_item.backorderable = false
+    stock_item.save!
+
+    user = create(:user)
+    order.user = user
+    order.update!
+
+    allow_any_instance_of(Spree::CheckoutController).to receive_messages(:current_order => order)
+    allow_any_instance_of(Spree::CheckoutController).to receive_messages(:try_spree_current_user => user)
+    allow_any_instance_of(Spree::CheckoutController).to receive_messages(:skip_state_validation? => true)
+    allow_any_instance_of(Spree::CheckoutController).to receive_messages(:ensure_sufficient_stock_lines => true)
+  end
+
+  it 'displays and removes' do
+    visit spree.checkout_state_path(:delivery)
+    expect(page).to have_content('Unshippable Items')
+
+    click_button "Save and Continue"
+
+    order.reload
+    expect(order.line_items.count).to eq 1
+  end
+end
+

data/spec/features/coupon_code_spec.rb

@@ -0,0 +1,227 @@
+require 'spec_helper'
+
+describe "Coupon code promotions", type: :feature, js: true do
+  let!(:country) { create(:country, :name => "United States of America", :states_required => true) }
+  let!(:state) { create(:state, :name => "Alabama", :country => country) }
+  let!(:zone) { create(:zone) }
+  let!(:shipping_method) { create(:shipping_method) }
+  let!(:payment_method) { create(:check_payment_method) }
+  let!(:product) { create(:product, :name => "RoR Mug", :price => 20) }
+
+  context "visitor makes checkout as guest without registration" do
+    def create_basic_coupon_promotion(code)
+      promotion = create(
+        :promotion,
+        name:       code.titleize,
+        code:       code,
+        starts_at:  1.day.ago,
+        expires_at: 1.day.from_now,
+      )
+
+      calculator = Spree::Calculator::FlatRate.new
+      calculator.preferred_amount = 10
+
+      action = Spree::Promotion::Actions::CreateItemAdjustments.new
+      action.calculator = calculator
+      action.promotion = promotion
+      action.save
+
+      promotion.reload # so that promotion.actions is available
+    end
+
+    let!(:promotion) { create_basic_coupon_promotion("onetwo") }
+
+    # OrdersController
+    context "on the payment page" do
+      before do
+
+        visit spree.root_path
+        click_link "RoR Mug"
+        click_button "add-to-cart-button"
+        click_button "Checkout"
+        fill_in "order_email", :with => "spree@example.com"
+        fill_in "First Name", :with => "John"
+        fill_in "Last Name", :with => "Smith"
+        fill_in "Street Address", :with => "1 John Street"
+        fill_in "City", :with => "City of John"
+        fill_in "Zip", :with => "01337"
+        select country.name, :from => "Country"
+        select state.name, :from => "order[bill_address_attributes][state_id]"
+        fill_in "Phone", :with => "555-555-5555"
+
+        # To shipping method screen
+        click_button "Save and Continue"
+        # To payment screen
+        click_button "Save and Continue"
+      end
+
+      it "informs about an invalid coupon code" do
+        fill_in "order_coupon_code", :with => "coupon_codes_rule_man"
+        click_button "Save and Continue"
+        expect(page).to have_content(Spree.t(:coupon_code_not_found))
+      end
+
+      it "can enter an invalid coupon code, then a real one" do
+        fill_in "order_coupon_code", :with => "coupon_codes_rule_man"
+        click_button "Save and Continue"
+        expect(page).to have_content(Spree.t(:coupon_code_not_found))
+        fill_in "order_coupon_code", :with => "onetwo"
+        click_button "Save and Continue"
+        expect(page).to have_content("Promotion (Onetwo)   -$10.00")
+      end
+
+      context "with a promotion" do
+        it "applies a promotion to an order" do
+          fill_in "order_coupon_code", :with => "onetwo"
+          click_button "Save and Continue"
+          expect(page).to have_content("Promotion (Onetwo)   -$10.00")
+        end
+      end
+    end
+
+    # CheckoutController
+    context "on the cart page" do
+
+      before do
+        visit spree.root_path
+        click_link "RoR Mug"
+        click_button "add-to-cart-button"
+      end
+
+      it "can enter a coupon code and receives success notification" do
+        fill_in "order_coupon_code", :with => "onetwo"
+        click_button "Update"
+        expect(page).to have_content(Spree.t(:coupon_code_applied))
+      end
+
+      it "can enter a promotion code with both upper and lower case letters" do
+        fill_in "order_coupon_code", :with => "ONETwO"
+        click_button "Update"
+        expect(page).to have_content(Spree.t(:coupon_code_applied))
+      end
+
+      it "informs the user about a coupon code which has exceeded its usage" do
+        expect_any_instance_of(Spree::Promotion).to receive(:usage_limit_exceeded?).and_return(true)
+
+        fill_in "order_coupon_code", :with => "onetwo"
+        click_button "Update"
+        expect(page).to have_content(Spree.t(:coupon_code_max_usage))
+      end
+
+      context "informs the user if the coupon code is not eligible" do
+        before do
+          rule = Spree::Promotion::Rules::ItemTotal.new
+          rule.promotion = promotion
+          rule.preferred_amount = 100
+          rule.save
+        end
+
+        specify do
+          visit spree.cart_path
+
+          fill_in "order_coupon_code", :with => "onetwo"
+          click_button "Update"
+          expect(page).to have_content(Spree.t(:item_total_less_than_or_equal, scope: [:eligibility_errors, :messages], amount: "$100.00"))
+        end
+      end
+
+      it "informs the user if the coupon is expired" do
+        promotion.expires_at = Date.today.beginning_of_week
+        promotion.starts_at = Date.today.beginning_of_week.advance(:day => 3)
+        promotion.save!
+        fill_in "order_coupon_code", :with => "onetwo"
+        click_button "Update"
+        expect(page).to have_content(Spree.t(:coupon_code_expired))
+      end
+
+      context "calculates the correct amount of money saved with flat percent promotions" do
+        before do
+          calculator = Spree::Calculator::FlatPercentItemTotal.new
+          calculator.preferred_flat_percent = 20
+          promotion.actions.first.calculator = calculator
+          promotion.save
+
+          create(:product, :name => "Spree Mug", :price => 10)
+        end
+
+        specify do
+          visit spree.root_path
+          click_link "Spree Mug"
+          click_button "add-to-cart-button"
+
+          visit spree.cart_path
+          fill_in "order_coupon_code", :with => "onetwo"
+          click_button "Update"
+
+          fill_in "order_line_items_attributes_0_quantity", :with => 2
+          fill_in "order_line_items_attributes_1_quantity", :with => 2
+          click_button "Update"
+
+
+          within '#cart_adjustments' do
+            # 20% of $40 = 8
+            # 20% of $20 = 4
+            # Therefore: promotion discount amount is $12.
+            expect(page).to have_content("Promotion (Onetwo) -$12.00")
+          end
+
+          within '.cart-total' do
+            expect(page).to have_content("$48.00")
+          end
+        end
+      end
+
+      context "calculates the correct amount of money saved with flat 100% promotions on the whole order" do
+        before do
+          calculator = Spree::Calculator::FlatPercentItemTotal.new
+          calculator.preferred_flat_percent = 100
+
+          action = Spree::Promotion::Actions::CreateAdjustment.new
+          action.calculator = calculator
+          action.promotion = promotion
+          action.save
+
+          promotion.promotion_actions = [action]
+          promotion.save
+
+          create(:product, name: "Spree Mug", price: 10)
+        end
+
+        specify do
+          visit spree.root_path
+          click_link "Spree Mug"
+          click_button "add-to-cart-button"
+
+          visit spree.cart_path
+
+          within '.cart-total' do
+            expect(page).to have_content("$30.00")
+          end
+
+          fill_in "order_coupon_code", with: "onetwo"
+          click_button "Update"
+
+          within '#cart_adjustments' do
+            expect(page).to have_content("Promotion (Onetwo) -$30.00")
+          end
+
+          within '.cart-total' do
+            expect(page).to have_content("$0.00")
+          end
+
+          fill_in "order_line_items_attributes_0_quantity", with: 2
+          fill_in "order_line_items_attributes_1_quantity", with: 2
+          click_button "Update"
+
+          within '#cart_adjustments' do
+            expect(page).to have_content("Promotion (Onetwo) -$60.00")
+          end
+
+          within '.cart-total' do
+            expect(page).to have_content("$0.00")
+          end
+        end
+      end
+    end
+  end
+end

data/spec/features/currency_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe "Switching currencies in backend", :type => :feature do
+  before do
+    create(:base_product, :name => "RoR Mug")
+  end
+
+  # Regression test for #2340
+  it "does not cause current_order to become nil", inaccessible: true do
+    visit spree.root_path
+    click_link "RoR Mug"
+    click_button "Add To Cart"
+    # Now that we have an order...
+    Spree::Config[:currency] = "AUD"
+    expect { visit spree.root_path }.not_to raise_error
+  end
+
+end

data/spec/features/free_shipping_promotions_spec.rb

@@ -0,0 +1,59 @@
+require 'spec_helper'
+
+describe "Free shipping promotions", :type => :feature, :js => true do
+  let!(:country) { create(:country, :name => "United States of America", :states_required => true) }
+  let!(:state) { create(:state, :name => "Alabama", :country => country) }
+  let!(:zone) { create(:zone) }
+  let!(:shipping_method) do
+    sm = create(:shipping_method)
+    sm.calculator.preferred_amount = 10
+    sm.calculator.save
+    sm
+  end
+
+  let!(:payment_method) { create(:check_payment_method) }
+  let!(:product) { create(:product, :name => "RoR Mug", :price => 20) }
+  let!(:promotion) do
+    promotion = Spree::Promotion.create!(:name       => "Free Shipping",
+                                         :starts_at  => 1.day.ago,
+                                         :expires_at => 1.day.from_now)
+
+    action = Spree::Promotion::Actions::FreeShipping.new
+    action.promotion = promotion
+    action.save
+
+    promotion.reload # so that promotion.actions is available
+  end
+
+  context "free shipping promotion automatically applied" do
+    before do
+
+      visit spree.root_path
+      click_link "RoR Mug"
+      click_button "add-to-cart-button"
+      click_button "Checkout"
+      fill_in "order_email", :with => "spree@example.com"
+      fill_in "First Name", :with => "John"
+      fill_in "Last Name", :with => "Smith"
+      fill_in "Street Address", :with => "1 John Street"
+      fill_in "City", :with => "City of John"
+      fill_in "Zip", :with => "01337"
+      select country.name, :from => "Country"
+      select state.name, :from => "order[bill_address_attributes][state_id]"
+      fill_in "Phone", :with => "555-555-5555"
+
+      # To shipping method screen
+      click_button "Save and Continue"
+      # To payment screen
+      click_button "Save and Continue"
+    end
+
+    # Regression test for #4428
+    it "applies the free shipping promotion" do
+      within("#checkout-summary") do
+        expect(page).to have_content("Shipping total:  $10.00")
+        expect(page).to have_content("Promotion (Free Shipping): -$10.00")
+      end
+    end
+  end
+end

data/spec/features/locale_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe 'setting locale', :type => :feature do
+  def with_locale(locale)
+    I18n.locale = locale
+    Spree::Frontend::Config[:locale] = locale
+    yield
+    I18n.locale = I18n.default_locale
+    Spree::Frontend::Config[:locale] = 'en'
+  end
+
+  context 'shopping cart link and page' do
+    before do
+      I18n.backend.store_translations(:fr,
+       :spree => {
+         :cart => 'Panier',
+         :shopping_cart => 'Panier'
+      })
+    end
+
+    it 'should be in french' do
+      with_locale('fr') do
+        visit spree.root_path
+        click_link 'Panier'
+        expect(page).to have_content('Panier')
+      end
+    end
+  end
+
+  context 'checkout form validation messages' do
+    include_context 'checkout setup'
+
+    let(:error_messages) do
+      {
+        'en' => 'This field is required.',
+        'fr' => 'Ce champ est obligatoire.',
+        'de' => 'Dieses Feld ist ein Pflichtfeld.',
+      }
+    end
+
+    def check_error_text(text)
+      %w(firstname lastname address1 city).each do |attr|
+        expect(find(".field#b#{attr} label.error").text).to eq(text)
+      end
+    end
+
+    it 'shows translated jquery.validate error messages', js: true do
+      visit spree.root_path
+      click_link mug.name
+      click_button 'add-to-cart-button'
+      error_messages.each do |locale, message|
+        with_locale(locale) do
+          visit '/checkout/address'
+          find('.form-buttons input[type=submit]').click
+          check_error_text message
+        end
+      end
+    end
+  end
+end

data/spec/features/order_spec.rb

@@ -0,0 +1,73 @@
+require 'spec_helper'
+
+describe 'orders', :type => :feature do
+  let(:order) { OrderWalkthrough.up_to(:complete) }
+  let(:user) { create(:user) }
+
+  before do
+    order.update_attribute(:user_id, user.id)
+    allow_any_instance_of(Spree::OrdersController).to receive_messages(:try_spree_current_user => user)
+  end
+
+  it "can visit an order" do
+    # Regression test for current_user call on orders/show
+    expect { visit spree.order_path(order) }.not_to raise_error
+  end
+
+  it "should display line item price" do
+    # Regression test for #2772
+    line_item = order.line_items.first
+    line_item.target_shipment = create(:shipment)
+    line_item.price = 19.00
+    line_item.save!
+
+    visit spree.order_path(order)
+
+    # Tests view spree/shared/_order_details
+    within 'td.price' do
+      expect(page).to have_content "19.00"
+    end
+  end
+
+  it "should have credit card info if paid with credit card" do
+    create(:payment, :order => order)
+    visit spree.order_path(order)
+    within '.payment-info' do
+      expect(page).to have_content "Ending in 1111"
+    end
+  end
+
+  it "should have payment method name visible if not paid with credit card" do
+    create(:check_payment, :order => order)
+    visit spree.order_path(order)
+    within '.payment-info' do
+      expect(page).to have_content "Check"
+    end
+  end
+
+  # Regression test for #2282
+  context "can support a credit card with blank information" do
+    before do
+      credit_card = create(:credit_card)
+      credit_card.update_column(:cc_type, '')
+      payment = order.payments.first
+      payment.source = credit_card
+      payment.save!
+    end
+
+    specify do
+      visit spree.order_path(order)
+      within '.payment-info' do
+        expect { find("img") }.to raise_error(Capybara::ElementNotFound)
+      end
+    end
+  end
+
+  it "should return the correct title when displaying a completed order" do
+    visit spree.order_path(order)
+
+    within '#order_summary' do
+      expect(page).to have_content("#{Spree.t(:order)} #{order.number}")
+    end
+  end
+end