solidus_backend 1.0.2 → 1.0.3

data/spec/features/admin/promotions/tiered_calculator_spec.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+
+feature "Tiered Calculator Promotions" do
+  stub_authorization!
+
+  let(:promotion) { create :promotion }
+
+  background do
+    visit spree.edit_admin_promotion_path(promotion)
+  end
+
+  scenario "adding a tiered percent calculator", js: true do
+    select2 "Create whole-order adjustment", from: "Add action of type"
+    within('#action_fields') { click_button "Add" }
+
+    select2 "Tiered Percent", from: "Calculator"
+    within('#actions_container') { click_button "Update" }
+
+    within("#actions_container .settings") do
+      expect(page).to have_content("BASE PERCENT")
+      expect(page).to have_content("TIERS")
+
+      click_button "Add"
+    end
+
+    fill_in "Base Percent", with: 5
+
+    within(".tier") do
+      find("input:last-child").set(100)
+      find("input:first-child").set(10)
+    end
+
+    within('#actions_container') { click_button "Update" }
+
+    first_action = promotion.actions.first
+    expect(first_action.class).to eq Spree::Promotion::Actions::CreateAdjustment
+
+    first_action_calculator = first_action.calculator
+    expect(first_action_calculator.class).to eq Spree::Calculator::TieredPercent
+    expect(first_action_calculator.preferred_base_percent).to eq 5
+    expect(first_action_calculator.preferred_tiers).to eq Hash[100.0 => 10.0]
+  end
+
+  context "with an existing tiered flat rate calculator" do
+    let(:promotion) { create :promotion, :with_order_adjustment }
+
+    background do
+      action = promotion.actions.first
+
+      action.calculator = Spree::Calculator::TieredFlatRate.new
+      action.calculator.preferred_base_amount = 5
+      action.calculator.preferred_tiers = Hash[100 => 10, 200 => 15, 300 => 20]
+      action.calculator.save!
+
+      visit spree.edit_admin_promotion_path(promotion)
+    end
+
+    scenario "deleting a tier", js: true do
+      within(".tier:nth-child(2)") do
+        find(".remove").click
+      end
+
+      within('#actions_container') { click_button "Update" }
+
+      calculator = promotion.actions.first.calculator
+      expect(calculator.preferred_tiers).to eq Hash[100.0 => 10.0, 300.0 => 20.0]
+    end
+  end
+end

data/spec/features/admin/reports_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+describe "Reports", :type => :feature do
+  stub_authorization!
+
+  context "visiting the admin reports page" do
+    it "should have the right content" do
+      visit spree.admin_path
+      click_link "Reports"
+      click_link "Sales Total"
+
+      expect(page).to have_content("Sales Totals")
+      expect(page).to have_content("Item Total")
+      expect(page).to have_content("Adjustment Total")
+      expect(page).to have_content("Sales Total")
+    end
+  end
+
+  context "searching the admin reports page" do
+    before do
+      order = create(:order)
+      order.update_columns({:adjustment_total => 100})
+      order.completed_at = Time.now
+      order.save!
+
+      order = create(:order)
+      order.update_columns({:adjustment_total => 200})
+      order.completed_at = Time.now
+      order.save!
+
+      #incomplete order
+      order = create(:order)
+      order.update_columns({:adjustment_total => 50})
+      order.save!
+
+      order = create(:order)
+      order.update_columns({:adjustment_total => 200})
+      order.completed_at = 3.years.ago
+      order.created_at = 3.years.ago
+      order.save!
+
+      order = create(:order)
+      order.update_columns({:adjustment_total => 200})
+      order.completed_at = 3.years.from_now
+      order.created_at = 3.years.from_now
+      order.save!
+    end
+
+    it "should allow me to search for reports" do
+      visit spree.admin_path
+      click_link "Reports"
+      click_link "Sales Total"
+
+      fill_in "q_completed_at_gt", :with => 1.week.ago
+      fill_in "q_completed_at_lt", :with => 1.week.from_now
+      click_button "Search"
+
+      expect(page).to have_content("$300.00")
+    end
+  end
+end

data/spec/features/admin/stock_transfer_spec.rb

@@ -0,0 +1,86 @@
+require 'spec_helper'
+
+describe 'Stock Transfers', :type => :feature, :js => true do
+  stub_authorization!
+
+  let(:admin_user) { create(:admin_user) }
+  let(:description) { 'Test stock transfer' }
+
+  before do
+    allow_any_instance_of(Spree::Admin::BaseController).to receive(:spree_current_user).and_return(admin_user)
+  end
+
+  describe 'create stock transfer' do
+    it 'can create a stock transfer' do
+      source_location = create(:stock_location_with_items, :name => 'NY')
+      destination_location = create(:stock_location, :name => 'SF')
+
+      visit spree.new_admin_stock_transfer_path
+      select "SF", from: 'stock_transfer[source_location_id]'
+      fill_in 'stock_transfer_description', with: description
+      click_button 'Continue'
+
+      expect(page.find('#stock_transfer_description').value).to eq description
+
+      select "NY", from: 'stock_transfer[destination_location_id]'
+      within "form.edit_stock_transfer" do
+        page.find('button').trigger('click')
+      end
+
+      expect(page).to have_content('Stock Transfer has been successfully updated')
+      expect(page).to have_css(:div, '#finalize-stock-transfer-warning')
+      expect(page).to have_content("NY")
+    end
+  end
+
+  describe 'ship stock transfer' do
+    let(:stock_transfer) { create(:stock_transfer_with_items) }
+
+    before do
+      stock_transfer.transfer_items do |item|
+        item.update_attributes(expected_quantity: 1)
+      end
+    end
+
+    describe "tracking info" do
+      it 'adds tracking number' do
+        visit spree.tracking_info_admin_stock_transfer_path(stock_transfer)
+
+        fill_in 'stock_transfer_tracking_number', :with => "12345"
+        click_button 'Save'
+
+        expect(page).to have_content('Stock Transfer has been successfully updated')
+        expect(stock_transfer.reload.tracking_number).to eq '12345'
+      end
+    end
+
+    describe 'with enough stock' do
+      it 'ships stock transfer' do
+        visit spree.tracking_info_admin_stock_transfer_path(stock_transfer)
+        click_link 'ship'
+
+        first('#confirm-ship-link', visible: false).click
+        expect(current_path).to eq spree.admin_stock_transfers_path
+        expect(stock_transfer.reload.shipped_at).to_not be_nil
+      end
+    end
+
+    describe 'without enough stock' do
+      before do
+        stock_transfer.transfer_items.each do |item|
+          stock_transfer.source_location.stock_item(item.variant).set_count_on_hand(0)
+        end
+      end
+
+      it 'does not ship stock transfer' do
+        visit spree.tracking_info_admin_stock_transfer_path(stock_transfer)
+
+        click_link 'ship'
+
+        first('#confirm-ship-link', visible: false).click
+        expect(current_path).to eq spree.tracking_info_admin_stock_transfer_path(stock_transfer)
+        expect(stock_transfer.reload.shipped_at).to be_nil
+      end
+    end
+  end
+end

data/spec/features/admin/store_credits_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+
+describe "Store credits admin" do
+  stub_authorization!
+  let!(:admin_user)   { create(:admin_user) }
+  let!(:store_credit) { create(:store_credit) }
+  let(:user)          { store_credit.user }
+
+  before do
+    allow(Spree.user_class).to receive(:find_by).
+      with(hash_including(:id)).
+      and_return(store_credit.user)
+  end
+
+  describe "visiting the store credits page" do
+    before do
+      visit spree.admin_path
+      click_link "Users"
+    end
+
+    it "should be on the store credits page" do
+      click_link store_credit.user.email
+      click_link "Store Credit"
+      expect(page.current_path).to eq spree.admin_user_store_credits_path(store_credit.user)
+
+      store_credit_table = page.find(".twelve.columns > table")
+      expect(store_credit_table.all('tr').count).to eq 1
+      expect(store_credit_table).to have_content(Spree::Money.new(store_credit.amount).to_s)
+      expect(store_credit_table).to have_content(Spree::Money.new(store_credit.amount_used).to_s)
+      expect(store_credit_table).to have_content(store_credit.category_name)
+      expect(store_credit_table).to have_content(store_credit.created_by_email)
+    end
+  end
+
+  describe "creating store credit" do
+    before do
+      visit spree.admin_path
+      click_link "Users"
+      click_link store_credit.user.email
+      click_link "Store Credit"
+      allow_any_instance_of(Spree::Admin::StoreCreditsController).to receive_messages(try_spree_current_user: admin_user)
+    end
+
+    it "should create store credit and associate it with the user" do
+      click_link "Add store credit"
+      page.fill_in "store_credit_amount", with: "102.00"
+      select "Exchange", from: "store_credit_category_id"
+      click_button "Create"
+
+      expect(page.current_path).to eq spree.admin_user_store_credits_path(store_credit.user)
+      store_credit_table = page.find(".twelve.columns > table")
+      expect(store_credit_table.all('tr').count).to eq 2
+      expect(Spree::StoreCredit.count).to eq 2
+    end
+  end
+
+  describe "updating store credit" do
+    let(:updated_amount) { "99.0" }
+    let!(:update_reason) { create(:store_credit_update_reason) }
+
+    before do
+      visit spree.admin_path
+      click_link "Users"
+      click_link store_credit.user.email
+      click_link "Store Credit"
+      allow_any_instance_of(Spree::Admin::StoreCreditsController).to receive_messages(try_spree_current_user: admin_user)
+    end
+
+    it "updates the store credit's amount" do
+      page.find(".twelve.columns > table td.actions a.fa-edit").click
+      expect(page).to have_content 'Store credit history'
+      click_link "Change amount"
+      expect(page).to have_content 'Editing store credit amount'
+      page.fill_in 'store_credit_amount', with: updated_amount
+      page.select update_reason.name, from: 'update_reason_id'
+      click_button "Update"
+      expect(page.find('#sc-detail-table')).to have_content "$99.00"
+      expect(store_credit.reload.amount.to_f).to eq updated_amount.to_f
+    end
+  end
+
+end

data/spec/features/admin/taxons_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe "Taxonomies and taxons", :type => :feature do
+  stub_authorization!
+
+  it "admin should be able to edit taxon" do
+
+    visit spree.new_admin_taxonomy_path
+
+    fill_in "Name", :with => "Hello"
+    click_button "Create"
+
+    @taxonomy = Spree::Taxonomy.last
+
+    visit spree.edit_admin_taxonomy_taxon_path(@taxonomy, @taxonomy.root.id)
+
+    fill_in "taxon_name", :with => "Shirt"
+    fill_in "taxon_description", :with => "Discover our new rails shirts"
+
+    fill_in "permalink_part", :with => "shirt-rails"
+    click_button "Update"
+    expect(page).to have_content("Taxon \"Shirt\" has been successfully updated!")
+  end
+
+  it "can see taxon tree", js: true do
+    taxonomy = create :taxonomy
+
+    visit spree.edit_admin_taxonomy_path(taxonomy)
+    expect(page).to have_content('Brand')
+  end
+end

data/spec/features/admin/users_spec.rb

@@ -0,0 +1,270 @@
+require 'spec_helper'
+
+describe 'Users', :type => :feature do
+  stub_authorization!
+  let!(:country) { create(:country) }
+  let!(:user_a) { create(:user_with_addreses, email: 'a@example.com') }
+  let!(:user_b) { create(:user_with_addreses, email: 'b@example.com') }
+
+  let(:order) { create(:completed_order_with_totals, user: user_a, number: "R123") }
+
+  let(:order_2) do
+    create(:completed_order_with_totals, user: user_a, number: "R456").tap do |o|
+      li = o.line_items.last
+      li.update_column(:price, li.price + 10)
+    end
+  end
+
+  let(:orders) { [order, order_2] }
+
+  shared_examples_for 'a user page' do
+    it 'has lifetime stats' do
+      orders
+      visit current_url # need to refresh after creating the orders for specs that did not require orders
+      within("#user-lifetime-stats") do
+        [:total_sales, :num_orders, :average_order_value, :member_since].each do |stat_name|
+          expect(page).to have_content Spree.t(stat_name)
+        end
+        expect(page).to have_content (order.total + order_2.total)
+        expect(page).to have_content orders.count
+        expect(page).to have_content (orders.sum(&:total) / orders.count)
+        expect(page).to have_content I18n.l(user_a.created_at.to_date)
+      end
+    end
+
+    it 'can go back to the users list' do
+      expect(page).to have_link Spree.t(:back_to_users_list), href: spree.admin_users_path
+    end
+
+    it 'can navigate to the account page' do
+      expect(page).to have_link Spree.t(:"admin.user.account"), href: spree.edit_admin_user_path(user_a)
+    end
+
+    it 'can navigate to the order history' do
+      expect(page).to have_link Spree.t(:"admin.user.orders"), href: spree.orders_admin_user_path(user_a)
+    end
+
+    it 'can navigate to the items purchased' do
+      expect(page).to have_link Spree.t(:"admin.user.items"), href: spree.items_admin_user_path(user_a)
+    end
+  end
+
+  shared_examples_for 'a sortable attribute' do
+    before { click_link sort_link }
+
+    it "can sort asc" do
+      within_table(table_id) do
+        expect(page).to have_text text_match_1
+        expect(page).to have_text text_match_2
+        expect(text_match_1).to appear_before text_match_2
+      end
+    end
+
+    it "can sort desc" do
+      within_table(table_id) do
+        click_link sort_link
+
+        expect(page).to have_text text_match_1
+        expect(page).to have_text text_match_2
+        expect(text_match_2).to appear_before text_match_1
+      end
+    end
+  end
+
+  before do
+    visit spree.admin_path
+    click_link 'Users'
+  end
+
+  context 'users index' do
+
+    context "email" do
+      it_behaves_like "a sortable attribute" do
+        let(:text_match_1) { user_a.email }
+        let(:text_match_2) { user_b.email }
+        let(:table_id) { "listing_users" }
+        let(:sort_link) { "users_email_title" }
+      end
+    end
+
+    it 'displays the correct results for a user search' do
+      fill_in 'q_email_cont', with: user_a.email
+      click_button 'Search'
+      within_table('listing_users') do
+        expect(page).to have_text user_a.email
+        expect(page).not_to have_text user_b.email
+      end
+    end
+  end
+
+  context 'editing users' do
+    before { click_link user_a.email }
+
+    it_behaves_like 'a user page'
+
+    it 'can edit the user email' do
+      fill_in 'user_email', with: 'a@example.com99'
+      click_button 'Update'
+
+      expect(user_a.reload.email).to eq 'a@example.com99'
+      expect(page).to have_text 'Account updated'
+      expect(find_field('user_email').value).to eq 'a@example.com99'
+    end
+
+    it 'can edit the user password' do
+      fill_in 'user_password', with: 'welcome'
+      fill_in 'user_password_confirmation', with: 'welcome'
+      click_button 'Update'
+
+      expect(page).to have_text 'Account updated'
+    end
+
+    it 'can edit user roles' do
+      Spree::Role.create name: "admin"
+      click_link user_a.email
+
+      check 'user_spree_role_admin'
+      click_button 'Update'
+      expect(page).to have_text 'Account updated'
+      expect(find_field('user_spree_role_admin')['checked']).to be true
+    end
+
+    it 'can edit user shipping address' do
+      click_link "Addresses"
+
+      within("#admin_user_edit_addresses") do
+        fill_in "user_ship_address_attributes_address1", with: "1313 Mockingbird Ln"
+        click_button 'Update'
+        expect(find_field('user_ship_address_attributes_address1').value).to eq "1313 Mockingbird Ln"
+      end
+
+      expect(user_a.reload.ship_address.address1).to eq "1313 Mockingbird Ln"
+    end
+
+    it 'can edit user billing address' do
+      click_link "Addresses"
+
+      within("#admin_user_edit_addresses") do
+        fill_in "user_bill_address_attributes_address1", with: "1313 Mockingbird Ln"
+        click_button 'Update'
+        expect(find_field('user_bill_address_attributes_address1').value).to eq "1313 Mockingbird Ln"
+      end
+
+      expect(user_a.reload.bill_address.address1).to eq "1313 Mockingbird Ln"
+    end
+
+    context 'no api key exists' do
+      it 'can generate a new api key' do
+        within("#admin_user_edit_api_key") do
+          expect(user_a.spree_api_key).to be_blank
+          click_button Spree.t('generate_key', :scope => 'api')
+        end
+
+        expect(user_a.reload.spree_api_key).to be_present
+      end
+    end
+
+    context 'an api key exists' do
+      before do
+        user_a.generate_spree_api_key!
+        expect(user_a.reload.spree_api_key).to be_present
+        visit current_path
+      end
+
+      it 'can clear an api key' do
+        within("#admin_user_edit_api_key") do
+          click_button Spree.t('clear_key', :scope => 'api')
+        end
+
+        expect(user_a.reload.spree_api_key).to be_blank
+        expect { find("#current-api-key") }.to raise_error Capybara::ElementNotFound
+      end
+
+      it 'can regenerate an api key' do
+        old_key = user_a.spree_api_key
+
+        within("#admin_user_edit_api_key") do
+          click_button Spree.t('regenerate_key', :scope => 'api')
+        end
+
+        expect(user_a.reload.spree_api_key).to be_present
+        expect(user_a.reload.spree_api_key).not_to eq old_key
+      end
+    end
+  end
+
+  context 'order history with sorting' do
+
+    before do
+      orders
+      click_link user_a.email
+      within("#sidebar") { click_link Spree.t(:"admin.user.orders") }
+    end
+
+    it_behaves_like 'a user page'
+
+    context "completed_at" do
+      it_behaves_like "a sortable attribute" do
+        let(:text_match_1) { I18n.l(order.completed_at.to_date) }
+        let(:text_match_2) { I18n.l(order_2.completed_at.to_date) }
+        let(:table_id) { "listing_orders" }
+        let(:sort_link) { "orders_completed_at_title" }
+      end
+    end
+
+    [:number, :state, :total].each do |attr|
+      context attr do
+        it_behaves_like "a sortable attribute" do
+          let(:text_match_1) { order.send(attr).to_s }
+          let(:text_match_2) { order_2.send(attr).to_s }
+          let(:table_id) { "listing_orders" }
+          let(:sort_link) { "orders_#{attr}_title" }
+        end
+      end
+    end
+  end
+
+  context 'items purchased with sorting' do
+
+    before do
+      orders
+      click_link user_a.email
+      within("#sidebar") { click_link Spree.t(:"admin.user.items") }
+    end
+
+    it_behaves_like 'a user page'
+
+    context "completed_at" do
+      it_behaves_like "a sortable attribute" do
+        let(:text_match_1) { I18n.l(order.completed_at.to_date) }
+        let(:text_match_2) { I18n.l(order_2.completed_at.to_date) }
+        let(:table_id) { "listing_items" }
+        let(:sort_link) { "orders_completed_at_title" }
+      end
+    end
+
+    [:number, :state].each do |attr|
+      context attr do
+        it_behaves_like "a sortable attribute" do
+          let(:text_match_1) { order.send(attr).to_s }
+          let(:text_match_2) { order_2.send(attr).to_s }
+          let(:table_id) { "listing_items" }
+          let(:sort_link) { "orders_#{attr}_title" }
+        end
+      end
+    end
+
+    it "has item attributes" do
+      items = order.line_items | order_2.line_items
+      expect(page).to have_table 'listing_items'
+      within_table('listing_items') do
+        items.each do |item|
+          expect(page).to have_selector(".item-name", text: item.product.name)
+          expect(page).to have_selector(".item-price", text: item.single_money.to_html)
+          expect(page).to have_selector(".item-quantity", text: item.quantity)
+          expect(page).to have_selector(".item-total", text: item.money.to_html)
+        end
+      end
+    end
+  end
+end