spree_frontend 3.3.0.rc1 → 3.3.0.rc2

data/spec/features/checkout_unshippable_spec.rb

@@ -1,34 +0,0 @@
-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

@@ -1,88 +0,0 @@
-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

@@ -1,18 +0,0 @@
-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

@@ -1,64 +0,0 @@
-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

@@ -1,59 +0,0 @@
-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

@@ -1,60 +0,0 @@
-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

@@ -1,107 +0,0 @@
-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

@@ -1,36 +0,0 @@
-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