solidus_backend 1.1.0 → 1.1.1

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,124 @@
+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("Display")
+        expect(all("th")[3].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)
+
+    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

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

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+describe "Tax Categories", :type => :feature do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+    click_link "Settings"
+  end
+
+  context "admin visiting tax categories list" do
+    it "should display the existing tax categories" do
+      create(:tax_category, :name => "Clothing", :tax_code => "CL001", :description => "For Clothing")
+      click_link "Tax Categories"
+      expect(page).to have_content("Listing Tax Categories")
+      within_row(1) do
+        expect(column_text(1)).to eq("Clothing")
+        expect(column_text(2)).to eq("CL001")
+        expect(column_text(3)).to eq("For Clothing")
+        expect(column_text(4)).to eq("No")
+      end
+    end
+  end
+
+  context "admin creating new tax category" do
+    before(:each) do
+      click_link "Tax Categories"
+      click_link "admin_new_tax_categories_link"
+    end
+
+    it "should be able to create new tax category" do
+      expect(page).to have_content("New Tax Category")
+      fill_in "tax_category_name", :with => "sports goods"
+      fill_in "tax_category_description", :with => "sports goods desc"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+    end
+
+    it "should show validation errors if there are any" do
+      click_button "Create"
+      expect(page).to have_content("Name can't be blank")
+    end
+  end
+
+  context "admin editing a tax category" do
+    it "should be able to update an existing tax category" do
+      create(:tax_category)
+      click_link "Tax Categories"
+      within_row(1) { click_icon :edit }
+      fill_in "tax_category_description", :with => "desc 99"
+      click_button "Update"
+      expect(page).to have_content("successfully updated!")
+      expect(page).to have_content("desc 99")
+    end
+  end
+end

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

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+describe "Tax Rates", :type => :feature do
+  stub_authorization!
+
+  let!(:tax_rate) { create(:tax_rate, :calculator => stub_model(Spree::Calculator)) }
+
+  before do
+    visit spree.admin_path
+    click_link "Settings"
+  end
+
+  # Regression test for #535
+  it "can see a tax rate in the list if the tax category has been deleted" do
+    tax_rate.tax_category.update_column(:deleted_at, Time.now)
+    expect { click_link "Tax Rates" }.not_to raise_error
+    within(:xpath, all("table tbody td")[2].path) do
+      expect(page).to have_content("N/A")
+    end
+  end
+
+  # Regression test for #1422
+  it "can create a new tax rate" do
+    click_link "Tax Rates"
+    click_link "New Tax Rate"
+    fill_in "Rate", :with => "0.05"
+    click_button "Create"
+    expect(page).to have_content("Tax Rate has been successfully created!")
+  end
+end

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

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe "Taxonomies", :type => :feature do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+    click_link "Settings"
+  end
+
+  context "show" do
+    it "should display existing taxonomies" do
+      create(:taxonomy, :name => 'Brand')
+      create(:taxonomy, :name => 'Categories')
+      click_link "Taxonomies"
+      within_row(1) { expect(page).to have_content("Brand") }
+      within_row(2) { expect(page).to have_content("Categories") }
+    end
+  end
+
+  context "create" do
+    before(:each) do
+      click_link "Taxonomies"
+      click_link "admin_new_taxonomy_link"
+    end
+
+    it "should allow an admin to create a new taxonomy" do
+      expect(page).to have_content("New Taxonomy")
+      fill_in "taxonomy_name", :with => "sports"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+    end
+
+    it "should display validation errors" do
+      fill_in "taxonomy_name", :with => ""
+      click_button "Create"
+      expect(page).to have_content("can't be blank")
+    end
+  end
+
+  context "edit" do
+    it "should allow an admin to update an existing taxonomy" do
+      create(:taxonomy)
+      click_link "Taxonomies"
+      within_row(1) { click_icon :edit }
+      fill_in "taxonomy_name", :with => "sports 99"
+      click_button "Update"
+      expect(page).to have_content("successfully updated!")
+      expect(page).to have_content("sports 99")
+    end
+  end
+end

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

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe "Zones", :type => :feature do
+  stub_authorization!
+
+  before(:each) do
+    Spree::Zone.delete_all
+    visit spree.admin_path
+    click_link "Settings"
+  end
+
+  context "show" do
+    it "should display existing zones" do
+      create(:zone, :name => "eastern", :description => "zone is eastern")
+      create(:zone, :name => "western", :description => "cool san fran")
+      click_link "Zones"
+
+      within_row(1) { expect(page).to have_content("eastern") }
+      within_row(2) { expect(page).to have_content("western") }
+
+      click_link "zones_order_by_description_title"
+
+      within_row(1) { expect(page).to have_content("western") }
+      within_row(2) { expect(page).to have_content("eastern") }
+    end
+  end
+
+  context "create" do
+    it "should allow an admin to create a new zone" do
+      click_link "Zones"
+      click_link "admin_new_zone_link"
+      expect(page).to have_content("New Zone")
+      fill_in "zone_name", :with => "japan"
+      fill_in "zone_description", :with => "japanese time zone"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+    end
+  end
+end

data/spec/features/admin/homepage_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+describe "Homepage", :type => :feature do
+
+  context 'as admin user' do
+    stub_authorization!
+
+    context "visiting the homepage" do
+      before(:each) do
+        visit spree.admin_path
+      end
+
+      it "should have a link to overview" do
+        within(:xpath, ".//figure[@data-hook='logo-wrapper']") { page.find(:xpath, "a[@href='/admin']") }
+      end
+
+      it "should have a link to orders" do
+        page.find_link("Orders")['/admin/orders']
+      end
+
+      it "should have a link to products" do
+        page.find_link("Products")['/admin/products']
+      end
+
+      it "should have a link to reports" do
+        page.find_link("Reports")['/admin/reports']
+      end
+
+      it "should have a link to configuration" do
+        page.find_link("Settings")['/admin/configurations']
+      end
+    end
+
+    context "visiting the products tab" do
+      before(:each) do
+        visit spree.admin_products_path
+      end
+
+      it "should have a link to products" do
+        within('#sub-menu') { page.find_link("Products")['/admin/products'] }
+      end
+
+      it "should have a link to option types" do
+        within('#sub-menu') { page.find_link("Option Types")['/admin/option_types'] }
+      end
+
+      it "should have a link to properties" do
+        within('#sub-menu') { page.find_link("Properties")['/admin/properties'] }
+      end
+
+      it "should have a link to prototypes" do
+        within('#sub-menu') { page.find_link("Prototypes")['/admin/prototypes'] }
+      end
+    end
+  end
+
+  context 'as fakedispatch user' do
+
+    before do
+      allow_any_instance_of(Spree::Admin::BaseController).to receive(:spree_current_user).and_return(nil)
+    end
+
+    custom_authorization! do |user|
+      can [:admin, :home], :dashboards
+      can [:admin, :edit, :index, :read], Spree::Order
+    end
+
+    it 'should only display tabs fakedispatch has access to' do
+      visit spree.admin_path
+      expect(page).to have_link('Orders')
+      expect(page).not_to have_link('Products')
+      expect(page).not_to have_link('Promotions')
+      expect(page).not_to have_link('Reports')
+      expect(page).not_to have_link('Settings')
+    end
+  end
+
+end

data/spec/features/admin/locale_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+describe "setting locale", :type => :feature do
+  stub_authorization!
+
+  before do
+    I18n.locale = I18n.default_locale
+    I18n.backend.store_translations(:fr,
+      :date => {
+        :month_names => [],
+      },
+      :spree => {
+        :admin => {
+          :tab => { :orders => "Ordres" }
+        },
+        :listing_orders => "Ordres",
+      })
+    Spree::Backend::Config[:locale] = "fr"
+  end
+
+  after do
+    I18n.locale = I18n.default_locale
+    Spree::Backend::Config[:locale] = "en"
+  end
+
+  it "should be in french" do
+    visit spree.admin_path
+    expect(page).to have_content("Ordres")
+  end
+end