Merge pull request #31 from turniere/ticket/TURNIERE-117

Check password for validity (length)

app/models/user.rb

@@ -4,11 +4,11 @@ class User < ApplicationRecord
   extend Devise::Models
 
   devise :database_authenticatable, :registerable,
-         :recoverable, :rememberable, :trackable
+         :recoverable, :rememberable, :trackable, :validatable
 
   include DeviseTokenAuth::Concerns::User
 
-  validates :username, presence: true, uniqueness: true
+  validates :username, presence: true, uniqueness: { case_sensitive: false }
 
   has_many :tournaments, dependent: :destroy
 end

spec/factories/users.rb

@@ -4,5 +4,6 @@ FactoryBot.define do
   factory :user do
     username { Faker::Internet.unique.username }
     email { Faker::Internet.unique.email }
+    password { Faker::Internet.password }
   end
 end

spec/models/user_spec.rb

@@ -7,6 +7,12 @@ RSpec.describe User, type: :model do
     it { should have_many :tournaments }
   end
 
+  describe 'validation' do
+    subject { create(:user) }
+    it { should validate_presence_of :username }
+    it { should validate_uniqueness_of(:username).case_insensitive }
+  end
+
   it 'has a valid factory' do
     expect(build(:user)).to be_valid
   end