From f3ae27649bf82ff049e6ccbdd7200418703d4532 Mon Sep 17 00:00:00 2001
From: wemeya <wemeya@163.com>
Date: Mon, 23 May 2016 14:24:42 +0800
Subject: [PATCH] modify test code

---
 dao/dao_test.go | 48 +++++++++++++++++++++++-------------------------
 1 file changed, 23 insertions(+), 25 deletions(-)

diff --git a/dao/dao_test.go b/dao/dao_test.go
index b631e427d..85b104853 100644
--- a/dao/dao_test.go
+++ b/dao/dao_test.go
@@ -702,6 +702,29 @@ func TestToggleAdminRole(t *testing.T) {
 	}
 }
 
+func TestChangeUserProfile(t *testing.T) {
+	user := models.User{UserID: currentUser.UserID, Email: username + "@163.com", Realname: "test", Comment: "Unit Test"}
+	err := ChangeUserProfile(user)
+	if err != nil {
+		t.Errorf("Error occurred in ChangeUserProfile: %v", err)
+	}
+	loginedUser, err := GetUser(models.User{UserID: currentUser.UserID})
+	if err != nil {
+		t.Errorf("Error occurred in GetUser: %v", err)
+	}
+	if loginedUser != nil {
+		if loginedUser.Email != username+"@163.com" {
+			t.Errorf("user email does not update, expected: %s, acutal: %s", username+"@163.com", loginedUser.Email)
+		}
+		if loginedUser.Realname != "test" {
+			t.Errorf("user realname does not update, expected: %s, acutal: %s", "test", loginedUser.Realname)
+		}
+		if loginedUser.Comment != "Unit Test" {
+			t.Errorf("user email does not update, expected: %s, acutal: %s", "Unit Test", loginedUser.Comment)
+		}
+	}
+}
+
 func TestDeleteUser(t *testing.T) {
 	err := DeleteUser(currentUser.UserID)
 	if err != nil {
@@ -716,31 +739,6 @@ func TestDeleteUser(t *testing.T) {
 	}
 }
 
-func TestChangeUserProfile(t *testing.T) {
-	user := models.User{UserID: currentUser.UserID, Email: currentUser.Username + "@163.com", Realname: "test", Comment: "Unit Test"}
-	err := ChangeUserProfile(user)
-	if err != nil {
-		t.Errorf("Error occurred in ChangeUserProfile: %v", err)
-	}
-	loginedUser, err := GetUser(models.User{UserID: currentUser.UserID})
-	if err != nil {
-		t.Errorf("Error occurred in GetUser: %v", err)
-	}
-	log.Debugf("loginedUser: %v", loginedUser)
-	log.Debugf("loginedUser: %v", loginedUser.Email)
-	log.Debugf("loginedUser: %v", loginedUser.Realname)
-	log.Debugf("loginedUser: %v", loginedUser.Comment)
-	if loginedUser.Email != username+"@163.com" {
-		t.Errorf("user email does not update, expected: %s, acutal: %s", username+"@163.com", loginedUser.Email)
-	}
-	if loginedUser.Realname != "test" {
-		t.Errorf("user realname does not update, expected: %s, acutal: %s", "test", loginedUser.Realname)
-	}
-	if loginedUser.Comment != "Unit Test" {
-		t.Errorf("user email does not update, expected: %s, acutal: %s", "Unit Test", loginedUser.Comment)
-	}
-}
-
 func TestGetRecentLogs(t *testing.T) {
 	logs, err := GetRecentLogs(10, "2016-05-13 00:00:00", time.Now().String())
 	if err != nil {