solidus_backend 2.10.5 → 2.11.0

data/spec/features/admin/reimbursements_spec.rb

@@ -1,18 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe 'Promotions', type: :feature do
-  stub_authorization!
-  let!(:reimbursement) { create(:reimbursement) }
-
-  it "should display the reimbursements table" do
-    visit spree.admin_order_reimbursement_path(reimbursement.order, reimbursement)
-    expect(page).to have_css('table thead tr th', text: 'Product')
-    expect(page).to have_css('table thead tr th', text: 'Preferred Reimbursement Type')
-    expect(page).to have_css('table thead tr th', text: 'Reimbursement Type Override')
-    expect(page).to have_css('table thead tr th', text: 'Exchange For')
-    expect(page).to have_css('table thead tr th', text: 'Amount Before Sales Tax')
-    expect(page).to have_css('table thead tr th', text: 'Total')
-  end
-end

data/spec/features/admin/store_credits_spec.rb

@@ -1,132 +0,0 @@
-# frozen_string_literal: true
-
-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(".sc-table")
-      expect(store_credit_table).to have_css('tr', count: 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(".sc-table")
-      expect(store_credit_table).to have_css('tr', count: 2)
-      expect(Spree::StoreCredit.count).to eq 2
-    end
-  end
-
-  describe "displaying a store credit details page" do
-    before do
-      visit spree.admin_path
-      click_link "Users"
-      click_link store_credit.user.email
-      click_link "Store Credit"
-      page.find(".sc-table td.actions a.fa-edit").click
-    end
-
-    it "shows the store credit's information" do
-      within ".content" do
-        expect(page).to have_content "Credited #{store_credit.display_amount}"
-        expect(page).to have_content "Created By #{store_credit.created_by.email}"
-        expect(page).to have_content "Type #{store_credit.category_name}"
-        expect(page).to have_content 'Store credit history'
-      end
-    end
-
-    it "lets edit store credit's memo", js: true do
-      allow_any_instance_of(Spree::Admin::StoreCreditsController)
-        .to receive(:try_spree_current_user)
-        .and_return(admin_user)
-
-      # When there are no errors
-      within '.store-credit-memo-row' do
-        click_button 'Edit'
-        fill_in 'store_credit_memo', with: 'Lottery Won'
-        click_button 'Save'
-        expect(page).to have_content "Memo Lottery Won", normalize_ws: true
-      end
-      expect(page).to have_content "Store Credit has been successfully updated!"
-
-      # When there are errors
-      allow_any_instance_of(Spree::StoreCredit).to receive(:save) { false }
-      allow_any_instance_of(Spree::StoreCredit)
-        .to receive_message_chain(:errors, :full_messages)
-        .and_return(["Memo is not valid"])
-
-      within '.store-credit-memo-row' do
-        find(:css, ".edit-memo").click
-        fill_in 'store_credit_memo', with: 'Lottery Won Twice'
-        find(:css, ".save-memo").click
-        expect(page).to have_content "Memo Lottery Won", normalize_ws: true
-        expect(page).not_to have_content "Lottery Won Twice"
-      end
-      expect(page).to have_content "Unable to update store credit [\"Memo is not valid\"]"
-    end
-  end
-
-  describe "updating store credit" do
-    let(:updated_amount) { "99.0" }
-    let!(:store_credit_reason) { create(:store_credit_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(".sc-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 store_credit_reason.name, from: 'store_credit_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/stores_spec.rb

@@ -1,37 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe 'Stores', type: :feature do
-  stub_authorization!
-
-  context 'when adding a store' do
-    before { visit spree.new_admin_store_path }
-
-    it 'admin should be able to set the default_currency' do
-      expect(find('#store_default_currency').value).to eq ''
-
-      fill_in 'store_name', with: 'Solidus Store'
-      fill_in 'store_code', with: 'solidus'
-      fill_in 'store_url', with: 'example.solidus.io'
-      fill_in 'store_mail_from_address', with: 'from@solidus.io'
-      select 'EUR', from: 'store_default_currency'
-      click_button 'Create'
-
-      @store = Spree::Store.last
-
-      expect(@store.default_currency).to eq 'EUR'
-    end
-  end
-
-  context 'when editing a store' do
-    let(:store) { create :store, default_currency: 'AUD' }
-    before { visit spree.edit_admin_store_path(store) }
-
-    it 'admin should be able to change the default_currency' do
-      select 'EUR', from: 'store_default_currency'
-      click_button 'Update'
-      expect(store.reload.default_currency).to eq 'EUR'
-    end
-  end
-end

data/spec/features/admin/style_guide_spec.rb

@@ -1,14 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Style guide", type: :feature do
-  stub_authorization!
-
-  it "should render successfully" do
-    visit "/admin/style_guide"
-
-    # Somewhere in a style guide you'd expect to talk about colors
-    expect(page).to have_text "Colors"
-  end
-end

data/spec/features/admin/taxons_spec.rb

@@ -1,90 +0,0 @@
-# frozen_string_literal: true
-
-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 view and add to taxon tree", js: true do
-    taxonomy = create :taxonomy
-
-    visit spree.edit_admin_taxonomy_path(taxonomy)
-    expect(page).to have_content('Brand')
-
-    click_on('Add taxon')
-    expect(page).to have_content('New node')
-
-    # Little tricky to select the right taxon. Since the text is technically
-    # inside the top-level li.
-    within '#taxonomy_tree li li', text: 'New node' do
-      click_icon :edit
-    end
-
-    expect(page).to have_current_path %r{/admin/taxonomies/\d+/taxons/\d+/edit}
-  end
-
-  context "inside sidebar menu" do
-    def only_one_selected_tab_inside?(sub_tab_selector, tab_name, tab_path)
-      within(sub_tab_selector) do
-        expect(page).to have_selector('.selected', count: 1)
-
-        within('.selected') { expect(page).to have_link(tab_name, href: tab_path) }
-      end
-    end
-
-    context "on display taxonomies page", js: true do
-      it "should be selected only one tab 'Taxonomies' in product sub tabs" do
-        visit spree.admin_taxonomies_path
-        only_one_selected_tab_inside?('[data-hook=admin_product_sub_tabs]', 'Taxonomies', spree.admin_taxonomies_path)
-      end
-    end
-
-    context "on edit taxonomy page", js: true do
-      it "should be selected only one tab 'Taxonomies' in product sub tabs" do
-        taxonomy = create :taxonomy
-
-        visit spree.edit_admin_taxonomy_path(taxonomy)
-        only_one_selected_tab_inside?('[data-hook=admin_product_sub_tabs]', 'Taxonomies', spree.admin_taxonomies_path)
-      end
-    end
-
-    context "on edit taxonomy's taxon page", js: true do
-      it "should be selected only one tab 'Taxonomies' in product sub tabs" do
-        taxonomy = create :taxonomy
-
-        visit spree.edit_admin_taxonomy_taxon_path(taxonomy, taxonomy.root.id)
-        only_one_selected_tab_inside?('[data-hook=admin_product_sub_tabs]', 'Taxonomies', spree.admin_taxonomies_path)
-      end
-    end
-  end
-
-  scenario "Removes attachments from taxon" do
-    taxon = create(:taxon)
-    taxon.update(icon: File.new(file_fixture('ror_ringer.jpeg')))
-
-    visit spree.edit_admin_taxonomy_taxon_path(taxon.taxonomy, taxon.id)
-    within('#taxon_icon_field') do
-      click_on 'Remove'
-    end
-
-    expect(page).to have_content("Icon has been successfully removed!")
-  end
-end

data/spec/features/admin/users_spec.rb

@@ -1,412 +0,0 @@
-# frozen_string_literal: true
-
-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!(:admin_role) { create(:role, name: 'admin') }
-  let!(:user_role) { create(:role, name: 'user') }
-
-  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)
-      o.recalculate
-    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 I18n.t(stat_name, scope: 'spree')
-        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::LegacyUser.model_name.human(count: Spree::I18N_GENERIC_PLURAL), href: spree.admin_users_path
-    end
-
-    it 'can navigate to the account page' do
-      expect(page).to have_link I18n.t("spree.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 I18n.t("spree.admin.user.order_history"), href: spree.orders_admin_user_path(user_a)
-    end
-
-    it 'can navigate to the items purchased' do
-      expect(page).to have_link I18n.t("spree.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_selector '.sort_link.asc'
-        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
-        # Ransack adds a ▲ to the sort link. With exact match Capybara is not able to find that link
-        click_link sort_link, exact: false
-      end
-
-      within_table(table_id) do
-        expect(page).to have_selector '.sort_link.desc'
-        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 by email' 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
-
-    context "member since" do
-      before do
-        user_a.update_column(:created_at, 2.weeks.ago)
-      end
-
-      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) { I18n.t('spree.member_since') }
-      end
-
-      it 'displays the correct results for a user search by creation date' do
-        fill_in 'q_created_at_lt', with: 1.week.ago
-        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 'with users having roles' do
-      before do
-        user_a.spree_roles << admin_role
-        user_b.spree_roles << user_role
-      end
-
-      it 'displays the correct results for a user search by role' do
-        select 'admin', from: Spree.user_class.human_attribute_name(:spree_roles)
-        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
-  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(page).to have_field('user_email', with: 'a@example.com99')
-    end
-
-    it 'can edit user roles' do
-      click_link 'Account'
-
-      check 'user_spree_role_admin'
-      click_button 'Update'
-      expect(page).to have_text 'Account updated'
-      expect(find_field('user_spree_role_admin')).to be_checked
-    end
-
-    it 'can delete user roles' do
-      user_a.spree_roles << Spree::Role.create(name: "dummy")
-      click_link 'Account'
-
-      user_a.spree_roles.each do |role|
-        uncheck "user_spree_role_#{role.name}"
-      end
-
-      click_button 'Update'
-      expect(page).to have_text 'Account updated'
-      expect(find_field('user_spree_role_dummy')).not_to be_checked
-      expect(user_a.reload.spree_roles).to be_empty
-    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(page).to have_field('user_ship_address_attributes_address1', with: "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(page).to have_field('user_bill_address_attributes_address1', with: "1313 Mockingbird Ln")
-      end
-
-      expect(user_a.reload.bill_address.address1).to eq "1313 Mockingbird Ln"
-    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
-
-    context 'without password permissions' do
-      custom_authorization! do |_user|
-        cannot [:update_password], Spree.user_class
-      end
-
-      it 'cannot edit the user password' do
-        expect(page).to_not have_text 'Password'
-      end
-    end
-
-    context 'invalid entry' do
-      around do |example|
-        ::AlwaysInvalidUser = Class.new(Spree.user_class) do
-          validate :always_invalid_email
-          def always_invalid_email
-            errors.add(:email, "is invalid")
-          end
-        end
-        orig_class = Spree.user_class
-        Spree.user_class = "AlwaysInvalidUser"
-
-        example.run
-
-        Spree.user_class = orig_class.name
-        Object.send(:remove_const, "AlwaysInvalidUser")
-      end
-
-      it 'should show validation errors' do
-        fill_in 'user_email', with: 'something'
-        click_button 'Update'
-
-        within('#errorExplanation') do
-          expect(page).to have_content("Email is invalid")
-        end
-
-        within('.flash.error') do
-          expect(page).to have_content("Email is invalid")
-        end
-      end
-    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 "Generate API key"
-        end
-
-        expect(user_a.reload.spree_api_key).to be_present
-
-        expect(page).to have_content('Key: (hidden)')
-      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
-        expect(page).to have_css('#current-api-key')
-
-        click_button "Clear key"
-
-        expect(page).to have_no_css('#current-api-key')
-
-        expect(user_a.reload.spree_api_key).to be_blank
-      end
-
-      it 'can regenerate an api key' do
-        old_key = user_a.spree_api_key
-
-        within("#admin_user_edit_api_key") do
-          click_button "Regenerate key"
-        end
-
-        expect(user_a.reload.spree_api_key).to be_present
-        expect(user_a.reload.spree_api_key).not_to eq old_key
-
-        expect(page).to have_content('Key: (hidden)')
-      end
-    end
-  end
-
-  context 'deleting users' do
-    let!(:an_user) { create(:user_with_addresses, email: 'an_user@example.com') }
-    let!(:order) { create(:completed_order_with_totals, user_id: an_user.id) }
-
-    context 'if an user has placed orders' do
-      before do
-        visit spree.admin_path
-        click_link 'Users'
-      end
-
-      it "can't be deleted" do
-        within "#spree_user_#{an_user.id}" do
-          expect(page).not_to have_selector('.fa-trash')
-        end
-      end
-    end
-  end
-
-  context 'order history with sorting' do
-    before do
-      orders
-      click_link user_a.email
-      within(".tabs") { click_link I18n.t("spree.admin.user.order_history") }
-    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
-
-    context :number do
-      it_behaves_like "a sortable attribute" do
-        let(:text_match_1) { order.number }
-        let(:text_match_2) { order_2.number }
-        let(:table_id) { "listing_orders" }
-        let(:sort_link) { "orders_number_title" }
-      end
-    end
-
-    context :total do
-      it_behaves_like "a sortable attribute" do
-        # Since Spree::Money renders each piece of the total in it's own span,
-        # we are just checking the dollar total matches. Mainly due to how
-        # RSpec matcher appear_before works since it can't index the broken up total.
-        let(:text_match_1) { order.total.to_i.to_s }
-        let(:text_match_2) { order_2.total.to_i.to_s }
-        let(:table_id) { "listing_orders" }
-        let(:sort_link) { "orders_total_title" }
-      end
-    end
-
-    context "state" do
-      let(:text_match_1) { "Complete" }
-      let(:text_match_2) { "Complete" }
-      let(:table_id) { "listing_orders" }
-      let(:sort_link) { "orders_#{attr}_title" }
-    end
-  end
-
-  context 'items purchased with sorting' do
-    before do
-      orders
-      click_link user_a.email
-      within(".tabs") { click_link I18n.t("spree.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
-
-    context "number" do
-      let(:text_match_1) { "R123" }
-      let(:text_match_2) { "R456" }
-      let(:table_id) { "listing_items" }
-      let(:sort_link) { "orders_#{attr}_title" }
-    end
-
-    context "state" do
-      let(:text_match_1) { "Complete" }
-      let(:text_match_2) { "Complete" }
-      let(:table_id) { "listing_items" }
-      let(:sort_link) { "orders_#{attr}_title" }
-    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(html_wrap: false))
-          expect(page).to have_selector(".item-quantity", text: item.quantity)
-          expect(page).to have_selector(".item-total", text: item.money.to_html(html_wrap: false))
-        end
-      end
-    end
-  end
-end