devise_openid_authenticatable 1.2.1 → 1.3.0

data/spec/scenario/config/environments/test.rb

@@ -8,7 +8,8 @@ Scenario::Application.configure do
   config.cache_classes = true
 
   # Log error messages when you accidentally call methods on nil.
-  config.whiny_nils = true
+  # DEPRECATION WARNING: config.whiny_nils option is deprecated and no longer works.
+  # config.whiny_nils = true
 
   # Show full error reports and disable caching
   config.consider_all_requests_local       = true
@@ -30,6 +31,6 @@ Scenario::Application.configure do
   config.action_dispatch.show_exceptions = false
 
   config.active_support.deprecation = :stderr
-  
+
   config.eager_load = false
 end

data/spec/scenario/config/initializers/devise.rb

@@ -0,0 +1,3 @@
+if Devise.respond_to?(:secret_key=)
+  Devise.secret_key = '9d516d28ce51f3647b770b7ce5b3e39fb171670dd68be0d0746cf33410050dceb033829316ea2b13ea774072945988f971488baf9647a6ec72959c1596487548'
+end

data/spec/scenario/config/initializers/secret_token.rb

@@ -1,2 +1,5 @@
 Rails.application.config.secret_token = 'ea942c41850d502f2c8283e26bdc57829f471bb18224ddff0a192c4f32cdf6cb5aa0d82b3a7a7adbeb640c4b06f3aa1cd5f098162d8240f669b39d6b49680571'
-Rails.application.config.session_store :cookie_store, :key => "_my_scenario"
+Rails.application.config.session_store :cookie_store, key: "_my_scenario"
+if Scenario::Application.config.respond_to?(:secret_key_base=)
+  Scenario::Application.config.secret_key_base = '67ecc07dd52ce53da248745c6e601603534de2250c8b7add5c7f9a791c1b4684d808d28a35ac6b31a6b12932b461afbe6fda8ccc2232466ee68a9bd16bac06b0'
+end

data/spec/spec_helper.rb

@@ -13,8 +13,14 @@ Webrat.configure do |config|
   config.open_error_files = false
 end
 
-RSpec.configure do |config| 
-  config.mock_with :mocha 
+RSpec.configure do |config|
+  %i(controller view request).each do |type|
+    config.include ::Rails::Controller::Testing::TestProcess,        type: type
+    config.include ::Rails::Controller::Testing::TemplateAssertions, type: type
+    config.include ::Rails::Controller::Testing::Integration,        type: type
+  end if Rails.version >= '5'
+  config.mock_with :mocha
+  config.infer_spec_type_from_file_location!
 end
 
 # This is mostly copied in from bin/rots; they don't provide a single app class we could just reuse, unfortunately
@@ -33,7 +39,7 @@ ROTS_CONFIG
 rots_server_options = {
   :storage => File.join(File.dirname(__FILE__), 'scenario', 'tmp', 'rots')
 }
-rots_server = Rack::Builder.new do 
+rots_server = Rack::Builder.new do
   use Rack::Lint
   map ("/%s" % rots_config['identity']) do
     run Rots::IdentityPageApp.new(rots_config, rots_server_options)
@@ -45,4 +51,4 @@ end
 
 ShamRack.mount(rots_server, 'openid.example.org')
 
-Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }

data/spec/strategy_spec.rb

@@ -39,11 +39,11 @@ describe Devise::Strategies::OpenidAuthenticatable do
     User.create! do |u|
       u.identity_url = "http://openid.example.org/myid"
     end
-    
+
     LegacyUser.create! do |u|
       u.identity_url = "http://openid.example.org/myid"
     end
-    
+
     DatabaseUser.create! do |u|
       u.email = "dbuser@example.com"
       u.password = "password"
@@ -104,18 +104,18 @@ describe Devise::Strategies::OpenidAuthenticatable do
       response.should redirect_to('http://openid.example.org/server')
     end
   end
-  
+
   describe "POST /users/sign_in (with rememberable)" do
     before do
       post '/users/sign_in', 'user' => { 'identity_url' => 'http://openid.example.org/myid', 'remember_me' => 1 }
     end
-    
+
     it 'should forward request to provider, with params preserved' do
       response.should be_redirect
       redirect_uri = URI.parse(response.header['Location'])
       redirect_uri.host.should == "openid.example.org"
       redirect_uri.path.should match(/^\/server/)
-      
+
       # Crack open the redirect URI and extract the return parameter from it, then parse it too
       req = Rack::Request.new(Rack::MockRequest.env_for(redirect_uri.to_s))
       return_req = Rack::Request.new(Rack::MockRequest.env_for(req.params['openid.return_to']))
@@ -163,11 +163,11 @@ describe Devise::Strategies::OpenidAuthenticatable do
     end
 
     it 'should update user-records with retrieved information' do
-      User.should have(1).record
+      expect(User.count).to eq 1
       User.first.email.should == 'dimitrij@example.com'
     end
   end
-  
+
   describe "POST /users/sign_in (from OpenID provider, success, rememberable)" do
 
     before do
@@ -182,12 +182,25 @@ describe Devise::Strategies::OpenidAuthenticatable do
     end
 
     it 'should update user-records with retrieved information and remember token' do
-      User.should have(1).record
+      expect(User.count).to eq 1
       User.first.email.should == 'dimitrij@example.com'
       User.first.remember_token.should_not be_nil
     end
   end
 
+  describe "POST /users/sign_in (from OpenID provider, success, NOT rememberable)" do
+    before do
+      stub_completion
+      post '/users/sign_in', openid_params.merge("_method"=>"post", "user" => { "remember_me" => 0 })
+    end
+
+    it 'should update user-records with retrieved information but not remember token' do
+      expect(User.count).to eq 1
+      User.first.email.should == 'dimitrij@example.com'
+      User.first.remember_token.should be_nil
+    end
+  end
+
   describe "POST /users/sign_in (from OpenID provider, success, new user)" do
 
     before do
@@ -203,26 +216,26 @@ describe Devise::Strategies::OpenidAuthenticatable do
     end
 
     it 'should auto-create user-records (if supported)' do
-      User.should have(2).records
+      User.count.should eq 2
     end
 
     it 'should update new user-records with retrieved information' do
       User.order(:id).last.email.should == 'dimitrij@example.com'
     end
   end
-  
+
   describe "POST /legacy_users/sign_in (from OpenID provider, success, new user)" do
 
     before do
       @previous_logger = Rails.logger
       @log_output = StringIO.new
       Rails.logger = Logger.new(@log_output)
-      
+
       @identity = 'http://openid.example.org/newid'
       stub_completion
       post '/legacy_users/sign_in', openid_params.merge("_method"=>"post")
     end
-    
+
     after do
       Rails.logger = @previous_logger
     end
@@ -234,32 +247,32 @@ describe Devise::Strategies::OpenidAuthenticatable do
     end
 
     it 'should auto-create user-records (if supported)' do
-      LegacyUser.should have(2).records
+      LegacyUser.count.should eq 2
     end
 
     it 'should update new user-records with retrieved information' do
       LegacyUser.order(:id).last.email.should == 'dimitrij@example.com'
     end
-    
+
     it 'should issue a deprecation warning' do
       @log_output.string.should =~ /DEPRECATION WARNING: create_from_identity_url/
     end
   end
 
   describe "POST /database_users/sign_in (using database authentication)" do
-    
+
     before do
       post '/database_users/sign_in', :database_user => { :email => "dbuser@example.com", :password => "password" }
     end
-    
+
     it 'should accept authentication with success' do
       response.should be_redirect
       response.should redirect_to('http://www.example.com/')
       flash[:notice].should match(/success/i)
     end
-    
+
   end
-  
+
   describe "POST /database_users/sign_in (using OpenID, begin_authentication)" do
     before do
       Rack::OpenID.any_instance.stubs(:begin_authentication).returns([302, {'Location' => 'http://openid.example.org/server'}, ['']])
@@ -271,7 +284,7 @@ describe Devise::Strategies::OpenidAuthenticatable do
       response.should redirect_to('http://openid.example.org/server')
     end
   end
-  
+
   describe "POST /database_users/sign_in (using OpenID, from provider, existing user)" do
     before do
       stub_completion
@@ -285,11 +298,11 @@ describe Devise::Strategies::OpenidAuthenticatable do
     end
 
     it 'should update user-records with retrieved information' do
-      DatabaseUser.should have(1).record
+      DatabaseUser.count.should eq 1
       DatabaseUser.first.email.should == 'dimitrij@example.com'
     end
   end
-  
+
   describe "POST /database_users/sign_in (using OpenID, from provider, existing email)" do
     before do
       DatabaseUser.delete_all
@@ -297,7 +310,7 @@ describe Devise::Strategies::OpenidAuthenticatable do
         u.email = "dimitrij@example.com"
         u.password = "password"
       end
-      
+
       stub_completion
       post '/database_users/sign_in', openid_params.merge("_method"=>"post")
     end
@@ -306,10 +319,10 @@ describe Devise::Strategies::OpenidAuthenticatable do
       response.should be_success
       response.should render_template("sessions/new")
       flash[:alert].should match(/email/i)
-      DatabaseUser.should have(1).record
+      DatabaseUser.count.should eq 1
     end
   end
-  
+
   describe "POST /database_users/sign_in (using OpenID, from provider, forgery attempt)" do
     before do
       DatabaseUser.delete_all
@@ -318,7 +331,7 @@ describe Devise::Strategies::OpenidAuthenticatable do
         u.password = "password"
         u.identity_url = "http://openid.example.org/different_id"
       end
-      
+
       stub_completion
       post '/database_users/sign_in', openid_params.merge("_method"=>"post")
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: devise_openid_authenticatable
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.3.0
 platform: ruby
 authors:
 - Nat Budin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-05 00:00:00.000000000 Z
+date: 2016-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack-openid
@@ -122,6 +122,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: roman-rots
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: OpenID authentication module for Devise using Rack::OpenID
 email:
 - natbudin@gmail.com
@@ -134,10 +148,21 @@ files:
 - CHANGELOG.rdoc
 - Gemfile
 - Gemfile.devise13
+- Gemfile.devise13.lock
 - Gemfile.devise14
+- Gemfile.devise14.lock
 - Gemfile.devise15
+- Gemfile.devise15.lock
 - Gemfile.devise21
+- Gemfile.devise21.lock
 - Gemfile.devise30
+- Gemfile.devise30.lock
+- Gemfile.devise33
+- Gemfile.devise33.lock
+- Gemfile.devise35
+- Gemfile.devise35.lock
+- Gemfile.devise42
+- Gemfile.devise42.lock
 - LICENSE
 - README.md
 - Rakefile
@@ -150,7 +175,72 @@ files:
 - lib/devise_openid_authenticatable/schema.rb
 - lib/devise_openid_authenticatable/strategy.rb
 - lib/devise_openid_authenticatable/version.rb
-- rails/init.rb
+- rails/Rakefile
+- rails/app/active_record/admin.rb
+- rails/app/active_record/shim.rb
+- rails/app/active_record/user.rb
+- rails/app/active_record/user_on_engine.rb
+- rails/app/active_record/user_on_main_app.rb
+- rails/app/active_record/user_without_email.rb
+- rails/app/controllers/admins/sessions_controller.rb
+- rails/app/controllers/admins_controller.rb
+- rails/app/controllers/application_controller.rb
+- rails/app/controllers/application_with_fake_engine.rb
+- rails/app/controllers/custom/registrations_controller.rb
+- rails/app/controllers/home_controller.rb
+- rails/app/controllers/publisher/registrations_controller.rb
+- rails/app/controllers/publisher/sessions_controller.rb
+- rails/app/controllers/users/omniauth_callbacks_controller.rb
+- rails/app/controllers/users_controller.rb
+- rails/app/helpers/application_helper.rb
+- rails/app/mailers/users/from_proc_mailer.rb
+- rails/app/mailers/users/mailer.rb
+- rails/app/mailers/users/reply_to_mailer.rb
+- rails/app/mongoid/admin.rb
+- rails/app/mongoid/shim.rb
+- rails/app/mongoid/user.rb
+- rails/app/mongoid/user_on_engine.rb
+- rails/app/mongoid/user_on_main_app.rb
+- rails/app/mongoid/user_without_email.rb
+- rails/app/views/admins/index.html.erb
+- rails/app/views/admins/sessions/new.html.erb
+- rails/app/views/home/admin_dashboard.html.erb
+- rails/app/views/home/index.html.erb
+- rails/app/views/home/join.html.erb
+- rails/app/views/home/private.html.erb
+- rails/app/views/home/user_dashboard.html.erb
+- rails/app/views/layouts/application.html.erb
+- rails/app/views/users/edit_form.html.erb
+- rails/app/views/users/index.html.erb
+- rails/app/views/users/mailer/confirmation_instructions.erb
+- rails/app/views/users/sessions/new.html.erb
+- rails/bin/bundle
+- rails/bin/rails
+- rails/bin/rake
+- rails/config.ru
+- rails/config/application.rb
+- rails/config/boot.rb
+- rails/config/database.yml
+- rails/config/environment.rb
+- rails/config/environments/development.rb
+- rails/config/environments/production.rb
+- rails/config/environments/test.rb
+- rails/config/initializers/backtrace_silencers.rb
+- rails/config/initializers/devise.rb
+- rails/config/initializers/inflections.rb
+- rails/config/initializers/secret_token.rb
+- rails/config/initializers/session_store.rb
+- rails/config/routes.rb
+- rails/db/migrate/20100401102949_create_tables.rb
+- rails/db/schema.rb
+- rails/lib/shared_admin.rb
+- rails/lib/shared_user.rb
+- rails/lib/shared_user_without_email.rb
+- rails/lib/shared_user_without_omniauth.rb
+- rails/public/404.html
+- rails/public/422.html
+- rails/public/500.html
+- rails/public/favicon.ico
 - spec/model_spec.rb
 - spec/scenario/app/controllers/application_controller.rb
 - spec/scenario/app/controllers/home_controller.rb
@@ -167,6 +257,7 @@ files:
 - spec/scenario/config/environments/development.rb
 - spec/scenario/config/environments/test.rb
 - spec/scenario/config/initializers/backtrace_silencers.rb
+- spec/scenario/config/initializers/devise.rb
 - spec/scenario/config/initializers/inflections.rb
 - spec/scenario/config/initializers/secret_token.rb
 - spec/scenario/config/routes.rb
@@ -186,7 +277,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -215,6 +306,7 @@ test_files:
 - spec/scenario/config/environments/development.rb
 - spec/scenario/config/environments/test.rb
 - spec/scenario/config/initializers/backtrace_silencers.rb
+- spec/scenario/config/initializers/devise.rb
 - spec/scenario/config/initializers/inflections.rb
 - spec/scenario/config/initializers/secret_token.rb
 - spec/scenario/config/routes.rb

data/rails/init.rb

@@ -1 +0,0 @@
-require "devise_openid_authenticatable"