diff --git a/app/models/user.rb b/app/models/user.rb index d5800e7b0..183003195 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -40,7 +40,7 @@ class User < ApplicationRecord length: { maximum: 50 }, email: true, allow_blank: true - validates :name, length: { maximum: 100 } + validates :name, length: { minimum: 1, maximum: 100 } validates :username, presence: true, uniqueness: { case_sensitive: false }, diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index 55d6c7712..234acb541 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -145,6 +145,13 @@ RSpec.describe User, type: :model do expect(user.old_old_username).to eq(old_username) end + it "does not allow too short or too long name" do + user.name = "" + expect(user).not_to be_valid + user.name = Faker::Lorem.paragraph_by_chars(200) + expect(user).not_to be_valid + end + it "does not accept invalid employer url" do user.employer_url = "ben.com" expect(user).not_to be_valid