spree_backend 3.0.5 → 3.0.6

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

@@ -0,0 +1,62 @@
+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_shipping_methods_path
+  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,68 @@
+require 'spec_helper'
+
+describe "States", type: :feature do
+  stub_authorization!
+
+  let!(:country) { create(:country) }
+
+  before(:each) do
+    @hungary = 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_selector('#new_state_link')
+    page.execute_script('$.fx.off = true')
+  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
+      set_select2_field("country", country.id)
+
+      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
+      set_select2_field "#country", @hungary.id
+      # Just so the change event actually gets triggered in this spec
+      # It is definitely triggered in the "real world"
+      page.execute_script("$('#country').change();")
+
+      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
+      set_select2_field("country", country.id)
+
+      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,48 @@
+require 'spec_helper'
+
+describe "Stock Locations", type: :feature, js: true do
+  stub_authorization!
+
+  before(:each) do
+    country = create(:country)
+    visit spree.admin_stock_locations_path
+  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" do
+    location = create(:stock_location)
+    visit current_path
+
+    expect(find('#listing_stock_locations')).to have_content("NY Warehouse")
+    accept_alert do
+      click_icon :delete
+    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, js: true do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+    click_link "Configuration"
+  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"
+      within("h1") { expect(page).to have_content("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,18 @@
+require 'spec_helper'
+
+describe "Tax Rates", type: :feature, js: true do
+  stub_authorization!
+
+  let!(:tax_rate) { create(:tax_rate, calculator: stub_model(Spree::Calculator)) }
+
+  # Regression test for #1422
+  it "can create a new tax rate" do
+    visit spree.admin_path
+    click_link "Configuration"
+    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/zones_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe "Zones", type: :feature, js: true do
+  stub_authorization!
+
+  before(:each) do
+    Spree::Zone.delete_all
+    visit spree.admin_path
+    click_link "Configuration"
+  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,81 @@
+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 the header text 'Orders'" do
+        within('h1') { expect(page).to have_content("Orders") }
+      end
+
+      it "should have a link to overview" do
+        within("header") { 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("Configuration")['/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('.sidebar') { page.find_link("Products")['/admin/products'] }
+      end
+
+      it "should have a link to option types" do
+        within('.sidebar') { page.find_link("Option Types")['/admin/option_types'] }
+      end
+
+      it "should have a link to properties" do
+        within('.sidebar') { page.find_link("Properties")['/admin/properties'] }
+      end
+
+      it "should have a link to prototypes" do
+        within('.sidebar') { 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, :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('Configurations')
+    end
+  end
+
+end

data/spec/features/admin/locale_spec.rb

@@ -0,0 +1,31 @@
+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
+    click_link "Ordres"
+    expect(page).to have_content("Ordres")
+  end
+end

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

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+describe "Adjustments Promotions", type: :feature do
+  stub_authorization!
+
+  before(:each) do
+    promotion = create(:promotion_with_item_adjustment,
+                         :name       => "$10 off",
+                         :path       => 'test',
+                         :code       => "10_off",
+                         :starts_at  => 1.day.ago,
+                         :expires_at => 1.day.from_now,
+                         :adjustment_rate => 10)
+
+    order = create(:order_with_totals)
+    line_item = order.line_items.first
+    # so we can be sure of a determinate price in our assertions
+    line_item.update_column(:price, 10)
+
+    visit spree.admin_order_adjustments_path(order)
+  end
+
+  context "admin adding a promotion" do
+    context "successfully" do
+      it "should create a new adjustment", js: true do
+        fill_in "coupon_code", with: "10_off"
+        click_button "Add Coupon Code"
+        expect(page).to have_content("$10 off")
+        expect(page).to have_content("-$10.00")
+      end
+    end
+
+    context "for non-existing promotion" do
+      it "should show an error message", js: true do
+        fill_in "coupon_code", with: "does_not_exist"
+        click_button "Add Coupon Code"
+        expect(page).to have_content("doesn't exist.")
+      end
+    end
+
+    context "for already applied promotion" do
+      it "should show an error message", js: true do
+        fill_in "coupon_code", with: "10_off"
+        click_button "Add Coupon Code"
+        expect(page).to have_content('-$10.00')
+
+        fill_in "coupon_code", with: "10_off"
+        click_button "Add Coupon Code"
+        expect(page).to have_content("already been applied")
+      end
+    end
+  end
+end