omniauth-remote-user 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 010df7958c3aa475eac85c8b461205366fc2ac4f
-  data.tar.gz: 6e2ea502ec8156f70ce56308d93edd475a3c507a
+  metadata.gz: bfd7c0c67a089fb6fa483ae0435e979cab1a4d38
+  data.tar.gz: a5a95d58c6692c0864e7ab732fd872b9203b1ad0
 SHA512:
-  metadata.gz: 1b3918704a355ad1f3c2deaf47d08313c579953d435996e6c66f0de02279bb7cd30d4171edd3f3f7ba54c55af23ac324be38921d874e2a78201fc760a9aa519c
-  data.tar.gz: 318aed62e2ac99cf01e7f0f65656201acd9650fa98c0bb34508057436cb83fafbada8912f41c0925c1133c6ffd6ef4ba61245881efcc74a62a3688ac1bbbdbb3
+  metadata.gz: 82013bb49295bc35dd7acead92634f6f2dbebbd60b686c6e2c5a61aa22e125e1710de84fe892d5bb4f3c7b4b9f9cb064983dd8f8fcddd63df9a198aa08e72afd
+  data.tar.gz: e91d77b0d6e9bc660585da0596f64a483cbc57152cc328b8e3710a986ae1a920528c25c1a20dec38df68599ccccdbfde1b2df9d4a793ab82f4b51c4e295eb3b5

data/lib/omniauth-remote-user/version.rb

@@ -1,5 +1,5 @@
 module Omniauth
   module RemoteUser
-    VERSION = '0.1.1'
+    VERSION = '0.1.2'
   end
 end

data/omniauth-remote-user.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |gem|
   gem.homepage = 'http://beta.softwarepublico.gov.br/gitlab/softwarepublico/omniauth-remote-user'
   gem.authors = ['Lucas Kanashiro', 'Thiago Ribeiro', 'Rodrigo Siqueira','Macartur Sousa', 'Antonio Terceiro']
   gem.require_paths = %w(lib)
-  gem.files = `git ls-files -z`.split("\x0").reject {|f| f.start_with?('spec/')}
+  gem.files = `git ls-files -z`.split("\x0")
   gem.test_files = `git ls-files -- {test,spec,feature}/*`.split("\n")
   gem.license = "Expat"
   gem.required_rubygems_version = '>= 1.3.5'

data/spec/omniauth/strategies/remote_user_spec.rb

@@ -0,0 +1,125 @@
+require 'spec_helper'
+
+describe 'Test Strategy Remote_User' do
+  let(:app) do
+    Rack::Builder.new do |b|
+      b.use Rack::Session::Cookie, :secret => 'abc123'
+      b.use OmniAuth::Strategies::RemoteUser
+      b.run  lambda { |_env| [200, {}, ['My body']] }
+    end.to_app
+  end
+
+  context 'Without HTTP_REMOTE_USER and not logged in' do
+    before(:each){
+      get '/', {}, {}
+    }
+
+    it 'Do nothing' do
+      expect(last_response.status).to eq(200)
+      expect(last_request.cookies['_remote_user']).to eq(nil)
+    end
+  end
+
+  context 'Without HTTP_REMOTE_USER and logged in' do
+    before(:each){
+      clear_cookies
+      set_cookie "_remote_user=test"
+      get '/', {}, {}
+    }
+
+    it 'Logout curreent user' do
+      expect(last_request.cookies['_remote_user']).to eq('test')
+      expect(last_response.status).to eq(302)
+      expect(last_response['Set-Cookie']).to include("_remote_user=")
+      expect(last_response['Set-Cookie']).to include("path=")
+    end
+  end
+
+  context 'With HTTP_REMOTE_USER and not logged in' do
+    before(:each){
+      get '/', {}, { 'HTTP_REMOTE_USER' => 'foobar' }
+    }
+
+    it 'logs HTTP_REMOTE_USER in' do
+      expect(last_response.status).to eq(302)
+      expect(last_response['Set-Cookie']).to include('_remote_user=foobar')
+      expect(last_response['Set-Cookie']).to include('path=')
+    end
+  end
+
+  context 'With HTTP_REMOTE_USER, logged in and current user equals HTTP_REMOTE_USER' do
+    before(:each){
+      clear_cookies
+      set_cookie "_remote_user=foobar"
+      get '/', {}, { 'HTTP_REMOTE_USER' => 'foobar' }
+    }
+
+    it 'Do nothing' do
+      expect(last_request.cookies['_remote_user']).to eq('foobar')
+      expect(last_response.status).to eq(200)
+      expect(last_response['Set-Cookie']).to eq(nil)
+    end
+  end
+
+  context 'With HTTP_REMOTE_USER, logged in and current user not equals HTTP_REMOTE_USER' do
+    before(:each){
+      clear_cookies
+      set_cookie "_remote_user=foobar"
+      get '/', {}, { 'HTTP_REMOTE_USER' => 'foobar2' }
+    }
+
+    it 'Logout current user and login HTTP_REMOTE_USER' do
+      expect(last_request.cookies['_remote_user']).to eq('foobar')
+      expect(last_response.status).to eq(302)
+    end
+  end
+
+  context 'Verify omniauth hash with HTTP_REMOTE_USER_DATA' do
+    before(:each){
+      clear_cookies
+      set_cookie "_remote_user=foobar"
+      post '/auth/RemoteUser/callback', {}, { 'HTTP_REMOTE_USER' => 'foobar',
+                                              'HTTP_REMOTE_USER_DATA' => JSON.dump({'name' => 'foobar barfoo', 'email' => 'foobar@test.com'})}
+    }
+
+    it 'Verify uid' do
+      expect(last_request.env['omniauth.auth']['uid']).to eq('foobar')
+    end
+
+    it 'Verify info' do
+      expect(last_request.env['omniauth.auth']['info']['nickname']).to eq('foobar')
+      expect(last_request.env['omniauth.auth']['info']['email']).to eq('foobar@test.com')
+      expect(last_request.env['omniauth.auth']['info']['lastname']).to eq('barfoo')
+      expect(last_request.env['omniauth.auth']['info']['firstname']).to eq('foobar')
+    end
+  end
+
+  context 'Verify omniauth.auth info without HTTP_REMOTE_USER_DATA' do
+    before(:each){
+      clear_cookies
+      set_cookie "_remote_user=foobar"
+      post '/auth/RemoteUser/callback', {}, { 'HTTP_REMOTE_USER' => 'foobar' }
+    }
+
+    it 'Verify uid' do
+      expect(last_request.env['omniauth.auth']['uid']).to eq('foobar')
+    end
+
+    it 'Verify info' do
+      expect(last_request.env['omniauth.auth']['info']).to eq({})
+    end
+  end
+
+  context 'With HTTP_REMOTE_USER and ' do
+    before(:each){
+      set_cookie "_remote_user=foobar"
+      get "auth/RemoteUser", {}, { 'HTTP_REMOTE_USER' => 'foobar' }
+    }
+
+    it 'redirect for callback' do
+      expect(last_response.status).to eq(302)
+      expect(last_response.location).to eq("/auth/RemoteUser/callback")
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,26 @@
+if RUBY_VERSION >= '1.9'
+	require 'simplecov'
+	require 'coveralls'
+	SimpleCov.formatters = [SimpleCov::Formatter::HTMLFormatter, Coveralls::SimpleCov::Formatter]
+	SimpleCov.start do
+		add_filter '/spec'
+		#minimum_coverage(90)
+	end
+end
+require 'rubygems'
+require 'bundler'
+require 'rack/test'
+require 'rspec'
+require 'rack/test'
+require 'omniauth'
+require 'omniauth/test'
+
+Bundler.setup :default, :development, :test
+
+require 'rack/test'
+require 'omniauth/remote-user'
+
+RSpec.configure do |config|
+	config.include Rack::Test::Methods
+	config.extend OmniAuth::Test::StrategyMacros, :type => :strategy
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-remote-user
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Lucas Kanashiro
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-18 00:00:00.000000000 Z
+date: 2016-03-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth
@@ -52,6 +52,8 @@ files:
 - lib/omniauth/strategies/remote_user.rb
 - omniauth-remote-user.gemspec
 - references/teste.rb
+- spec/omniauth/strategies/remote_user_spec.rb
+- spec/spec_helper.rb
 - test_notes.txt
 homepage: http://beta.softwarepublico.gov.br/gitlab/softwarepublico/omniauth-remote-user
 licenses:
@@ -73,9 +75,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.5
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Authentication with HTTP Remote User
 test_files: []
-has_rdoc: