solidus_backend_devise_token_auth 2.8.0.alpha.2

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

@@ -0,0 +1,62 @@
+# 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

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

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Tax Categories", type: :feature do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+    click_link "Taxes"
+  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("SettingsTaxesTax 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,31 @@
+# frozen_string_literal: true
+
+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 "Taxes"
+  end
+
+  # Regression test for https://github.com/spree/spree/issues/535
+  it "can see a tax rate in the list if the tax category has been deleted" do
+    tax_rate.tax_categories.first.update_column(:deleted_at, Time.current)
+    click_link "Tax Rates"
+
+    expect(find("table tbody td:nth-child(3)")).to have_content('N/A')
+  end
+
+  # Regression test for https://github.com/spree/spree/issues/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,54 @@
+# frozen_string_literal: true
+
+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_nav "Products", "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_nav "Products", "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_nav "Products", "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,41 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Zones", type: :feature do
+  stub_authorization!
+
+  before(:each) do
+    Spree::Zone.delete_all
+    visit spree.admin_path
+    click_link "Zones"
+  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,86 @@
+# frozen_string_literal: true
+
+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(".admin-nav-header") { expect(page).to have_link(nil, href: "/admin") }
+      end
+
+      it "should have a link to orders" do
+        expect(page).to have_link("Orders", href: "/admin/orders")
+      end
+
+      it "should have a link to products" do
+        expect(page).to have_link("Products", href: "/admin/products", count: 2)
+      end
+
+      it "should have a link to configuration" do
+        expect(page).to have_link("Settings", href: "/admin/stores")
+      end
+
+      it "should have a link to promotions" do
+        expect(page).to have_link("Promotions", href: "/admin/promotions", count: 2)
+      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('.selected .admin-subnav') { expect(page).to have_link("Products", href: "/admin/products") }
+      end
+
+      it "should have a link to option types" do
+        within('.selected .admin-subnav') { expect(page).to have_link("Option Types", href: "/admin/option_types") }
+      end
+
+      it "should have a link to property types" do
+        within('.selected .admin-subnav') { expect(page).to have_link("Property Types", href: "/admin/properties") }
+      end
+    end
+
+    context "visiting the promotions tab" do
+      before(:each) do
+        visit spree.admin_promotions_path
+      end
+
+      it "should have a link to promotions" do
+        within('.selected .admin-subnav') { expect(page).to have_link("Promotions", href: "/admin/promotions") }
+      end
+
+      it "should have a link to promotion categories" do
+        within('.selected .admin-subnav') { expect(page).to have_link("Promotion Categories", href: "/admin/promotion_categories") }
+      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('Settings')
+    end
+  end
+end

data/spec/features/admin/javascript_format_money_spec.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe 'JS Spree.formatMoney', js: true do
+  stub_authorization!
+
+  # This is a slightly hacky spec to ensure that our JS will format money in
+  # the same was as our ruby code.
+  # This should probably replaced with a pure JS test in the future.
+  it 'should behave identically to Spree::Money#to_s' do
+    visit '/admin'
+
+    Money::Currency.all.map(&:id).map(&:to_s).map(&:upcase).uniq.each do |currency|
+      money = Spree::Money.new(1234, currency: currency)
+
+      js_result = page.evaluate_script("Spree.formatMoney(#{money.to_d}, '#{currency}')")
+
+      expect(js_result).to eq money.to_s
+    end
+  end
+end

data/spec/features/admin/locale_spec.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "setting locale", type: :feature do
+  stub_authorization!
+
+  before do
+    ActionView::Base.raise_on_missing_translations = false
+    I18n.locale = I18n.default_locale
+    I18n.backend.store_translations(:fr,
+      date: {
+        month_names: []
+      },
+      spree: {
+        i18n: { this_file_language: "Français" },
+        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"
+    ActionView::Base.raise_on_missing_translations = true
+  end
+
+  it "should be in french" do
+    visit spree.admin_path
+    expect(page).to have_content("Ordres")
+  end
+end

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

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Adjustments Promotions", type: :feature do
+  stub_authorization!
+
+  before(:each) do
+    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,130 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Adjustments", type: :feature do
+  stub_authorization!
+
+  let!(:ship_address) { create(:address) }
+  let!(:tax_zone) { create(:global_zone) } # will include the above address
+  let!(:tax_rate) { create(:tax_rate, amount: 0.20, zone: tax_zone, tax_categories: [tax_category]) }
+
+  let!(:order) do
+    create(
+      :completed_order_with_totals,
+      line_items_attributes: [{ price: 10, variant: variant }] * 5,
+      ship_address: ship_address,
+    )
+  end
+  let!(:line_item) { order.line_items[0] }
+
+  let(:tax_category) { create(:tax_category) }
+  let(:variant) { create(:variant, tax_category: tax_category) }
+
+  let!(:adjustment) { order.adjustments.create!(order: order, label: 'Rebate', amount: 10) }
+
+  before(:each) do
+    order.recalculate
+
+    visit spree.admin_path
+    click_link "Orders"
+    within_row(1) { click_icon :edit }
+    click_link "Adjustments"
+  end
+
+  context "admin managing adjustments" do
+    it "should display the correct values for existing order adjustments" do
+      within first('table tr', text: 'Tax') do
+        expect(column_text(2)).to match(/TaxCategory - \d+ 20\.000%/)
+        expect(column_text(3)).to eq("$2.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"
+
+        order.reload.all_adjustments.each do |adjustment|
+          expect(adjustment.order_id).to equal(order.id)
+        end
+      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" do
+    before(:each) do
+      within('table tr', text: 'Rebate') do
+        click_icon :edit
+      end
+    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: $259.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
+    context 'when the adjustment is finalized' do
+      let!(:adjustment) { super().tap(&:finalize!) }
+
+      it 'should not be possible' do
+        within('table tr', text: 'Rebate') do
+          expect(page).not_to have_css('.fa-trash')
+        end
+      end
+    end
+
+    it "should update the total", js: true do
+      accept_alert do
+        within('table tr', text: 'Rebate') do
+          click_icon(:trash)
+        end
+      end
+
+      expect(page).to have_content(/Total: ?\$170\.00/)
+    end
+  end
+end

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

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Cancelling + Resuming", type: :feature do
+  stub_ability_authorization!
+
+  let(:user) { create(:user) }
+
+  before do
+    allow(user).to receive(:has_spree_role?).and_return(true)
+    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
+      expect(find('dt#order_status + dd')).to have_content("Canceled")
+    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
+        expect(find('dt#order_status + dd')).to have_content("Resumed")
+      end
+    end
+  end
+end

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

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Cancel items" do
+  stub_authorization!
+
+  let!(:order) do
+    create(
+      :order_ready_to_ship,
+      number: "R100",
+      state: "complete",
+      line_items_count: 1
+    )
+  end
+
+  def visit_order
+    visit spree.admin_path
+    click_link "Orders"
+    within_row(1) do
+      click_link "R100"
+    end
+  end
+
+  context "when some items are cancelable" do
+    it "can cancel the item" do
+      visit_order
+
+      click_link 'Cancel Items'
+
+      within_row(1) do
+        check 'inventory_unit_ids[]'
+      end
+
+      click_button "Cancel Items"
+      expect(page).to have_content("Inventory canceled")
+      expect(page).to have_content("1 x Canceled")
+    end
+  end
+
+  context "when all items are not cancelable" do
+    before { order.inventory_units.each(&:cancel!) }
+
+    it "does not display the link to cancel items" do
+      visit_order
+
+      expect(page).to have_no_content('Cancel Items')
+    end
+  end
+end