solidus_backend 2.10.2 → 2.11.0

data/spec/controllers/spree/admin/variants_controller_spec.rb

@@ -1,71 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-module Spree
-  module Admin
-    describe VariantsController, type: :controller do
-      stub_authorization!
-
-      describe "#index" do
-        let(:product) { create(:product) }
-        let(:params) { { product_id: product.slug } }
-
-        subject { get :index, params: params }
-
-        context "the value of @parent" do
-          it "is the product" do
-            subject
-            expect(assigns(:parent)).to eq product
-          end
-
-          context "with a deleted product" do
-            before { product.discard }
-
-            it "is the product" do
-              subject
-              expect(assigns(:parent)).to eq product
-            end
-          end
-        end
-
-        context "the value of @collection" do
-          let!(:variant) { create(:variant, product: product) }
-          let!(:deleted_variant) { create(:variant, product: product) }
-
-          context "with soft-deleted variants" do
-            before { deleted_variant.discard }
-
-            context "when deleted is not requested" do
-              it "excludes deleted variants" do
-                subject
-                expect(assigns(:collection)).to include variant
-                expect(assigns(:collection)).not_to include deleted_variant
-              end
-            end
-
-            context "when deleted is requested" do
-              let(:params) { { product_id: product.slug, deleted: "on" } }
-
-              it "includes deleted variants" do
-                subject
-                expect(assigns(:collection)).to include variant
-                expect(assigns(:collection)).to include deleted_variant
-              end
-            end
-          end
-        end
-      end
-
-      describe "#delete" do
-        let!(:variant) { create(:variant) }
-        let(:product) { variant.product }
-
-        it "can be deleted" do
-          delete :destroy, params: { product_id: product.to_param, id: variant.to_param }
-          expect(variant.reload).to be_discarded
-        end
-      end
-    end
-  end
-end

data/spec/features/admin/configuration/payment_methods_spec.rb

@@ -1,176 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Payment Methods", type: :feature do
-  stub_authorization!
-
-  before(:each) do
-    visit spree.admin_path
-    click_link "Settings"
-  end
-
-  context "admin visiting payment methods listing page" do
-    before do
-      create(:check_payment_method)
-    end
-
-    it "should display existing payment methods" do
-      click_link "Payments"
-      expect(page).to have_link 'Payment Methods'
-      within("table#listing_payment_methods") do
-        expect(all("th")[1].text).to eq("Name")
-        expect(all("th")[2].text).to eq("Type")
-        expect(all("th")[3].text).to eq("Available to Users")
-        expect(all("th")[4].text).to eq("Available to Admin")
-        expect(all("th")[5].text).to eq("State")
-      end
-
-      within('table#listing_payment_methods') do
-        expect(page).to have_content(Spree::PaymentMethod::Check.model_name.human)
-      end
-    end
-  end
-
-  context "admin creating a new payment method", :js do
-    it "creates a new payment method and disables the form" do
-      click_link "Payments"
-      expect(page).to have_link 'Payment Methods'
-      click_link "admin_new_payment_methods_link"
-      expect(page).to have_content("New Payment Method")
-      fill_in "payment_method_name", with: "check90"
-      fill_in "payment_method_description", with: "check90 desc"
-      select Spree::PaymentMethod::Check.model_name.human, from: "Type"
-      click_button "Create"
-      expect(page).to have_content("successfully created!")
-
-      visit spree.new_admin_payment_method_path
-      page.execute_script "$('form').submit(function(e) { e.preventDefault()})"
-      fill_in "payment_method_name", with: "check90"
-      fill_in "payment_method_description", with: "check90 desc"
-      select Spree::PaymentMethod::Check.model_name.human, from: "Type"
-      click_button "Create"
-
-      expect(page).to have_button("Create", disabled: true)
-    end
-  end
-
-  context "admin editing a payment method" do
-    let!(:payment_method) { create(:check_payment_method) }
-
-    before do
-      click_link "Payments"
-      expect(page).to have_link 'Payment Methods'
-      within("table#listing_payment_methods") do
-        click_icon(:edit)
-      end
-    end
-
-    it "should be able to edit an existing payment method" do
-      fill_in "payment_method_name", with: "Payment 99"
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-      expect(page).to have_field("payment_method_name", with: "Payment 99")
-    end
-
-    context "with multiple stores available" do
-      before do
-        create(:store, name: "Default Store", url: "spreestore.example.com")
-        visit current_path
-      end
-
-      it "should be able to change the associated stores" do
-        select "Default Store", from: "Stores"
-        click_button "Update"
-        expect(page).to have_content("successfully updated!")
-        expect(page).to have_select("payment_method_store_ids", selected: "Default Store")
-      end
-    end
-
-    it "should display validation errors" do
-      fill_in "payment_method_name", with: ""
-      click_button "Update"
-      expect(page).to have_content("Name can't be blank")
-    end
-
-    context 'with payment method having hash and array as preference' do
-      class ComplexPayments < Spree::PaymentMethod
-        preference :name, :string
-        preference :mapping, :hash
-        preference :recipients, :array
-      end
-
-      let!(:payment_method) { ComplexPayments.create!(name: 'Complex Payments') }
-
-      it "does not display preference fields that are hash or array" do
-        expect(page).to have_field("Name")
-        expect(page).to_not have_field("Mapping")
-        expect(page).to_not have_field("Recipients")
-      end
-    end
-  end
-
-  context "changing type and payment_source", js: true do
-    before do
-      create(:credit_card_payment_method)
-    end
-
-    it "displays message when changing type" do
-      click_link "Payments"
-      expect(page).to have_link 'Payment Methods'
-      click_icon :edit
-      expect(page).to have_content('Test Mode')
-
-      select Spree::PaymentMethod::Check.model_name.human, from: 'Type'
-      expect(page).to have_content('you must save first')
-      expect(page).to have_no_content('Test Mode')
-
-      # change back
-      select Spree::PaymentMethod::BogusCreditCard.model_name.human, from: 'Type'
-      expect(page).to have_no_content('you must save first')
-      expect(page).to have_content('Test Mode')
-    end
-
-    it "displays message when changing preference source" do
-      Spree::Config.static_model_preferences.add(Spree::PaymentMethod::BogusCreditCard, 'my_prefs', {})
-
-      click_link "Payments"
-      expect(page).to have_link 'Payment Methods'
-      click_icon :edit
-      expect(page).to have_content('Test Mode')
-
-      select 'my_prefs', from: 'Preference Source'
-      expect(page).to have_content('you must save first')
-      expect(page).to have_no_content('Test Mode')
-
-      # change back
-      select '(custom)', from: 'Preference Source'
-      expect(page).to have_no_content('you must save first')
-      expect(page).to have_content('Test Mode')
-    end
-
-    it "updates successfully and keeps secrets" do
-      Spree::Config.static_model_preferences.add(Spree::PaymentMethod::BogusCreditCard, 'my_prefs', { server: 'secret' })
-
-      click_link "Payments"
-      expect(page).to have_link 'Payment Methods'
-      click_icon :edit
-
-      select 'my_prefs', from: 'Preference Source'
-      click_on 'Update'
-      expect(page).to have_content('Using static preferences')
-      expect(page).to have_no_content('secret')
-
-      # change back
-      select '(custom)', from: 'Preference Source'
-      click_on 'Update'
-      expect(page).to have_content('Test Mode')
-      expect(page).to have_no_content('secret')
-    end
-
-    after do
-      # cleanup
-      Spree::Config.static_model_preferences.for_class(Spree::PaymentMethod::BogusCreditCard).clear
-    end
-  end
-end

data/spec/features/admin/configuration/shipping_methods_spec.rb

@@ -1,126 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Shipping Methods", type: :feature do
-  stub_authorization!
-  let!(:zone) { create(:global_zone) }
-  let!(:shipping_method) { create(:shipping_method, zones: [zone]) }
-
-  before do
-    visit spree.admin_path
-    click_link "Settings"
-    click_link "Shipping"
-  end
-
-  context "show" do
-    it "should display existing shipping methods" do
-      within_row(1) do
-        expect(column_text(1)).to eq(shipping_method.name)
-        expect(column_text(2)).to eq(zone.name)
-        expect(column_text(3)).to eq("Flat Rate")
-        expect(column_text(4)).to eq("Yes")
-      end
-    end
-  end
-
-  context "create", js: true do
-    it "creates a new shipping method and disables the submit button", :js do
-      click_link "New Shipping Method"
-
-      fill_in "shipping_method_name", with: "bullock cart"
-
-      within("#shipping_method_categories_field") do
-        check first("input[type='checkbox']")["name"]
-      end
-
-      click_on "Create"
-      expect(current_path).to eql(spree.edit_admin_shipping_method_path(Spree::ShippingMethod.last))
-
-      visit spree.new_admin_shipping_method_path
-      page.execute_script "$('form').submit(function(e) { e.preventDefault()})"
-      fill_in "shipping_method_name", with: "bullock cart"
-
-      within("#shipping_method_categories_field") do
-        check first("input[type='checkbox']")["name"]
-      end
-
-      click_on "Create"
-
-      expect(page).to have_button("Create", disabled: true)
-    end
-
-    context 'with shipping method having a calculator with array or hash preference type' do
-      before do
-        class ComplexShipments < Spree::ShippingCalculator
-          preference :amount, :decimal
-          preference :currency, :string
-          preference :mapping, :hash
-          preference :list, :array
-
-          def self.description
-            "Complex Shipments"
-          end
-        end
-        @calculators = Rails.application.config.spree.calculators.shipping_methods
-        Rails.application.config.spree.calculators.shipping_methods = [ComplexShipments]
-      end
-
-      after do
-        Rails.application.config.spree.calculators.shipping_methods = @calculators
-      end
-
-      it "does not show array and hash form fields" do
-        click_link "New Shipping Method"
-
-        fill_in "shipping_method_name", with: "bullock cart"
-
-        within("#shipping_method_categories_field") do
-          check first("input[type='checkbox']")["name"]
-        end
-
-        click_on "Create"
-        select 'Complex Shipments', from: 'Base Calculator'
-        click_on "Update"
-
-        expect(page).to have_field('Amount')
-        expect(page).to have_field('Currency')
-        expect(page).to_not have_field('Mapping')
-        expect(page).to_not have_field('List')
-      end
-    end
-  end
-
-  # Regression test for https://github.com/spree/spree/issues/1331
-  context "update" do
-    it "can update the existing calculator", js: true do
-      within("#listing_shipping_methods") do
-        click_icon :edit
-      end
-
-      fill_in 'Amount', with: 20
-
-      click_button "Update"
-
-      expect(page).to have_content 'successfully updated'
-      expect(page).to have_field 'Amount', with: '20.0'
-    end
-
-    it "can change the calculator", js: true do
-      within("#listing_shipping_methods") do
-        click_icon :edit
-      end
-
-      select 'Flexible Rate per Package Item', from: 'Base Calculator'
-
-      fill_in 'First Item', with: 10
-      fill_in 'Additional Item', with: 20
-
-      click_button "Update"
-
-      expect(page).to have_content 'successfully updated'
-      expect(page).to have_field 'First Item', with: '10.0'
-      expect(page).to have_field 'Additional Item', with: '20.0'
-    end
-  end
-end

data/spec/features/admin/configuration/stock_locations_spec.rb

@@ -1,54 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Stock Locations", type: :feature do
-  stub_authorization!
-
-  before(:each) do
-    create(:country)
-    visit spree.admin_path
-    click_link "Settings"
-    click_link "Shipping"
-    click_link "Stock Locations"
-  end
-
-  it "can create a new stock location" do
-    click_link "New Stock Location"
-    fill_in "Name", with: "London"
-    check "Active"
-    click_button "Create"
-
-    expect(page).to have_content("successfully created")
-    expect(page).to have_content("London")
-  end
-
-  it "can delete an existing stock location", js: true do
-    create(:stock_location)
-    visit current_path
-
-    expect(find('#listing_stock_locations')).to have_content("NY Warehouse")
-    accept_alert do
-      click_icon :trash
-    end
-
-    expect(page).to have_content('Stock Location "NY Warehouse" has been successfully removed')
-
-    visit current_path
-    expect(page).to have_content("No Stock Locations found")
-  end
-
-  it "can update an existing stock location" do
-    create(:stock_location)
-    visit current_path
-
-    expect(page).to have_content("NY Warehouse")
-
-    click_icon :edit
-    fill_in "Name", with: "London"
-    click_button "Update"
-
-    expect(page).to have_content("successfully updated")
-    expect(page).to have_content("London")
-  end
-end

data/spec/features/admin/configuration/store_spec.rb

@@ -1,62 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Store", type: :feature, js: true do
-  stub_authorization!
-
-  let!(:store) do
-    create(:store, name: 'Test Store', url: 'test.example.org',
-           mail_from_address: 'test@example.org')
-  end
-
-  let!(:vat_country) { create(:country, iso: "DE", name: "Germany") }
-
-  before(:each) do
-    visit spree.admin_path
-    click_link "Settings"
-    within('.admin-nav') do
-      click_link "Stores"
-    end
-  end
-
-  context "visiting general store settings" do
-    it "should have the right content" do
-      expect(page).to have_field("store_name", with: "Test Store")
-      expect(page).to have_field("store_url", with: "test.example.org")
-      expect(page).to have_field("store_mail_from_address", with: "test@example.org")
-    end
-  end
-
-  context "editing general store settings" do
-    it "should be able to update the site name" do
-      fill_in "store_name", with: "Spree Demo Site99"
-      fill_in "store_mail_from_address", with: "spree@example.org"
-      click_button "Update"
-
-      expect(page).to have_content "successfully updated"
-      expect(page).to have_field("store_name", with: "Spree Demo Site99")
-      expect(page).to have_field("store_mail_from_address", with: "spree@example.org")
-    end
-
-    it "should be able to update the default cart tax country" do
-      expect(page).to have_select('Tax Country for Empty Carts', selected: 'No taxes on carts without address')
-
-      select "Germany", from: "Tax Country for Empty Carts"
-      click_button "Update"
-
-      expect(page).to have_content("has been successfully updated")
-      expect(page).to have_select("Tax Country for Empty Carts", selected: "Germany")
-    end
-  end
-
-  context "update fails" do
-    it "should display the error" do
-      fill_in "Site Name", with: " "
-      click_button "Update"
-
-      expect(page).to have_content("can't be blank")
-      expect(page).to have_field("Site Name", with: " ")
-    end
-  end
-end