From 17d6795921c70b1e2707f5e7793a835d73800b47 Mon Sep 17 00:00:00 2001
From: zchh < zc123zhangchi@gmail.com>
Date: Mon, 18 Jan 2021 19:56:07 +0800
Subject: [PATCH] =?UTF-8?q?=E5=AE=8C=E5=96=84=E4=BB=A3=E7=A0=81?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 core/codes/codes.go      | 5 ++++-
 core/error/error.go      | 7 ++++---
 core/error/error_test.go | 3 ++-
 3 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/core/codes/codes.go b/core/codes/codes.go
index 7938fea7..77f59220 100644
--- a/core/codes/codes.go
+++ b/core/codes/codes.go
@@ -1,11 +1,14 @@
 package codes
 
+// A Code is an unsigned 32-bit error code as defined in the beego spec.
 type Code uint32
 
 const (
+	// SessionSessionStartError means func SessionStart error in session module.
 	SessionSessionStartError Code = 5001001
 )
 
+// CodeToStr is a map about Code and Code's message
 var CodeToStr = map[Code]string{
 	SessionSessionStartError : `"SESSION_MODULE_SESSION_START_ERROR"`,
-}
+}
\ No newline at end of file
diff --git a/core/error/error.go b/core/error/error.go
index 50cce244..6caf9839 100644
--- a/core/error/error.go
+++ b/core/error/error.go
@@ -6,9 +6,10 @@ import (
 	"strconv"
 )
 
-// The `Error`type defines custom error for Beego. It is used by every module
+// Error type defines custom error for Beego. It is used by every module
 // in Beego. Each `Error` message contains three pieces of data: error code,
-// error message. More docs http://beego.me/docs/module/error.md
+// error message.
+// More docs http://beego.me/docs/module/error.md.
 type Error struct {
     Code codes.Code
     Msg string
@@ -35,7 +36,7 @@ func (e *Error) Error() string {
 	return fmt.Sprintf("beego error: code = %s desc = %s", codeSrt, e.GetMessage())
 }
 
-// GetCode returns Error's Code
+// GetCode returns Error's Code.
 func (e *Error) GetCode() codes.Code {
 	if e != nil {
 		return e.Code
diff --git a/core/error/error_test.go b/core/error/error_test.go
index 30b61164..630b277c 100644
--- a/core/error/error_test.go
+++ b/core/error/error_test.go
@@ -118,7 +118,7 @@ func TestErrorf(t *testing.T) {
 		wantErr bool
 	}{
 		// TODO: Add test cases.
-		{name: "1", args: args{codes.SessionSessionStartError, "%s", []interface{}{codes.CodeToStr[codes.SessionSessionStartError]}}},
+		{name: "1", args: args{codes.SessionSessionStartError, "%s", []interface{}{codes.CodeToStr[codes.SessionSessionStartError]}}, wantErr: true},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
@@ -140,6 +140,7 @@ func TestNew(t *testing.T) {
 		want *Error
 	}{
 		// TODO: Add test cases.
+		{name: "1", args: args{codes.SessionSessionStartError, codes.CodeToStr[codes.SessionSessionStartError]}, want: &Error{Code:codes.SessionSessionStartError,  Msg:codes.CodeToStr[codes.SessionSessionStartError]}},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {