solidus_subscriptions-alpha 0.0.3 → 0.0.4

data/lib/solidus_subscriptions/engine.rb

@@ -1,56 +1,56 @@
-module SolidusSubscriptions
-  class Engine < Rails::Engine
-    require 'spree/core'
-    require 'solidus_subscriptions/permitted_attributes'
-    require 'solidus_subscriptions/config'
-    require 'solidus_subscriptions/processor'
-
-    isolate_namespace SolidusSubscriptions
-    engine_name 'solidus_subscriptions'
-
-    # use rspec for tests
-    config.generators do |g|
-      g.test_framework :rspec
-    end
-
-    config.autoload_paths << config.root.join('app', 'jobs')
-
-    initializer 'configure spree subcription permitted attributes', after: 'require subscription lib helpers' do
-      PermittedAttributes.update_spree_permiteed_attributes
-    end
-
-    initializer 'solidus_subscriptions.configs', before: "require subscription lib helpers" do
-    end
-
-    initializer 'register_subscription_promotion_rule', after: 'spree.promo.register.promotion.rules' do |app|
-      app.config.spree.promotions.rules << 'SolidusSubscriptions::SubscriptionPromotionRule'
-      app.config.spree.promotions.rules << 'SolidusSubscriptions::SubscriptionOrderPromotionRule'
-    end
-
-    initializer 'subscriptions_backend' do
-      next unless Spree::Backend::Config.respond_to?(:menu_items)
-      Spree::Backend::Config.configure do |config|
-        config.menu_items << config.class::MenuItem.new(
-          [:subscriptions],
-          'repeat',
-          url: :admin_subscriptions_path,
-          condition: ->{ can?(:admin, SolidusSubscriptions::Subscription) }
-        )
-      end
-    end
-
-    def self.activate
-      Dir.glob(File.join(File.dirname(__FILE__), '../../app/decorators/**/*.rb')) do |c|
-        Rails.configuration.cache_classes ? require(c) : load(c)
-      end
-
-      Spree::Ability.register_ability(SolidusSubscriptions::Ability)
-    end
-
-    config.to_prepare(&method(:activate).to_proc)
-  end
-
-  def self.table_name_prefix
-    'solidus_subscriptions_'
-  end
-end
+module SolidusSubscriptions
+  class Engine < Rails::Engine
+    require 'spree/core'
+    require 'solidus_subscriptions/permitted_attributes'
+    require 'solidus_subscriptions/config'
+    require 'solidus_subscriptions/processor'
+
+    isolate_namespace SolidusSubscriptions
+    engine_name 'solidus_subscriptions'
+
+    # use rspec for tests
+    config.generators do |g|
+      g.test_framework :rspec
+    end
+
+    config.autoload_paths << config.root.join('app', 'jobs')
+
+    initializer 'configure spree subcription permitted attributes', after: 'require subscription lib helpers' do
+      PermittedAttributes.update_spree_permiteed_attributes
+    end
+
+    initializer 'solidus_subscriptions.configs', before: "require subscription lib helpers" do
+    end
+
+    initializer 'register_subscription_promotion_rule', after: 'spree.promo.register.promotion.rules' do |app|
+      app.config.spree.promotions.rules << 'SolidusSubscriptions::SubscriptionPromotionRule'
+      app.config.spree.promotions.rules << 'SolidusSubscriptions::SubscriptionOrderPromotionRule'
+    end
+
+    initializer 'subscriptions_backend' do
+      next unless Spree::Backend::Config.respond_to?(:menu_items)
+      Spree::Backend::Config.configure do |config|
+        config.menu_items << config.class::MenuItem.new(
+          [:subscriptions],
+          'repeat',
+          url: :admin_subscriptions_path,
+          condition: ->{ can?(:admin, SolidusSubscriptions::Subscription) }
+        )
+      end
+    end
+
+    def self.activate
+      Dir.glob(File.join(File.dirname(__FILE__), '../../app/decorators/**/*.rb')) do |c|
+        Rails.configuration.cache_classes ? require(c) : load(c)
+      end
+
+      Spree::Ability.register_ability(SolidusSubscriptions::Ability)
+    end
+
+    config.to_prepare(&method(:activate).to_proc)
+  end
+
+  def self.table_name_prefix
+    'solidus_subscriptions_'
+  end
+end

data/lib/solidus_subscriptions/permitted_attributes.rb

@@ -1,36 +1,36 @@
-# This module is responsible for managing what attributes can be updated
-# through the api. It also overrides Spree::Permitted attributes to allow the
-# solidus api to accept nested params for subscription models as well
-module SolidusSubscriptions
-  module PermittedAttributes
-    class << self
-      def update_spree_permiteed_attributes
-        Spree::PermittedAttributes.line_item_attributes << {
-          subscription_line_items_attributes: nested(
-            subscription_line_item_attributes
-          )
-        }
-
-        Spree::PermittedAttributes.user_attributes << {
-          subscriptions_attributes: nested(subscription_attributes)
-        }
-      end
-
-      def subscription_line_item_attributes
-        [Config.subscription_line_item_attributes]
-      end
-
-      def subscription_attributes
-        Config.subscription_attributes | [
-          { line_items_attributes: nested(subscription_line_item_attributes) - [:subscribable_id] }
-        ]
-      end
-
-      private
-
-      def nested(attributes)
-        attributes | [:id]
-      end
-    end
-  end
-end
+# This module is responsible for managing what attributes can be updated
+# through the api. It also overrides Spree::Permitted attributes to allow the
+# solidus api to accept nested params for subscription models as well
+module SolidusSubscriptions
+  module PermittedAttributes
+    class << self
+      def update_spree_permiteed_attributes
+        Spree::PermittedAttributes.line_item_attributes << {
+          subscription_line_items_attributes: nested(
+            subscription_line_item_attributes
+          )
+        }
+
+        Spree::PermittedAttributes.user_attributes << {
+          subscriptions_attributes: nested(subscription_attributes)
+        }
+      end
+
+      def subscription_line_item_attributes
+        [Config.subscription_line_item_attributes]
+      end
+
+      def subscription_attributes
+        Config.subscription_attributes | [
+          { line_items_attributes: nested(subscription_line_item_attributes) - [:subscribable_id] }
+        ]
+      end
+
+      private
+
+      def nested(attributes)
+        attributes | [:id]
+      end
+    end
+  end
+end

data/lib/solidus_subscriptions/processor.rb

@@ -1,108 +1,108 @@
-# This class is responsible for finding subscriptions and installments
-# which need to be processed. It will group them together by user and attempts
-# to process them together. Subscriptions will also be grouped by their
-# shiping address id.
-#
-# This class passes the reponsibility of actually creating the order off onto
-# the consolidated installment class.
-#
-# This class generates `ProcessInstallmentsJob`s
-module SolidusSubscriptions
-  class Processor
-    class << self
-      # Find all actionable subscriptions and intallments, group them together
-      # by user, and schedule a processing job for the group as a batch
-      def run
-        batched_users_to_be_processed.each { |batch| new(batch).build_jobs }
-      end
-
-      private
-
-      def batched_users_to_be_processed
-        subscriptions = SolidusSubscriptions::Subscription.arel_table
-        installments = SolidusSubscriptions::Installment.arel_table
-
-        Spree::User.
-          joins(:subscriptions).
-          joins(
-            subscriptions.
-              join(installments, Arel::Nodes::OuterJoin).
-              on(subscriptions[:id].eq(installments[:subscription_id])).
-              join_sources
-          ).
-          where(
-            SolidusSubscriptions::Subscription.actionable.arel.constraints.reduce(:and).
-              or(SolidusSubscriptions::Installment.actionable.arel.constraints.reduce(:and))
-          ).
-          distinct.
-          find_in_batches
-      end
-    end
-
-    # @return [Array<Spree.user_class>]
-    attr_reader :users
-
-    # Get a new instance of the SolidusSubscriptions::Processor
-    #
-    # @param users [Array<Spree.user_class>] A list of users with actionable
-    #   subscriptions or installments
-    #
-    # @return [SolidusSubscriptions::Processor]
-    def initialize(users)
-      @users = users
-      @installments = {}
-    end
-
-    # Create `ProcessInstallmentsJob`s for the users used to initalize the
-    # instance
-    def build_jobs
-      users.map do |user|
-        installemts_by_address_and_user = installments(user).group_by do |i|
-          i.subscription.shipping_address_id
-        end
-
-        installemts_by_address_and_user.values.each do |grouped_installments|
-          ProcessInstallmentsJob.perform_later grouped_installments.map(&:id)
-        end
-      end
-    end
-
-    private
-
-    def subscriptions_by_id
-      @subscriptions_by_id ||= Subscription.
-        actionable.
-        includes(:line_items, :user).
-        where(user_id: user_ids).
-        group_by(&:user_id)
-    end
-
-    def retry_installments
-      @failed_installments ||= Installment.
-        actionable.
-        includes(:subscription).
-        where(solidus_subscriptions_subscriptions: { user_id: user_ids }).
-        group_by { |i| i.subscription.user_id }
-    end
-
-    def installments(user)
-      @installments[user.id] ||= retry_installments.fetch(user.id, []) + new_installments(user)
-    end
-
-    def new_installments(user)
-      ActiveRecord::Base.transaction do
-        subscriptions_by_id.fetch(user.id, []).map do |sub|
-          sub.successive_skip_count = 0
-          sub.advance_actionable_date
-          sub.cancel! if sub.pending_cancellation?
-          sub.deactivate! if sub.can_be_deactivated?
-          sub.installments.create!
-        end
-      end
-    end
-
-    def user_ids
-      @user_ids ||= users.map(&:id)
-    end
-  end
-end
+# This class is responsible for finding subscriptions and installments
+# which need to be processed. It will group them together by user and attempts
+# to process them together. Subscriptions will also be grouped by their
+# shiping address id.
+#
+# This class passes the reponsibility of actually creating the order off onto
+# the consolidated installment class.
+#
+# This class generates `ProcessInstallmentsJob`s
+module SolidusSubscriptions
+  class Processor
+    class << self
+      # Find all actionable subscriptions and intallments, group them together
+      # by user, and schedule a processing job for the group as a batch
+      def run
+        batched_users_to_be_processed.each { |batch| new(batch).build_jobs }
+      end
+
+      private
+
+      def batched_users_to_be_processed
+        subscriptions = SolidusSubscriptions::Subscription.arel_table
+        installments = SolidusSubscriptions::Installment.arel_table
+
+        Spree::User.
+          joins(:subscriptions).
+          joins(
+            subscriptions.
+              join(installments, Arel::Nodes::OuterJoin).
+              on(subscriptions[:id].eq(installments[:subscription_id])).
+              join_sources
+          ).
+          where(
+            SolidusSubscriptions::Subscription.actionable.arel.constraints.reduce(:and).
+              or(SolidusSubscriptions::Installment.actionable.arel.constraints.reduce(:and))
+          ).
+          distinct.
+          find_in_batches
+      end
+    end
+
+    # @return [Array<Spree.user_class>]
+    attr_reader :users
+
+    # Get a new instance of the SolidusSubscriptions::Processor
+    #
+    # @param users [Array<Spree.user_class>] A list of users with actionable
+    #   subscriptions or installments
+    #
+    # @return [SolidusSubscriptions::Processor]
+    def initialize(users)
+      @users = users
+      @installments = {}
+    end
+
+    # Create `ProcessInstallmentsJob`s for the users used to initalize the
+    # instance
+    def build_jobs
+      users.map do |user|
+        installemts_by_address_and_user = installments(user).group_by do |i|
+          i.subscription.shipping_address_id
+        end
+
+        installemts_by_address_and_user.values.each do |grouped_installments|
+          ProcessInstallmentsJob.perform_later grouped_installments.map(&:id)
+        end
+      end
+    end
+
+    private
+
+    def subscriptions_by_id
+      @subscriptions_by_id ||= Subscription.
+        actionable.
+        includes(:line_items, :user).
+        where(user_id: user_ids).
+        group_by(&:user_id)
+    end
+
+    def retry_installments
+      @failed_installments ||= Installment.
+        actionable.
+        includes(:subscription).
+        where(solidus_subscriptions_subscriptions: { user_id: user_ids }).
+        group_by { |i| i.subscription.user_id }
+    end
+
+    def installments(user)
+      @installments[user.id] ||= retry_installments.fetch(user.id, []) + new_installments(user)
+    end
+
+    def new_installments(user)
+      ActiveRecord::Base.transaction do
+        subscriptions_by_id.fetch(user.id, []).map do |sub|
+          sub.successive_skip_count = 0
+          sub.advance_actionable_date
+          sub.cancel! if sub.pending_cancellation?
+          sub.deactivate! if sub.can_be_deactivated?
+          sub.installments.create!
+        end
+      end
+    end
+
+    def user_ids
+      @user_ids ||= users.map(&:id)
+    end
+  end
+end

data/lib/solidus_subscriptions/testing_support/factories.rb

@@ -1,5 +1,5 @@
-require 'factory_girl'
-require 'spree/testing_support/factories'
-
-factory_path = "#{File.dirname(__FILE__)}/factories/**/*_factory.rb"
-Dir[factory_path].each { |f| require File.expand_path(f) }
+require 'factory_girl'
+require 'spree/testing_support/factories'
+
+factory_path = "#{File.dirname(__FILE__)}/factories/**/*_factory.rb"
+Dir[factory_path].each { |f| require File.expand_path(f) }

data/lib/solidus_subscriptions/testing_support/factories/installment_detail_factory.rb

@@ -1,7 +1,7 @@
-FactoryGirl.define do
-  factory :installment_detail, class: 'SolidusSubscriptions::InstallmentDetail' do
-    installment
-
-    trait(:success) { success true }
-  end
-end
+FactoryGirl.define do
+  factory :installment_detail, class: 'SolidusSubscriptions::InstallmentDetail' do
+    installment
+
+    trait(:success) { success true }
+  end
+end

data/lib/solidus_subscriptions/testing_support/factories/installment_factory.rb

@@ -1,21 +1,21 @@
-FactoryGirl.define do
-  factory :installment, class: 'SolidusSubscriptions::Installment' do
-    transient { subscription_traits [] }
-    subscription { build :subscription, :with_line_item, *subscription_traits }
-
-    trait :failed do
-      actionable_date { Time.zone.yesterday }
-      details { build_list(:installment_detail, 1, installment: @instance) }
-    end
-
-    trait :success do
-      transient do
-        order { create :completed_order_with_totals }
-      end
-
-      details do
-        build_list(:installment_detail, 1, :success, installment: @instance, order: order)
-      end
-    end
-  end
-end
+FactoryGirl.define do
+  factory :installment, class: 'SolidusSubscriptions::Installment' do
+    transient { subscription_traits [] }
+    subscription { build :subscription, :with_line_item, *subscription_traits }
+
+    trait :failed do
+      actionable_date { Time.zone.yesterday }
+      details { build_list(:installment_detail, 1, installment: @instance) }
+    end
+
+    trait :success do
+      transient do
+        order { create :completed_order_with_totals }
+      end
+
+      details do
+        build_list(:installment_detail, 1, :success, installment: @instance, order: order)
+      end
+    end
+  end
+end