money_mover 0.0.3

data/spec/lib/money_mover/dwolla/transfer_spec.rb

@@ -0,0 +1,94 @@
+require 'spec_helper'
+
+describe MoneyMover::Dwolla::Transfer do
+  let(:funding_source_token) { 'funding_source_token' }
+  let(:destination_source_token) { 'destination_source_token' }
+  let(:amount) { '12.45' }
+  let(:metadata) { { ach_tranfer_id: 123 } }
+
+  let(:dwolla_errors) { [ ['key1', 'error1'], ['key2', 'error2'] ] }
+  let(:dwolla_response) { double 'dwolla response', success?: transfer_success?, resource_location: resource_location, resource_id: resource_id, errors: dwolla_errors }
+  let(:api_url) { 'https:://api/url' }
+  let(:dwolla_client) { double 'dwolla client', post: dwolla_response, api_url: api_url}
+
+  let(:resource_location) { double 'resource location' }
+  let(:resource_id) { double 'resource id' }
+
+  let(:transfer_success?) { true }
+
+  let(:attrs) {{
+    sender_funding_source_token: funding_source_token,
+    destination_funding_source_token: destination_source_token,
+    transfer_amount: amount,
+    metadata: metadata
+  }}
+
+  subject { described_class.new(attrs) }
+
+  before do
+    allow(MoneyMover::Dwolla::AccountClient).to receive(:new) { dwolla_client }
+  end
+
+  it { should validate_presence_of(:sender_funding_source_token) }
+  it { should validate_presence_of(:destination_funding_source_token) }
+  it { should validate_presence_of(:transfer_amount) }
+
+  describe '#save' do
+    let(:transfer_success?) { true }
+
+    let(:transfer_params) {
+      {
+        _links: {
+          destination: {
+            href: "#{api_url}/funding-sources/#{destination_source_token}"
+          },
+          source: {
+            href: "#{api_url}/funding-sources/#{funding_source_token}"
+          }
+        },
+        amount: {
+          value: amount.to_s,
+          currency: "USD"
+        },
+        metadata: metadata
+      }
+    }
+
+    context 'success' do
+      it 'returns true and sets resource_location and id' do
+        expect(dwolla_client).to receive(:post).with("/transfers", transfer_params)
+
+        expect(subject.save).to eq(true)
+
+        expect(subject.errors).to be_empty
+        expect(subject.resource_location).to eq(resource_location)
+        expect(subject.id).to eq(resource_id)
+      end
+    end
+
+    context 'validations fail' do
+      let(:funding_source_token) { nil }
+      let(:destination_source_token) { nil }
+      let(:amount) { nil }
+
+      it 'does not attempt to create transfer and returns false with errors' do
+        expect(dwolla_client).to_not receive(:post)
+
+        expect(subject.save).to eq(false)
+        expect(subject.errors.full_messages).to eq(["Sender funding source token can't be blank", "Destination funding source token can't be blank", "Transfer amount can't be blank"])
+      end
+    end
+
+    context 'dwolla error' do
+      let(:transfer_success?) { false }
+
+      it 'returns false and has errors' do
+        expect(dwolla_client).to receive(:post).with("/transfers", transfer_params)
+
+        expect(subject.save).to eq(false)
+        expect(subject.errors[:key1]).to eq(['error1'])
+        expect(subject.errors[:key2]).to eq(['error2'])
+      end
+    end
+  end
+end

data/spec/lib/money_mover/dwolla/webhook_subscription_spec.rb

@@ -0,0 +1,199 @@
+require 'spec_helper'
+
+describe MoneyMover::Dwolla::WebhookSubscription do
+  let(:dwolla_client) { double 'dwolla client' }
+  let(:response_body) { nil }
+
+  before do
+    allow(MoneyMover::Dwolla::ApplicationClient).to receive(:new) { dwolla_client }
+  end
+
+  describe '.all' do
+    let(:id1) { 'tokenizedid1' }
+    let(:id2) { 'tokenizedid2' }
+    let(:response_body) { { _embedded: { :'webhook-subscriptions' => [ sub1, sub2 ] } } }
+    let(:sub1) { { id: 'sub1', url: 'url1' } }
+    let(:sub2) { { id: 'sub2', url: 'url2' } }
+
+    let(:dwolla_response) { double 'dwolla response', body: response_body }
+
+    subject { described_class }
+
+    before do
+      allow(dwolla_client).to receive(:get).with('/webhook-subscriptions') { dwolla_response }
+    end
+
+    it 'returns expected ids' do
+      subscriptions = subject.all
+
+      expect(subscriptions.length).to eq(2)
+      expect(subscriptions[0].id).to eq(sub1[:id])
+      expect(subscriptions[0].url).to eq(sub1[:url])
+      expect(subscriptions[1].id).to eq(sub2[:id])
+      expect(subscriptions[1].url).to eq(sub2[:url])
+    end
+  end
+
+  describe '#save' do
+    let(:callback_url) { double 'callback url' }
+    let(:webhook_secret_key) { double 'webhook secret key' }
+
+    let(:resource_location) { double 'resource location' }
+    let(:resource_id) { double 'resource id' }
+    let(:response_errors) { [] }
+    let(:dwolla_response) { double 'dwolla response', success?: save_success?, errors: response_errors, resource_location: resource_location, resource_id: resource_id }
+
+    subject { described_class.new(url: callback_url, secret: webhook_secret_key) }
+
+    before do
+      allow(dwolla_client).to receive(:post).with('/webhook-subscriptions', { url: callback_url, secret: webhook_secret_key }) { dwolla_response }
+    end
+
+    context 'success' do
+      let(:save_success?) { true }
+
+      it 'sets resource properties and returns true' do
+        expect(subject.save).to eq(true)
+        expect(subject.id).to eq(resource_id)
+        expect(subject.resource_location).to eq(resource_location)
+      end
+    end
+
+    context 'failure' do
+      let(:save_success?) { false }
+      let(:response_errors) { [ [:base, 'Error1'], [:base, 'Error2'] ] }
+
+      it 'returns false and has errors' do
+        expect(subject.save).to eq(false)
+        expect(subject.errors[:base]).to eq(['Error1', 'Error2'])
+      end
+    end
+  end
+
+  describe '#destroy' do
+    let(:subscription_id) { 'dwollatokenid' }
+    let(:response_errors) { [] }
+    let(:dwolla_response) { double 'dwolla response', success?: delete_success?, errors: response_errors }
+
+    subject { described_class.new(id: subscription_id) }
+
+    before do
+      allow(dwolla_client).to receive(:delete).with("/webhook-subscriptions/#{subscription_id}") { dwolla_response }
+    end
+
+    context 'success' do
+      let(:delete_success?) { true }
+
+      it 'returns true' do
+        expect(subject.destroy).to eq(true)
+      end
+    end
+
+    context 'failure' do
+      let(:delete_success?) { false }
+      let(:response_errors) { [ [:base, 'Error1'], [:base, 'Error2'] ] }
+
+      it 'returns false and has errors' do
+        expect(subject.destroy).to eq(false)
+        expect(subject.errors[:base]).to eq(['Error1', 'Error2'])
+      end
+    end
+  end
+end
+
+# INTEGRATION BLACK BOX TEST - making sure that application token is used instead of account token
+describe MoneyMover::Dwolla::WebhookSubscription do
+  describe '.all' do
+    let(:id1) { 'tokenizedid1' }
+    let(:id2) { 'tokenizedid2' }
+    let(:webhooks_response) { { _embedded: { :'webhook-subscriptions' => [ sub1, sub2 ] } } }
+    let(:sub1) { { id: 'sub1', url: 'url1' } }
+    let(:sub2) { { id: 'sub2', url: 'url2' } }
+
+    let(:dwolla_response) { double 'dwolla response', body: response_body }
+
+    subject { described_class }
+
+    before do
+      dwolla_helper.stub_get_token_request
+      dwolla_helper.stub_get_webhook_subscriptions_request(webhooks_response)
+    end
+
+    it 'returns expected ids' do
+      subscriptions = subject.all
+
+      expect(subscriptions.length).to eq(2)
+      expect(subscriptions[0].id).to eq(sub1[:id])
+      expect(subscriptions[0].url).to eq(sub1[:url])
+      expect(subscriptions[1].id).to eq(sub2[:id])
+      expect(subscriptions[1].url).to eq(sub2[:url])
+    end
+  end
+
+  describe '#save' do
+    let(:callback_url) { "https://callbackurl.com" }
+    let(:webhook_secret_key) { "some-secret-key" }
+
+    let(:create_params) {{
+      url: callback_url,
+      secret: webhook_secret_key
+    }}
+
+    let(:webhook_token) { 'some-webhook-subscription-token' }
+    let(:webhook_location) { dwolla_helper.webhook_subscription_url(webhook_token) }
+
+    subject { described_class.new(url: callback_url, secret: webhook_secret_key) }
+
+    before do
+      dwolla_helper.stub_get_token_request
+      dwolla_helper.stub_create_webhook_subscription_request(create_params, create_response)
+    end
+
+    context 'success' do
+      let(:create_response) { dwolla_helper.create_webhook_success_response(webhook_token) }
+
+      it 'sets resource properties and returns true' do
+        expect(subject.save).to eq(true)
+        expect(subject.id).to eq(webhook_token)
+        expect(subject.resource_location).to eq(webhook_location)
+      end
+    end
+
+    context 'failure' do
+      let(:create_response) { dwolla_helper.error_response({ message: 'some error' }) }
+
+      it 'returns false and has errors' do
+        expect(subject.save).to eq(false)
+        expect(subject.errors[:base]).to eq(['some error'])
+      end
+    end
+  end
+
+  describe '#destroy' do
+    let(:subscription_id) { 'some-subscription-id' }
+
+    subject { described_class.new(id: subscription_id) }
+
+    before do
+      dwolla_helper.stub_get_token_request
+      dwolla_helper.stub_delete_webhook_subscription_request(subscription_id, delete_response)
+    end
+
+    context 'success' do
+      let(:delete_response) { dwolla_helper.resource_deleted_response }
+
+      it 'returns true' do
+        expect(subject.destroy).to eq(true)
+      end
+    end
+
+    context 'failure' do
+      let(:delete_response) { dwolla_helper.error_response({ message: 'some error' }) }
+
+      it 'returns false and has errors' do
+        expect(subject.destroy).to eq(false)
+        expect(subject.errors[:base]).to eq(['some error'])
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,121 @@
+require 'shoulda-matchers'
+require 'webmock/rspec'
+require 'rack/test'
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# The generated `.rspec` file contains `--require spec_helper` which will cause
+# this file to always be loaded, without a need to explicitly require it in any
+# files.
+#
+# Given that it is always loaded, you are encouraged to keep this file as
+# light-weight as possible. Requiring heavyweight dependencies from this file
+# will add to the boot time of your test suite on EVERY test run, even for an
+# individual file that may not need all of that loaded. Instead, consider making
+# a separate helper file that requires the additional dependencies and performs
+# the additional setup, and require it from the spec files that actually need
+# it.
+#
+# The `.rspec` file also contains a few flags that are not defaults but that
+# users commonly want.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+
+require_relative '../lib/money_mover'
+
+Dir[File.expand_path('../support/**/*.rb', __FILE__)].each { |f| require f }
+
+RSpec.configure do |config|
+  # rspec-expectations config goes here. You can use an alternate
+  # assertion/expectation library such as wrong or the stdlib/minitest
+  # assertions if you prefer.
+  config.expect_with :rspec do |expectations|
+    # This option will default to `true` in RSpec 4. It makes the `description`
+    # and `failure_message` of custom matchers include text for helper methods
+    # defined using `chain`, e.g.:
+    #     be_bigger_than(2).and_smaller_than(4).description
+    #     # => "be bigger than 2 and smaller than 4"
+    # ...rather than:
+    #     # => "be bigger than 2"
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  config.include(Shoulda::Matchers::ActiveModel)
+
+  # rspec-mocks config goes here. You can use an alternate test double
+  # library (such as bogus or mocha) by changing the `mock_with` option here.
+  config.mock_with :rspec do |mocks|
+    # Prevents you from mocking or stubbing a method that does not exist on
+    # a real object. This is generally recommended, and will default to
+    # `true` in RSpec 4.
+    mocks.verify_partial_doubles = true
+  end
+
+  config.before do
+    MoneyMover::Dwolla.config_provider = TestConfigProvider.new
+  end
+
+  # This option will default to `:apply_to_host_groups` in RSpec 4 (and will
+  # have no way to turn it off -- the option exists only for backwards
+  # compatibility in RSpec 3). It causes shared context metadata to be
+  # inherited by the metadata hash of host groups and examples, rather than
+  # triggering implicit auto-inclusion in groups with matching metadata.
+  config.shared_context_metadata_behavior = :apply_to_host_groups
+
+# The settings below are suggested to provide a good initial experience
+# with RSpec, but feel free to customize to your heart's content.
+=begin
+  # This allows you to limit a spec run to individual examples or groups
+  # you care about by tagging them with `:focus` metadata. When nothing
+  # is tagged with `:focus`, all examples get run. RSpec also provides
+  # aliases for `it`, `describe`, and `context` that include `:focus`
+  # metadata: `fit`, `fdescribe` and `fcontext`, respectively.
+  config.filter_run_when_matching :focus
+
+  # Allows RSpec to persist some state between runs in order to support
+  # the `--only-failures` and `--next-failure` CLI options. We recommend
+  # you configure your source control system to ignore this file.
+  config.example_status_persistence_file_path = "spec/examples.txt"
+
+  # Limits the available syntax to the non-monkey patched syntax that is
+  # recommended. For more details, see:
+  #   - http://rspec.info/blog/2012/06/rspecs-new-expectation-syntax/
+  #   - http://www.teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
+  #   - http://rspec.info/blog/2014/05/notable-changes-in-rspec-3/#zero-monkey-patching-mode
+  config.disable_monkey_patching!
+
+  # This setting enables warnings. It's recommended, but in some cases may
+  # be too noisy due to issues in dependencies.
+  config.warnings = true
+
+  # Many RSpec users commonly either run the entire suite or an individual
+  # file, and it's useful to allow more verbose output when running an
+  # individual spec file.
+  if config.files_to_run.one?
+    # Use the documentation formatter for detailed output,
+    # unless a formatter has already been configured
+    # (e.g. via a command-line flag).
+    config.default_formatter = 'doc'
+  end
+
+  # Print the 10 slowest examples and example groups at the
+  # end of the spec run, to help surface which specs are running
+  # particularly slow.
+  config.profile_examples = 10
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = :random
+
+  # Seed global randomization in this process using the `--seed` CLI option.
+  # Setting this allows you to use `--seed` to deterministically reproduce
+  # test failures related to randomization by passing the same `--seed` value
+  # as the one that triggered the failure.
+  Kernel.srand config.seed
+=end
+end
+
+def dwolla_helper
+  @dwolla_helper ||= DwollaHelper.new
+end