authpwn_rails 0.17.0 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e9776c5aa80187c9368f044da7036cb46c872e3e
-  data.tar.gz: 2df0b77fc051007d52ea662efb289bedcbbd6ead
+  metadata.gz: 32109345928f563a79ad4bcf0eb49d53012a0ee9
+  data.tar.gz: 23a263174be62eb9853a7a74aea220156c319cd9
 SHA512:
-  metadata.gz: 3a3c415f26386f64090d662398eee705a573ecf6f10cff87ae754eb3f2e65d26f729e4cc6ac91d494c8f36acc4a560a1520b711b99d933e6609dee2d2c151e26
-  data.tar.gz: 261ff309c4d7c7b8bc404d13b4fe7da501801f68c2b4881d9fa55a573ef253bb7f54c16106aa17b2d51d6b53d2843887afa1f9b1e0d52bf5752275f8c1561201
+  metadata.gz: c1fe2c3cbc72389b9d776c54a081e90b09bb988f7bc24bfe9efcba25372372b61e1788a02e88f8cd4ac9f8e2079228cf22c2cd2513ba935cf5a1eb084c61f569
+  data.tar.gz: 7033ce674a9e6e87dcb8c6c693d80677e12f4c96cc0a9b397814ac7745313af7bf0e85dda9138645f4f3f044b62e9b33cfe5a9cf8e2968446a8d802ca524c12e

data/VERSION

@@ -1 +1 @@
-0.17.0
+0.17.1

data/authpwn_rails.gemspec

@@ -2,11 +2,11 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: authpwn_rails 0.17.0 ruby lib
+# stub: authpwn_rails 0.17.1 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "authpwn_rails"
-  s.version = "0.17.0"
+  s.version = "0.17.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]

data/lib/authpwn_rails/generators/templates/session_controller_test.rb

@@ -140,4 +140,14 @@ class SessionControllerTest < ActionController::TestCase
     assert_nil Tokens::Base.with_code(old_token.code).first,
                'old session not purged'
   end
+
+  test "OmniAuth login via developer strategy and new account" do
+    request.env['omniauth.auth'] = {
+        'provider' => @omniauth_credential.provider,
+        'uid' => 'new_user_gmail_com_uid',
+        'info' => { 'email' => 'new_user@gmail.com' } }
+    post :omniauth, provider: @omniauth_credential.provider
+    assert_not_nil session_current_user, 'session'
+    assert_redirected_to session_url
+  end
 end

data/lib/authpwn_rails/user_extensions/password_field.rb

@@ -12,7 +12,7 @@ module PasswordField
   extend ActiveSupport::Concern
 
   included do
-    validates :password, presence: { on: :create },
+    validates :password, length: { minimum: 1, on: :create, allow_nil: true },
                          confirmation: { allow_nil: true }
   end
 

data/lib/authpwn_rails/user_model.rb

@@ -94,8 +94,9 @@ module UserModel
     def create_from_omniauth(omniauth_hash)
       info_hash = omniauth_hash['info']
       return nil unless email = info_hash && info_hash['email']
-      user = User.create!
-      Credentials::Email.create! user: user, email: email, verified: true
+      user = User.new
+      user.credentials << Credentials::Email.new(email: email, verified: true)
+      user.save!
       user
     end
   end  # module Authpwn::UserModel::ClassMethods

data/test/user_extensions/password_field_test.rb

@@ -8,31 +8,36 @@ class PasswordFieldTest < ActiveSupport::TestCase
   def setup
     @user = UserWithPassword.new password: 'awesome',
                                  password_confirmation: 'awesome'
-    
+
     @john = UserWithPassword.find_by_id(users(:john).id)
     @jane = UserWithPassword.find_by_id(users(:jane).id)
     @bill = UserWithPassword.find_by_id(users(:bill).id)
   end
-  
+
   test 'setup' do
     assert @user.valid?
   end
-  
-  test 'password required' do
-    @user.password = @user.password_confirmation = nil
+
+  test 'password can be missing for non-password logins' do
+    user = UserWithPassword.new
+    assert user.valid?
+  end
+
+  test 'password cannot be empty' do
+    @user.password = @user.password_confirmation = ''
     assert !@user.valid?
   end
-  
+
   test 'password assumed ok for existing records' do
     @john.save!
     assert @john.valid?
   end
-  
+
   test 'password confirmation' do
     @user.password_confirmation = 'not awesome'
     assert !@user.valid?
   end
-  
+
   test 'password_credential' do
     assert_equal credentials(:john_password), @john.password_credential
     assert_equal credentials(:jane_password), @jane.password_credential

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: authpwn_rails
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.17.1
 platform: ruby
 authors:
 - Victor Costan