solidus_core 2.1.1 → 2.2.0.beta1

data/spec/models/spree/wallet_payment_source_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+describe Spree::WalletPaymentSource, type: :model do
+  subject { Spree::WalletPaymentSource }
+
+  describe "validation" do
+    context 'with a non-PaymentSource model' do
+      with_model 'NonPaymentSource', scope: :all do
+        model do
+          # We have to set this up or else `inverse_of` prevents us from testing our code
+          has_many :wallet_payment_sources, class_name: 'Spree::WalletPaymentSource', as: :payment_source, inverse_of: :payment_source
+        end
+      end
+
+      let(:payment_source) { NonPaymentSource.create! }
+
+      it "errors when `payment_source` is not a `Spree::PaymentSource`" do
+        wallet_payment_source = Spree::WalletPaymentSource.new(
+          payment_source: payment_source,
+          user: create(:user)
+        )
+
+        expect(wallet_payment_source).not_to be_valid
+        expect(wallet_payment_source.errors.messages).to eq(
+          { payment_source: ["has to be a Spree::PaymentSource"] }
+        )
+      end
+    end
+
+    it "is valid with a `credit_card` as `payment_source`" do
+      valid_attrs = {
+        payment_source: create(:credit_card),
+        user: create(:user)
+      }
+      expect(subject.new(valid_attrs)).to be_valid
+    end
+
+    it "is valid with `store_credit` as `payment_source`" do
+      valid_attrs = {
+        payment_source: create(:store_credit),
+        user: create(:user)
+      }
+      expect(subject.new(valid_attrs)).to be_valid
+    end
+  end
+end

data/spec/models/spree/wallet_spec.rb

@@ -0,0 +1,128 @@
+require 'spec_helper'
+
+describe Spree::Wallet, type: :model do
+  let(:user) { create(:user) }
+  let(:credit_card) { create(:credit_card, user_id: user.id) }
+  let(:store_credit) { create(:store_credit, user_id: user.id) }
+  subject(:wallet) { Spree::Wallet.new(user) }
+
+  describe "#add" do
+    context "with valid payment source" do
+      it "creates a wallet_payment_source for this user's wallet" do
+        expect { subject.add(credit_card) }.to change(Spree::WalletPaymentSource, :count).by(1)
+      end
+
+      it "only creates the payment source once" do
+        subject.add(credit_card)
+        expect { subject.add(credit_card) }.to_not change(Spree::WalletPaymentSource, :count)
+      end
+    end
+  end
+
+  describe "#remove" do
+    context "with payment_source not in the wallet" do
+      it "will raise a RecordNotFound exception" do
+        expect { subject.remove(credit_card) }.to raise_error(ActiveRecord::RecordNotFound)
+      end
+    end
+
+    context "with the payment_source in the wallet" do
+      before { subject.add(credit_card) }
+      it "will remove the payment source from the wallet" do
+        expect { subject.remove(credit_card) }.to change(Spree::WalletPaymentSource, :count).by(-1)
+      end
+    end
+  end
+
+  describe "#wallet_payment_sources" do
+    context "with no payment sources" do
+      it "returns an empty Array" do
+        expect(subject.wallet_payment_sources).to eql []
+      end
+    end
+
+    context "with payment sources" do
+      let!(:wallet_credit_card) { subject.add(credit_card) }
+      let!(:wallet_store_credit) { subject.add(store_credit) }
+
+      it "returns all the payment sources" do
+        expect(subject.wallet_payment_sources).to include(wallet_credit_card, wallet_store_credit)
+      end
+    end
+  end
+
+  describe "#find" do
+    context "with no payment sources in the wallet" do
+      it "returns nil" do
+        expect(subject.find(1)).to be_nil
+      end
+    end
+
+    context "with payment sources in the wallet" do
+      let(:wallet_credit_card) { subject.add(credit_card) }
+
+      it "finds the wallet payment source by id" do
+        expect(subject.find(wallet_credit_card.id)).to eql wallet_credit_card
+      end
+    end
+  end
+
+  describe "#default_wallet_payment_source=" do
+    context "with no current default" do
+      let!(:wallet_credit_card) { subject.add(credit_card) }
+
+      it "marks the passed in payment source as default" do
+        expect { subject.default_wallet_payment_source = wallet_credit_card }.to(
+          change(subject, :default_wallet_payment_source).
+            from(nil).
+            to(wallet_credit_card)
+        )
+      end
+    end
+
+    context "with other payment source already default" do
+      let!(:wallet_credit_card) { subject.add(credit_card) }
+      let!(:wallet_store_credit) { subject.add(store_credit) }
+
+      before { subject.default_wallet_payment_source = wallet_credit_card }
+
+      it "sets the new payment source as the default" do
+        expect { subject.default_wallet_payment_source = wallet_store_credit }.to(
+          change(subject, :default_wallet_payment_source).
+            from(wallet_credit_card).
+            to(wallet_store_credit)
+        )
+      end
+    end
+
+    context 'with a wallet payment source that does not belong to the wallet' do
+      let(:other_wallet_credit_card) { other_wallet.add(credit_card) }
+      let(:other_wallet) { Spree::Wallet.new(other_user) }
+      let(:other_credit_card) { create(:credit_card, user_id: other_user.id) }
+      let(:other_user) { create(:user) }
+
+      it 'raises an error' do
+        expect {
+          wallet.default_wallet_payment_source = other_wallet_credit_card
+        }.to raise_error(Spree::Wallet::Unauthorized)
+      end
+    end
+  end
+
+  describe "#default_wallet_payment_source" do
+    context "with no default payment source present" do
+      it "will return nil" do
+        expect(subject.default_wallet_payment_source).to be_nil
+      end
+    end
+
+    context "with a default payment source" do
+      let!(:wallet_credit_card) { subject.add(credit_card) }
+      before { subject.default_wallet_payment_source = wallet_credit_card }
+
+      it "will return the wallet payment source" do
+        expect(subject.default_wallet_payment_source).to eql wallet_credit_card
+      end
+    end
+  end
+end

data/spec/support/concerns/payment_source.rb

@@ -0,0 +1,64 @@
+RSpec.shared_examples "a payment source" do
+  subject(:payment_source) { described_class.new }
+
+  describe 'attributes' do
+    it { is_expected.to respond_to(:imported) }
+    it { is_expected.to respond_to(:name) }
+  end
+
+  describe "#associations" do
+    it { is_expected.to respond_to(:payments) }
+    it { is_expected.to respond_to(:user) }
+  end
+
+  describe "#can_capture?" do
+    it "should be true if payment is pending" do
+      payment = mock_model(Spree::Payment, pending?: true, created_at: Time.current)
+      expect(payment_source.can_capture?(payment)).to be true
+    end
+
+    it "should be true if payment is checkout" do
+      payment = mock_model(Spree::Payment, pending?: false, checkout?: true, created_at: Time.current)
+      expect(payment_source.can_capture?(payment)).to be true
+    end
+  end
+
+  describe "#can_void?" do
+    it "should be true if payment is not void" do
+      payment = mock_model(Spree::Payment, failed?: false, void?: false)
+      expect(payment_source.can_void?(payment)).to be true
+    end
+  end
+
+  describe "#can_credit?" do
+    it "should be false if payment is not completed" do
+      payment = mock_model(Spree::Payment, completed?: false)
+      expect(payment_source.can_credit?(payment)).to be false
+    end
+
+    it "should be false when credit_allowed is zero" do
+      payment = mock_model(Spree::Payment, completed?: true, credit_allowed: 0, order: mock_model(Spree::Order, payment_state: 'credit_owed'))
+      expect(payment_source.can_credit?(payment)).to be false
+    end
+  end
+
+  describe "#first_name" do
+    before do
+      payment_source.name = "Ludwig van Beethoven"
+    end
+
+    it "extracts the first name" do
+      expect(payment_source.first_name).to eq "Ludwig"
+    end
+  end
+
+  describe "#last_name" do
+    before do
+      payment_source.name = "Ludwig van Beethoven"
+    end
+
+    it "extracts the last name" do
+      expect(payment_source.last_name).to eq "van Beethoven"
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_core
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.2.0.beta1
 platform: ruby
 authors:
 - Solidus Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-12-12 00:00:00.000000000 Z
+date: 2017-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemerchant
@@ -132,22 +132,22 @@ dependencies:
   name: kaminari
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.15'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.15.1
+        version: '0.17'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.15'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.15.1
+        version: '0.17'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: monetize
   requirement: !ruby/object:Gem::Requirement
@@ -166,16 +166,22 @@ dependencies:
   name: paperclip
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '4.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '4.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '6'
 - !ruby/object:Gem::Dependency
   name: paranoia
   requirement: !ruby/object:Gem::Requirement
@@ -298,16 +304,22 @@ dependencies:
   name: twitter_cldr
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5'
 - !ruby/object:Gem::Dependency
   name: email_spec
   requirement: !ruby/object:Gem::Requirement
@@ -332,12 +344,13 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- app/assets/config/solidus_core_manifest.js
 - app/assets/images/logo/solidus_logo.png
 - app/assets/images/noimage/large.png
 - app/assets/images/noimage/mini.png
 - app/assets/images/noimage/product.png
 - app/assets/images/noimage/small.png
-- app/assets/javascripts/spree.js.coffee.erb
+- app/assets/javascripts/spree.js.erb
 - app/controllers/spree/base_controller.rb
 - app/helpers/spree/base_helper.rb
 - app/helpers/spree/checkout_helper.rb
@@ -345,9 +358,11 @@ files:
 - app/helpers/spree/products_helper.rb
 - app/helpers/spree/store_helper.rb
 - app/helpers/spree/taxons_helper.rb
+- app/jobs/spree/promotion_code_batch_job.rb
 - app/mailers/spree/base_mailer.rb
 - app/mailers/spree/carton_mailer.rb
 - app/mailers/spree/order_mailer.rb
+- app/mailers/spree/promotion_code_batch_mailer.rb
 - app/mailers/spree/reimbursement_mailer.rb
 - app/mailers/spree/test_mailer.rb
 - app/models/concerns/spree/adjustment_source.rb
@@ -426,6 +441,7 @@ files:
 - app/models/spree/payment_method.rb
 - app/models/spree/payment_method/check.rb
 - app/models/spree/payment_method/store_credit.rb
+- app/models/spree/payment_source.rb
 - app/models/spree/preference.rb
 - app/models/spree/preferences/configuration.rb
 - app/models/spree/preferences/preferable.rb
@@ -457,11 +473,11 @@ files:
 - app/models/spree/promotion/rules/user_logged_in.rb
 - app/models/spree/promotion/rules/user_role.rb
 - app/models/spree/promotion_action.rb
-- app/models/spree/promotion_builder.rb
 - app/models/spree/promotion_category.rb
 - app/models/spree/promotion_chooser.rb
 - app/models/spree/promotion_code.rb
-- app/models/spree/promotion_code/code_builder.rb
+- app/models/spree/promotion_code/batch_builder.rb
+- app/models/spree/promotion_code_batch.rb
 - app/models/spree/promotion_handler/cart.rb
 - app/models/spree/promotion_handler/coupon.rb
 - app/models/spree/promotion_handler/free_shipping.rb
@@ -564,8 +580,10 @@ files:
 - app/models/spree/variant_property_rule.rb
 - app/models/spree/variant_property_rule_condition.rb
 - app/models/spree/variant_property_rule_value.rb
+- app/models/spree/wallet.rb
 - app/models/spree/wallet/add_payment_sources_to_wallet.rb
 - app/models/spree/wallet/default_payment_builder.rb
+- app/models/spree/wallet_payment_source.rb
 - app/models/spree/zone.rb
 - app/models/spree/zone_member.rb
 - app/views/layouts/spree/base_mailer.html.erb
@@ -575,6 +593,8 @@ files:
 - app/views/spree/order_mailer/confirm_email.html.erb
 - app/views/spree/order_mailer/confirm_email.text.erb
 - app/views/spree/order_mailer/inventory_cancellation_email.text.erb
+- app/views/spree/promotion_code_batch_mailer/promotion_code_batch_errored.text.erb
+- app/views/spree/promotion_code_batch_mailer/promotion_code_batch_finished.text.erb
 - app/views/spree/reimbursement_mailer/reimbursement_email.html.erb
 - app/views/spree/reimbursement_mailer/reimbursement_email.text.erb
 - app/views/spree/shared/_base_mailer_footer.html.erb
@@ -585,7 +605,6 @@ files:
 - app/views/spree/test_mailer/test_email.text.erb
 - config/initializers/assets.rb
 - config/initializers/friendly_id.rb
-- config/initializers/premailer_assets.rb
 - config/locales/en.yml
 - db/default/spree/countries.rb
 - db/default/spree/return_reasons.rb
@@ -883,6 +902,8 @@ files:
 - db/migrate/20160308000300_disallow_adjustment_finalized_nulls.rb
 - db/migrate/20160318145302_add_timestamps_to_prices.rb
 - db/migrate/20160330204846_add_missing_timestamp_columns.rb
+- db/migrate/20160420044191_create_spree_wallet_payment_sources.rb
+- db/migrate/20160420181916_migrate_credit_cards_to_wallet_payment_sources.rb
 - db/migrate/20160509181311_add_country_iso_to_prices.rb
 - db/migrate/20160527070401_rename_shipment_address_field.rb
 - db/migrate/20160608162651_ensure_default_store.rb
@@ -893,7 +914,10 @@ files:
 - db/migrate/20160924135758_remove_is_default_from_prices.rb
 - db/migrate/20161009141333_remove_currency_from_line_items.rb
 - db/migrate/20161014221052_add_available_to_columns_and_remove_display_on_from_payment_methods.rb
+- db/migrate/20161017102621_create_spree_promotion_code_batch.rb
 - db/migrate/20161123154034_add_available_to_users_and_remove_display_on_from_shipping_methods.rb
+- db/migrate/20161129035810_add_index_to_spree_payments_number.rb
+- db/migrate/20170223235001_remove_spree_store_credits_column.rb
 - db/seeds.rb
 - lib/generators/spree/custom_user/custom_user_generator.rb
 - lib/generators/spree/custom_user/templates/authentication_helpers.rb.tt
@@ -940,7 +964,6 @@ files:
 - lib/spree/core/search/base.rb
 - lib/spree/core/search/variant.rb
 - lib/spree/core/stock_configuration.rb
-- lib/spree/core/unreturned_item_charger.rb
 - lib/spree/core/validators/email.rb
 - lib/spree/core/version.rb
 - lib/spree/deprecation.rb
@@ -1057,7 +1080,6 @@ files:
 - lib/spree_core.rb
 - lib/tasks/core.rake
 - lib/tasks/email.rake
-- lib/tasks/exchanges.rake
 - lib/tasks/migrations/copy_order_bill_address_to_credit_card.rake
 - lib/tasks/migrations/copy_shipped_shipments_to_cartons.rake
 - lib/tasks/migrations/create_vat_prices.rake
@@ -1072,6 +1094,7 @@ files:
 - spec/helpers/order_helper_spec.rb
 - spec/helpers/products_helper_spec.rb
 - spec/helpers/taxons_helper_spec.rb
+- spec/jobs/promotion_code_batch_job_spec.rb
 - spec/lib/calculated_adjustments_spec.rb
 - spec/lib/i18n_spec.rb
 - spec/lib/search/base_spec.rb
@@ -1151,15 +1174,14 @@ files:
 - spec/lib/spree/core/testing_support/factories/variant_property_rule_value_factory_spec.rb
 - spec/lib/spree/core/testing_support/factories/zone_factory_spec.rb
 - spec/lib/spree/core/testing_support/preferences_spec.rb
-- spec/lib/spree/core/unreturned_item_charger_spec.rb
 - spec/lib/spree/core/validators/email_spec.rb
 - spec/lib/spree/core/version_spec.rb
 - spec/lib/spree/localized_number_spec.rb
 - spec/lib/spree/migrations_spec.rb
 - spec/lib/spree/money_spec.rb
+- spec/lib/spree/permission_sets/default_customer_spec.rb
 - spec/lib/tasks/dummy_task.rake
 - spec/lib/tasks/dummy_task_spec.rb
-- spec/lib/tasks/exchanges_spec.rb
 - spec/lib/tasks/migrations/copy_shipped_shipments_to_cartons_spec.rb
 - spec/lib/tasks/migrations/create_vat_prices_spec.rb
 - spec/lib/tasks/migrations/migrate_shipping_rate_taxes_spec.rb
@@ -1167,6 +1189,7 @@ files:
 - spec/lib/tasks/upgrade_spec.rb
 - spec/mailers/carton_mailer_spec.rb
 - spec/mailers/order_mailer_spec.rb
+- spec/mailers/promotion_code_batch_mailer_spec.rb
 - spec/mailers/reimbursement_mailer_spec.rb
 - spec/mailers/test_mailer_spec.rb
 - spec/models/spree/ability_spec.rb
@@ -1287,9 +1310,9 @@ files:
 - spec/models/spree/promotion/rules/user_role_spec.rb
 - spec/models/spree/promotion/rules/user_spec.rb
 - spec/models/spree/promotion_action_spec.rb
-- spec/models/spree/promotion_builder_spec.rb
 - spec/models/spree/promotion_category_spec.rb
-- spec/models/spree/promotion_code/code_builder_spec.rb
+- spec/models/spree/promotion_code/batch_builder_spec.rb
+- spec/models/spree/promotion_code_batch_spec.rb
 - spec/models/spree/promotion_code_spec.rb
 - spec/models/spree/promotion_handler/cart_spec.rb
 - spec/models/spree/promotion_handler/coupon_spec.rb
@@ -1370,11 +1393,14 @@ files:
 - spec/models/spree/variant_property_rule_spec.rb
 - spec/models/spree/variant_property_rule_value_spec.rb
 - spec/models/spree/variant_spec.rb
+- spec/models/spree/wallet_payment_source_spec.rb
+- spec/models/spree/wallet_spec.rb
 - spec/models/spree/zone_spec.rb
 - spec/shared_examples/calculator_shared_examples.rb
 - spec/spec_helper.rb
 - spec/support/big_decimal.rb
 - spec/support/concerns/default_price.rb
+- spec/support/concerns/payment_source.rb
 - spec/support/concerns/working_factories.rb
 - spec/support/dummy_ability.rb
 - spec/support/shared_contexts/rake.rb
@@ -1402,7 +1428,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.8.23
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 2.6.10
 signing_key: 
 specification_version: 4
 summary: Essential models, mailers, and classes for the Solidus e-commerce project.

data/app/assets/javascripts/spree.js.coffee.erb

@@ -1,64 +0,0 @@
-#= require jsuri
-class window.Spree
-  @ready: (callback) ->
-    if typeof Turbolinks isnt 'undefined' and Turbolinks.supported
-      jQuery(document).on 'turbolinks:load', -> callback(jQuery)
-    else
-      jQuery(document).ready(callback)
-
-  @mountedAt: ->
-    "<%= Rails.application.routes.url_helpers.spree_path(trailing_slash: true) %>"
-
-  @pathFor: (path) ->
-    locationOrigin = "#{window.location.protocol}//#{window.location.hostname}" + (if window.location.port then ":#{window.location.port}" else "")
-    "#{locationOrigin}#{@mountedAt()}#{path}"
-
-  # Helper function to take a URL and add query parameters to it
-  # Uses the JSUri library from here: https://github.com/derek-watson/jsUri
-  # Thanks to Jake Moffat for the suggestion: https://twitter.com/jakeonrails/statuses/321776992221544449
-  @url: (uri, query) ->
-    if Spree.env == 'development' || Spree.env == 'test'
-      console.warn 'Spree.url is deprecated, please use Spree.ajax for your request instead.'
-    if uri.path == undefined
-      uri = new Uri(uri)
-    if query
-      $.each query, (key, value) ->
-        uri.addQueryParam(key, value)
-    return uri
-
-  # This function automatically appends the API token
-  # for the user to the end of any URL.
-  # Immediately after, this string is then passed to jQuery.ajax.
-  #
-  # Spree.ajax works in two ways to support common jQuery syntax:
-  #
-  # Spree.ajax("url", {settings: 'go here'})
-  # or:
-  # Spree.ajax({url: "url", settings: 'go here'})
-  #
-  # Spree.getJSON has the same method signature as $.getJSON
-  @ajax: (url, options) ->
-    if typeof(url) == "object"
-      options = url
-      url = undefined
-
-    options = options || {}
-
-    options = $.extend(options, { headers: { "X-Spree-Token": Spree.api_key } })
-    $.ajax(url, options)
-
-  @routes:
-    states_search: @pathFor('api/states')
-    apply_coupon_code: (order_id) ->
-      Spree.pathFor("api/orders/#{order_id}/apply_coupon_code")
-
-  @getJSON: (url, data, success) ->
-    if typeof data is 'function'
-      success = data
-      data = undefined
-    @ajax(
-      dataType: "json",
-      url: url,
-      data: data,
-      success: success
-    )

data/app/models/spree/promotion_builder.rb

@@ -1,55 +0,0 @@
-class Spree::PromotionBuilder
-  include ActiveModel::Model
-
-  attr_reader :promotion
-  attr_accessor :base_code, :number_of_codes, :user
-
-  validates :number_of_codes,
-    numericality: { only_integer: true, greater_than: 0 },
-    allow_nil: true
-
-  validate :promotion_validity
-
-  class_attribute :code_builder_class
-  self.code_builder_class = ::Spree::PromotionCode::CodeBuilder
-
-  # @param promotion_attrs [Hash] The desired attributes for the newly promotion
-  # @param attributes [Hash] The desired attributes for this builder
-  # @param user [Spree::User] The user who triggered this promotion build
-  def initialize(attributes = {}, promotion_attributes = {})
-    @promotion = Spree::Promotion.new(promotion_attributes)
-    super(attributes)
-  end
-
-  def perform
-    if can_build_codes?
-      @promotion.codes = code_builder.build_promotion_codes
-    end
-
-    return false unless valid?
-
-    @promotion.save
-  end
-
-  def number_of_codes=(value)
-    @number_of_codes = value.presence.try(:to_i)
-  end
-
-  private
-
-  def promotion_validity
-    if !@promotion.valid?
-      @promotion.errors.each do |attribute, error|
-        errors[attribute].push error
-      end
-    end
-  end
-
-  def can_build_codes?
-    @base_code && @number_of_codes
-  end
-
-  def code_builder
-    self.class.code_builder_class.new(@promotion, @base_code, @number_of_codes)
-  end
-end

data/app/models/spree/promotion_code/code_builder.rb

@@ -1,62 +0,0 @@
-# A class responsible for building PromotionCodes
-class ::Spree::PromotionCode::CodeBuilder
-  attr_reader :promotion, :num_codes, :base_code
-
-  class_attribute :random_code_length
-  self.random_code_length = 6
-
-  # Requres a +promotion+, +base_code+ and +num_codes+
-  #
-  # +promotion+ Must be a Spree::Promotion.
-  # +base_code+ Must be a String.
-  # +num_codes+ Must be a positive integer greater than zero.
-  def initialize(promotion, base_code, num_codes)
-    @base_code = base_code
-    @num_codes = num_codes
-    @promotion = promotion
-  end
-
-  # Builds and returns an array of Spree::PromotionCode's
-  def build_promotion_codes
-    codes.map do |code|
-      promotion.codes.build(value: code)
-    end
-  end
-
-  private
-
-  def codes
-    if num_codes > 1
-      generate_random_codes
-    else
-      [base_code]
-    end
-  end
-
-  def generate_random_codes
-    valid_codes = Set.new
-
-    while valid_codes.size < num_codes
-      new_codes = Array.new(num_codes) { generate_random_code }.to_set
-      valid_codes += get_unique_codes(new_codes)
-    end
-
-    valid_codes.to_a.take(num_codes)
-  end
-
-  def generate_random_code
-    suffix = Array.new(self.class.random_code_length) do
-      sample_characters.sample
-    end.join
-
-    "#{@base_code}_#{suffix}"
-  end
-
-  def sample_characters
-    @sample_characters ||= ('a'..'z').to_a
-  end
-
-  def get_unique_codes(code_set)
-    code_set - Spree::PromotionCode.where(value: code_set.to_a).pluck(:value)
-  end
-end