myfinance-rails 0.4.5 → 0.4.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6e1735e3dd9132665f12c10469b78c2c6133f051
-  data.tar.gz: 922810fc3721c021937396772c9f94bf8b90e58c
+  metadata.gz: 358d3ebb84924f69546c77e26f18fd0d9cc4dcfd
+  data.tar.gz: 2f84365f90be4cee592f2e7a3720ffb33d4287d1
 SHA512:
-  metadata.gz: 97940e757fb5d3d640ac84ebc250154e03b3f9d5602420dc5e48f1fc18d2bf1229b6b7f6e4e4f07dfcca956f681f1f0a8ace8e3a53333a21dcf6458d300d2ce3
-  data.tar.gz: 8700f2158948320b3f05660d9198358db8dd6098e0eecccd3aaec428147c976dd4ab44db262db84faa063c9e5441a094e03ca6aff2dfa5e255c3a1eb37531048
+  metadata.gz: e8d77829d1e5f3a617bc0419fd92a6c3b829a8e7e469cc9868dd0fb65e055306c03e818faf6d9042c17dab10b4502e06e9ea422c8b5cc2732d0b7dd2f6a70b7a
+  data.tar.gz: 6f97183cffe953e9adb361274621e26ab9ea371e03e0f329a97d72a93cb1503dcc4395f83e5aea173a9ab715ee51fd1b1230540df3f3a7a5278bba990ae8fa00

data/lib/myfinance/pessoa.rb

@@ -1,36 +1,19 @@
 module Myfinance
 
   def self.pessoa_id(cnpj_ou_nome)
-    @everyone ||= all_people
-    mid = nil
-    @everyone.each do | pessoa |
-      if @account_id.to_i == pessoa['account_id']
-        if pessoa['federation_subscription_number_only_numbers'] == cnpj_ou_nome or pessoa['federation_subscription_number'] == cnpj_ou_nome or pessoa['name'].casecmp(cnpj_ou_nome) == 0
-          mid = pessoa['id']
-          break
-        end
-      end
-    end
-    mid
+    person = pessoa(cnpj_ou_nome)
+    person.nil? ? nil : person["id"]
   end
 
-  def self.all_people
-    everyone = []
-    response = lget "/people.json"
-    total_pages = get_last_page(response)
-    for page in 1..total_pages
-      response = lget "/people.json?page=#{page}"
-      response.each do | people |
-        everyone << people['person']
-      end
-    end
-    everyone
+  def self.pessoa(cnpj_ou_nome)
+    person = pesquisa_pessoa("federation_subscription_number", cnpj_ou_nome)
+    person ||= pesquisa_pessoa("name", cnpj_ou_nome)
+    person
   end
 
-  def self.pessoa(cnpj_ou_nome)
-    mid = pessoa_id(cnpj_ou_nome)
-    response = lget "/people/#{mid}.json"
-    response['person'] if response
+  def self.pesquisa_pessoa(campo, valor)
+    people =  Myfinance.lget URI.encode("/people.json?search[#{campo}]=#{valor}")
+    people.empty? ? nil : people[0]["person"]
   end
 
   def self.cria_pessoa( pessoa )
@@ -46,18 +29,5 @@ module Myfinance
     @everyone = nil
     lput "/people/#{people_id}.json", pessoa
   end
-
-
-  def self.get_last_page(response)
-    total_pages = 1
-    links = (response.header['link'].split(',') rescue [])
-    link = links.find{ |l| l.include?('last') }
-    if link
-      total_pages = (link.split(";")[0].split('?')[1].gsub('page=','').to_i rescue 1)
-    end
-    total_pages
-  end
-
-
 end
 

data/lib/myfinance/version.rb

@@ -1,3 +1,3 @@
 module Myfinance
-  VERSION = '0.4.5'
+  VERSION = '0.4.6'
 end

data/spec/features/pessoas_spec.rb

@@ -37,39 +37,75 @@ describe 'Manipulando Pessoas', type: :feature do
 
   end
 
-  it 'Se procurar por um cnpj não cadastrado, deve voltar nulo' do
-    Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
-    id = Myfinance.pessoa_id('67977504999999')
-    expect(id).to be_nil
-  end
+  describe "#pessoa_id" do
+    it 'Se procurar por um cnpj não cadastrado, deve voltar nulo' do
+      Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
+      id = Myfinance.pessoa_id('67977504999999')
+      expect(id).to be_nil
+    end
 
-  it 'Se procurar por um cnpj cadastrado, deve voltar id' do
-    Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
-    id = Myfinance.pessoa_id('27206831000170')
-    expect(id).to_not be_nil
-  end
+    it 'Se procurar por um cnpj cadastrado, deve voltar id' do
+      Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
+      id = Myfinance.pessoa_id('27206831000170')
+      expect(id).to_not be_nil
+    end
 
-  it 'Se procurar por nome cadastrado, deve voltar id' do
-    Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
-    id = Myfinance.pessoa_id('Ciclano')
-    expect(id).to_not be_nil
-  end
+    it 'Se procurar por nome cadastrado, deve voltar id' do
+      Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
+      id = Myfinance.pessoa_id('Ciclano')
+      expect(id).to_not be_nil
+    end
 
-  it 'Se procurar por nome cadastrado, mas em case diferente, deve voltar id' do
-    Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
-    id = Myfinance.pessoa_id('CiClAnO')
-    expect(id).to_not be_nil
+    it 'Se procurar por nome cadastrado, mas em case diferente, deve voltar id' do
+      Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
+      id = Myfinance.pessoa_id('CiClAnO')
+      expect(id).to_not be_nil
+    end
+
+    it 'Se procurar por nome não cadastrado, deve voltar id nulo' do
+      Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
+      id = Myfinance.pessoa_id('Ciclano Ciclano')
+      expect(id).to be_nil
+    end
   end
 
-  it 'Se procurar por nome não cadastrado, deve voltar id nulo' do
-    Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
-    id = Myfinance.pessoa_id('Ciclano Ciclano')
-    expect(id).to be_nil
+  describe "#pessoa" do
+    let!(:person_double) { double("person") }
+    it "Retorna pessoa quando encontra cnpj" do
+      expect(Myfinance).to receive(:pesquisa_pessoa).with("federation_subscription_number", "67977504000137").and_return person_double
+      expect(Myfinance.pessoa("67977504000137")).to eql person_double
+    end
+    it "Retorna pessoa quando encontra nome" do
+      expect(Myfinance).to receive(:pesquisa_pessoa).with("federation_subscription_number", "nome").and_return nil
+      expect(Myfinance).to receive(:pesquisa_pessoa).with("name", "nome").and_return person_double
+      expect(Myfinance.pessoa("nome")).to eql person_double
+    end
+    it "Retorna nil quando não encontra pessoa" do
+      expect(Myfinance).to receive(:pesquisa_pessoa).with("federation_subscription_number", "nome").and_return nil
+      expect(Myfinance).to receive(:pesquisa_pessoa).with("name", "nome").and_return nil
+      expect(Myfinance.pessoa("nome")).to be_nil
+    end
   end
 
-  it 'Se procurar por um cnpj cadastrado de outra conta, não deve voltar id' do
-    Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277', false, 1)
-    id = Myfinance.pessoa_id('27206831000170')
-    expect(id).to be_nil
+  describe "#pesquisa_pessoa" do
+    before do
+      Myfinance.setup('2acecbb483842ebbfb2c638070bf019b70e757190166d277')
+    end
+    it "retorna pessoal quando encontra cnpj com pontuação" do
+      pessoa = Myfinance.pesquisa_pessoa("federation_subscription_number", "67.977.504/0001-37")
+      expect(pessoa["id"]).to eql 192058
+    end
+    it "retorna pessoal quando encontra cnpj sem pontuação" do
+      pessoa = Myfinance.pesquisa_pessoa("federation_subscription_number", "67977504000137")
+      expect(pessoa["id"]).to eql 192058
+    end
+    it "retorna pessoal quando encontra nome" do
+      pessoa = Myfinance.pesquisa_pessoa("name", "AGROPECUARIA POTENZA LTDA.")
+      expect(pessoa["id"]).to eql 192058
+    end
+    it "retorna nil quando não encontra cnpj" do
+      pessoa = Myfinance.pesquisa_pessoa("name", "AGROPECUARIA POTENZA")
+      expect(pessoa).to be_nil
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: myfinance-rails
 version: !ruby/object:Gem::Version
-  version: 0.4.5
+  version: 0.4.6
 platform: ruby
 authors:
 - José Lopes Neto
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-24 00:00:00.000000000 Z
+date: 2016-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler