Use a user's full name as their display name if a nickname hasn't been specified

1 files changed, 31 insertions, 1 deletions

diff --git a/model/user_test.go b/model/user_test.go
index a0c62676a..a48c3f2e7 100644
--- a/model/user_test.go
+++ b/model/user_test.go
@@ -108,13 +108,43 @@ func TestUserIsValid(t *testing.T) {
 	}
 }
 
+func TestUserGetFullName(t *testing.T) {
+	user := User{}
+
+	if fullName := user.GetFullName(); fullName != "" {
+		t.Fatal("Full name should be blank")
+	}
+
+	user.FirstName = "first"
+	if fullName := user.GetFullName(); fullName != "first" {
+		t.Fatal("Full name should be first name")
+	}
+
+	user.FirstName = ""
+	user.LastName = "last"
+	if fullName := user.GetFullName(); fullName != "last" {
+		t.Fatal("Full name should be last name")
+	}
+
+	user.FirstName = "first"
+	if fullName := user.GetFullName(); fullName != "first last" {
+		t.Fatal("Full name should be first name and last name")
+	}
+}
+
 func TestUserGetDisplayName(t *testing.T) {
-	user := User{FirstName: "first", LastName: "last", Username: "user"}
+	user := User{Username: "user"}
 
 	if displayName := user.GetDisplayName(); displayName != "user" {
 		t.Fatal("Display name should be username")
 	}
 
+	user.FirstName = "first"
+	user.LastName = "last"
+	if displayName := user.GetDisplayName(); displayName != "first last" {
+		t.Fatal("Display name should be full name")
+	}
+
 	user.Nickname = "nickname"
 	if displayName := user.GetDisplayName(); displayName != "nickname" {
 		t.Fatal("Display name should be nickname")