solidus_backend 1.0.2 → 1.0.3

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

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,126 @@
+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_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_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"
+
+        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_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: $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
+    it "should not be possible if adjustment is closed" do
+      within_row(1) do
+        expect(page).not_to have_css('.fa-trash')
+      end
+    end
+
+    it "should update the total", :js => true do
+      accept_alert do
+        within_row(2) do
+          click_icon(:trash)
+        end
+      end
+
+      expect(page).to have_content(/TOTAL: ?\$170\.00/)
+    end
+  end
+end