coins_paid_api 1.0.6 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 020d45404c81037d89565ba5e7dcb231982db444abdbd00a6f1535bfd9e73170
-  data.tar.gz: 84f0ab1f61a80fe45dedca5cc0d0ce8467e5328315fc75d0f3812702e84774a4
+  metadata.gz: 14dc12db1d0e7f9f90746feb4e0ec39f491bdf2533984282e197cc87babad6dd
+  data.tar.gz: 34baccc03be0e0fd327f7240f71a58fa181805ae835a105e158db3863722610b
 SHA512:
-  metadata.gz: cfd4a488e8434dd0ccc78b68feed059545f1d193bbaf2b226e597da371b04b454f459c4a157f55d56750805181048135322ef1671c09f82c78bb8e83186fc50f
-  data.tar.gz: c7ad6dea7bc2c80882df508506df798d7864cb59073db39f14cf46df55ea8d904d20a4416daf4e26ce8c5eee44570cd3c41b8a451760b2642f1ad1b9dc6761b9
+  metadata.gz: cb016be8a69caefbb956d4d7939164f78e4d6fed5b6f3d3b1dba050284a57e6c882725e60a7f2439a80483aad724e3eb9b6647ba76286ffff696bfa1edd70e2d
+  data.tar.gz: 345be14c726a21cd9f4f21ac98dc11aa46ff1a95b92cf690a445f0d2dbad90252106bf1d4fb7aec0f1f41fc59d42f497f83b04bdcfaeec7b303f498a8cacab50

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.6'
+  s.version = '1.2.0'
   s.files = `git ls-files`.split("\n")
   s.summary = 'Coins Paid Integration'
   s.license = 'MIT'

data/lib/coins_paid/api/callback_data.rb

@@ -23,10 +23,12 @@ module CoinsPaid
       end
 
       attribute? :currency_sent do
+        attribute :currency, Types::String
         attribute :amount, Types::String
       end
 
       attribute? :currency_received do
+        attribute :currency, Types::String
         attribute :amount, Types::String
         attribute? :amount_minus_fee, Types::String
       end

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.rb

@@ -35,7 +35,7 @@ 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

data/spec/callback_spec.rb

@@ -42,11 +42,8 @@ describe CoinsPaid::API, '.callback' do
           { transaction_type: 'blockchain', type: 'deposit', id: 714576 },
           { transaction_type: 'exchange', type: 'exchange', id: 714577 }
         ],
-        currency_sent: { amount: '0.01000000' },
-        currency_received: {
-          amount_minus_fee: '90',
-          amount: '84.17070222'
-        }
+        currency_sent: { currency: 'BTC', amount: '0.01000000' },
+        currency_received: { currency: 'EUR', amount_minus_fee: '90', amount: '84.17070222' }
       }
     end
 
@@ -91,10 +88,8 @@ describe CoinsPaid::API, '.callback' do
           { transaction_type: 'exchange', type: 'exchange', id: 1 },
           { transaction_type: 'blockchain', type: 'withdrawal', id: 1 }
         ],
-        currency_sent: { amount: '381' },
-        currency_received: {
-          amount: '0.01000000'
-        }
+        currency_sent: { currency: 'EUR', amount: '381' },
+        currency_received: { currency: 'BTC', amount: '0.01000000' }
       }
     end
 
@@ -132,8 +127,8 @@ describe CoinsPaid::API, '.callback' do
         status: 'confirmed',
         foreign_id: '',
         error: '',
-        currency_sent: { amount: "0.56114000"},
-        currency_received: { amount: "80.21790617"},
+        currency_sent: { currency: 'ETH', amount: '0.56114000'},
+        currency_received: { currency: 'EUR', amount: '80.21790617'},
         transactions: [
           { transaction_type: 'exchange', type: 'exchange', id: 714576 },
         ]

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,10 +2,10 @@
 
 require_relative './request_examples'
 
-RSpec.shared_examples 'CoinsPaid API withdrawal' do |request_data:|
-  endpoint = 'https://app.coinspaid.com/api/v2/withdrawal/crypto'
+RSpec.shared_examples 'CoinsPaid API withdrawal' do
+  let(:endpoint) { 'https://app.coinspaid.com/api/v2/withdrawal/crypto' }
 
-  include_context 'CoinsPaid API request', request_data: request_data
+  include_context 'CoinsPaid API request'
 
   let(:response_data) do
     {
@@ -44,19 +44,24 @@ RSpec.shared_examples 'CoinsPaid API withdrawal' do |request_data:|
 end
 
 describe CoinsPaid::API, '.withdraw' do
-  base_request_data = {
-    foreign_id: 'user-id:2048',
-    amount: '0.01',
-    currency: 'EUR',
-    convert_to: 'BTC',
-    address: 'abc123'
-  }
+  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
-    it_behaves_like 'CoinsPaid API withdrawal', request_data: base_request_data.merge(tag: 'thetag')
+    let(:request_data) { base_request_data.merge(tag: 'thetag') }
+    it_behaves_like 'CoinsPaid API withdrawal'
   end
 
   context 'request does not include tag parameter' do
-    it_behaves_like 'CoinsPaid API withdrawal', request_data: base_request_data
+    let(:request_data) { base_request_data }
+    it_behaves_like 'CoinsPaid API withdrawal'
   end
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: coins_paid_api
 version: !ruby/object:Gem::Version
-  version: 1.0.6
+  version: 1.2.0
 platform: ruby
 authors:
 - Artem Biserov(artembiserov)
 - Oleg Ivanov(morhekil)
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-09-24 00:00:00.000000000 Z
+date: 2022-03-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dry-struct
@@ -81,8 +81,8 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.7'
-description:
-email:
+description: 
+email: 
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -90,6 +90,7 @@ files:
 - ".circleci/config.yml"
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - Gemfile
 - coins_paid_api.gemspec
 - lib/coins_paid/api.rb
@@ -112,11 +113,11 @@ files:
 - spec/support/shared/data.rb
 - spec/take_address_spec.rb
 - spec/withdrawal_spec.rb
-homepage:
+homepage: 
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -131,9 +132,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project:
+rubyforge_project: 
 rubygems_version: 2.7.6
-signing_key:
+signing_key: 
 specification_version: 4
 summary: Coins Paid Integration
 test_files: []