payola-payments 1.2.0 → 1.2.1

data/spec/factories/subscription_plan.rb

@@ -6,4 +6,11 @@ FactoryGirl.define do
     interval "month"
     interval_count 1
   end
+
+  factory :subscription_plan_without_interval_count do
+    sequence(:name) { |n| "Foo #{n}" }
+    sequence(:stripe_id) { |n| "foo-#{n}" }
+    amount 100
+    interval "month"
+  end
 end

data/spec/mailers/payola/admin_mailer_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+module Payola
+  describe AdminMailer do
+    let(:sale) { create(:sale) }
+
+    describe '#receipt' do
+      it "should send a receipt notification" do
+        mail = AdminMailer.receipt(sale.guid)
+        expect(mail.subject).to eq 'Receipt'
+      end
+    end
+
+    describe '#refund' do
+      it "should send a refund notification" do
+        mail = AdminMailer.refund(sale.guid)
+        expect(mail.subject).to eq 'Refund'
+      end
+    end
+
+    describe '#dispute' do
+      it "should send a dispute notification" do
+        mail = AdminMailer.dispute(sale.guid)
+        expect(mail.subject).to eq 'Dispute'
+      end
+    end
+
+    describe '#failure' do
+      it "should send a failure notification" do
+        mail = AdminMailer.failure(sale.guid)
+        expect(mail.subject).to eq 'Failure'
+      end
+    end
+  end
+end

data/spec/mailers/payola/receipt_mailer_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+module Payola
+  describe ReceiptMailer do
+    let(:sale) { create(:sale) }
+
+    describe '#receipt' do
+      it 'should send a receipt' do
+        Payola.pdf_receipt = false
+        mail = Payola::ReceiptMailer.receipt(sale.guid)
+        expect(mail.subject).to eq 'Purchase Receipt'
+      end
+
+      it 'should send a receipt with a pdf' do
+        Payola.pdf_receipt = true
+        mail = Payola::ReceiptMailer.receipt(sale.guid)
+        expect(mail.attachments["receipt-#{sale.guid}.pdf"]).to_not be nil
+      end
+
+      it "should allow product to override subject" do
+        expect_any_instance_of(Product).to receive(:receipt_subject).and_return("Override Subject")
+        mail = Payola::ReceiptMailer.receipt(sale.guid)
+        expect(mail.subject).to eq 'Override Subject'
+      end
+
+      it "should allow product to override from address" do
+        expect_any_instance_of(Product).to receive(:receipt_from_address).and_return("Override <override@example.com>")
+        mail = Payola::ReceiptMailer.receipt(sale.guid)
+        expect(mail.from.first).to eq 'override@example.com'
+      end
+    end
+
+    describe '#refund' do
+      it "should send refund email" do
+        mail = Payola::ReceiptMailer.refund(sale.guid)
+        expect(mail.subject).to eq 'Refund Confirmation'
+      end
+
+      it "should allow product to override subject" do
+        expect_any_instance_of(Product).to receive(:refund_subject).and_return("Override Subject")
+        mail = Payola::ReceiptMailer.refund(sale.guid)
+        expect(mail.subject).to eq 'Override Subject'
+      end
+
+      it "should allow product to override from address" do
+        expect_any_instance_of(Product).to receive(:refund_from_address).and_return("Override <override@example.com>")
+        mail = Payola::ReceiptMailer.refund(sale.guid)
+        expect(mail.from.first).to eq 'override@example.com'
+      end
+    end
+  end
+end

data/spec/models/payola/subscription_spec.rb

@@ -36,6 +36,9 @@ module Payola
         trial_start = subscription.trial_start
         trial_end = subscription.trial_end
 
+        now = Time.now.to_i
+        expect(stripe_sub).to receive(:canceled_at).and_return(now).at_least(1)
+
         subscription.sync_with!(stripe_sub)
 
         subscription.reload
@@ -44,6 +47,24 @@ module Payola
         expect(subscription.current_period_start).to_not eq old_start
         expect(subscription.current_period_end).to eq Time.at(stripe_sub.current_period_end)
         expect(subscription.current_period_end).to_not eq old_end
+        expect(subscription.canceled_at).to eq Time.at(now)
+      end
+
+      it "should sync non-timestamp fields" do
+        plan = create(:subscription_plan)
+        subscription = build(:subscription, plan: plan)
+        stripe_sub = Stripe::Customer.create.subscriptions.create(plan: plan.stripe_id, card: StripeMock.generate_card_token(last4: '1234', exp_year: Time.now.year + 1))
+
+        expect(stripe_sub).to receive(:quantity).and_return(10).at_least(1)
+        expect(stripe_sub).to receive(:cancel_at_period_end).and_return(true).at_least(1)
+
+        subscription.sync_with!(stripe_sub)
+
+        subscription.reload
+
+        expect(subscription.quantity).to eq 10
+        expect(subscription.stripe_status).to eq 'active'
+        expect(subscription.cancel_at_period_end).to eq true
       end
     end
   end

data/spec/payola_spec.rb

@@ -1,6 +1,12 @@
 require 'spec_helper'
 
 module Payola
+  class FakeWorker
+    def self.can_run?
+      false
+    end
+  end
+
   describe "#configure" do
     it "should pass the class back to the given block" do
       Payola.configure do |payola|
@@ -42,7 +48,6 @@ module Payola
     before do
       Payola.reset!
 
-      class FakeWorker; end
       Payola::Worker.registry ||= {}
       Payola::Worker.registry[:fake] = FakeWorker
     end
@@ -88,7 +93,6 @@ module Payola
     before do
       Payola.reset!
 
-      class FakeWorker; end
       Payola::Worker.registry ||= {}
       Payola::Worker.registry[:fake] = FakeWorker
       Payola.background_worker = :fake

data/spec/services/payola/create_plan_spec.rb

@@ -18,6 +18,15 @@ module Payola
         expect(plan.currency).to eq 'usd'
         expect(plan.trial_period_days).to eq @subscription_plan.trial_period_days
       end
+
+      it "should default interval_count" do
+        our_plan = create(:subscription_plan_without_interval_count)
+
+        expect(our_plan.respond_to?(:interval_count)).to eq false
+
+        plan = Stripe::Plan.retrieve(our_plan.stripe_id)
+        expect(plan.interval_count).to be_nil
+      end
     end
 
   end

data/spec/services/payola/process_sale_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+module Payola
+  describe ProcessSale do
+    describe "#call" do
+      it "should call process!" do
+        sale = create(:sale)
+        expect(Payola::Sale).to receive(:find_by).with(guid: sale.guid).and_return(sale)
+        expect(sale).to receive(:process!)
+
+        Payola::ProcessSale.call(sale.guid)
+      end
+    end
+  end
+end

data/spec/services/payola/process_subscription_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+module Payola
+  describe ProcessSubscription do
+    describe "#call" do
+      it "should call process!" do
+        sale = create(:sale)
+        expect(Payola::Subscription).to receive(:find_by).with(guid: sale.guid).and_return(sale)
+        expect(sale).to receive(:process!)
+
+        Payola::ProcessSubscription.call(sale.guid)
+      end
+    end
+  end
+end

data/spec/services/payola/send_mail_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+module Payola
+
+  class TestMailer < ActionMailer::Base
+    def test_mail(to, text)
+      mail(
+        to: to,
+        from: 'from@example.com',
+        body: text
+      )
+    end
+  end
+  
+  describe SendMail do
+    describe "#call" do
+      it "should send a mail" do
+        mail = double
+        expect(TestMailer).to receive(:test_mail).with('to@example.com', 'Some Text').and_return(mail)
+        expect(mail).to receive(:deliver)
+        Payola::SendMail.call('Payola::TestMailer', 'test_mail', 'to@example.com', 'Some Text')
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,3 +1,16 @@
+require 'simplecov'
+require "codeclimate-test-reporter"
+
+if ENV['CIRCLE_ARTIFACTS']
+  dir = File.join("..", "..", "..", ENV['CIRCLE_ARTIFACTS'], "coverage")
+  SimpleCov.coverage_dir(dir)
+  CodeClimate::TestReporter.start
+end
+SimpleCov.start 'rails' do
+  add_filter 'app/secrets'
+end
+
+
 # This file is copied to spec/ when you run 'rails generate rspec:install'
 ENV["RAILS_ENV"] ||= 'test'
 require File.expand_path("../dummy/config/environment", __FILE__)

data/spec/worker_spec.rb

@@ -8,6 +8,29 @@ module Payola
     end
   end
 
+  describe Worker do
+    describe "#autofind" do
+      it "should return ActiveJob if available" do
+        expect(Payola::Worker::ActiveJob).to receive(:can_run?).and_return(true)
+        expect(Payola::Worker.autofind).to eq Payola::Worker::ActiveJob
+      end
+      it "should return something else if available" do
+        expect(Payola::Worker::ActiveJob).to receive(:can_run?).and_return(false)
+        expect(Payola::Worker::Sidekiq).to receive(:can_run?).and_return(false)
+        expect(Payola::Worker::SuckerPunch).to receive(:can_run?).and_return(true)
+        expect(Payola::Worker.autofind).to eq Payola::Worker::SuckerPunch
+      end
+
+      it "should raise if nothing available" do
+        expect(Payola::Worker::ActiveJob).to receive(:can_run?).and_return(false).at_least(1)
+        expect(Payola::Worker::Sidekiq).to receive(:can_run?).and_return(false)
+        expect(Payola::Worker::SuckerPunch).to receive(:can_run?).and_return(false)
+
+        expect { Payola::Worker.autofind }.to raise_error("No eligable background worker systems found.")
+      end
+    end
+  end
+
   describe Worker::Sidekiq do
     before do
       module ::Sidekiq
@@ -32,6 +55,24 @@ module Payola
     end
   end
 
+  describe Worker::SuckerPunch do
+    describe "#can_run?" do
+      it "should return true if SuckerPunch is defined" do
+        expect(Payola::Worker::SuckerPunch.can_run?).to be_truthy
+      end
+    end
+
+    describe "#call" do
+      it "should call async" do
+        worker = double
+        expect(Payola::Worker::SuckerPunch).to receive(:new).and_return(worker)
+        expect(worker).to receive(:async).and_return(worker)
+        expect(worker).to receive(:perform)
+        Payola::Worker::SuckerPunch.call(Payola::TestService, double)
+      end
+    end
+  end
+
   describe Worker::ActiveJob do
     before do
       module ::ActiveJob
@@ -52,4 +93,18 @@ module Payola
       end
     end
   end
+
+  describe Worker::BaseWorker do
+
+    class SomeTestService
+      def self.call; end
+    end
+    
+    describe "#perform" do
+      it "should call the given service" do
+        expect(SomeTestService).to receive(:call)
+        Payola::Worker::BaseWorker.new.perform('Payola::SomeTestService')
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: payola-payments
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Pete Keen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-17 00:00:00.000000000 Z
+date: 2014-11-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -56,28 +56,28 @@ dependencies:
   name: aasm
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.4.0
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.4.0
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: stripe_event
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.3.0
 - !ruby/object:Gem::Dependency
@@ -150,6 +150,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 1.2.1
+- !ruby/object:Gem::Dependency
+  name: docverter
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: One-off and subscription payments for your Rails application
 email:
 - pete@payola.io
@@ -164,6 +178,8 @@ files:
 - app/assets/javascripts/payola/form.js
 - app/assets/javascripts/payola/subscription_form.js
 - app/assets/stylesheets/payola/application.css
+- app/controllers/concerns/payola/affiliate_behavior.rb
+- app/controllers/concerns/payola/async_behavior.rb
 - app/controllers/payola/application_controller.rb
 - app/controllers/payola/subscriptions_controller.rb
 - app/controllers/payola/transactions_controller.rb
@@ -171,6 +187,7 @@ files:
 - app/helpers/payola/price_helper.rb
 - app/mailers/payola/admin_mailer.rb
 - app/mailers/payola/receipt_mailer.rb
+- app/models/concerns/payola/guid_behavior.rb
 - app/models/concerns/payola/plan.rb
 - app/models/concerns/payola/sellable.rb
 - app/models/payola/affiliate.rb
@@ -184,6 +201,7 @@ files:
 - app/services/payola/create_plan.rb
 - app/services/payola/create_sale.rb
 - app/services/payola/create_subscription.rb
+- app/services/payola/invoice_behavior.rb
 - app/services/payola/invoice_failed.rb
 - app/services/payola/invoice_paid.rb
 - app/services/payola/process_sale.rb
@@ -260,6 +278,7 @@ files:
 - spec/dummy/app/models/owner.rb
 - spec/dummy/app/models/product.rb
 - spec/dummy/app/models/subscription_plan.rb
+- spec/dummy/app/models/subscription_plan_without_interval_count.rb
 - spec/dummy/app/views/buy/index.html.erb
 - spec/dummy/app/views/home/index.html.erb
 - spec/dummy/app/views/layouts/application.html.erb
@@ -293,6 +312,7 @@ files:
 - spec/dummy/db/migrate/20141001230848_create_products.rb
 - spec/dummy/db/migrate/20141029140518_create_owners.rb
 - spec/dummy/db/migrate/20141105010234_create_subscription_plans.rb
+- spec/dummy/db/migrate/20141120170744_create_subscription_plan_without_interval_counts.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/log/development.log
@@ -334,6 +354,8 @@ files:
 - spec/factories/sales.rb
 - spec/factories/subscription_plan.rb
 - spec/helpers/payola/price_helper_spec.rb
+- spec/mailers/payola/admin_mailer_spec.rb
+- spec/mailers/payola/receipt_mailer_spec.rb
 - spec/models/payola/sale_spec.rb
 - spec/models/payola/stripe_webhook_spec.rb
 - spec/models/payola/subscription_spec.rb
@@ -346,6 +368,9 @@ files:
 - spec/services/payola/create_subscription_spec.rb
 - spec/services/payola/invoice_failed_spec.rb
 - spec/services/payola/invoice_paid_spec.rb
+- spec/services/payola/process_sale_spec.rb
+- spec/services/payola/process_subscription_spec.rb
+- spec/services/payola/send_mail_spec.rb
 - spec/services/payola/start_subscription_spec.rb
 - spec/services/payola/sync_subscription_spec.rb
 - spec/services/payola/update_card_spec.rb
@@ -394,6 +419,7 @@ test_files:
 - spec/dummy/app/models/owner.rb
 - spec/dummy/app/models/product.rb
 - spec/dummy/app/models/subscription_plan.rb
+- spec/dummy/app/models/subscription_plan_without_interval_count.rb
 - spec/dummy/app/views/buy/index.html.erb
 - spec/dummy/app/views/home/index.html.erb
 - spec/dummy/app/views/layouts/application.html.erb
@@ -427,6 +453,7 @@ test_files:
 - spec/dummy/db/migrate/20141001230848_create_products.rb
 - spec/dummy/db/migrate/20141029140518_create_owners.rb
 - spec/dummy/db/migrate/20141105010234_create_subscription_plans.rb
+- spec/dummy/db/migrate/20141120170744_create_subscription_plan_without_interval_counts.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/log/development.log
@@ -470,6 +497,8 @@ test_files:
 - spec/factories/sales.rb
 - spec/factories/subscription_plan.rb
 - spec/helpers/payola/price_helper_spec.rb
+- spec/mailers/payola/admin_mailer_spec.rb
+- spec/mailers/payola/receipt_mailer_spec.rb
 - spec/models/payola/sale_spec.rb
 - spec/models/payola/stripe_webhook_spec.rb
 - spec/models/payola/subscription_spec.rb
@@ -482,6 +511,9 @@ test_files:
 - spec/services/payola/create_subscription_spec.rb
 - spec/services/payola/invoice_failed_spec.rb
 - spec/services/payola/invoice_paid_spec.rb
+- spec/services/payola/process_sale_spec.rb
+- spec/services/payola/process_subscription_spec.rb
+- spec/services/payola/send_mail_spec.rb
 - spec/services/payola/start_subscription_spec.rb
 - spec/services/payola/sync_subscription_spec.rb
 - spec/services/payola/update_card_spec.rb