diff --git a/app/models/user.rb b/app/models/user.rb index 5a14e885..d548853e 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -53,6 +53,10 @@ class User < ActiveRecord::Base }, presence: true + def name + self[:name] || username + end + def subscribed? conference self.subscriptions.find_by(conference_id: conference.id).present? end diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index 36c6153a..01038a6e 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -33,6 +33,13 @@ describe User do expect(another_user.roles[1]).to eq(cfp_role) end + describe '#name' do + it 'returns the username as name if there is not name' do + user = create(:user, name: nil) + expect(user.name).to eq(user.username) + end + end + describe '#has_role?' do describe 'when user has a role' do it 'returns true when the user has the role' do