solidus_backend_devise_token_auth 2.8.0.alpha.2

data/spec/helpers/admin/navigation_helper_spec.rb

@@ -0,0 +1,137 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::NavigationHelper, type: :helper do
+  describe "#tab" do
+    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
+
+    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('class="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('class="selected"')
+        end
+
+        it "should be selected if the current path" do
+          allow(helper).to receive(:request).and_return(double(ActionDispatch::Request, fullpath: "/admin/orders"))
+          expect(subject).to include('class="selected"')
+        end
+
+        it "should not be selected if not current path" do
+          allow(helper).to receive(:request).and_return(double(ActionDispatch::Request, fullpath: "/admin/products"))
+          expect(subject).not_to include('class="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, match_path: '/orders')
+          expect(tab).to include('class="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, match_path: /orders$|orders\//)
+          expect(tab).to include('class="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, match_path: '/shady')
+          expect(tab).not_to include('class="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, match_path: /shady$|shady\//)
+          expect(tab).not_to include('class="selected"')
+        end
+      end
+    end
+
+    it "should accept a block of content to append" do
+      admin_tab = helper.tab(:orders){ 'foo' }
+      expect(admin_tab).to end_with("foo</li>")
+    end
+  end
+
+  describe "#link_to_delete" do
+    let!(:item) { create(:stock_item) }
+    let(:options) { {} }
+    let(:link) { subject }
+
+    subject { helper.link_to_delete(item, options) }
+
+    # object_url is provided by the ResourceController abstract controller.
+    # as we cannot set a custom controller for helper tests, we need to fake it
+    before do
+      allow(helper).to receive(:object_url) do |o|
+        "/stock_items/#{o.to_param}"
+      end
+    end
+
+    it "generates a deletion link for the resource" do
+      expect(link).to include("href=\"/stock_items/#{item.to_param}\"")
+      expect(link).to include("data-action=\"remove\"")
+      expect(link).to include("data-confirm=\"Are you sure?\"")
+      expect(link).to include("<span class=\"text\">Delete</span>")
+    end
+
+    it "allows customization of the url" do
+      options[:url] = "/test/url"
+      expect(link).to include("href=\"/test/url\"")
+    end
+
+    it "allows customization of the link name" do
+      options[:name] = "Delete Item"
+      expect(link).to include("name=\"Delete Item\"")
+      expect(link).to include("<span class=\"text\">Delete Item</span>")
+    end
+
+    it "allows customization of the confirmation message" do
+      options[:confirm] = "Please confirm."
+      expect(link).to include("data-confirm=\"Please confirm.\"")
+    end
+  end
+
+  describe "#solidus_icon" do
+    context "if given an icon_name" do
+      subject(:solidus_icon) { helper.solidus_icon('example-icon-name') }
+
+      it { is_expected.to eq "<i class=\"example-icon-name\"></i>" }
+    end
+
+    context "if not given nil icon_name" do
+      subject(:solidus_icon) { helper.solidus_icon(nil) }
+
+      it { is_expected.to eq "" }
+    end
+  end
+
+  describe "#icon" do
+    subject(:icon) { helper.icon('icon-name') }
+
+    it "is a deprecated way to use #solidus_icon" do
+      expect(Spree::Deprecation).to receive(:warn).with("icon is deprecated and will be removed from Solidus 3.0 (use solidus_icon instead)", instance_of(Array))
+      expect(subject).to eq helper.solidus_icon('icon-name')
+    end
+  end
+end

data/spec/helpers/admin/stock_movements_helper_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::StockMovementsHelper, type: :helper do
+  describe "#pretty_originator" do
+    let!(:stock_location) { create(:stock_location_with_items) }
+    let!(:stock_item)     { stock_location.stock_items.first }
+    let(:stock_movement)  { create(:stock_movement, stock_item: stock_item, originator: originator) }
+
+    subject { helper.pretty_originator(stock_movement) }
+
+    context "originator has a number" do
+      let(:originator) { create(:order) }
+
+      it "returns the originator's number" do
+        expect(subject).to eq originator.number
+      end
+    end
+
+    context "originator doesn't have a number" do
+      let(:originator) { create(:user) }
+
+      it "returns an empty string" do
+        expect(subject).to eq ""
+      end
+    end
+  end
+end

data/spec/helpers/admin/store_credit_events_helper_spec.rb

@@ -0,0 +1,97 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Spree::Admin::StoreCreditEventsHelper, type: :helper do
+  describe "#store_credit_event_admin_action_name" do
+    let(:store_credit_event) { create(:store_credit_event, action: action) }
+
+    subject { store_credit_event_admin_action_name(store_credit_event) }
+
+    context "capture event" do
+      let(:action) { Spree::StoreCredit::CAPTURE_ACTION }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Used"
+      end
+    end
+
+    context "authorize event" do
+      let(:action) { Spree::StoreCredit::AUTHORIZE_ACTION }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Authorized"
+      end
+    end
+
+    context "eligible event" do
+      let(:action) { Spree::StoreCredit::ELIGIBLE_ACTION }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Eligibility Verified"
+      end
+    end
+
+    context "allocation event" do
+      let(:action) { Spree::StoreCredit::ALLOCATION_ACTION }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Added"
+      end
+    end
+
+    context "void event" do
+      let(:action) { Spree::StoreCredit::VOID_ACTION }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Voided"
+      end
+    end
+
+    context "credit event" do
+      let(:action) { Spree::StoreCredit::CREDIT_ACTION }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Credit"
+      end
+    end
+  end
+
+  describe "#store_credit_event_originator_link" do
+    let(:store_credit_event) { create(:store_credit_adjustment_event, originator: originator) }
+
+    subject { store_credit_event_originator_link(store_credit_event) }
+
+    context "originator is a user" do
+      let(:originator) { create(:user) }
+
+      it "returns a link to the user's edit page" do
+        expect(subject).to eq %(<a target=\"_blank\" href=\"/admin/users/#{originator.id}/edit\">User - #{originator.email}</a>)
+      end
+    end
+
+    context "originator is a payment" do
+      let(:originator) { create(:payment) }
+
+      it "returns a link to the order's payments page" do
+        expect(subject).to eq %(<a target=\"_blank\" href=\"/admin/orders/#{originator.order.number}/payments/#{originator.id}\">Payment - Order ##{originator.order.number}</a>)
+      end
+    end
+
+    context "originator is a refund" do
+      let(:originator) { create(:refund, amount: 1.0) }
+
+      it "returns a link to the order's payments page" do
+        expect(subject).to eq %(<a target=\"_blank\" href=\"/admin/orders/#{originator.payment.order.number}/payments\">Refund - Order ##{originator.payment.order.number}</a>)
+      end
+    end
+
+    context "originator is not specifically handled" do
+      let(:originator) { create(:store_credit_update_reason) }
+
+      it "raises an error" do
+        expect { subject }.to raise_error(RuntimeError, "Unexpected originator type Spree::StoreCreditUpdateReason")
+      end
+    end
+  end
+end

data/spec/helpers/promotion_rules_helper_spec.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+module Spree
+ describe Spree::PromotionRulesHelper, type: :helper do
+   it "does not include existing rules in options" do
+     promotion = Spree::Promotion.new
+     promotion.promotion_rules << Spree::Promotion::Rules::ItemTotal.new
+
+     options = helper.options_for_promotion_rule_types(promotion)
+     expect(options).not_to match(/ItemTotal/)
+   end
+ end
+end

data/spec/javascripts/fixtures/_boot.html.erb

@@ -0,0 +1,8 @@
+<%= javascript_include_tag *@suite.spec_assets, debug: @suite.config.expand_assets, allow_non_precompiled: true %>
+<%= render "spree/admin/shared/js_locale_data" %>
+<%= javascript_tag do -%>
+  Spree.env = "<%= Rails.env %>";
+<% end %>
+<script type="text/javascript">
+  Teaspoon.onWindowLoad(Teaspoon.execute);
+</script>

data/spec/javascripts/fixtures/number_with_currency/with_currency_select.html.erb

@@ -0,0 +1,3 @@
+<%= fields_for 'price', Spree::Price.new do |f| %>
+  <%= render 'spree/admin/shared/number_with_currency', f: f %>
+<% end %>

data/spec/javascripts/fixtures/number_with_currency/without_select.html.erb

@@ -0,0 +1,3 @@
+<%= fields_for 'price', Spree::Price.new() do |f| %>
+  <%= render 'spree/admin/shared/number_with_currency', f: f, currency: 'USD' %>
+<% end %>

data/spec/javascripts/format_money_spec.js

@@ -0,0 +1,48 @@
+describe('Spree.formatMoney', function() {
+  it('can format USD', function() {
+    expect(Spree.formatMoney(10, 'USD')).to.equal('$10.00');
+    expect(Spree.formatMoney('10', 'USD')).to.equal('$10.00');
+    expect(Spree.formatMoney(12.34, 'USD')).to.equal('$12.34');
+    expect(Spree.formatMoney('12.34', 'USD')).to.equal('$12.34');
+    expect(Spree.formatMoney(1000, 'USD')).to.equal('$1,000.00');
+  });
+
+  it('can format CAD', function() {
+    expect(Spree.formatMoney(1000, 'CAD')).to.equal('$1,000.00');
+  });
+
+  it('can format GBP', function() {
+    expect(Spree.formatMoney(1000, 'GBP')).to.equal('£1,000.00');
+  });
+
+  it('can format EUR', function() {
+    expect(Spree.formatMoney(1000, 'EUR')).to.equal('€1,000.00');
+  });
+
+  it('can format YEN', function() {
+    expect(Spree.formatMoney(1000, 'JPY')).to.equal('¥1,000');
+  });
+
+  describe('with comma as decimal', function() {
+    support.withTranslations({
+      currency_delimiter: ".",
+      currency_separator: ","
+    });
+
+    it('can format USD', function() {
+      expect(Spree.formatMoney(10, 'USD')).to.equal('$10,00');
+      expect(Spree.formatMoney('10', 'USD')).to.equal('$10,00');
+      expect(Spree.formatMoney(12.34, 'USD')).to.equal('$12,34');
+      expect(Spree.formatMoney('12.34', 'USD')).to.equal('$12,34');
+      expect(Spree.formatMoney(1000, 'GBP')).to.equal('£1.000,00');
+    });
+
+    it('can format EUR', function() {
+      expect(Spree.formatMoney(1000, 'EUR')).to.equal('€1.000,00');
+    });
+
+    it('can format YEN', function() {
+      expect(Spree.formatMoney(1000, 'JPY')).to.equal('¥1.000');
+    });
+  });
+});

data/spec/javascripts/spec_helper.js

@@ -0,0 +1,11 @@
+//= require ./support/show_errors
+//= require support/sinon
+//= require support/chai
+//= require support/chai-jq-0.0.7
+
+//= require_self
+//= require spree/backend
+//= require_tree ./support
+
+window.support = {};
+window.expect = chai.expect;

data/spec/javascripts/support/show_errors.js

@@ -0,0 +1,8 @@
+/* Teaspoon doesn't show errors by default */
+window.onerror = function(message) {
+  Teaspoon.log(JSON.stringify({
+    _teaspoon: true,
+    type: "exception",
+    message:  message
+  }));
+}

data/spec/javascripts/support/with_translations.js

@@ -0,0 +1,11 @@
+support.withTranslations = function(translations) {
+  beforeEach(function() {
+    this._oldTranslations = Spree.translations;
+    Spree.translations = translations;
+  });
+
+  afterEach(function() {
+    Spree.translations = this._oldTranslations;
+  });
+};
+

data/spec/javascripts/translation_spec.js

@@ -0,0 +1,58 @@
+describe('Spree.t', function() {
+  support.withTranslations({
+    simple: "simple",
+    nested: {
+      key: "nested key"
+    },
+    deeply: {
+      nested: {
+        key: "deeply nested key"
+      }
+    }
+  })
+
+  it('can get a simple key', function() {
+    expect(Spree.t('simple')).to.equal('simple');
+  });
+
+  it('can get a nested key', function() {
+    expect(Spree.t('nested.key')).to.equal('nested key');
+  });
+
+  it('can get a nested key using scope', function() {
+    expect(Spree.t('key', {scope: 'nested'})).to.equal('nested key');
+  });
+
+  it('can get a deeply nested key', function() {
+    expect(Spree.t('deeply.nested.key')).to.equal('deeply nested key');
+  });
+
+  it('can get a deeply nested key using scope', function() {
+    expect(Spree.t('nested.key', {scope: 'deeply'})).to.equal('deeply nested key');
+    expect(Spree.t('key', {scope: 'deeply.nested'})).to.equal('deeply nested key');
+  });
+
+  it('ignores default when key exists', function() {
+    expect(Spree.t('simple', {default: 'foo'})).to.equal('simple');
+  });
+
+  it('returns a default for a missing key', function() {
+    expect(Spree.t('does_not_exist', {default: 'foo'})).to.equal('foo');
+  });
+});
+
+describe('Spree.human_attribute_name', function() {
+  support.withTranslations({
+    activerecord: {
+      attributes: {
+        "spree/model": {
+          "name": "Name"
+        }
+      }
+    }
+  });
+
+  it('can get attribute names', function() {
+    expect(Spree.human_attribute_name('spree/model', "name")).to.equal('Name');
+  });
+});

data/spec/javascripts/views/number_with_currency_spec.js

@@ -0,0 +1,72 @@
+fixture.preload("number_with_currency/with_currency_select", "number_with_currency/without_select");
+
+describe("Spree.Views.NumberWithCurrency", function() {
+  var view, $el;
+  var $symbol, $input, $select, $addon;
+
+  var loadFixture = function(path) {
+    var fixtures = fixture.load(path, true);
+    $el = $(fixtures[0]);
+  }
+
+  var loadView = function() {
+    view = new Spree.Views.NumberWithCurrency({el: $el});
+    view.render();
+    $symbol = $el.find('.number-with-currency-symbol');
+    $input = $el.find('input');
+    $select = $el.find('select');
+    $addon = $el.find('.number-with-currency-addon');
+  }
+
+  describe("with currency selector", function() {
+    beforeEach(function() {
+      loadFixture("number_with_currency/with_currency_select");
+      loadView();
+    });
+
+    it("has a default currency selected", function() {
+      expect($select).to.have.$val('USD');
+      expect($symbol).to.have.$text('$');
+    });
+
+    it("can select CAD", function() {
+      $select.val('CAD').trigger('change');
+
+      expect($select).to.have.$val('CAD');
+      expect($symbol).to.have.$text('$');
+    });
+
+    it("can select GBP", function() {
+      $select.val('GBP').trigger('change');
+
+      expect($select).to.have.$val('GBP');
+      expect($symbol).to.have.$text('£');
+    });
+
+    it("can select JPY", function() {
+      $select.val('JPY').trigger('change');
+
+      expect($select).to.have.$val('JPY');
+      expect($symbol).to.have.$text('¥');
+    });
+  });
+
+  describe("without currency selector", function() {
+    beforeEach(function() {
+      loadFixture("number_with_currency/without_select");
+      loadView();
+    });
+
+    it("uses USD format", function() {
+      $el.find('[data-currency]').data('currency', 'USD');
+      view.render();
+      expect($symbol).to.have.$text('$');
+    });
+
+    it("uses JPY format", function() {
+      $el.find('[data-currency]').data('currency', 'JPY');
+      view.render();
+      expect($symbol).to.have.$text('¥');
+    });
+  });
+});

data/spec/spec_helper.rb

@@ -0,0 +1,123 @@
+# frozen_string_literal: true
+
+if ENV["COVERAGE"]
+  require 'simplecov'
+  SimpleCov.start('rails')
+end
+
+# This file is copied to ~/spec when you run 'ruby script/generate rspec'
+# from the project root directory.
+ENV["RAILS_ENV"] ||= 'test'
+
+require 'solidus_backend'
+require 'spree/testing_support/dummy_app'
+DummyApp.setup(
+  gem_root: File.expand_path('..', __dir__),
+  lib_name: 'solidus_backend'
+)
+
+require 'rails-controller-testing'
+require 'rspec-activemodel-mocks'
+require 'rspec/rails'
+require 'pry'
+
+# Requires supporting files with custom matchers and macros, etc,
+# in ./support/ and its subdirectories.
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+
+require 'database_cleaner'
+require 'with_model'
+
+require 'spree/testing_support/authorization_helpers'
+require 'spree/testing_support/factories'
+require 'spree/testing_support/preferences'
+require 'spree/testing_support/controller_requests'
+require 'spree/testing_support/flash'
+require 'spree/testing_support/url_helpers'
+require 'spree/testing_support/order_walkthrough'
+require 'spree/testing_support/capybara_ext'
+
+require 'capybara-screenshot/rspec'
+Capybara.save_path = ENV['CIRCLE_ARTIFACTS'] if ENV['CIRCLE_ARTIFACTS']
+Capybara.exact = true
+
+require "selenium/webdriver"
+
+Capybara.register_driver :selenium_chrome_headless do |app|
+  browser_options = ::Selenium::WebDriver::Chrome::Options.new
+  browser_options.args << '--headless'
+  browser_options.args << '--disable-gpu'
+  browser_options.args << '--window-size=1440,1080'
+  Capybara::Selenium::Driver.new(app, browser: :chrome, options: browser_options)
+end
+
+Capybara.javascript_driver = (ENV['CAPYBARA_DRIVER'] || :selenium_chrome_headless).to_sym
+
+ActionView::Base.raise_on_missing_translations = true
+
+Capybara.default_max_wait_time = ENV['DEFAULT_MAX_WAIT_TIME'].to_f if ENV['DEFAULT_MAX_WAIT_TIME'].present?
+
+ActiveJob::Base.queue_adapter = :test
+
+RSpec.configure do |config|
+  config.color = true
+  config.infer_spec_type_from_file_location!
+  config.expect_with :rspec do |c|
+    c.syntax = :expect
+  end
+  config.mock_with :rspec do |c|
+    c.syntax = :expect
+  end
+
+  # If you're not using ActiveRecord, or you'd prefer not to run each of your
+  # examples within a transaction, comment the following line or assign false
+  # instead of true.
+  config.use_transactional_fixtures = true
+
+  config.before :suite do
+    DatabaseCleaner.clean_with :truncation
+  end
+
+  config.when_first_matching_example_defined(type: :feature) do
+    config.before :suite do
+      # Preload assets
+      # This should avoid capybara timeouts, and avoid counting asset compilation
+      # towards the timing of the first feature spec.
+      Rails.application.precompiled_assets
+    end
+  end
+
+  config.before do
+    Rails.cache.clear
+    reset_spree_preferences
+    if RSpec.current_example.metadata[:js] && page.driver.browser.respond_to?(:url_blacklist)
+      page.driver.browser.url_blacklist = ['http://fonts.googleapis.com']
+    end
+  end
+
+  config.include BaseFeatureHelper, type: :feature
+
+  config.after(:each, type: :feature) do |example|
+    missing_translations = page.body.scan(/translation missing: #{I18n.locale}\.(.*?)[\s<\"&]/)
+    if missing_translations.any?
+      puts "Found missing translations: #{missing_translations.inspect}"
+      puts "In spec: #{example.location}"
+    end
+  end
+
+  config.include FactoryBot::Syntax::Methods
+  config.include ActiveJob::TestHelper
+
+  config.include Spree::TestingSupport::Preferences
+  config.include Spree::TestingSupport::UrlHelpers
+  config.include Spree::TestingSupport::ControllerRequests, type: :controller
+  config.include Spree::TestingSupport::Flash
+
+  config.extend WithModel
+
+  config.example_status_persistence_file_path = "./spec/examples.txt"
+
+  config.order = :random
+
+  Kernel.srand config.seed
+end

data/spec/support/appear_before_matcher.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+require 'rspec/expectations'
+
+RSpec::Matchers.define :appear_before do |expected|
+  match do |actual|
+    raise "Page instance required to use the appear_before matcher" unless page
+    page.body.index(actual) <= page.body.index(expected)
+  end
+end

data/spec/support/feature/base_feature_helper.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module BaseFeatureHelper
+  def click_nav(nav_text, subnav_text = nil)
+    primary_nav = find(".admin-nav-menu>ul>li>a", text: /#{nav_text}/i)
+    if subnav_text
+      primary_nav.find('+ul>li>a', text: /#{subnav_text}/i).click
+    else
+      primary_nav.click
+    end
+  end
+end