spree_frontend 3.2.9 → 3.3.0.rc1

data/spec/features/checkout_unshippable_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe "checkout with unshippable items", type: :feature, inaccessible: true do
+  let!(:stock_location) { create(:stock_location) }
+  let(:order) { OrderWalkthrough.up_to(:delivery) }
+
+  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_with_updater!
+
+    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,88 @@
+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) }
+  let!(:store) { create(:store) }
+
+  context "visitor makes checkout as guest without registration" do
+    def create_basic_coupon_promotion(code)
+      promotion = Spree::Promotion.create!(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 "informs the user about a coupon code which has exceeded its usage" do
+        promotion.update_column(:usage_limit, 5)
+        allow_any_instance_of(promotion.class).to receive_messages(credits_count: 10)
+
+        fill_in "order_coupon_code", with: "onetwo"
+        click_button "Save and Continue"
+        expect(page).to have_content(Spree.t(:coupon_code_max_usage))
+      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
+  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/delivery_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe 'Delivery', type: :feature, inaccessible: true, js: true do
+  include_context 'checkout setup'
+
+  let(:country) { create(:country, name: 'United States of America', iso_name: 'UNITED STATES') }
+  let(:state) { create(:state, name: 'Alabama', abbr: 'AL', country: country) }
+  let!(:product2) { create(:product) }
+  let(:user) { create(:user) }
+  let!(:shipping_method2) do
+    sm = create(:shipping_method, name: 'Shipping Method2')
+    sm.calculator.preferred_amount = 20
+    sm.calculator.save
+    sm
+  end
+
+  before do
+    shipping_method.calculator.preferred_amount = 10
+    shipping_method.calculator.save
+  end
+
+  describe 'shipping total gets updated when shipping method is changed in the delivery step' do
+    before do
+      add_mug_to_cart
+      click_button 'Checkout'
+
+      fill_in 'order_email', with: 'test@example.com'
+      click_on 'Continue'
+      fill_in_address
+
+      click_button 'Save and Continue'
+    end
+
+    it 'should contain the shipping total' do
+      expect(page).to have_content('Shipping total: $10.00')
+    end
+
+    context 'shipping method is changed' do
+      before { choose(shipping_method2.name) }
+
+      it 'shipping total and order total both are updates' do
+        expect(page).to have_content("Shipping total: $20.00")
+      end
+    end
+  end
+
+  def fill_in_address
+    address = "order_bill_address_attributes"
+    fill_in "#{address}_firstname", with: FFaker::Name.first_name
+    fill_in "#{address}_lastname", with: FFaker::Name.last_name
+    fill_in "#{address}_address1", with: FFaker::Address.street_address
+    fill_in "#{address}_city", with: FFaker::Address.city
+    select country.name, from: "#{address}_country_id"
+    select state.name, from: "#{address}_state_id"
+    fill_in "#{address}_zipcode", with: FFaker::AddressUS.zip_code
+    fill_in "#{address}_phone", with: FFaker::PhoneNumber.phone_number
+  end
+
+  def add_mug_to_cart
+    visit spree.root_path
+    click_link mug.name
+    click_button "add-to-cart-button"
+  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 = 'en'
+    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("#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,107 @@
+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)
+    order.shipments.destroy_all
+    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
+
+  # Regression test for #6733
+  context "address_requires_state preference" do
+    context "when set to true" do
+      before do
+        configure_spree_preferences { |config| config.address_requires_state = true }
+      end
+
+      it "should show state text" do
+        visit spree.order_path(order)
+
+        within '#order' do
+          expect(page).to have_content(order.bill_address.state_text)
+          expect(page).to have_content(order.ship_address.state_text)
+        end
+      end
+    end
+
+    context "when set to false" do
+      before do
+        configure_spree_preferences { |config| config.address_requires_state = false }
+      end
+
+      it "should not show state text" do
+        visit spree.order_path(order)
+
+        within '#order' do
+          expect(page).not_to have_content(order.bill_address.state_text)
+          expect(page).not_to have_content(order.ship_address.state_text)
+        end
+      end
+    end
+  end
+end

data/spec/features/page_promotions_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe 'page promotions', type: :feature do
+  let!(:product) { create(:product, name: "RoR Mug", price: 20) }
+  before do
+    promotion = Spree::Promotion.create!(name:       "$10 off",
+                                         path:       'test',
+                                         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.create(calculator: calculator)
+   promotion.actions << action
+
+   visit spree.root_path
+   click_link "RoR Mug"
+   click_button "add-to-cart-button"
+  end
+
+  it "automatically applies a page promotion upon visiting" do
+    expect(page).not_to have_content("Promotion ($10 off) -$10.00")
+    visit '/content/test'
+    visit '/cart'
+    expect(page).to have_content("Promotion ($10 off) -$10.00")
+    expect(page).to have_content("Subtotal (1 item) $20.00")
+  end
+
+  it "does not activate an adjustment for a path that doesn't have a promotion" do
+    expect(page).not_to have_content("Promotion ($10 off) -$10.00")
+    visit '/content/cvv'
+    visit '/cart'
+    expect(page).not_to have_content("Promotion ($10 off) -$10.00")
+  end
+end