solidus_backend 1.0.2 → 1.0.3

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

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+module Spree
+  module Admin
+    describe VariantsController, :type => :controller do
+      stub_authorization!
+
+      describe "#index" do
+        let(:product) { create(:product) }
+        let!(:variant_1) { create(:variant, product: product) }
+        let!(:variant_2) { create(:variant, product: product) }
+
+        context "deleted is not requested" do
+          it "assigns the variants for a requested product" do
+            spree_get :index, product_id: product.slug
+            expect(assigns(:collection)).to include variant_1
+            expect(assigns(:collection)).to include variant_2
+          end
+        end
+
+        context "deleted is requested" do
+          before { variant_2.destroy }
+          it "assigns only deleted variants for a requested product" do
+            spree_get :index, product_id: product.slug, deleted: "on"
+            expect(assigns(:collection)).not_to include variant_1
+            expect(assigns(:collection)).to include variant_2
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe "Analytics Tracker", :type => :feature do
+  stub_authorization!
+
+  context "index" do
+    before(:each) do
+      2.times { create(:tracker, :environment => "test") }
+      visit spree.admin_path
+      click_link "Settings"
+      click_link "Analytics Tracker"
+    end
+
+    it "should have the right content" do
+      expect(page).to have_content("Analytics Trackers")
+    end
+
+    it "should have the right tabular values displayed" do
+      within_row(1) do
+        expect(column_text(1)).to eq("A100")
+        expect(column_text(2)).to eq("Test")
+        expect(column_text(3)).to eq("Yes")
+      end
+
+      within_row(2) do
+        expect(column_text(1)).to eq("A100")
+        expect(column_text(2)).to eq("Test")
+        expect(column_text(3)).to eq("Yes")
+      end
+    end
+   end
+
+  context "create" do
+    before(:each) do
+      visit spree.admin_path
+      click_link "Settings"
+      click_link "Analytics Tracker"
+    end
+
+    it "should be able to create a new analytics tracker" do
+      click_link "admin_new_tracker_link"
+      fill_in "tracker_analytics_id", :with => "A100"
+      select "Test", :from => "tracker-env"
+      click_button "Create"
+
+      expect(page).to have_content("successfully created!")
+      within_row(1) do
+        expect(column_text(1)).to eq("A100")
+        expect(column_text(2)).to eq("Test")
+        expect(column_text(3)).to eq("Yes")
+      end
+    end
+  end
+end

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

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+module Spree
+  describe "Countries", type: :feature do
+    stub_authorization!
+
+    it "deletes a country", js: true do
+      visit spree.admin_countries_path
+      click_link "New Country"
+
+      fill_in "Name", with: "Brazil"
+      fill_in "Iso Name", with: "BRL"
+      click_button "Create"
+
+      accept_alert do
+        click_icon :trash
+      end
+
+      expect(page).to have_content 'Country "Brazil" has been successfully removed!'
+    end
+  end
+end

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

@@ -0,0 +1,45 @@
+require 'spec_helper'
+
+describe "General Settings", type: :feature, js: true do
+  stub_authorization!
+
+  before(:each) do
+    store = create(:store, name: 'Test Store', url: 'test.example.org',
+                           mail_from_address: 'test@example.org')
+    visit spree.admin_path
+    click_link "Settings"
+    click_link "General Settings"
+  end
+
+  context "visiting general settings (admin)" do
+    it "should have the right content" do
+      expect(page).to have_content("General Settings")
+      expect(find("#store_name").value).to eq("Test Store")
+      expect(find("#store_url").value).to eq("test.example.org")
+      expect(find("#store_mail_from_address").value).to eq("test@example.org")
+    end
+  end
+
+  context "editing general settings (admin)" 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"
+
+      assert_successful_update_message(:general_settings)
+      expect(find("#store_name").value).to eq("Spree Demo Site99")
+      expect(find("#store_mail_from_address").value).to eq("spree@example.org")
+    end
+  end
+
+  context "clearing the cache" do
+    it "should clear the cache" do
+      expect(page).to_not have_content(Spree.t(:clear_cache_ok))
+      expect(page).to have_content(Spree.t(:clear_cache_warning))
+
+      click_button "Clear Cache"
+
+      expect(page).to have_content(Spree.t(:clear_cache_ok))
+    end
+  end
+end

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

@@ -0,0 +1,125 @@
+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
+    it "should display existing payment methods" do
+      create(:check_payment_method)
+      click_link "Payment Methods"
+
+      within("table#listing_payment_methods") do
+        expect(all("th")[0].text).to eq("Name")
+        expect(all("th")[1].text).to eq("Provider")
+        expect(all("th")[2].text).to eq("Environment")
+        expect(all("th")[3].text).to eq("Display")
+        expect(all("th")[4].text).to eq("Active")
+      end
+
+      within('table#listing_payment_methods') do
+        expect(page).to have_content("Spree::PaymentMethod::Check")
+      end
+    end
+  end
+
+  context "admin creating a new payment method" do
+    it "should be able to create a new payment method" do
+      click_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 "PaymentMethod::Check", :from => "gtwy-type"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+    end
+  end
+
+  context "admin editing a payment method" do
+    before(:each) do
+      create(:check_payment_method)
+      click_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(find_field("payment_method_name").value).to eq("Payment 99")
+    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
+  end
+
+  context "changing type and payment_source", js: true do
+    after do
+      # cleanup
+      Spree::Config.static_model_preferences.for_class(Spree::Gateway::Bogus).clear
+    end
+
+    it "displays message when changing type" do
+      create(:credit_card_payment_method)
+      click_link "Payment Methods"
+      click_icon :edit
+      expect(page).to have_content('TEST MODE')
+
+      select2_search 'Spree::PaymentMethod::Check', from: 'Provider'
+      expect(page).to have_content('you must save first')
+      expect(page).to have_no_content('TEST MODE')
+
+      # change back
+      select2_search 'Spree::Gateway::Bogus', from: 'Provider'
+      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::Gateway::Bogus, 'my_prefs', {})
+
+      create(:credit_card_payment_method)
+      click_link "Payment Methods"
+      click_icon :edit
+      expect(page).to have_content('TEST MODE')
+
+      select2_search 'my_prefs', from: 'Preference Source'
+      expect(page).to have_content('you must save first')
+      expect(page).to have_no_content('TEST MODE')
+
+      # change back
+      select2_search '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::Gateway::Bogus, 'my_prefs', {server: 'secret'})
+
+      create(:credit_card_payment_method)
+      click_link "Payment Methods"
+      click_icon :edit
+
+      select2_search '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
+      select2_search 'Custom', from: 'Preference Source'
+      click_on 'Update'
+      expect(page).to have_content('TEST MODE')
+      expect(page).to have_no_content('secret')
+    end
+  end
+end

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

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe "Shipping Methods", :type => :feature do
+  stub_authorization!
+  let!(:zone) { create(:global_zone) }
+  let!(:shipping_method) { create(:shipping_method, :zones => [zone]) }
+
+  after do
+    Capybara.ignore_hidden_elements = true
+  end
+
+  before do
+    Capybara.ignore_hidden_elements = false
+    # HACK: To work around no email prompting on check out
+    allow_any_instance_of(Spree::Order).to receive_messages(:require_email => false)
+    create(:check_payment_method, :environment => 'test')
+
+    visit spree.admin_path
+    click_link "Settings"
+    click_link "Shipping Methods"
+  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("Both")
+      end
+    end
+  end
+
+  context "create" do
+    it "should be able to create a new shipping method" 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))
+    end
+  end
+
+  # Regression test for #1331
+  context "update" do
+    it "can change the calculator", :js => true do
+      within("#listing_shipping_methods") do
+        click_icon :edit
+      end
+
+      expect(find(:css, ".calculator-settings-warning")).not_to be_visible
+      select2_search('Flexible Rate', :from => 'Calculator')
+      expect(find(:css, ".calculator-settings-warning")).to be_visible
+
+      click_button "Update"
+      expect(page).not_to have_content("Shipping method is not found")
+    end
+  end
+end

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

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe "States", :type => :feature do
+  stub_authorization!
+
+  let!(:country) { create(:country) }
+
+  let!(:hungary) do
+    Spree::Country.create!(:name => "Hungary", :iso_name => "Hungary")
+  end
+
+  def go_to_states_page
+    visit spree.admin_country_states_path(country)
+    expect(page).to have_css("#new_state_link")
+  end
+
+  context "admin visiting states listing" do
+    let!(:state) { create(:state, :country => country) }
+
+    it "should correctly display the states" do
+      visit spree.admin_country_states_path(country)
+      expect(page).to have_content(state.name)
+    end
+  end
+
+  context "creating and editing states" do
+    it "should allow an admin to edit existing states", :js => true do
+      go_to_states_page
+      select2 country.name, from: 'Country'
+
+      click_link "new_state_link"
+      fill_in "state_name", :with => "Calgary"
+      fill_in "Abbreviation", :with => "CL"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+      expect(page).to have_content("Calgary")
+    end
+
+    it "should allow an admin to create states for non default countries", :js => true do
+      go_to_states_page
+      select2 hungary.name, from: 'Country'
+
+      click_link "new_state_link"
+      fill_in "state_name", :with => "Pest megye"
+      fill_in "Abbreviation", :with => "PE"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+      expect(page).to have_content("Pest megye")
+      expect(find("#s2id_country span").text).to eq("Hungary")
+    end
+
+    it "should show validation errors", :js => true do
+      go_to_states_page
+      select2 country.name, from: 'Country'
+
+      click_link "new_state_link"
+
+      fill_in "state_name", :with => ""
+      fill_in "Abbreviation", :with => ""
+      click_button "Create"
+      expect(page).to have_content("Name can't be blank")
+    end
+  end
+end

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

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe "Stock Locations", :type => :feature do
+  stub_authorization!
+
+  before(:each) do
+    country = create(:country)
+    visit spree.admin_path
+    click_link "Settings"
+    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
+    location = create(:stock_location)
+    visit current_path
+
+    expect(find('#listing_stock_locations')).to have_content("NY Warehouse")
+    accept_alert do
+      click_icon :trash
+    end
+    # Wait for API request to complete.
+    wait_for_ajax
+    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