sale 0.1.6 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 450588d444cde46db5fc732c7d89d10633e3372e2dff6e4fc88d33a2f2d0c269
-  data.tar.gz: '030684795a3a9e69fae7bf98a845811d6e159bbedc8574af59ae3791135f7f91'
+  metadata.gz: d2bdb714aece35dd7f04de403c5f36e973572a390938d0da5af1368b3d20563f
+  data.tar.gz: 61c118538568e2566b1f75a1a579f36cb0504b98a30404441a3967154188a476
 SHA512:
-  metadata.gz: 5778d3a4424179c537181faf397a02960f798f35df3f8987f80811c774e30cc9d3d03b1795c0dd4a923b798fe3d3b78d54c3c2ff170669755ed2f7382b395292
-  data.tar.gz: e70e56d8d47f6f7067605ca08e1d3178d362218099973daa5161f21ada54561883d72a33b6ab5a4696dc039ffc1dc1e50f7a5bea6ee88a4846d0b8261272a38c
+  metadata.gz: 2616e683c8f148bdb90a6ada8d151104fb092d38232fad25e428aee84360702b51c3cf4347aca41e1447e923b4d7e575ee31e104fbac5e81b0c37b1309829fa9
+  data.tar.gz: bf78adc18c95bc390289302ebfeafe9002e14f2e8e55ec6b0da1d0eeabd2d2215ab82387fa1661b7d254ec46905fc6865c322be6798cfecb1b172e6cea5a5cc7

data/lib/sale/base.rb

@@ -8,6 +8,7 @@ module Bsale
   class APIBase
     ENDPOINT = 'https://api.bsale.cl'.freeze
     VERSION  = 'v1'.freeze
+    MAX_RETRIES = 5
 
     def initialize(token)
       @token = token or raise 'Token not set!'
@@ -34,7 +35,7 @@ module Bsale
       wrap_response(resp)
     end
 
-    def request(method, path, body = nil)
+    def request(method, path, body = nil, attempt = 1)
       headers = { 'access_token' => @token }
       url = path['://'] ? path : [ ENDPOINT, VERSION, path ].join('/')
 
@@ -43,6 +44,13 @@ module Bsale
       else
         Dagger.request(method, url, body, { json: true, follow: 3, headers: headers })
       end
+    rescue InvalidResponseError, Errno::ECONNREFUSED, Errno::ECONNRESET, Errno::ETIMEDOUT, Errno::EINVAL, Timeout::Error, \
+      SocketError, EOFError, Net::HTTPBadResponse, Net::HTTPHeaderSyntaxError, Net::ProtocolError, OpenSSL::SSL::SSLError => e
+
+      raise if attempt > MAX_RETRIES
+      puts "Connection refused. Retrying in a few secs... (attempt #{attempt})"
+      sleep(attempt * 5) # 5 secs, 10 secs, 15 secs, 20 secs
+      request(method, path, body, attempt + 1)
     end
 
     def connect

data/lib/sale/buyers.rb

@@ -18,6 +18,8 @@ class Bsale::Buyers
       end
 
       return with_code.first
+    # else
+    #   raise BuyerNotFoundError.new("No matches with email #{email}, code #{code}")
     end
   end
 

data/lib/sale/entity.rb

@@ -1,19 +1,16 @@
 class Entity
+  include Enumerable
 
   attr_reader :data
 
   def initialize(data, client, original_params = nil)
-    @data, @client, @original_params = data, client, original_params
+    @data, @client, @original_params = maybe_stringify(data), client, original_params
   end
 
   def to_s
     @data.inspect
   end
 
-  def to_a
-    data['items'] ? items : nil
-  end
-
   def [](key)
     data[key.to_s]
   end
@@ -22,26 +19,10 @@ class Entity
     data[key.to_s] = obj
   end
 
-  def size
-    (to_a || []).size
-  end
-
-  alias_method :count, :size
-
-  def each(&block)
-    to_a.each(&block)
-  end
-
-  def each_with_index(&block)
-    to_a.each_with_index &block
-  end
-
-  def map(&block)
-    to_a.map(&block)
+  def to_a
+    data['items'] ? items : []
   end
 
-  alias_method :collect, :map
-
   def load(params = nil)
     client.get(data['href'], params)
   end
@@ -51,6 +32,16 @@ class Entity
     load # just return a new instance
   end
 
+  def each(&block)
+    to_a.each(&block)
+  end
+
+  def count
+    to_a.size
+  end
+
+  alias_method :size, :count
+
   def items
     return if data['items'].nil?
     @items ||= data['items'].map { |obj| Entity.new(obj, client) }
@@ -80,20 +71,15 @@ class Entity
     data['next'] ? follow_link(data['next'], @original_params) : nil
   end
 
-  def method_missing(name)
+  def method_missing(name, *args)
     name = name.to_s
-
-    if data[name]
-      # if is_link?(data[name])
-      #   follow_link(data[name]['href'])
-      if is_hash?(data[name])
-        Entity.new(data[name], client)
-      else
-        data[name]
-      end
+    if is_hash?(data[name])
+      Entity.new(data[name], client)
+    elsif data[name]
+      data[name]
     else
-      # puts data.inspect
-      # puts "Unknown method: #{name}"
+      nil
+      # raise "Not found: #{name} in #{data}"
     end
   end
 
@@ -104,10 +90,22 @@ class Entity
   #   is_hash?(obj) && obj['href'] && obj.keys == ['href']
   # end
 
+  # def is_integer?(val)
+  #   Integer(val)
+  #   true
+  # rescue
+  #   false
+  # end
+
   def is_hash?(obj)
     obj.is_a?(Hash)
   end
 
+  def maybe_stringify(obj)
+    return obj unless is_hash?(obj)
+    obj.inject({}) { |memo,(k,v) | memo[k.to_s] = v; memo }
+  end
+
   def follow_link(href, params = nil)
     client.get(href, params)
   end

data/lib/sale/fake_rut.rb

@@ -1,3 +1,5 @@
+require 'digest/sha1'
+
 module FakeRUT
 
   def self.for(str)

data/sale.gemspec

@@ -3,7 +3,7 @@
 
 Gem::Specification.new do |s|
   s.name        = "sale"
-  s.version     = '0.1.6'
+  s.version     = '0.2.0'
   s.platform    = Gem::Platform::RUBY
   s.authors     = ['Tomás Pollak']
   s.email       = ['tomas@bootic.net']

data/spec/base_spec.rb

@@ -0,0 +1,21 @@
+require 'bundler/setup'
+require_relative '../lib/sale'
+
+RSpec.configure do |config|
+  config.color = true
+end
+
+describe 'APIBase' do
+
+  let(:subject) { Bsale::APIBase.new(token) }
+  let(:token) { '123456' }
+
+  describe '#request' do
+    it 'explodes with 401 error if invalid token' do
+      expect do
+        resp = subject.get('foo')
+      end.to raise_error(Bsale::InvalidResponseError)
+    end
+  end
+
+end

data/spec/buyers_spec.rb

@@ -56,12 +56,14 @@ describe 'Bsale' do
 
         it 'returns result, without code' do
           res = client.find_by_email_and_code('aaa@foo.com')
-          expect(res).to eql(OpenStruct.new({ code: 123, accumulatePoints: '1' }))
+          expect(res).to be_a(Entity) # (OpenStruct.new({ code: 123, accumulatePoints: '1' }))
+          expect(res.code).to eq(123)
         end
 
         it 'returns result, with matching code' do
           res = client.find_by_email_and_code('aaa@foo.com', '123')
-          expect(res).to eql(OpenStruct.new({ code: 123, accumulatePoints: '1' }))
+          expect(res).to be_a(Entity)
+          expect(res.code).to eq(123)
         end
 
         it 'returns nil, with non-matching code' do
@@ -81,12 +83,14 @@ describe 'Bsale' do
 
         it 'returns result, without code' do
           res = client.find_by_email_and_code('aaa@foo.com')
-          expect(res).to eql(OpenStruct.new({ code: 123, accumulatePoints: '1' }))
+          expect(res).to be_a(Entity)
+          expect(res.code).to eq(123)
         end
 
         it 'returns result, with matching code' do
           res = client.find_by_email_and_code('aaa@foo.com', '123')
-          expect(res).to eql(OpenStruct.new({ code: 123, accumulatePoints: '1' }))
+          expect(res).to be_a(Entity)
+          expect(res.code).to eq(123)
         end
 
         it 'returns nil, with non-matching code' do
@@ -138,7 +142,8 @@ describe 'Bsale' do
 
         it 'returns result, when code matches one' do
           res = client.find_by_email_and_code('aaa@foo.com', '123')
-          expect(res).to eql(OpenStruct.new({ code: 123, accumulatePoints: '1' }))
+          expect(res).to be_a(Entity)
+          expect(res.code).to eq(123)
         end
 
         it 'returns nil, when code doesnt match any' do

data/spec/entity_spec.rb

@@ -0,0 +1,47 @@
+require 'bundler/setup'
+require_relative '../lib/sale'
+
+RSpec.configure do |config|
+  config.color = true
+end
+
+describe 'Entity' do
+
+  describe '#to_a' do
+    it 'returns empty array if no items' do
+      obj = Entity.new({ foo: 'bar '}, nil)
+      expect(obj.to_a).to eq([])
+    end
+  end
+
+  describe '#count' do
+    it 'returns 0 array if no items' do
+      obj = Entity.new({ foo: 'bar '}, nil)
+      expect(obj.count).to eq(0)
+    end
+
+    it 'returns number if items' do
+      obj = Entity.new({ items: [1] }, nil)
+      expect(obj.count).to eq(1)
+    end
+  end
+
+  describe '#each_with_index' do
+    it 'works beautifully' do
+      obj = Entity.new({ foo: 'bar', items: [1, 2, 3] }, nil)
+      res = []
+      obj.each_with_index { |el, i| res.push([el.data, i]) }
+      expect(res).to eq([[1,0], [2,1], [3,2]])
+    end
+  end
+
+  describe '#select' do
+    it 'works beautifully' do
+      obj = Entity.new({ foo: 'bar', items: [1, 2, 3] }, nil)
+      res = obj.select { |el| el.data % 2 == 0 }
+      expect(res.map(&:data)).to eq([2])
+    end
+  end
+
+end
+

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sale
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.2.0
 platform: ruby
 authors:
 - Tomás Pollak
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-08-23 00:00:00.000000000 Z
+date: 2019-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -97,7 +97,9 @@ files:
 - lib/sale/invoices.rb
 - lib/sale/root.rb
 - sale.gemspec
+- spec/base_spec.rb
 - spec/buyers_spec.rb
+- spec/entity_spec.rb
 homepage: https://github.com/github/bsale-api-wrapper
 licenses: []
 metadata: {}