spree_backend 3.0.5 → 3.0.6

data/spec/features/admin/stock_transfer_spec.rb

@@ -0,0 +1,75 @@
+require 'spec_helper'
+
+describe 'Stock Transfers', type: :feature, js: true do
+  stub_authorization!
+
+  it 'transfer between 2 locations' do
+    source_location = create(:stock_location_with_items, name: 'NY')
+    destination_location = create(:stock_location, name: 'SF')
+    variant = Spree::Variant.last
+
+    visit spree.admin_stock_transfers_path
+    click_on 'New Stock Transfer'
+    fill_in 'reference', with: 'PO 666'
+
+    select2_search variant.name, from: 'Variant'
+
+    click_button 'Add'
+    click_button 'Transfer Stock'
+
+    expect(page).to have_content('Reference PO 666')
+    expect(page).to have_content('NY')
+    expect(page).to have_content('SF')
+    expect(page).to have_content(variant.name)
+
+    transfer = Spree::StockTransfer.last
+    expect(transfer.stock_movements.size).to eq 2
+  end
+
+  describe 'received stock transfer' do
+    def it_is_received_stock_transfer(page)
+      expect(page).to have_content('Reference PO 666')
+      expect(page).not_to have_selector("#stock-location-source")
+      expect(page).to have_selector("#stock-location-destination")
+
+      transfer = Spree::StockTransfer.last
+      expect(transfer.stock_movements.size).to eq 1
+      expect(transfer.source_location).to be_nil
+    end
+
+    it 'receive stock to a single location' do
+      source_location = create(:stock_location_with_items, name: 'NY')
+      destination_location = create(:stock_location, name: 'SF')
+      variant = Spree::Variant.last
+
+      visit spree.new_admin_stock_transfer_path
+
+      fill_in 'reference', with: 'PO 666'
+      check 'transfer_receive_stock'
+      select('NY', from: 'transfer_destination_location_id')
+      select2_search variant.name, from: 'Variant'
+
+      click_button 'Add'
+      click_button 'Transfer Stock'
+
+      it_is_received_stock_transfer page
+    end
+
+    it 'forced to only receive there is only one location' do
+      source_location = create(:stock_location_with_items, name: 'NY')
+      variant = Spree::Variant.last
+
+      visit spree.new_admin_stock_transfer_path
+
+      fill_in 'reference', with: 'PO 666'
+
+      select('NY', from: 'transfer_destination_location_id')
+      select2_search variant.name, from: 'Variant'
+
+      click_button 'Add'
+      click_button 'Transfer Stock'
+
+      it_is_received_stock_transfer page
+    end
+  end
+end

data/spec/features/admin/taxons_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+feature "Taxonomies and taxons" do
+  stub_authorization!
+
+  scenario "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
+
+  scenario "admin should be able to remove a product from a taxon", js: true do
+    taxon_1 = create(:taxon, name: 'Clothing')
+    product = create(:product)
+    product.taxons << taxon_1
+
+    visit spree.admin_taxons_path
+    select_clothing_from_select2
+
+    find('.product').hover
+    find('.product .dropdown-toggle').click
+    click_link "Delete From Taxon"
+    wait_for_ajax
+
+    visit current_path
+    select_clothing_from_select2
+
+    expect(page).to have_content("No results")
+  end
+
+  def select_clothing_from_select2
+    targetted_select2_search "Clothing", from: "#s2id_taxon_id"
+    wait_for_ajax
+  end
+end

data/spec/features/admin/users_spec.rb

@@ -0,0 +1,284 @@
+require 'spec_helper'
+
+describe 'Users', type: :feature do
+  stub_authorization!
+
+  let!(:country) { create(:country) }
+  let!(:user_a) { create(:user_with_addresses, email: 'a@example.com') }
+  let!(:user_b) { create(:user_with_addresses, 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] }
+
+  before do
+    stub_const('Spree::User', create(:user, email: 'example@example.com').class)
+  end
+
+  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, visible: false
+      click_button 'Search', visible: false
+      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 'Users'
+      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
+
+        within("#admin_user_edit_api_key") do
+          expect(find("#current-api-key").text).to match /Key: #{user_a.spree_api_key}/
+        end
+      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
+
+        within("#admin_user_edit_api_key") do
+          expect(find("#current-api-key").text).to match /Key: #{user_a.spree_api_key}/
+        end
+      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

data/spec/helpers/admin/base_helper_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Spree::Admin::BaseHelper, :type => :helper do
+  include Spree::Admin::BaseHelper
+
+  context "#datepicker_field_value" do
+    it "should return nil when date is empty" do
+      date = nil
+      expect(datepicker_field_value(date)).to be_nil
+    end
+
+    it "should return a formatted date when date is present" do
+      date = "2013-08-14".to_time
+      expect(datepicker_field_value(date)).to eq("2013/08/14")
+    end
+  end
+
+  context "#plural_resource_name" do
+    it "should return correct form of class" do
+      resource_class = Spree::Product
+      expect(plural_resource_name(resource_class)).to eq("Products")
+    end
+  end
+end

data/spec/helpers/admin/navigation_helper_spec.rb

@@ -0,0 +1,106 @@
+# coding: UTF-8
+require 'spec_helper'
+
+describe Spree::Admin::NavigationHelper, type: :helper do
+
+  describe "#tab" do
+    before do
+      allow(helper).to receive(:cannot?).and_return false
+    end
+
+    context "creating an admin tab" do
+      it "should capitalize the first letter of each word in the tab's label" do
+        admin_tab = helper.tab(:orders)
+        expect(admin_tab).to include("Orders")
+      end
+    end
+
+    it "should accept options with label and capitalize each word of it" do
+      admin_tab = helper.tab(:orders, label: "delivered orders")
+      expect(admin_tab).to include("Delivered Orders")
+    end
+
+    it "should capitalize words with unicode characters" do
+      # overview
+      admin_tab = helper.tab(:orders, label: "přehled")
+      expect(admin_tab).to include("Přehled")
+    end
+
+    describe "selection" do
+      context "when match_path option is not supplied" do
+        subject(:tab) { helper.tab(:orders) }
+
+        it "should be selected if the controller matches" do
+          allow(controller).to receive(:controller_name).and_return("orders")
+          expect(subject).to include('selected')
+        end
+
+        it "should not be selected if the controller does not match" do
+          allow(controller).to receive(:controller_name).and_return("bonobos")
+          expect(subject).not_to include('selected')
+        end
+
+      end
+
+      context "when match_path option is supplied" do
+        before do
+          allow(helper).to receive(:request).and_return(double(ActionDispatch::Request, fullpath: "/admin/orders/edit/1"))
+        end
+
+        it "should be selected if the fullpath matches" do
+          allow(controller).to receive(:controller_name).and_return("bonobos")
+          tab = helper.tab(:orders, label: "delivered orders", match_path: '/orders')
+          expect(tab).to include('selected')
+        end
+
+        it "should be selected if the fullpath matches a regular expression" do
+          allow(controller).to receive(:controller_name).and_return("bonobos")
+          tab = helper.tab(:orders, label: "delivered orders", match_path: /orders$|orders\//)
+          expect(tab).to include('selected')
+        end
+
+        it "should not be selected if the fullpath does not match" do
+          allow(controller).to receive(:controller_name).and_return("bonobos")
+          tab = helper.tab(:orders, label: "delivered orders", match_path: '/shady')
+          expect(tab).not_to include('selected')
+        end
+
+        it "should not be selected if the fullpath does not match a regular expression" do
+          allow(controller).to receive(:controller_name).and_return("bonobos")
+          tab = helper.tab(:orders, label: "delivered orders", match_path: /shady$|shady\//)
+          expect(tab).not_to include('selected')
+        end
+      end
+    end
+  end
+
+  describe '#klass_for' do
+
+    it 'returns correct klass for Spree model' do
+      expect(klass_for(:products)).to eq(Spree::Product)
+      expect(klass_for(:product_properties)).to eq(Spree::ProductProperty)
+    end
+
+    it 'returns correct klass for non-spree model' do
+      class MyUser
+      end
+      expect(klass_for(:my_users)).to eq(MyUser)
+
+      Object.send(:remove_const, 'MyUser')
+    end
+
+    it 'returns correct namespaced klass for non-spree model' do
+      module My
+        class User
+        end
+      end
+
+      expect(klass_for(:my_users)).to eq(My::User)
+
+      My.send(:remove_const, 'User')
+      Object.send(:remove_const, 'My')
+    end
+
+  end
+
+end