Skip to content

Commit 80e06eb

Browse files
committed
feat: Use email as fallback for User.full_name
1 parent 456b26a commit 80e06eb

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

app/models/user.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ def last_name
5656
end
5757

5858
def full_name
59-
return "" if questionnaire.blank?
59+
return email if questionnaire.blank?
6060
questionnaire.full_name
6161
end
6262

test/models/user_test.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@ class UserTest < ActiveSupport::TestCase
4848
assert_equal "Alpha Beta", questionnaire.user.full_name
4949
end
5050

51-
should "return blank when no questionnaire exists" do
52-
user = create(:user)
53-
assert_equal "", user.full_name
51+
should "return email when no questionnaire exists" do
52+
user = create(:user, email: "[email protected]")
53+
assert_equal "[email protected]", user.full_name
5454
end
5555
end
5656

0 commit comments

Comments
 (0)