susply 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7b3829fb3458ce9daf9f83a8155fc4c2f0410385
-  data.tar.gz: 244df39807a39516b83e48c70ecf8d502f61a2c3
+  metadata.gz: d45e6b7802160d8581a0d6a13dd282b9d04fad14
+  data.tar.gz: 2978c1e66d5bb7334cbfe7b7f052e7136056508e
 SHA512:
-  metadata.gz: f5f794f190d0a7f75efb993fed32272b4dd132f07a3ff8f2747326384e5c9aaf1208bbf9e9578f6786e83bded40c25d75a6fa1c3e1abd8df1c5996b1f038c197
-  data.tar.gz: 7958f7464dcfc9efd7a0e1b075f57a9994c6a5f5872f6d51785337f3c464911d055f4e077533c3a846d62718971ebcee2f680bfdf56a2f7283d6cf7d6438a6cc
+  metadata.gz: da42b7708ef8a48d7600717c7843906ccda69817b78fccd21101390973610fcb99f5a4cb4ab3c83a201a05196f80159ff2db7abdcac5897df070cd390b7076aa
+  data.tar.gz: 52029ee2fde8fab1b1b0deb44e2f917b5071d2c530dc245fedee9ff2232d392512bbbfcf04a6a2781cca851402b3df1a7ec426554c2373d3406b75d0ddd9ac53

data/README.rdoc

@@ -1,6 +1,7 @@
 = Susply
 
 {<img alt="Status?branch=master" src="https://codeship.com/projects/9749e880-e313-0132-4e7b-0e8782f5ebb8/status?branch=master" />}[https://codeship.com/projects/81636]
+{<img src="https://badge.fury.io/rb/susply.svg" alt="Gem Version" />}[http://badge.fury.io/rb/susply]
 {<img src="https://codeclimate.com/github/wcpaez/susply/badges/gpa.svg" />}[https://codeclimate.com/github/wcpaez/susply]
 {<img src="https://codeclimate.com/github/wcpaez/susply/badges/coverage.svg" />}[https://codeclimate.com/github/wcpaez/susply/coverage]
 

data/app/controllers/susply/application_controller.rb

@@ -1,4 +1,4 @@
 module Susply
-  class ApplicationController < ActionController::Base
+  class ApplicationController < ::ApplicationController
   end
 end

data/app/controllers/susply/change_plans_controller.rb

@@ -0,0 +1,32 @@
+require_dependency "susply/application_controller"
+
+module Susply
+  class ChangePlansController < ApplicationController
+    def create
+      plan = Susply::Plan.find(params[:plan_id])
+      if current_owner.can_change_plan?(plan)
+        Susply::ChangeSubscription.call(current_owner, plan)
+        redirect_to after_change_plan_success_path,
+          alert: t('susply.messages.success_changed_plan')
+      else
+        redirect_to after_change_plan_fail_path,
+          alert: t('susply.messages.failed_changed_plan')
+      end
+    end
+
+    private
+    def current_owner
+      @current_owner ||= send(Susply.billable_entity)
+    end
+
+    def after_change_plan_fail_path
+      return super(@current_owner) if defined?(super)
+      susply.owner_path(@current_owner)
+    end
+
+    def after_change_plan_success_path
+      return super(@current_owner) if defined?(super)
+      susply.owner_path(@current_owner)
+    end
+  end
+end

data/app/controllers/susply/renovations_controller.rb

@@ -0,0 +1,37 @@
+require_dependency "susply/application_controller"
+
+module Susply
+  class RenovationsController < ApplicationController
+    before_filter :load_owner
+
+    def create
+      subscription = Susply::RenewsSubscription.call(@owner)
+      if subscription
+        redirect_to after_renovation_success_path,
+          notice: t('susply.messages.success_renovation')
+      else
+        redirect_to after_renovation_fail_path,
+          alert: t('susply.messages.failed_renovation')
+      end
+    end
+
+    private
+    def load_owner
+      unless params[:owner_id].nil?
+        searched_owner = Susply.subscription_owner_class.constantize.
+          find(params[:owner_id])
+        @owner = searched_owner
+      end
+    end
+
+    def after_renovation_fail_path
+      return super(@owner) if defined?(super)
+      susply.owner_path(@owner)
+    end
+
+    def after_renovation_success_path
+      return super(@owner) if defined?(super)
+      susply.owner_path(@owner)
+    end
+  end
+end

data/app/models/susply/subscription.rb

@@ -19,5 +19,13 @@ module Susply
     def active?
       deactivated_at.nil?
     end
+
+    def expired?
+      current_period_end < Time.zone.now
+    end
+
+    def allowed_to_renew?
+      active? && expired?
+    end
   end
 end

data/app/services/susply/renews_subscription.rb

@@ -1,10 +1,10 @@
 module Susply
   class RenewsSubscription
     def self.call(owner)
-      if owner.has_active_subscription?
-        subscription = owner.active_subscription
-        Susply::CreatePayment.call(subscription, "plan_renovation")
-        calculate_renewed_subscription(subscription)
+      active_subscription = owner.active_subscription
+      if active_subscription && active_subscription.expired?
+        Susply::CreatePayment.call(active_subscription, "plan_renovation")
+        calculate_renewed_subscription(active_subscription)
       else
         nil
       end

data/config/locales/susply.en.yml

@@ -0,0 +1,8 @@
+en:
+  susply:
+    messages:
+      failed_renovation: 'Failed renovation'
+      success_renovation: 'Success renovation'
+      failed_changed_plan: 'Failed change plan'
+      success_changed_plan: 'Success change plan'
+

data/config/routes.rb

@@ -1,2 +1,6 @@
 Susply::Engine.routes.draw do
+  resource :change_plan, only: [:create]
+  resources Susply.owner_resource, as: :owner, only: [:show] do
+    resources :renovations, only: [:create]
+  end
 end

data/lib/generators/susply/install_generator.rb

@@ -9,12 +9,23 @@ module Susply
       @subscription_owner_model.capitalize
     end
 
+    def billable_entity
+      @subscription_owner_model.downcase
+    end
+
     def install
       template "config/initializers/susply.rb"
 
       inject_into_class "app/models/#{subscription_owner_model.downcase}.rb", 
         subscription_owner_model.downcase.camelize.constantize,
         "# Added by Susply\n  include Susply::OwnerMethods  \n\n"
+
+      copy_locales
+    end
+
+    def copy_locales
+      copy_file "../../../config/locales/susply.en.yml",
+        "config/locales/susply.en.yml"
     end
   end
 end

data/lib/generators/templates/config/initializers/susply.rb

@@ -1,3 +1,4 @@
 Susply.setup do |config|
   config.subscription_owner_class = "<%= subscription_owner_model %>"
+  config.billable_entity = "current_<%= billable_entity %>"
 end

data/lib/susply.rb

@@ -3,8 +3,14 @@ require "decorators"
 
 module Susply
   mattr_accessor :subscription_owner_class
+  mattr_accessor :billable_entity
 
   def self.setup(&block)
     yield self
   end
+
+  #ejm :organizations
+  def self.owner_resource
+    subscription_owner_class.downcase.pluralize.to_sym
+  end
 end

data/lib/susply/version.rb

@@ -1,3 +1,3 @@
 module Susply
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/spec/controllers/susply/change_plans_controller_spec.rb

@@ -0,0 +1,95 @@
+require 'spec_helper'
+
+module Susply
+  describe ChangePlansController, type: :controller do
+    let(:time) { Time.zone.now }
+
+    describe "#create" do
+      before do
+        @routes = Susply::Engine.routes
+      end
+
+      it "raises errors on not found plan" do
+        expect{
+          post :create, plan_id: 1
+        }.to raise_error(ActiveRecord::RecordNotFound)
+      end
+
+      it "raises errors when there is not a current billable entity" do
+        plan = create(:susply_plan)
+        expect{
+          post :create, plan_id: plan.id
+        }.to raise_error(NoMethodError)
+      end
+
+      context "when billable entity cannot change plan" do
+        before do
+          @owner = Organization.create
+
+          allow(@owner).to receive(:can_change_plan?).and_return(false)
+          allow_any_instance_of(ApplicationController).
+            to receive(:current_organization).and_return(@owner)
+
+          @plan = create(:susply_plan, sku: 'new_plan')
+        end
+
+        it "redirects to owner path by default" do
+          post :create, plan_id: @plan.id
+
+          expect(response).to redirect_to owner_path(@owner)
+        end
+
+        it "redirects to provided path when set" do
+          path = "/after_change_plan_fail"
+          allow_any_instance_of(ApplicationController).
+            to receive(:after_change_plan_fail_path).and_return(path)
+
+          post :create, plan_id: @plan.id
+          expect(response).to redirect_to path
+        end
+
+        it "sets the flash alert message" do
+          post :create, plan_id: @plan.id
+
+          expect(flash[:alert]).
+            to eq I18n.t('susply.messages.failed_changed_plan')
+        end
+      end
+
+      context "when billable entity change plan" do
+        before do
+          @owner = Organization.create
+          create(:susply_subscription, :active, owner: @owner)
+
+          allow(@owner).to receive(:can_change_plan?).and_return(true)
+          allow_any_instance_of(ApplicationController).
+            to receive(:current_organization).and_return(@owner)
+
+          @plan = create(:susply_plan, sku: 'new_plan')
+        end
+
+        it "redirects to owner path by default" do
+          post :create, plan_id: @plan.id
+
+          expect(response).to redirect_to owner_path(@owner)
+        end
+
+        it "redirects to provided path when set" do
+          path = "/after_change_plan_success"
+          allow_any_instance_of(ApplicationController).
+            to receive(:after_change_plan_success_path).and_return(path)
+
+          post :create, plan_id: @plan.id
+          expect(response).to redirect_to path
+        end
+
+        it "sets the flash alert message" do
+          post :create, plan_id: @plan.id
+
+          expect(flash[:alert]).
+            to eq I18n.t('susply.messages.success_changed_plan')
+        end
+      end
+    end
+  end
+end

data/spec/controllers/susply/renovations_controller_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+
+module Susply
+  describe RenovationsController, type: :controller do
+    let(:time) { Time.zone.now }
+
+    describe "#create" do
+      before do
+        @routes = Susply::Engine.routes
+      end
+
+      it "raise an exception when no owner is finded" do
+        expect{
+          post :create, owner_id: 1
+        }.to raise_error()
+      end
+
+      it "sets the owner variable to the searched one" do
+        owner = Susply.subscription_owner_class.constantize.create
+        post :create, owner_id: owner.id
+
+        expect(assigns(:owner)).to eq owner
+      end
+
+      context "success response" do
+        it "redirects to owner path by default" do
+          owner = Susply.subscription_owner_class.constantize.create
+          create(:susply_subscription, :active, owner: owner,
+                 current_period_end: time - 1.day)
+          post :create, owner_id: owner.id
+
+          expect(response).to redirect_to owner_path(owner)
+        end
+
+        it "redirects to provided path when set" do
+          path = "/new_after_success_path"
+          allow_any_instance_of(ApplicationController).
+            to receive(:after_renovation_success_path).and_return(path)
+          owner = Susply.subscription_owner_class.constantize.create
+          create(:susply_subscription, :active, owner: owner,
+                 current_period_end: time - 1.day)
+          post :create, owner_id: owner.id
+
+          expect(response).to redirect_to path
+        end
+
+        it "sets the notice alert message" do
+          owner = Susply.subscription_owner_class.constantize.create
+          create(:susply_subscription, :active, owner: owner,
+                 current_period_end: time - 1.day)
+          post :create, owner_id: owner.id
+
+          expect(flash[:notice]).
+            to eq I18n.t('susply.messages.success_renovation')
+        end
+      end
+
+      context "fail response" do
+        it "redirects to owner path by default" do
+          owner = Susply.subscription_owner_class.constantize.create
+          post :create, owner_id: owner.id
+
+          expect(response).to redirect_to owner_path(owner)
+        end
+        
+        it "redirects to provided path when set" do
+          path = "/new_after_fail_path"
+          allow_any_instance_of(ApplicationController).
+            to receive(:after_renovation_fail_path).and_return(path)
+
+          owner = Susply.subscription_owner_class.constantize.create
+          post :create, owner_id: owner.id
+
+          expect(response).to redirect_to path
+        end
+
+        it "sets the alert flash message" do
+          owner = Susply.subscription_owner_class.constantize.create
+          post :create, owner_id: owner.id
+
+          expect(flash[:alert]).
+            to eq I18n.t('susply.messages.failed_renovation')
+        end
+      end
+    end
+  end
+end

data/spec/dummy/app/assets/javascripts/companies.js

@@ -0,0 +1,2 @@
+// Place all the behaviors and hooks related to the matching controller here.
+// All this logic will automatically be available in application.js.

data/spec/dummy/app/assets/stylesheets/companies.css

@@ -0,0 +1,4 @@
+/*
+  Place all the styles related to the matching controller here.
+  They will automatically be included in application.css.
+*/

data/spec/dummy/app/assets/stylesheets/scaffold.css

@@ -0,0 +1,56 @@
+body { background-color: #fff; color: #333; }
+
+body, p, ol, ul, td {
+  font-family: verdana, arial, helvetica, sans-serif;
+  font-size:   13px;
+  line-height: 18px;
+}
+
+pre {
+  background-color: #eee;
+  padding: 10px;
+  font-size: 11px;
+}
+
+a { color: #000; }
+a:visited { color: #666; }
+a:hover { color: #fff; background-color:#000; }
+
+div.field, div.actions {
+  margin-bottom: 10px;
+}
+
+#notice {
+  color: green;
+}
+
+.field_with_errors {
+  padding: 2px;
+  background-color: red;
+  display: table;
+}
+
+#error_explanation {
+  width: 450px;
+  border: 2px solid red;
+  padding: 7px;
+  padding-bottom: 0;
+  margin-bottom: 20px;
+  background-color: #f0f0f0;
+}
+
+#error_explanation h2 {
+  text-align: left;
+  font-weight: bold;
+  padding: 5px 5px 5px 15px;
+  font-size: 12px;
+  margin: -7px;
+  margin-bottom: 0px;
+  background-color: #c00;
+  color: #fff;
+}
+
+#error_explanation ul li {
+  font-size: 12px;
+  list-style: square;
+}