Permalink
Browse files

Revert "Revert "New password default minimum length is now 8""

This reverts commit b1f490a.
Actually, the implementation was good!
  • Loading branch information...
1 parent b1f490a commit bdf0bc7b1e66d1a70e4ecda8507145658a4f1a96 @josevalim josevalim committed Jul 24, 2012
Showing with 5 additions and 5 deletions.
  1. +1 −1 lib/devise.rb
  2. +1 −1 lib/generators/templates/devise.rb
  3. +3 −3 test/models/validatable_test.rb
View
@@ -91,7 +91,7 @@ module Strategies
# Range validation for password length
mattr_accessor :password_length
- @@password_length = 8..128
+ @@password_length = 6..128
# The time the user will be remembered without asking for credentials again.
mattr_accessor :remember_for
@@ -122,7 +122,7 @@
# ==> Configuration for :validatable
# Range for password length. Default is 8..128.
- # config.password_length = 8..128
+ config.password_length = 8..128
# Email regex used to validate email formats. It simply asserts that
# an one (and only one) @ exists in the given string. This is mainly
@@ -76,10 +76,10 @@ class ValidatableTest < ActiveSupport::TestCase
assert_equal 'doesn\'t match confirmation', user.errors[:password].join
end
- test 'should require a password with minimum of 8 characters' do
- user = new_user(:password => '1234567', :password_confirmation => '1234567')
+ test 'should require a password with minimum of 6 characters' do
+ user = new_user(:password => '12345', :password_confirmation => '12345')
assert user.invalid?
- assert_equal 'is too short (minimum is 8 characters)', user.errors[:password].join
+ assert_equal 'is too short (minimum is 6 characters)', user.errors[:password].join
end
test 'should require a password with maximum of 128 characters long' do

0 comments on commit bdf0bc7

Please sign in to comment.