spree_backend 3.2.1 → 3.2.2

data/spec/features/admin/orders/payments_spec.rb

@@ -1,231 +0,0 @@
-require 'spec_helper'
-
-describe 'Payments', type: :feature, js: true do
-  stub_authorization!
-
-  context "with a pre-existing payment" do
-
-    let!(:payment) do
-      create(:payment,
-        order:          order,
-        amount:         order.outstanding_balance,
-        payment_method: create(:credit_card_payment_method),
-        state:          state
-      )
-    end
-
-    let(:order) { create(:completed_order_with_totals, number: 'R100', line_items_count: 5) }
-    let(:state) { 'checkout' }
-
-    before do
-      visit spree.edit_admin_order_path(order)
-      click_link 'Payments'
-    end
-
-    def refresh_page
-      visit current_path
-    end
-
-    # Regression tests for #1453
-    context 'with a check payment' do
-      let(:order) { create(:completed_order_with_totals, number: 'R100') }
-      let!(:payment) do
-        create(:payment,
-          order:          order,
-          amount:         order.outstanding_balance,
-          payment_method: create(:check_payment_method)  # Check
-        )
-      end
-
-      it 'capturing a check payment from a new order' do
-        click_icon(:capture)
-        expect(page).not_to have_content('Cannot perform requested operation')
-        expect(page).to have_content('Payment Updated')
-      end
-
-      it 'voids a check payment from a new order' do
-        click_icon(:void)
-        expect(page).to have_content('Payment Updated')
-      end
-    end
-
-    it 'should list all captures for a payment' do
-      Spree::ShippingMethod.delete_all
-      capture_amount = order.outstanding_balance/2 * 100
-      payment.capture!(capture_amount)
-
-      visit spree.admin_order_payment_path(order, payment)
-      expect(page).to have_content 'Capture events'
-      within '#capture_events' do
-        within_row(1) do
-          expect(page).to have_content(capture_amount / 100)
-        end
-      end
-    end
-
-    it 'lists and create payments for an order' do
-      within_row(1) do
-        expect(column_text(3)).to eq('$150.00')
-        expect(column_text(4)).to eq('Credit Card')
-        expect(column_text(6)).to eq('checkout')
-      end
-
-      click_icon :void
-      expect(find('#payment_status').text).to eq('balance due')
-      expect(page).to have_content('Payment Updated')
-
-      within_row(1) do
-        expect(column_text(3)).to eq('$150.00')
-        expect(column_text(4)).to eq('Credit Card')
-        expect(column_text(6)).to eq('void')
-      end
-
-      click_on 'New Payment'
-      expect(page).to have_content('New Payment')
-      click_button 'Update'
-      expect(page).to have_content('successfully created!')
-
-      click_icon(:capture)
-      wait_for_ajax
-      expect(find('#payment_status').text).to eq('paid')
-
-      expect(page).not_to have_selector('#new_payment_section')
-    end
-
-    # Regression test for #1269
-    it 'cannot create a payment for an order with no payment methods', js: false do
-      Spree::PaymentMethod.delete_all
-      order.payments.delete_all
-
-      click_on 'New Payment'
-      expect(page).to have_content('You cannot create a payment for an order without any payment methods defined.')
-      expect(page).to have_content('Please define some payment methods first.')
-    end
-
-    %w[checkout pending].each do |state|
-      context "payment is #{state.inspect}" do
-        let(:state) { state }
-
-        it 'allows the amount to be edited by clicking on the edit button then saving' do
-          within_row(1) do
-            click_icon(:edit)
-            fill_in('amount', with: '$1')
-            click_icon(:save)
-            expect(page).to have_selector('td.amount span', text: '$1.00')
-            expect(payment.reload.amount).to eq(1.00)
-          end
-        end
-
-        it 'allows the amount to be edited by clicking on the amount then saving' do
-          within_row(1) do
-            find('td.amount span').click
-            fill_in('amount', with: '$1.01')
-            click_icon(:save)
-            expect(page).to have_selector('td.amount span', text: '$1.01')
-            expect(payment.reload.amount).to eq(1.01)
-          end
-        end
-
-        it 'allows the amount change to be cancelled by clicking on the cancel button' do
-          within_row(1) do
-            click_icon(:edit)
-
-            # Can't use fill_in here, as under poltergeist that will unfocus (and
-            # thus submit) the field under poltergeist
-            find('td.amount input').click
-            page.execute_script("$('td.amount input').val('$1')")
-
-            click_icon(:cancel)
-            expect(page).to have_selector('td.amount span', text: '$150.00')
-            expect(payment.reload.amount).to eq(150.00)
-          end
-        end
-
-        it 'displays an error when the amount is invalid' do
-          within_row(1) do
-            click_icon(:edit)
-            fill_in('amount', with: 'invalid')
-            click_icon(:save)
-            expect(find('td.amount input').value).to eq('invalid')
-            expect(payment.reload.amount).to eq(150.00)
-          end
-          expect(page).to have_selector('.alert-error', text: 'Invalid resource. Please fix errors and try again.')
-        end
-      end
-    end
-
-    context 'payment is completed', js: false do
-      let(:state) { 'completed' }
-
-      it 'does not allow the amount to be edited' do
-        within_row(1) do
-          expect(page).not_to have_selector('.fa-edit')
-          expect(page).not_to have_selector('td.amount span')
-        end
-      end
-    end
-  end
-
-  context "with no prior payments" do
-    let(:order) { create(:order_with_line_items, line_items_count: 1) }
-    let!(:payment_method) { create(:credit_card_payment_method) }
-
-    # Regression tests for #4129
-    context "with a credit card payment method" do
-      before do
-        visit spree.admin_order_payments_path(order)
-      end
-
-      it "is able to create a new credit card payment with valid information" do
-        fill_in "Card Number", with: "4111 1111 1111 1111"
-        fill_in "Name", with: "Test User"
-        fill_in "Expiration", with: "09 / #{Time.current.year + 1}"
-        fill_in "Card Code", with: "007"
-        # Regression test for #4277
-        sleep(1)
-        expect(find('.ccType', visible: false).value).to eq('visa')
-        click_button "Continue"
-        expect(page).to have_content("Payment has been successfully created!")
-      end
-
-      it "is unable to create a new payment with invalid information" do
-        click_button "Continue"
-        expect(page).to have_content("Payment could not be created.")
-        expect(page).to have_content("Number can't be blank")
-        expect(page).to have_content("Name can't be blank")
-        expect(page).to have_content("Verification Value can't be blank")
-        expect(page).to have_content("Month is not a number")
-        expect(page).to have_content("Year is not a number")
-      end
-    end
-
-    context "user existing card" do
-      let!(:cc) do
-        create(:credit_card, user_id: order.user_id, payment_method: payment_method, gateway_customer_profile_id: "BGS-RFRE")
-      end
-
-      before { visit spree.admin_order_payments_path(order) }
-
-      it "is able to reuse customer payment source", js: false do
-        expect(find("#card_#{cc.id}")).to be_checked
-        click_button "Continue"
-        expect(page).to have_content("Payment has been successfully created!")
-      end
-    end
-
-    context "with a check" do
-      let!(:payment_method) { create(:check_payment_method) }
-
-      before do
-        visit spree.admin_order_payments_path(order.reload)
-      end
-
-      it "can successfully be created and captured" do
-        click_on 'Continue'
-        expect(page).to have_content("Payment has been successfully created!")
-        click_icon(:capture)
-        expect(page).to have_content("Payment Updated")
-      end
-    end
-  end
-end

data/spec/features/admin/orders/risk_analysis_spec.rb

@@ -1,48 +0,0 @@
-require 'spec_helper'
-
-describe 'Order Risk Analysis', type: :feature do
-  stub_authorization!
-
-  let!(:order) do
-    create(:completed_order_with_pending_payment)
-  end
-
-  def visit_order
-    visit spree.admin_path
-    click_link 'Orders'
-    within_row(1) do
-      click_link order.number
-    end
-  end
-
-  context "the order is considered risky" do
-    before do
-      allow_any_instance_of(Spree::Admin::BaseController).to receive_messages try_spree_current_user: create(:user)
-
-      order.payments.first.update_column(:avs_response, 'X')
-      order.considered_risky!
-      visit_order
-    end
-
-    it "displays 'Risk Analysis' box" do
-      expect(page).to have_content 'Risk Analysis'
-    end
-
-    it "can be approved" do
-      click_button('Approve')
-      expect(page).to have_content 'Approver'
-      expect(page).to have_content 'Approved at'
-      expect(page).to have_content 'Status: complete'
-    end
-  end
-
-  context "the order is not considered risky" do
-    before do
-      visit_order
-    end
-
-    it "does not display 'Risk Analysis' box" do
-      expect(page).to_not have_content 'Risk Analysis'
-    end
-  end
-end

data/spec/features/admin/orders/shipments_spec.rb

@@ -1,64 +0,0 @@
-require 'spec_helper'
-
-describe "Shipments", type: :feature do
-  stub_authorization!
-
-  let!(:order) { create(:order_ready_to_ship, number: "R100", state: "complete", line_items_count: 5) }
-
-  # Regression test for #4025
-  context "a shipment without a shipping method" do
-    before do
-      order.shipments.each do |s|
-        # Deleting the shipping rates causes there to be no shipping methods
-        s.shipping_rates.delete_all
-      end
-    end
-
-    it "can still be displayed" do
-      expect { visit spree.edit_admin_order_path(order) }.not_to raise_error
-    end
-  end
-
-  context "shipping an order", js: true do
-    before(:each) do
-      visit spree.admin_orders_path
-      within_row(1) do
-        click_link "R100"
-      end
-    end
-
-    it "can ship a completed order" do
-      click_on "Ship"
-      wait_for_ajax
-
-      expect(page).to have_content("shipped package")
-      expect(order.reload.shipment_state).to eq("shipped")
-    end
-  end
-
-  context "moving variants between shipments", js: true do
-    let!(:la) { create(:stock_location, name: "LA") }
-    before(:each) do
-      visit spree.admin_orders_path
-      within_row(1) do
-        click_link "R100"
-      end
-    end
-
-    it "can move a variant to a new and to an existing shipment" do
-      expect(order.shipments.count).to eq(1)
-
-      within_row(1) { click_icon :split }
-      targetted_select2 'LA', from: '#s2id_item_stock_location'
-      click_icon :save
-      wait_for_ajax
-      expect(page.find("#shipment_#{order.shipments.first.id}")).to be_present
-
-      within_row(2) { click_icon :split }
-      targetted_select2 "LA(#{order.reload.shipments.last.number})", from: '#s2id_item_stock_location'
-      click_icon :save
-      wait_for_ajax
-      expect(page.find("#shipment_#{order.reload.shipments.last.id}")).to be_present
-    end
-  end
-end

data/spec/features/admin/orders/state_changes_spec.rb

@@ -1,21 +0,0 @@
-require 'spec_helper'
-
-describe "Order - State Changes", type: :feature do
-  stub_authorization!
-
-  let!(:order) { create(:order_with_line_items) }
-
-  context "for completed order" do
-    before do
-      order.next!
-      visit spree.admin_order_state_changes_path(order)
-    end
-    it 'are viewable' do
-      within_row(1) do
-        within('td:nth-child(1)') { expect(page).to have_content('Order') }
-        within('td:nth-child(2)') { expect(page).to have_content('Cart') }
-        within('td:nth-child(3)') { expect(page).to have_content('Address') }
-      end
-    end
-  end
-end

data/spec/features/admin/products/edit/images_spec.rb

@@ -1,86 +0,0 @@
-require 'spec_helper'
-
-describe "Product Images", type: :feature, js: true do
-  stub_authorization!
-
-  let(:file_path) { Rails.root + "../../spec/support/ror_ringer.jpeg" }
-
-  before do
-    # Ensure attachment style keys are symbolized before running all tests
-    # Otherwise this would result in this error:
-    # undefined method `processors' for \"48x48>\
-    Spree::Image.attachment_definitions[:attachment][:styles].symbolize_keys!
-  end
-
-  context "uploading, editing, and deleting an image" do
-    it "should allow an admin to upload and edit an image for a product" do
-      Spree::Image.attachment_definitions[:attachment].delete :storage
-
-      create(:product)
-
-      visit spree.admin_products_path
-      click_icon(:edit)
-      click_link "Images"
-      click_link "new_image_link"
-      attach_file('image_attachment', file_path)
-      click_button "Create"
-      expect(page).to have_content("successfully created!")
-
-      click_icon(:edit)
-      fill_in "image_alt", with: "ruby on rails t-shirt"
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-      expect(page).to have_content("ruby on rails t-shirt")
-
-      accept_alert do
-        click_icon :delete
-      end
-      expect(page).not_to have_content("ruby on rails t-shirt")
-    end
-  end
-
-  # Regression test for #2228
-  it "should see variant images", js: false do
-    variant = create(:variant)
-    variant.images.create!(attachment: File.open(file_path))
-    visit spree.admin_product_images_path(variant.product)
-
-    expect(page).not_to have_content("No Images Found.")
-    within("table.table") do
-      expect(page).to have_content(variant.options_text)
-
-      #ensure no duplicate images are displayed
-      expect(page).to have_css("tbody tr", count: 1)
-
-      #ensure variant header is displayed
-      within("thead") do
-        expect(page.body).to have_content("Variant")
-      end
-
-      #ensure variant header is displayed
-      within("tbody") do
-        expect(page).to have_content("Size: S")
-      end
-    end
-  end
-
-  it "should not see variant column when product has no variants", js: false do
-    product = create(:product)
-    product.images.create!(attachment: File.open(file_path))
-    visit spree.admin_product_images_path(product)
-
-    expect(page).not_to have_content("No Images Found.")
-    within("table.table") do
-      #ensure no duplicate images are displayed
-      expect(page).to have_css("tbody tr", count: 1)
-
-      #ensure variant header is not displayed
-      within("thead") do
-        expect(page).not_to have_content("Variant")
-      end
-
-      #ensure correct cell count
-      expect(page).to have_css("thead th", count: 3)
-    end
-  end
-end

data/spec/features/admin/products/edit/products_spec.rb

@@ -1,64 +0,0 @@
-# encoding: UTF-8
-require 'spec_helper'
-
-describe 'Product Details', type: :feature, js: true do
-  stub_authorization!
-
-  context 'editing a product' do
-    it 'should list the product details' do
-      create(:product, name: 'Bún thịt nướng', sku: 'A100',
-              description: 'lorem ipsum', available_on: '2013-08-14 01:02:03')
-
-      visit spree.admin_products_path
-      within_row(1) { click_icon :edit }
-
-      click_link 'Details'
-
-      expect(find('.content-header h1').text.strip).to eq('Products / Bún thịt nướng')
-      expect(find('input#product_name').value).to eq('Bún thịt nướng')
-      expect(find('input#product_slug').value).to eq('bun-th-t-n-ng')
-      expect(find('textarea#product_description').text.strip).to eq('lorem ipsum')
-      expect(find('input#product_price').value).to eq('19.99')
-      expect(find('input#product_cost_price').value).to eq('17.00')
-      expect(find('input#product_available_on').value).to eq("2013/08/14")
-      expect(find('input#product_sku').value).to eq('A100')
-    end
-
-    it "should handle slug changes" do
-      create(:product, name: 'Bún thịt nướng', sku: 'A100',
-               description: 'lorem ipsum', available_on: '2011-01-01 01:01:01')
-
-      visit spree.admin_products_path
-      within('table.table tbody tr:nth-child(1)') do
-        click_icon(:edit)
-      end
-
-      fill_in "product_slug", with: 'random-slug-value'
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-
-      fill_in "product_slug", with: 'a'
-      click_button "Update"
-      within('#product_slug_field') { expect(page).to have_content("is too short") }
-
-      fill_in "product_slug", with: 'another-random-slug-value'
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-    end
-  end
-
-  # Regression test for #3385
-  context "deleting a product" do
-    it "is still able to find the master variant" do
-      create(:product)
-
-      visit spree.admin_products_path
-      within_row(1) do
-        accept_alert do
-          click_icon :delete
-        end
-      end
-      wait_for_ajax
-    end
-  end
-end

data/spec/features/admin/products/edit/taxons_spec.rb

@@ -1,41 +0,0 @@
-require 'spec_helper'
-
-describe "Product Taxons", type: :feature, js: true do
-  stub_authorization!
-
-  after do
-    Capybara.ignore_hidden_elements = true
-  end
-
-  before do
-    Capybara.ignore_hidden_elements = false
-  end
-
-  context "managing taxons" do
-    def selected_taxons
-      find("#product_taxon_ids").value.split(',').map(&:to_i).uniq
-    end
-
-    it "should allow an admin to manage taxons" do
-      taxon_1 = create(:taxon)
-      taxon_2 = create(:taxon, name: 'Clothing')
-      product = create(:product)
-      product.taxons << taxon_1
-
-      visit spree.admin_products_path
-      within_row(1) { click_icon :edit }
-
-      expect(find(".select2-search-choice").text).to eq(taxon_1.name)
-      expect(selected_taxons).to match_array([taxon_1.id])
-
-      select2_search "Clothing", from: "Taxons"
-      click_button "Update"
-      expect(selected_taxons).to match_array([taxon_1.id, taxon_2.id])
-
-      # Regression test for #2139
-      sleep(1)
-      expect(first(".select2-search-choice", text: taxon_1.name)).to be_present
-      expect(first(".select2-search-choice", text: taxon_2.name)).to be_present
-    end
-  end
-end

data/spec/features/admin/products/edit/variants_spec.rb

@@ -1,56 +0,0 @@
-require 'spec_helper'
-
-describe "Product Variants", type: :feature, js: true do
-  stub_authorization!
-
-  let!(:product) { create(:product) }
-
-  before(:each) do
-    visit spree.admin_products_path
-  end
-
-  context "editing variant option types" do
-    it "should allow an admin to create option types for a variant" do
-      within_row(1) { click_icon :edit }
-
-      within('#sidebar') { click_link "Variants" }
-      expect(page).to have_content("To add variants, you must first define")
-    end
-
-    it "allows admin to create a variant if there are option types" do
-      click_link "Products"
-      click_link "Option Types"
-      click_link "new_option_type_link"
-      fill_in "option_type_name", with: "shirt colors"
-      fill_in "option_type_presentation", with: "colors"
-      click_button "Create"
-      expect(page).to have_content("successfully created!")
-
-      page.find('#option_type_option_values_attributes_0_name').set('color')
-      page.find('#option_type_option_values_attributes_0_presentation').set('black')
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-
-      visit spree.admin_products_path
-      within_row(1) { click_icon :edit }
-
-      select2_search "shirt", from: "Option Types"
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-
-      within('#sidebar') { click_link "Variants" }
-      click_link "New Variant"
-
-      targetted_select2 "black", from: "#s2id_variant_option_value_ids"
-      fill_in "variant_sku", with: "A100"
-      click_button "Create"
-      expect(page).to have_content("successfully created!")
-
-      within(".table") do
-        expect(page).to have_content("19.99")
-        expect(page).to have_content("black")
-        expect(page).to have_content("A100")
-      end
-    end
-  end
-end