Unify all locations where we determine a user's display named based off of their nickname/username into a helper function

1 files changed, 13 insertions, 0 deletions

diff --git a/model/user_test.go b/model/user_test.go
index 4db099f97..a0c62676a 100644
--- a/model/user_test.go
+++ b/model/user_test.go
@@ -107,3 +107,16 @@ func TestUserIsValid(t *testing.T) {
 		t.Fatal(err)
 	}
 }
+
+func TestUserGetDisplayName(t *testing.T) {
+	user := User{FirstName: "first", LastName: "last", Username: "user"}
+
+	if displayName := user.GetDisplayName(); displayName != "user" {
+		t.Fatal("Display name should be username")
+	}
+
+	user.Nickname = "nickname"
+	if displayName := user.GetDisplayName(); displayName != "nickname" {
+		t.Fatal("Display name should be nickname")
+	}
+}