bloom_remit 0.0.1 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 329b69261f0dd3ce1c80717973da1c342393f21d
-  data.tar.gz: 6a1a4a41a31bbe5186508b63c9cfaeef30aa4046
+  metadata.gz: 66d45aad8f97251312abcdf0da5ecb2e90365aae
+  data.tar.gz: 21a83f8b40228a62e0055e749dad4ab5416ee4ca
 SHA512:
-  metadata.gz: 9a789f5dd11c5966934e18b8135a7d947a5a4d7b53ff28d8b50c744de0ee600dd1c4516f31b4dc09d92c38a6636de9fa86d78db4a59d45581ba0e419c694f961
-  data.tar.gz: 82ac77e0ebf73065012728771226e532a8976835e7d651181a29d615e77004f5b2adff9f15eb9c81479fef25e7b8e14dbcc0a64dda6abefd8847f7e184251260
+  metadata.gz: 4e1397e6e12435c937e28d50d786798210c093de3873fa2c511c516106d13ee806dcad6cd4492466a8debb1a4247e87888ac6df1b161e7da7deb3096de50195f
+  data.tar.gz: c2fd6eb776e1ca1aefc521b1fc37ad9b2c78642c4b947aae1fcf6d090b8f467ee1f7bad213119b5eab9195613fbd370bc27a424c01e109f87f3733e9d7a2021b

data/app/concepts/bloom_remit/txns/contracts/create.rb

@@ -3,13 +3,15 @@ module BloomRemit
     module Contracts
       class Create < Reform::Form
 
-        property :recipient_type
-        property :recipient_id
+        model ::BloomRemit::Txn
         property :sender_type
         property :sender_id
         property :amount
         property :account_name
         property :account_id
+        property :target_slug
+        property :owner_id
+        property :owner_type
 
       end
     end

data/app/concepts/bloom_remit/txns/contracts/update.rb

@@ -3,6 +3,7 @@ module BloomRemit
     module Contracts
       class Update < Reform::Form
 
+        model ::BloomRemit::Txn
         property :status
 
       end

data/app/concepts/bloom_remit/txns/operations/create.rb

@@ -9,7 +9,9 @@ module BloomRemit
         contract Contracts::Create
 
         include Dispatch
-        callback :after_create, Callbacks::AfterCreate
+        callback :after_create do
+          on_change :after_create
+        end
 
         def process(params)
           validate(params[:txn]) do |f|
@@ -19,6 +21,14 @@ module BloomRemit
           end
         end
 
+        private
+
+        def after_create(form, opts)
+          ActiveRecord::Base.after_transaction do
+            PayoutJob.perform_async(form.model.id)
+          end
+        end
+
       end
     end
   end

data/app/concepts/bloom_remit/txns/operations/update.rb

@@ -8,18 +8,39 @@ module BloomRemit
 
         contract Contracts::Update
 
+        include Callback
+        callback :after_update do
+          on_change :after_update
+        end
+
         def process(params)
           validate(params[:txn]) do |f|
-            f.save
+            if f.save
+              dispatch! :after_update
+            end
           end
         end
 
         private
 
+        def setup_params!(params)
+          params[:txn] ||= {}
+          params[:txn].merge!(
+            status: params[:status],
+            secret: params[:secret],
+          )
+        end
+
         def model!(params)
           Txn.find_by!(id: params[:id], secret: params[:secret])
         end
 
+        def after_update(form, opts)
+          ActiveRecord::Base.after_transaction do
+            OnUpdateJob.perform_async(form.model.id, opts[:params][:txn])
+          end
+        end
+
       end
     end
   end

data/app/jobs/bloom_remit/txns/on_update_job.rb

@@ -0,0 +1,15 @@
+module BloomRemit
+  module Txns
+    class OnUpdateJob
+
+      include Sidekiq::Worker
+      sidekiq_options retry: false
+
+      def perform(txn_id, params)
+        txn = Txn.find(txn_id)
+        OnUpdate.(txn, params)
+      end
+
+    end
+  end
+end

data/app/jobs/bloom_remit/txns/payout_job.rb

@@ -6,7 +6,7 @@ module BloomRemit
       sidekiq_options retry: false
 
       def perform(txn_id)
-        txn = Txn.find(txn_id)
+        txn = ::BloomRemit::Txn.find(txn_id)
         Payout.(txn)
       end
 

data/app/models/bloom_remit/response.rb

@@ -0,0 +1,9 @@
+module BloomRemit
+  class Response < ActiveRecord::Base
+
+    store :body, coder: JSON
+    belongs_to :txn, class_name: "BloomRemit::Txn"
+    validates :body, presence: true
+
+  end
+end

data/app/models/bloom_remit/txn.rb

@@ -1,8 +1,9 @@
 module BloomRemit
   class Txn < ActiveRecord::Base
 
-    belongs_to :recipient, polymorphic: true
     belongs_to :sender, polymorphic: true
+    belongs_to :owner, polymorphic: true
+    has_many :responses, class_name: "BloomRemit::Response", dependent: :destroy
 
     enum status: {
       bank_error: -2,

data/app/services/bloom_remit/create_txn.rb

@@ -1,15 +1,16 @@
 module BloomRemit
   class CreateTxn
 
-    def self.call(recipient:, sender:, amount:, account_name:, account_id:)
+    def self.call(target_slug:, sender:, owner:, amount:, account_name:, account_id:)
       Txns::Operations::Create.(txn: {
-        recipient_id: recipient.id,
-        recipient_type: recipient.class.name,
         sender_id: sender.id,
-        sender_type: sender.class.name,
+        sender_type: sender.class.base_class.name,
         amount: amount,
         account_name: account_name,
         account_id: account_id,
+        target_slug: target_slug,
+        owner_id: owner.id,
+        owner_type: owner.class.base_class.name,
       })
     end
 

data/app/services/bloom_remit/txns/handle_response.rb

@@ -0,0 +1,16 @@
+module BloomRemit
+  module Txns
+    class HandleResponse
+
+      extend LightService::Action
+      expects :txn, :remote_response
+      promises :response
+
+      executed do |c|
+        body = c.remote_response.body
+        c.response = c.txn.responses.create!(body: body)
+      end
+
+    end
+  end
+end

data/app/services/bloom_remit/txns/on_update.rb

@@ -0,0 +1,12 @@
+module BloomRemit
+  module Txns
+    class OnUpdate
+
+      def self.call(txn, params)
+        return if BloomRemit.on_txn_response.nil?
+        BloomRemit.on_txn_response.(txn, params)
+      end
+
+    end
+  end
+end

data/app/services/bloom_remit/txns/payout.rb

@@ -8,6 +8,8 @@ module BloomRemit
         with(txn: txn).reduce(
           InitializeClient,
           Post,
+          HandleResponse,
+          RunHook,
         )
       end
 

data/app/services/bloom_remit/txns/post.rb

@@ -4,18 +4,22 @@ module BloomRemit
 
       extend LightService::Action
       expects :client, :txn
+      promises :remote_response
 
       executed do |c|
         txn = c.txn
+        callback_url =
+          Engine.routes.url_helpers.api_v1_txn_url(txn, secret: txn.secret)
 
-        c.client.create_payment(
+        c.remote_response = c.client.create_payment(
           sender_id: txn.sender.send(BloomRemit.sender_id_method),
           account_name: txn.account_name,
           account_number: txn.account_id,
-          callback_url: Engine.routes.url_helpers.api_v1_txn_url(txn),
+          callback_url: callback_url,
           dest_currency: "PHP",
           orig_currency: "PHP",
-          payout_method: txn.recipient.slug,
+          payout_method: txn.target_slug,
+          paid_in_orig_currency: txn.amount,
           receivable_in_dest_currency: txn.amount,
         )
       end

data/app/services/bloom_remit/txns/run_hook.rb

@@ -0,0 +1,14 @@
+module BloomRemit
+  module Txns
+    class RunHook
+
+      extend LightService::Action
+      expects :txn, :response
+
+      executed do |c|
+        OnUpdate.(c.txn, c.response.body)
+      end
+
+    end
+  end
+end

data/db/migrate/20160805004223_drop_recipient_from_txns.rb

@@ -0,0 +1,6 @@
+class DropRecipientFromTxns < ActiveRecord::Migration
+  def change
+    remove_column :bloom_remit_txns, :recipient_id
+    remove_column :bloom_remit_txns, :recipient_type
+  end
+end

data/db/migrate/20160805004350_add_target_slug_to_bloom_remit_txns.rb

@@ -0,0 +1,5 @@
+class AddTargetSlugToBloomRemitTxns < ActiveRecord::Migration
+  def change
+    add_column :bloom_remit_txns, :target_slug, :string, null: false
+  end
+end

data/db/migrate/20160805040950_add_owner_to_bloom_remit_txns.rb

@@ -0,0 +1,7 @@
+class AddOwnerToBloomRemitTxns < ActiveRecord::Migration
+  def change
+    add_column :bloom_remit_txns, :owner_id, :string
+    add_column :bloom_remit_txns, :owner_type, :string
+    add_index :bloom_remit_txns, %i[owner_id owner_type]
+  end
+end

data/db/migrate/20160805095308_change_txns_sender_id_to_string.rb

@@ -0,0 +1,5 @@
+class ChangeTxnsSenderIdToString < ActiveRecord::Migration
+  def change
+    change_column :bloom_remit_txns, :sender_id, :string
+  end
+end

data/db/migrate/20160806054012_create_bloom_remit_responses.rb

@@ -0,0 +1,9 @@
+class CreateBloomRemitResponses < ActiveRecord::Migration
+  def change
+    create_table :bloom_remit_responses do |t|
+      t.integer :txn_id, null: false, index: true
+      t.text :body, null: false
+      t.timestamps null: false
+    end
+  end
+end

data/lib/bloom_remit/factories/response.rb

@@ -0,0 +1,7 @@
+FactoryGirl.define do
+
+  factory :bloom_remit_response, class: BloomRemit::Response do
+    association :txn, factory: :bloom_remit_txn
+  end
+
+end

data/lib/bloom_remit/factories/txn.rb

@@ -0,0 +1,9 @@
+FactoryGirl.define do
+
+  factory :bloom_remit_txn, class: BloomRemit::Txn do
+    target_slug "TARGET_SLUG"
+    amount 2500
+    secret { SecureRandom.uuid }
+  end
+
+end

data/lib/bloom_remit/factories.rb

@@ -0,0 +1,2 @@
+Dir[File.expand_path("../factories/*.rb", __FILE__)].each {|f| require f}
+require 'bloom_remit_client/factories'

data/lib/bloom_remit/version.rb

@@ -1,3 +1,3 @@
 module BloomRemit
-  VERSION = "0.0.1"
+  VERSION = "0.1.0"
 end

data/lib/bloom_remit.rb

@@ -1,3 +1,4 @@
+require "ar_after_transaction"
 require "storext"
 require "sidekiq"
 require "light-service"
@@ -21,6 +22,7 @@ module BloomRemit
   mattr_accessor :agent_id
   mattr_accessor :url
   mattr_reader :sender_id_method_default
+  mattr_accessor :on_txn_response
   @@sender_id_method_default = :bloom_remit_id
   mattr_accessor :sender_id_method
   self.sender_id_method = self.sender_id_method_default

data/spec/bloom_remit_spec.rb

@@ -0,0 +1,29 @@
+require 'rails_helper'
+
+RSpec.describe BloomRemit do
+
+  it "is configurable" do
+    BloomRemit.configure do |c|
+      c.token = "token"
+      c.secret = "secret"
+      c.url = "https://url.com"
+      c.agent_id = "agent_id"
+      c.on_txn_response = OnTxnResponse
+    end
+
+    expect(described_class.token).to eq "token"
+    expect(described_class.secret).to eq "secret"
+    expect(described_class.url).to eq "https://url.com"
+    expect(described_class.agent_id).to eq "agent_id"
+    expect(described_class.sender_id_method).to eq :bloom_remit_id
+    expect(described_class.on_txn_response).to eq OnTxnResponse
+  end
+
+  describe ".sender_id_method" do
+    it "is configuratble" do
+      described_class.configure {|c| c.sender_id_method = "asd"}
+      expect(described_class.sender_id_method).to eq "asd"
+    end
+  end
+
+end

data/spec/concepts/bloom_remit/txns/operations/create_spec.rb

@@ -0,0 +1,36 @@
+require 'rails_helper'
+
+module BloomRemit
+  module Txns
+    module Operations
+      RSpec.describe Create, cleaning_strategy: :truncation do
+
+        let(:user) { create(:bloom_remit_dummy_sub_user) }
+        let(:biller) { create(:bloom_remit_dummy_biller, slug: "PLDT") }
+        let(:payment) { create(:bloom_remit_dummy_sub_payment) }
+
+        it "creates a txn with a secret and enqueues the PayoutJob" do
+          op = described_class.(txn: {
+            target_slug: biller.slug,
+            sender_id: user.id,
+            sender_type: user.class.base_class.name,
+            amount: 800.0,
+            account_name: "028109090",
+            account_id: "Hooli X",
+            owner_id: payment.id,
+            owner_type: payment.class.base_class.name,
+          })
+          txn = op.model
+          expect(txn.target_slug).to eq biller.slug
+          expect(txn.sender).to eq user
+          expect(txn.amount).to eq 800.0
+          expect(txn.account_name).to eq "028109090"
+          expect(txn.account_id).to eq "Hooli X"
+          expect(txn.owner).to eq payment
+          expect(PayoutJob).to have_enqueued_job(txn.id)
+        end
+
+      end
+    end
+  end
+end

data/spec/concepts/bloom_remit/txns/operations/update_spec.rb

@@ -0,0 +1,25 @@
+require 'rails_helper'
+
+module BloomRemit
+  module Txns
+    module Operations
+      RSpec.describe Update, cleaning_strategy: :truncation do
+
+        let(:user) { create(:bloom_remit_dummy_user) }
+        let(:txn) { create(:bloom_remit_txn, sender: user) }
+
+        it "creates the txn, and triggers BloomRemit.on_txn_update" do
+          op = described_class.
+            (id: txn.id, secret: txn.secret, status: "paid")
+          txn = op.model
+          expect(txn.status).to eq "paid"
+          expect(OnUpdateJob).to have_enqueued_job(txn.id, {
+            status: "paid",
+            secret: txn.secret,
+          })
+        end
+
+      end
+    end
+  end
+end

data/spec/dummy/README.rdoc

@@ -0,0 +1,28 @@
+== README
+
+This README would normally document whatever steps are necessary to get the
+application up and running.
+
+Things you may want to cover:
+
+* Ruby version
+
+* System dependencies
+
+* Configuration
+
+* Database creation
+
+* Database initialization
+
+* How to run the test suite
+
+* Services (job queues, cache servers, search engines, etc.)
+
+* Deployment instructions
+
+* ...
+
+
+Please feel free to use a different markup language if you do not plan to run
+<tt>rake doc:app</tt>.

data/spec/dummy/Rakefile

@@ -0,0 +1,6 @@
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+
+require File.expand_path('../config/application', __FILE__)
+
+Rails.application.load_tasks

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

@@ -0,0 +1,13 @@
+// This is a manifest file that'll be compiled into application.js, which will include all the files
+// listed below.
+//
+// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
+// or any plugin's vendor/assets/javascripts directory can be referenced here using a relative path.
+//
+// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
+// compiled file.
+//
+// Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
+// about supported directives.
+//
+//= require_tree .

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

@@ -0,0 +1,15 @@
+/*
+ * This is a manifest file that'll be compiled into application.css, which will include all the files
+ * listed below.
+ *
+ * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
+ * or any plugin's vendor/assets/stylesheets directory can be referenced here using a relative path.
+ *
+ * You're free to add application-wide styles to this file and they'll appear at the bottom of the
+ * compiled file so the styles you add here take precedence over styles defined in any styles
+ * defined in the other CSS/SCSS files in this directory. It is generally better to create a new
+ * file per style scope.
+ *
+ *= require_tree .
+ *= require_self
+ */

data/spec/dummy/app/controllers/application_controller.rb

@@ -0,0 +1,5 @@
+class ApplicationController < ActionController::Base
+  # Prevent CSRF attacks by raising an exception.
+  # For APIs, you may want to use :null_session instead.
+  protect_from_forgery with: :exception
+end

data/spec/dummy/app/helpers/application_helper.rb

@@ -0,0 +1,2 @@
+module ApplicationHelper
+end

data/spec/dummy/app/models/biller.rb

@@ -0,0 +1,3 @@
+class Biller < ActiveRecord::Base
+
+end

data/spec/dummy/app/models/payment.rb

@@ -0,0 +1,3 @@
+class Payment < ActiveRecord::Base
+  
+end

data/spec/dummy/app/models/sub_payment.rb

@@ -0,0 +1,3 @@
+class SubPayment < Payment
+
+end

data/spec/dummy/app/models/sub_user.rb

@@ -0,0 +1,3 @@
+class SubUser < User
+
+end

data/spec/dummy/app/models/user.rb

@@ -0,0 +1,3 @@
+class User < ActiveRecord::Base
+
+end

data/spec/dummy/app/views/layouts/application.html.erb

@@ -0,0 +1,14 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>Dummy</title>
+  <%= stylesheet_link_tag    'application', media: 'all', 'data-turbolinks-track' => true %>
+  <%= javascript_include_tag 'application', 'data-turbolinks-track' => true %>
+  <%= csrf_meta_tags %>
+</head>
+<body>
+
+<%= yield %>
+
+</body>
+</html>

data/spec/dummy/bin/bundle

@@ -0,0 +1,3 @@
+#!/usr/bin/env ruby
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', __FILE__)
+load Gem.bin_path('bundler', 'bundle')

data/spec/dummy/bin/rails

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+APP_PATH = File.expand_path('../../config/application', __FILE__)
+require_relative '../config/boot'
+require 'rails/commands'

data/spec/dummy/bin/rake

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+require_relative '../config/boot'
+require 'rake'
+Rake.application.run

data/spec/dummy/bin/setup

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+require 'pathname'
+
+# path to your application root.
+APP_ROOT = Pathname.new File.expand_path('../../',  __FILE__)
+
+Dir.chdir APP_ROOT do
+  # This script is a starting point to setup your application.
+  # Add necessary setup steps to this file:
+
+  puts "== Installing dependencies =="
+  system "gem install bundler --conservative"
+  system "bundle check || bundle install"
+
+  # puts "\n== Copying sample files =="
+  # unless File.exist?("config/database.yml")
+  #   system "cp config/database.yml.sample config/database.yml"
+  # end
+
+  puts "\n== Preparing database =="
+  system "bin/rake db:setup"
+
+  puts "\n== Removing old logs and tempfiles =="
+  system "rm -f log/*"
+  system "rm -rf tmp/cache"
+
+  puts "\n== Restarting application server =="
+  system "touch tmp/restart.txt"
+end

data/spec/dummy/config/application.rb

@@ -0,0 +1,26 @@
+require File.expand_path('../boot', __FILE__)
+
+require 'rails/all'
+
+Bundler.require(*Rails.groups)
+require "bloom_remit"
+
+module Dummy
+  class Application < Rails::Application
+    # Settings in config/environments/* take precedence over those specified here.
+    # Application configuration should go into files in config/initializers
+    # -- all .rb files in that directory are automatically loaded.
+
+    # Set Time.zone default to the specified zone and make Active Record auto-convert to this zone.
+    # Run "rake -D time" for a list of tasks for finding time zone names. Default is UTC.
+    # config.time_zone = 'Central Time (US & Canada)'
+
+    # The default locale is :en and all translations from config/locales/*.rb,yml are auto loaded.
+    # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s]
+    # config.i18n.default_locale = :de
+
+    # Do not swallow errors in after_commit/after_rollback callbacks.
+    config.active_record.raise_in_transactional_callbacks = true
+  end
+end
+

data/spec/dummy/config/boot.rb

@@ -0,0 +1,5 @@
+# Set up gems listed in the Gemfile.
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../../../Gemfile', __FILE__)
+
+require 'bundler/setup' if File.exist?(ENV['BUNDLE_GEMFILE'])
+$LOAD_PATH.unshift File.expand_path('../../../../lib', __FILE__)