spree_backend 3.2.9 → 3.3.0.rc1

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 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", 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 :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", js: true 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_credit_categories_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe "Store Credit Categories", type: :feature, js: true do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+    click_link "Configuration"
+  end
+
+  context "admin visiting store credit categories list" do
+    it "should display existing store credit categories" do
+      create(:store_credit_category)
+      click_link "Store Credit Categories"
+
+      within_row(1) { expect(page).to have_content("Exchange") }
+    end
+  end
+
+  context "admin creating a new store credit category" do
+    before(:each) do
+      click_link "Store Credit Categories"
+      click_link "admin_new_store_credit_category_link"
+    end
+
+    it "should be able to create a new store credit category" do
+      expect(page).to have_content("New Store Credit Category")
+      fill_in "store_credit_category_name", with: "Return"
+      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 store credit category" do
+    it "should be able to update an existing store credit category" do
+      create(:store_credit_category)
+      click_link "Store Credit Categories"
+      within_row(1) { click_icon :edit }
+      fill_in "store_credit_category_name", with: "Return"
+      click_button "Update"
+      expect(page).to have_content("successfully updated!")
+      expect(page).to have_content("Return")
+    end
+  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 "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", js: true 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 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 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,89 @@
+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 "has header text 'Orders'" do
+        within('h1') { expect(page).to have_content("Orders") }
+      end
+
+      it "has a link to overview" do
+        within("header") { page.find(:xpath, "//a[@href='/admin']") }
+      end
+        
+      it "has a link to orders" do
+        page.find_link("Orders")['/admin/orders']
+      end
+
+      it "has a link to products" do
+        page.find_link("Products")['/admin/products']
+      end
+
+      it "has a link to reports" do
+        page.find_link("Reports")['/admin/reports']
+      end
+
+      it "has a link to configuration" do
+        page.find_link("Configuration")['/admin/configurations']
+      end
+
+      it "has a link to return authorizations" do
+        within('.sidebar') { page.find_link("Return Authorizations")['/admin/return_authorizations'] }
+      end
+
+      it "has a link to customer returns" do
+        within('.sidebar') { page.find_link("Customer Returns")['/admin/customer_returns'] }
+      end
+    end
+
+    context "visiting the products tab" do
+      before(:each) do
+        visit spree.admin_products_path
+      end
+
+      it "has a link to products" do
+        within('.sidebar') { page.find_link("Products")['/admin/products'] }
+      end
+
+      it "has a link to option types" do
+        within('.sidebar') { page.find_link("Option Types")['/admin/option_types'] }
+      end
+
+      it "has a link to properties" do
+        within('.sidebar') { page.find_link("Properties")['/admin/properties'] }
+      end
+
+      it "has 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 'only displays 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

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

@@ -0,0 +1,124 @@
+require 'spec_helper'
+
+describe "Adjustments", type: :feature do
+  stub_authorization!
+
+  let!(:order) { create(:completed_order_with_totals, line_items_count: 5) }
+  let!(:line_item) do
+    line_item = order.line_items.first
+    # so we can be sure of a determinate price in our assertions
+    line_item.update_column(:price, 10)
+    line_item
+  end
+
+  let!(:tax_adjustment) do
+    create(:tax_adjustment,
+      adjustable: line_item,
+      state: 'closed',
+      order: order,
+      label: "VAT 5%",
+      amount: 10)
+  end
+
+  let!(:adjustment) { order.adjustments.create!(order: order, label: 'Rebate', amount: 10) }
+
+  before(:each) do
+    # To ensure the order totals are correct
+    order.update_totals
+    order.persist_totals
+
+    visit spree.admin_orders_path
+    within_row(1) { click_on order.number }
+    click_on "Adjustments"
+  end
+
+  after :each do
+    order.reload.all_adjustments.each do |adjustment|
+      expect(adjustment.order_id).to equal(order.id)
+    end
+  end
+
+  context "admin managing adjustments" do
+    it "should display the correct values for existing order adjustments" do
+      within_row(1) do
+        expect(column_text(2)).to eq("VAT 5%")
+        expect(column_text(3)).to eq("$10.00")
+      end
+    end
+
+    it "only shows eligible adjustments" do
+      expect(page).not_to have_content("ineligible")
+    end
+  end
+
+  context "admin creating a new adjustment" do
+    before(:each) do
+      click_link "New Adjustment"
+    end
+
+    context "successfully" do
+      it "should create a new adjustment" do
+        fill_in "adjustment_amount", with: "10"
+        fill_in "adjustment_label", with: "rebate"
+        click_button "Continue"
+        expect(page).to have_content("successfully created!")
+        expect(page).to have_content("Total: $80.00")
+      end
+    end
+
+    context "with validation errors" do
+      it "should not create a new adjustment" do
+        fill_in "adjustment_amount", with: ""
+        fill_in "adjustment_label", with: ""
+        click_button "Continue"
+        expect(page).to have_content("Label can't be blank")
+        expect(page).to have_content("Amount is not a number")
+      end
+    end
+  end
+
+  context "admin editing an adjustment", js: true do
+
+    before(:each) do
+      within_row(2) { click_icon :edit }
+    end
+
+    context "successfully" do
+      it "should update the adjustment" do
+        fill_in "adjustment_amount", with: "99"
+        fill_in "adjustment_label", with: "rebate 99"
+        click_button "Continue"
+        expect(page).to have_content("successfully updated!")
+        expect(page).to have_content("rebate 99")
+        within(".adjustments") do
+          expect(page).to have_content("$99.00")
+        end
+
+        expect(page).to have_content("Total: $159.00")
+      end
+    end
+
+    context "with validation errors" do
+      it "should not update the adjustment" do
+        fill_in "adjustment_amount", with: ""
+        fill_in "adjustment_label", with: ""
+        click_button "Continue"
+        expect(page).to have_content("Label can't be blank")
+        expect(page).to have_content("Amount is not a number")
+      end
+    end
+  end
+
+  context "deleting an adjustment" do
+    it "should update the total", js: true do
+      accept_alert do
+        within_row(2) do
+          click_icon(:delete)
+        end
+      end
+
+      wait_for_ajax
+      expect(page).to have_content(/Total: ?\$170\.00/)
+    end
+  end
+end

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

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe "Cancelling + Resuming", type: :feature do
+
+  stub_authorization!
+
+  let(:user) { double(id: 123, has_spree_role?: true, spree_api_key: 'fake') }
+
+  before do
+    allow_any_instance_of(Spree::Admin::BaseController).to receive(:try_spree_current_user).and_return(user)
+  end
+
+  let(:order) do
+    order = create(:order)
+    order.update_columns({
+      state: 'complete',
+      completed_at: Time.current
+    })
+    order
+  end
+
+  it "can cancel an order" do
+    visit spree.edit_admin_order_path(order.number)
+    click_button 'Cancel'
+    within(".additional-info") do
+      within(".state") do
+        expect(page).to have_content("canceled")
+      end
+    end
+  end
+
+  context "with a cancelled order" do
+    before do
+      order.update_column(:state, 'canceled')
+    end
+
+    it "can resume an order" do
+      visit spree.edit_admin_order_path(order.number)
+      click_button 'Resume'
+      within(".additional-info") do
+        within(".state") do
+          expect(page).to have_content("resumed")
+        end
+      end
+    end
+  end
+end