omniauth-boletosimples 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b0060b4ecb03a3d5ca498fec9ed35b8cd5efd2a09176aad4f4edc4907dd0471e
-  data.tar.gz: 9f653f20372ee7e87bfffb0026c340d3a3e4f5cea7dd7389e16016a3d13fb0a2
+  metadata.gz: d7fab2120bc0f923910c4a0e8cbe9b2e8fd01f5bca3862a41837261f289e9d6b
+  data.tar.gz: d99836911a51afcb1c86842d7ef79045c92b4b7cee52c9d54a22c470e967a124
 SHA512:
-  metadata.gz: ddb8e8d2309e00ca27ee4ae84c140a25dda8fdf7b654656e965a91600d808cf6be9eb8889618d1fdbe73fada3a45c96f8cd0c0fdaf71a02fff94b23365aff1cc
-  data.tar.gz: e529b3e2762e57790ef2b10f13baa481dc2cc071ed6a2ff2f9d028514ec355e5d521634d4780379d2074fe6e5b39931cca5bfe6dba4809da9ec3d139e773268e
+  metadata.gz: ccc83583be35f54546050a7e3af1d2fae79556978d4a4eaf17e074ff5cec25781a9469b6244653a75c9ca6387a3ede8ce48ca27f242764f7690fddbd77bb5d14
+  data.tar.gz: fd32fcbaec477e1c5ec38453e487dac801c089137579d5f542f058d00c9d4cd59e528aead9a98644aba558ee138c9173e63be39aff8701da292f1b245abe8068

data/.rubocop.yml

@@ -0,0 +1,2 @@
+AllCops:
+  NewCops: enable

data/.ruby-version

@@ -1 +1 @@
-2.6.3
+3.0.1

data/Gemfile

@@ -5,3 +5,6 @@ source 'https://rubygems.org'
 gemspec
 
 gem 'json'
+gem 'rubocop'
+gem 'rubocop-rake'
+gem 'rubocop-rspec'

data/Gemfile.lock

@@ -1,48 +1,80 @@
 PATH
   remote: .
   specs:
-    omniauth-boletosimples (0.1.0)
+    omniauth-boletosimples (0.1.1)
       omniauth-oauth2 (~> 1.6)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    diff-lcs (1.3)
-    faraday (0.15.4)
+    ast (2.4.2)
+    diff-lcs (1.4.4)
+    faraday (1.3.0)
+      faraday-net_http (~> 1.0)
       multipart-post (>= 1.2, < 3)
-    hashie (3.6.0)
-    json (2.2.0)
-    jwt (2.2.1)
-    multi_json (1.13.1)
+      ruby2_keywords
+    faraday-net_http (1.0.1)
+    hashie (4.1.0)
+    json (2.5.1)
+    jwt (2.2.2)
+    multi_json (1.15.0)
     multi_xml (0.6.0)
     multipart-post (2.1.1)
-    oauth2 (1.4.1)
-      faraday (>= 0.8, < 0.16.0)
+    oauth2 (1.4.7)
+      faraday (>= 0.8, < 2.0)
       jwt (>= 1.0, < 3.0)
       multi_json (~> 1.3)
       multi_xml (~> 0.5)
       rack (>= 1.2, < 3)
-    omniauth (1.9.0)
-      hashie (>= 3.4.6, < 3.7.0)
+    omniauth (2.0.4)
+      hashie (>= 3.4.6)
       rack (>= 1.6.2, < 3)
-    omniauth-oauth2 (1.6.0)
-      oauth2 (~> 1.1)
-      omniauth (~> 1.9)
-    rack (2.0.7)
-    rake (12.3.2)
-    rspec (3.8.0)
-      rspec-core (~> 3.8.0)
-      rspec-expectations (~> 3.8.0)
-      rspec-mocks (~> 3.8.0)
-    rspec-core (3.8.2)
-      rspec-support (~> 3.8.0)
-    rspec-expectations (3.8.4)
+      rack-protection
+    omniauth-oauth2 (1.7.1)
+      oauth2 (~> 1.4)
+      omniauth (>= 1.9, < 3)
+    parallel (1.20.1)
+    parser (3.0.1.0)
+      ast (~> 2.4.1)
+    rack (2.2.3)
+    rack-protection (2.1.0)
+      rack
+    rainbow (3.0.0)
+    rake (13.0.3)
+    regexp_parser (2.1.1)
+    rexml (3.2.5)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-mocks (3.8.1)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-support (3.8.2)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.2)
+    rubocop (1.12.1)
+      parallel (~> 1.10)
+      parser (>= 3.0.0.0)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml
+      rubocop-ast (>= 1.2.0, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.4.1)
+      parser (>= 2.7.1.5)
+    rubocop-rake (0.5.1)
+      rubocop
+    rubocop-rspec (2.2.0)
+      rubocop (~> 1.0)
+      rubocop-ast (>= 1.1.0)
+    ruby-progressbar (1.11.0)
+    ruby2_keywords (0.0.4)
+    unicode-display_width (2.0.0)
 
 PLATFORMS
   ruby
@@ -52,6 +84,9 @@ DEPENDENCIES
   omniauth-boletosimples!
   rake
   rspec
+  rubocop
+  rubocop-rake
+  rubocop-rspec
 
 BUNDLED WITH
-   2.0.2
+   2.2.15

data/lib/omniauth/boletosimples/version.rb

@@ -2,6 +2,6 @@
 
 module OmniAuth
   module BoletoSimples
-    VERSION = '0.1.0'
+    VERSION = '0.1.1'
   end
 end

data/lib/omniauth/strategies/boletosimples.rb

@@ -58,13 +58,14 @@ module OmniAuth
         environment = options.environment || :production
         options.client_options[:site] = ENVIRONMENTS[environment.to_sym]
         options.client_options[:authorize_url] = "#{ENVIRONMENTS[environment.to_sym]}/api/v1/oauth2/authorize"
-        if options.user_agent
-          options.client_options[:connection_opts] = {
-            headers: {
-              'User-Agent' => options.user_agent
-            }
+
+        return unless options.user_agent
+
+        options.client_options[:connection_opts] = {
+          headers: {
+            'User-Agent' => options.user_agent
           }
-        end
+        }
       end
 
       def raw_info
@@ -84,9 +85,6 @@ module OmniAuth
       private
 
       def load_identity
-        access_token.options[:mode] = :query
-        access_token.options[:param_name] = :access_token
-        access_token.options[:grant_type] = :authorization_code
         access_token.get('/api/v1/userinfo').parsed
       end
     end

data/spec/omniauth/strategies/boletosimples_spec.rb

@@ -10,43 +10,58 @@ describe OmniAuth::Strategies::BoletoSimples do
 
   it_should_behave_like 'an oauth2 strategy'
 
-  describe '#client' do
-    it 'has correct BoletoSimples site' do
-      subject.setup_phase
-      expect(subject.client.site).to eq('https://boletosimples.com.br')
-    end
+  it 'has correct BoletoSimples site' do
+    subject.setup_phase
+    expect(subject.client.site).to eq('https://boletosimples.com.br')
+  end
 
-    it 'has correct BoletoSimples sandbox site' do
-      @options = { environment: :sandbox }
-      subject.setup_phase
-      expect(subject.client.site).to eq('https://sandbox.boletosimples.com.br')
-    end
+  it 'has correct BoletoSimples sandbox site' do
+    @options = { environment: :sandbox }
+    subject.setup_phase
+    expect(subject.client.site).to eq('https://sandbox.boletosimples.com.br')
+  end
 
-    it 'has correct authorize url' do
-      subject.setup_phase
-      expect(subject.client.options[:authorize_url]).to eq('https://boletosimples.com.br/api/v1/oauth2/authorize')
-    end
+  it 'has correct authorize url' do
+    subject.setup_phase
+    expect(subject.client.options[:authorize_url]).to eq('https://boletosimples.com.br/api/v1/oauth2/authorize')
+  end
 
-    it 'has correct sandbox authorize url' do
-      @options = { environment: :sandbox }
-      subject.setup_phase
-      expect(subject.client.options[:authorize_url]).to eq('https://sandbox.boletosimples.com.br/api/v1/oauth2/authorize')
-    end
+  it 'has correct sandbox authorize url' do
+    @options = { environment: :sandbox }
+    subject.setup_phase
+    expect(subject.client.options[:authorize_url]).to eq('https://sandbox.boletosimples.com.br/api/v1/oauth2/authorize')
+  end
+
+  it 'has correct token url' do
+    expect(subject.client.options[:token_url]).to eq('/api/v1/oauth2/token')
+  end
 
-    it 'has correct token url' do
-      expect(subject.client.options[:token_url]).to eq('/api/v1/oauth2/token')
+  it 'has correct connection_opts' do
+    @options = { user_agent: 'email@example.com' }
+    subject.setup_phase
+    expect(subject.client.options[:connection_opts]).to eq(headers: { "User-Agent": 'email@example.com' })
+  end
+
+  describe '#callback_url' do
+    let(:base_url) { 'https://example.com' }
+
+    it 'has the correct default callback path' do
+      allow(subject).to receive(:full_host) { base_url }
+      allow(subject).to receive(:script_name) { '' }
+      expect(subject.send(:callback_url)).to eq("#{base_url}/auth/boletosimples/callback")
     end
 
-    it 'has correct connection_opts' do
-      @options = { user_agent: 'email@example.com' }
-      subject.setup_phase
-      expect(subject.client.options[:connection_opts]).to eq(headers: { "User-Agent": 'email@example.com' })
+    it 'should set the callback path with script_name if present' do
+      allow(subject).to receive(:full_host) { base_url }
+      allow(subject).to receive(:script_name) { '/v1' }
+      expect(subject.send(:callback_url)).to eq("#{base_url}/v1/v1/auth/boletosimples/callback")
     end
-  end
 
-  describe '#callback_path' do
-    it 'has the correct callback path' do
-      expect(subject.callback_path).to eq('/auth/boletosimples/callback')
+    it 'should set the callback_path parameter if present' do
+      @options = { callback_path: '/auth/foo/callback' }
+      allow(subject).to receive(:full_host) { base_url }
+      allow(subject).to receive(:script_name) { '' }
+      expect(subject.send(:callback_url)).to eq("#{base_url}/auth/foo/callback")
     end
   end
 end

data/spec/spec_helper.rb

@@ -3,7 +3,7 @@
 require 'bundler/setup'
 require 'rspec'
 require 'omniauth'
-Dir[File.expand_path('support/**/*', __dir__)].each { |f| require f }
+Dir[File.expand_path('support/**/*', __dir__)].sort.each { |f| require f }
 
 RSpec.configure do |_config|
   OmniAuth.config.test_mode = true

data/spec/support/shared_examples.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# NOTE it would be useful if this lived in omniauth-oauth2 eventually
+# NOTE: it would be useful if this lived in omniauth-oauth2 eventually
 shared_examples 'an oauth2 strategy' do
   describe '#client' do
     it 'should be initialized with symbolized client_options' do

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-boletosimples
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Rafael Lima
 - Kivanio Barbosa
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-07-10 00:00:00.000000000 Z
+date: 2021-04-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth-oauth2
@@ -53,7 +53,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description:
 email:
 - contato@rafael.adm.br
 - kivanio@gmail.com
@@ -62,6 +62,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rubocop.yml"
 - ".ruby-gemset"
 - ".ruby-version"
 - CHANGELOG.md
@@ -83,7 +84,7 @@ homepage: https://github.com/BoletoSimples/omniauth-boletosimples
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -98,8 +99,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.15
+signing_key:
 specification_version: 4
 summary: BoletoSimples strategy for OmniAuth
 test_files: