solidus_subscriptions 1.0.0.rc1

data/solidus_subscriptions.gemspec

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require_relative 'lib/solidus_subscriptions/version'
+
+Gem::Specification.new do |spec|
+  spec.name = 'solidus_subscriptions'
+  spec.version = SolidusSubscriptions::VERSION
+  spec.authors = ['Solidus Team']
+  spec.email = 'contact@solidus.io'
+
+  spec.summary = 'Add subscription support to Solidus'
+  spec.description = 'Add subscription support to Solidus'
+  spec.homepage = 'https://github.com/solidusio-contrib/solidus_subscriptions'
+  spec.license = 'BSD-3-Clause'
+
+  spec.metadata['homepage_uri'] = spec.homepage
+  spec.metadata['source_code_uri'] = 'https://github.com/solidusio-contrib/solidus_subscriptions'
+  spec.metadata['changelog_uri'] = 'https://github.com/solidusio-contrib/solidus_subscriptions/blob/master/CHANGELOG.md'
+
+  spec.required_ruby_version = Gem::Requirement.new('~> 2.5')
+
+  # Specify which files should be added to the gem when it is released.
+  # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
+  files = Dir.chdir(__dir__) { `git ls-files -z`.split("\x0") }
+
+  spec.files = files.grep_v(%r{^(test|spec|features)/})
+  spec.test_files = files.grep(%r{^(test|spec|features)/})
+  spec.bindir = "exe"
+  spec.executables = files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.require_paths = ["lib"]
+
+  spec.add_dependency 'deface'
+  spec.add_dependency 'httparty', '~> 0.18'
+  spec.add_dependency 'i18n'
+  spec.add_dependency 'solidus_core', ['>= 2.0.0', '< 3']
+  spec.add_dependency 'solidus_support', '~> 0.7'
+  spec.add_dependency 'state_machines'
+
+  spec.add_development_dependency 'rspec-activemodel-mocks'
+  spec.add_development_dependency 'shoulda-matchers', '~> 4.4'
+  spec.add_development_dependency 'solidus_dev_support', '~> 2.0'
+  spec.add_development_dependency 'timecop'
+  spec.add_development_dependency 'vcr'
+  spec.add_development_dependency 'versioncake'
+  spec.add_development_dependency 'webmock'
+  spec.add_development_dependency 'yard'
+end

data/spec/controllers/spree/admin/subscriptions_controller_spec.rb

@@ -0,0 +1,202 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Spree::Admin::SubscriptionsController, type: :request do
+  extend Spree::TestingSupport::AuthorizationHelpers::Request
+  stub_authorization!
+
+  before do
+    ActionController::Base.allow_forgery_protection = false
+  end
+
+  after do
+    ActionController::Base.allow_forgery_protection = true
+  end
+
+  describe 'get /admin/subscriptions' do
+    subject do
+      get spree.admin_subscriptions_path
+      response
+    end
+
+    it { is_expected.to be_successful }
+  end
+
+  describe 'GET :new' do
+    subject do
+      get spree.new_admin_subscription_path
+      response
+    end
+
+    it { is_expected.to be_successful }
+  end
+
+  describe 'GET :edit' do
+    subject do
+      get spree.edit_admin_subscription_path(subscription)
+      response
+    end
+
+    let(:subscription) { create :subscription, :actionable }
+
+    it { is_expected.to be_successful }
+  end
+
+  describe 'PUT :update' do
+    it 'redirects to edit subscription page' do
+      subscription = create :subscription
+      subscription_params = { subscription: { interval_length: 1 } }
+
+      expect(put(spree.admin_subscription_path(subscription), params: subscription_params)).
+        to redirect_to spree.edit_admin_subscription_path(subscription)
+    end
+
+    it 'updates the subscription attributes', :aggregate_failures do
+      expected_date = DateTime.parse('2001/11/12')
+      subscription = create :subscription, :actionable
+      subscription_params = { subscription: { actionable_date: expected_date } }
+
+      expect { put spree.admin_subscription_path(subscription), params: subscription_params }.
+        to change { subscription.reload.actionable_date }.
+        to expected_date
+    end
+
+    it 'does not duplicate line items' do
+      variant = create :variant, subscribable: true
+      subscription = create :subscription
+      subscription_params = {
+        subscription: {
+          line_items_attributes: [
+            { subscribable_id: variant.id, quantity: 1 }
+          ]
+        }
+      }
+
+      expect { put spree.admin_subscription_path(subscription), params: subscription_params }.
+        to change { subscription.reload.line_items.count }.
+        by 1
+    end
+
+    context 'when updating the payment method' do
+      it 'updates the subscription payment method' do
+        check_payment_method = create :check_payment_method
+        subscription = create :subscription
+        subscription_params = { subscription: { payment_method_id: check_payment_method.id } }
+
+        put spree.admin_subscription_path(subscription), params: subscription_params
+
+        expect(subscription.reload).to have_attributes(
+          payment_method: check_payment_method,
+          payment_source: nil,
+        )
+      end
+
+      it 'updates the subscription payment source if payment method requires source' do
+        payment = create :credit_card_payment
+        payment_source = payment.source
+        payment_method = payment.payment_method
+
+        subscription = create :subscription, user: payment.order.user
+        subscription_params = {
+          subscription: {
+            payment_method_id: payment_method.id,
+            payment_source_id: payment_source.id,
+          }
+        }
+
+        put spree.admin_subscription_path(subscription), params: subscription_params
+
+        expect(subscription.reload.payment_source).to eq(payment_source)
+      end
+    end
+  end
+
+  describe 'POST cancel' do
+    subject(:delete_subscription) { delete spree.cancel_admin_subscription_path(subscription) }
+
+    context 'when the subscription can be canceled' do
+      let(:subscription) { create :subscription, :actionable }
+
+      it { is_expected.to redirect_to spree.admin_subscriptions_path }
+
+      it 'has a message' do
+        delete_subscription
+        expect(flash[:notice]).to be_present
+      end
+
+      it 'cancels the subscription' do
+        expect { delete_subscription }.to change { subscription.reload.state }.from('active').to('canceled')
+      end
+    end
+
+    context 'when the subscription cannot be canceled' do
+      let(:subscription) { create :subscription, :canceled }
+
+      it { is_expected.to redirect_to spree.admin_subscriptions_path }
+
+      it 'has a message' do
+        delete_subscription
+        expect(flash[:notice]).to be_present
+      end
+
+      it 'cancels the subscription' do
+        expect { delete_subscription }.not_to(change { subscription.reload.state })
+      end
+    end
+  end
+
+  describe 'POST activate' do
+    subject(:activate) { post spree.activate_admin_subscription_path(subscription) }
+
+    context 'when the subscription can be activated' do
+      let(:subscription) { create :subscription, :canceled, :with_line_item }
+
+      it { is_expected.to redirect_to spree.admin_subscriptions_path }
+
+      it 'has a message' do
+        activate
+        expect(flash[:notice]).to be_present
+      end
+
+      it 'cancels the subscription' do
+        expect { activate }.to change { subscription.reload.state }.from('canceled').to('active')
+      end
+    end
+
+    context 'when the subscription cannot be activated' do
+      let(:subscription) { create :subscription, :actionable, :with_line_item }
+
+      it { is_expected.to redirect_to spree.admin_subscriptions_path }
+
+      it 'has a message' do
+        activate
+        expect(flash[:notice]).to be_present
+      end
+
+      it 'cancels the subscription' do
+        expect { activate }.not_to(change{ subscription.reload.state })
+      end
+    end
+  end
+
+  describe 'POST skip' do
+    subject(:skip) { post spree.skip_admin_subscription_path(subscription) }
+
+    let(:subscription) { create :subscription, :actionable, :with_line_item }
+    let!(:expected_date) { subscription.next_actionable_date }
+
+    it { is_expected.to redirect_to spree.admin_subscriptions_path }
+
+    it 'has a message' do
+      skip
+      expect(flash[:notice]).to be_present
+    end
+
+    it 'advances the actionable_date' do
+      expect { skip }.
+        to change { subscription.reload.actionable_date }.
+        from(subscription.actionable_date).to(expected_date)
+    end
+  end
+end

data/spec/controllers/spree/api/line_items_controller_spec.rb

@@ -0,0 +1,103 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'spree/api/testing_support/helpers'
+
+RSpec.describe Spree::Api::LineItemsController, type: :controller do
+  include Spree::Api::TestingSupport::Helpers
+  routes { Spree::Core::Engine.routes }
+
+  describe 'POST :create' do
+    subject(:post_create) { post :create, params: params }
+
+    let(:params) { line_item_params }
+    let!(:variant) { create :variant }
+    let!(:order) { create :order }
+
+    let(:line_item_params) do
+      {
+        order_id: order.number,
+        order_token: order.guest_token,
+        format: 'json',
+        line_item: {
+          quantity: 1,
+          variant_id: variant.id
+        }
+      }
+    end
+
+    shared_examples 'a new line item' do
+      it { is_expected.to be_created }
+
+      it 'creates a line item' do
+        expect { post_create }.
+          to change { Spree::LineItem.count }.
+          from(0).to(1)
+      end
+    end
+
+    context 'with subscription_line_item params' do
+      let(:params) { line_item_params.merge(subscription_line_item_params) }
+      let(:subscription_line_item_params) do
+        {
+          subscription_line_item: {
+            quantity: 2,
+            end_date: '1990/10/12',
+            subscribable_id: variant.id,
+            interval_length: 30,
+            interval_units: "day"
+          }
+        }
+      end
+
+      it_behaves_like 'a new line item'
+
+      it 'creates a new subscription line item' do
+        expect { post_create }.
+          to change { SolidusSubscriptions::LineItem.count }.
+          from(0).to(1)
+      end
+    end
+
+    context 'without subscription_line_item params' do
+      it_behaves_like 'a new line item'
+    end
+  end
+
+  describe 'patch :update' do
+    subject(:patch_create) { patch :create, params: params }
+
+    let(:params) { line_item_params }
+    let!(:variant) { create :variant }
+    let!(:order) { create :order }
+    let!(:line_item) { create :line_item, order: order, variant: variant }
+
+    let(:line_item_params) do
+      {
+        id: line_item.id,
+        order_id: order.number,
+        order_token: order.guest_token,
+        format: 'json',
+        line_item: {
+          quantity: 1,
+          variant_id: variant.id
+        },
+        subscription_line_item: {
+          quantity: 2,
+          end_date: '1990/10/12',
+          subscribable_id: variant.id,
+          interval_length: 30,
+          interval_units: "day"
+        }
+      }
+    end
+
+    it { is_expected.to be_successful }
+
+    it 'creates a new subscription line item' do
+      expect { patch_create }.
+        to change { SolidusSubscriptions::LineItem.count }.
+        from(0).to(1)
+    end
+  end
+end

data/spec/controllers/spree/api/orders_controller_spec.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'spree/api/testing_support/helpers'
+
+RSpec.describe Spree::Api::OrdersController, type: :controller do
+  include Spree::Api::TestingSupport::Helpers
+  routes { Spree::Core::Engine.routes }
+
+  let(:order) { create :order }
+  let(:variant) { create :variant }
+
+  describe 'patch /update' do
+    subject(:subscription_line_items) do
+      patch :update, params: params
+      order.subscription_line_items.reload
+    end
+
+    before { stub_authentication! }
+
+    let(:params) do
+      {
+        order: { line_items_attributes: [line_items_params] },
+        id: order.to_param,
+        format: 'json',
+        order_token: order.guest_token
+      }
+    end
+
+    let(:line_items_params) do
+      {
+        variant_id: variant.id,
+        quantity: 1,
+        subscription_line_items_attributes: [subscription_line_items_params]
+      }
+    end
+
+    let(:subscription_line_items_params) do
+      {
+        quantity: 1,
+        subscribable_id: variant.id,
+        interval_length: 30,
+        interval_units: "day"
+      }
+    end
+
+    it 'is a successful response' do
+      subscription_line_items
+      expect(response).to be_successful
+    end
+
+    it 'create the correct number of subscription line items' do
+      expect(subscription_line_items.length).
+        to eq line_items_params[:subscription_line_items_attributes].length
+    end
+  end
+end

data/spec/controllers/spree/api/users_controller_spec.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'spree/api/testing_support/helpers'
+
+RSpec.describe Spree::Api::UsersController, type: :controller do
+  include Spree::Api::TestingSupport::Helpers
+  routes { Spree::Core::Engine.routes }
+
+  let!(:user) do
+    create(:user, &:generate_spree_api_key).tap(&:save)
+  end
+  let!(:subscription) { create :subscription, :with_line_item, user: user }
+
+  describe 'patch /update' do
+    subject(:update_user) { patch :update, params: params }
+
+    let(:params) do
+      {
+        id: user.id,
+        token: user.spree_api_key,
+        format: 'json',
+        user: {
+          subscriptions_attributes: [{
+            id: subscription.id,
+            line_items_attributes: [line_item_attributes]
+          }]
+        }
+      }
+    end
+
+    let(:line_item_attributes) do
+      {
+        id: subscription.line_item_ids.first,
+        quantity: 6,
+        interval_length: 1,
+        interval_units: 'month'
+      }
+    end
+
+    it 'updates the subscription line items' do
+      update_user
+      line_item = subscription.line_items.reload.first
+
+      expect(line_item).to have_attributes(line_item_attributes)
+    end
+  end
+end