coins_paid_api 1.0.5 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f347724eaccb1361f9082352da01fdcdcf6becc50ba56e98f4826cac56f92e87
-  data.tar.gz: f018f55ad92352b0d10fe4ff5358876e8df68a5505f0fc7c49f7600f4c292fb6
+  metadata.gz: ccc1bf1ca520dd1c060f67d7cafbbb031d361afa643bed0018f600ea34e4af69
+  data.tar.gz: fbf3c4991c2906cf6071c27e1896c0d97b2439392ef8a2672e5da114301af951
 SHA512:
-  metadata.gz: 37fdf652b56173c030c62a01fed2eb7fccded736e441da735bf60aa7ab0ea9a577072333f663ff4572db7775e6784f9656f2505992c4639ea8d8037e1a7ca5f4
-  data.tar.gz: 1cbc72a4c4b5c45125c84a6b825a2cebe033a36ae4307608ae1bdf201847c555eb8385040624269908ceac14947059045ae83f52ba40ff1cef28cf6ed1cd1d78
+  metadata.gz: 12a086063c8d671e6c9efbfc186e801bb3e099116adc02ee98eed280a82b675c836acf817dccf660ef772ee66d4420ebc9e9717bf1c693b971e4b1d2509e29af
+  data.tar.gz: ce88ae1135b1bd64bb809311072e899b357c31634de096a11e14ca2d39427c71b4769ba47e620a5c693f9e204ed9d4bcf210b2c98d28d32164cd7c227d2f5020

data/.gitignore

@@ -1,2 +1,3 @@
 /*.gem
 /Gemfile.lock
+spec/examples.txt

data/.rubocop.yml

@@ -0,0 +1,64 @@
+require:
+  - rubocop-performance
+  - rubocop-rails
+AllCops:
+  TargetRubyVersion: 2.7
+  NewCops: enable
+  SuggestExtensions: false
+
+Naming/MethodParameterName:
+  MinNameLength: 1
+Naming/VariableNumber:
+  Enabled: false
+
+Lint/MissingCopEnableDirective:
+  Enabled: false
+
+Layout/EmptyLineAfterMagicComment:
+  Enabled: false
+Layout/LineLength:
+  Max: 120
+Layout/MultilineOperationIndentation:
+  Enabled: false
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented_relative_to_receiver
+
+Style/AsciiComments:
+  Enabled: false
+Style/ClassAndModuleChildren:
+  Enabled: false
+Style/Documentation:
+  Enabled: false
+Style/FormatStringToken:
+  Enabled: false
+Style/Lambda:
+  Enabled: false
+Style/LambdaCall:
+  Enabled: false
+Style/ModuleFunction:
+  Enabled: false
+Style/PercentLiteralDelimiters:
+  PreferredDelimiters:
+    '%i': '()'
+    '%I': '()'
+    '%r': '{}'
+    '%w': '()'
+    '%W': '()'
+Style/SignalException:
+  EnforcedStyle: 'only_raise'
+
+Metrics/AbcSize:
+  Exclude:
+    - 'config/**/*'
+    - 'db/**/*'
+    - 'spec/**/*'
+Metrics/BlockLength:
+  Exclude:
+    - 'config/**/*'
+    - 'db/**/*'
+    - 'spec/**/*'
+Metrics/MethodLength:
+  Exclude:
+    - 'config/**/*'
+    - 'db/**/*'
+    - 'spec/**/*'

data/coins_paid_api.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
   s.name = 'coins_paid_api'
   s.authors = ['Artem Biserov(artembiserov)', 'Oleg Ivanov(morhekil)']
-  s.version = '1.0.5'
+  s.version = '1.1.0'
   s.files = `git ls-files`.split("\n")
   s.summary = 'Coins Paid Integration'
   s.license = 'MIT'

data/lib/coins_paid/api/take_address.rb

@@ -6,13 +6,18 @@ module CoinsPaid
       class Request < Dry::Struct
         attribute :foreign_id, Types::Coercible::String
         attribute :currency, Types::String
-        attribute :convert_to, Types::String
+        attribute :convert_to, Types::String.optional
+
+        def to_hash
+          convert_value = convert_to != currency ? convert_to : nil
+          super().merge(convert_to: convert_value).compact
+        end
       end
 
       class Response < Dry::Struct
         attribute :external_id, Types::Integer
         attribute :address, Types::String
-        attribute :tag, Types::String.optional
+        attribute :tag, Types::Coercible::String.optional
       end
 
       PATH = 'addresses/take'

data/lib/coins_paid/api/withdrawal.rb

@@ -9,6 +9,7 @@ module CoinsPaid
         attribute :currency, Types::String
         attribute :convert_to, Types::String
         attribute :address, Types::String
+        attribute? :tag, Types::String
       end
 
       class Response < Dry::Struct

data/lib/coins_paid/api.rb

@@ -35,18 +35,15 @@ module CoinsPaid
       yield self
     end
 
-    def take_address(foreign_id:, currency:, convert_to:)
+    def take_address(foreign_id:, currency:, convert_to: nil)
       Requester.call(
         TakeAddress,
         foreign_id: foreign_id, currency: currency, convert_to: convert_to
       )
     end
 
-    def withdraw(foreign_id:, amount:, currency:, convert_to:, address:)
-      Requester.call(
-        Withdrawal,
-        foreign_id: foreign_id, amount: amount, currency: currency, convert_to: convert_to, address: address
-      )
+    def withdraw(data)
+      Requester.call(Withdrawal, data)
     end
 
     def currencies_list

data/spec/currencies_list_spec.rb

@@ -3,7 +3,8 @@
 require_relative './request_examples'
 
 describe CoinsPaid::API, '.currencies_list' do
-  endpoint = 'https://app.coinspaid.com/api/v2/currencies/list'
+  let(:endpoint) { 'https://app.coinspaid.com/api/v2/currencies/list' }
+  let(:request_body) { "{}" }
   include_context 'CoinsPaid API request'
 
   let(:expected_currencies) do
@@ -65,5 +66,5 @@ describe CoinsPaid::API, '.currencies_list' do
     expect(response).to match_array currencies
   end
 
-  it_behaves_like 'CoinsPaid API error handling', endpoint: endpoint
+  it_behaves_like 'CoinsPaid API error handling'
 end

data/spec/request_examples.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-RSpec.shared_context 'CoinsPaid API request' do |request_data: {}|
+RSpec.shared_context 'CoinsPaid API request' do
   let(:signature) { 'c01dc0ffee' }
   let(:request_signature_headers) do
     {
@@ -10,11 +10,11 @@ RSpec.shared_context 'CoinsPaid API request' do |request_data: {}|
   end
 
   before do
-    allow(CoinsPaid::API::Signature).to receive(:generate).with(request_data.to_json).and_return signature
+    allow(CoinsPaid::API::Signature).to receive(:generate).with(request_body).and_return signature
   end
 end
 
-RSpec.shared_examples 'CoinsPaid API error handling' do |endpoint:, request_body: '{}'|
+RSpec.shared_examples 'CoinsPaid API error handling' do
   context 'when coins paid responded with validation errors' do
     let(:response_data) do
       {

data/spec/spec_helper.rb

@@ -10,3 +10,10 @@ require 'webmock/rspec'
 require 'pry'
 require 'coins_paid_api'
 Dir['./spec/support/**/*.rb'].each { |f| require f }
+
+RSpec.configure do |config|
+  config.filter_run focus: true
+  config.alias_example_to :fit, focus: true
+  config.run_all_when_everything_filtered = true
+  config.example_status_persistence_file_path = 'spec/examples.txt'
+end

data/spec/take_address_spec.rb

@@ -3,43 +3,69 @@
 require_relative './request_examples'
 
 describe CoinsPaid::API, '.take_address' do
-  endpoint = 'https://app.coinspaid.com/api/v2/addresses/take'
-  request_data = {
-    foreign_id: 'user-id:2048',
-    currency: 'BTC',
-    convert_to: 'EUR'
-  }
-  include_context 'CoinsPaid API request', request_data: request_data
-
-  let(:expected_address_attributes) do
-    {
-      external_id: 1,
-      address: '12983h13ro1hrt24it432t',
-      tag: 'tag-123'
-    }
+  let(:endpoint) { 'https://app.coinspaid.com/api/v2/addresses/take' }
+
+  let(:foreign_id) { 'user-id:2048' }
+  let(:currency) { 'BTC' }
+
+  let(:request_data) do
+    { foreign_id: foreign_id, currency: currency, convert_to: api_convert_to }.compact
   end
-  subject(:take_address) { described_class.take_address(request_data) }
+  let(:request_body) { request_data.to_json }
+
+  include_context 'CoinsPaid API request'
 
   let(:response_data) do
     {
       'data' => {
         'id' => 1,
         'currency' => 'BTC',
-        'convert_to' => 'EUR',
+        'convert_to' => api_convert_to,
         'address' => '12983h13ro1hrt24it432t',
-        'tag' => 'tag-123',
+        'tag' => 123,
         'foreign_id' => 'user-id:2048'
-      }
+      }.compact
     }
   end
 
-  it 'returns valid response if successful' do
-    stub_request(:post, endpoint)
-      .with(body: request_data, headers: request_signature_headers)
-      .to_return(status: 201, body: response_data.to_json)
+  let(:args) do
+    { foreign_id: 'user-id:2048', currency: 'BTC', convert_to: args_convert_to }.compact
+  end
+  subject(:take_address) { described_class.take_address(args) }
+
+  shared_examples 'successful request' do
+    it 'returns valid response' do
+      stub_request(:post, endpoint)
+        .with(body: request_body, headers: request_signature_headers)
+        .to_return(status: 201, body: response_data.to_json)
+
+      expected_address_attributes = {
+        external_id: 1,
+        address: '12983h13ro1hrt24it432t',
+        tag: '123'
+      }
+      expect(take_address).to be_struct_with_params(
+        CoinsPaid::API::TakeAddress::Response, expected_address_attributes
+      )
+    end
+  end
 
-    expect(take_address).to be_struct_with_params(CoinsPaid::API::TakeAddress::Response, expected_address_attributes)
+  context 'with explicit convert_to not matching address currency' do
+    let(:args_convert_to) { 'EUR' }
+    let(:api_convert_to) { 'EUR' }
+    it_behaves_like 'successful request'
+    it_behaves_like 'CoinsPaid API error handling'
   end
 
-  it_behaves_like 'CoinsPaid API error handling', endpoint: endpoint, request_body: request_data
+  context 'with explicit convert_to matching address currency' do
+    let(:args_convert_to) { 'BTC' }
+    let(:api_convert_to) { nil }
+    it_behaves_like 'successful request'
+  end
+
+  context 'with nil convert_to' do
+    let(:args_convert_to) { nil }
+    let(:api_convert_to) { nil }
+    it_behaves_like 'successful request'
+  end
 end

data/spec/withdrawal_spec.rb

@@ -2,16 +2,10 @@
 
 require_relative './request_examples'
 
-describe CoinsPaid::API, '.withdraw' do
-  endpoint = 'https://app.coinspaid.com/api/v2/withdrawal/crypto'
-  request_data = {
-    foreign_id: 'user-id:2048',
-    amount: '0.01',
-    currency: 'EUR',
-    convert_to: 'BTC',
-    address: 'abc123'
-  }
-  include_context 'CoinsPaid API request', request_data: request_data
+RSpec.shared_examples 'CoinsPaid API withdrawal' do
+  let(:endpoint) { 'https://app.coinspaid.com/api/v2/withdrawal/crypto' }
+
+  include_context 'CoinsPaid API request'
 
   let(:response_data) do
     {
@@ -48,3 +42,26 @@ describe CoinsPaid::API, '.withdraw' do
     end
   end
 end
+
+describe CoinsPaid::API, '.withdraw' do
+  let(:base_request_data) do
+    {
+      foreign_id: 'user-id:2048',
+      amount: '0.01',
+      currency: 'EUR',
+      convert_to: 'BTC',
+      address: 'abc123'
+    }
+  end
+  let(:request_body) { request_data.to_json }
+
+  context 'request includes tag parameter' do
+    let(:request_data) { base_request_data.merge(tag: 'thetag') }
+    it_behaves_like 'CoinsPaid API withdrawal'
+  end
+
+  context 'request does not include tag parameter' do
+    let(:request_data) { base_request_data }
+    it_behaves_like 'CoinsPaid API withdrawal'
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coins_paid_api
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.1.0
 platform: ruby
 authors:
 - Artem Biserov(artembiserov)
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-16 00:00:00.000000000 Z
+date: 2022-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dry-struct
@@ -90,6 +90,7 @@ files:
 - ".circleci/config.yml"
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - Gemfile
 - coins_paid_api.gemspec
 - lib/coins_paid/api.rb