tests: email tool add more test case

Author: 201430098137

tool/email/sendmail.go

@@ -52,7 +52,6 @@ type sendMailResponse struct {
 }
 
 // sendMail performs the send mail operation.
-// go smtp not support context, one send one mail, can't stop
 func (e *emailToolSet) sendMail(ctx context.Context, req *sendMailRequest) (rsp *sendMailResponse, err error) {
 	rsp = &sendMailResponse{}
 
@@ -81,7 +80,7 @@ func (e *emailToolSet) sendMail(ctx context.Context, req *sendMailRequest) (rsp
 		opts...,
 	)
 	if err != nil {
-		rsp.Message = fmt.Sprintf("the address or password is incorrect,please check: %v", err)
+		rsp.Message = fmt.Sprintf("the server address err: %v", err)
 		return rsp, nil
 	}
 

tool/email/sendmail_test.go

@@ -7,7 +7,7 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
-func TestMailTool_sendMail(t *testing.T) {
+func Test_emailToolSet_sendMail(t *testing.T) {
 	toolSet, err := NewToolSet(
 		WithSendEmailEnabled(true),
 	)
@@ -21,6 +21,7 @@ func TestMailTool_sendMail(t *testing.T) {
 		ToEmail  string
 		Subject  string
 		Content  string
+		wantErr  bool
 	}{
 
 		{
@@ -29,6 +30,7 @@ func TestMailTool_sendMail(t *testing.T) {
 			ToEmail:  "[email protected]",
 			Subject:  "test",
 			Content:  "test",
+			wantErr:  false,
 		},
 
 		{
@@ -37,14 +39,29 @@ func TestMailTool_sendMail(t *testing.T) {
 			ToEmail:  "[email protected]",
 			Subject:  "test",
 			Content:  "test",
+			wantErr:  false,
+		},
+		// error case
+		{
+			Name:     "18503@[email protected]",
+			Password: "",
+			ToEmail:  "[email protected]",
+			Subject:  "test",
+			Content:  "test",
+			wantErr:  true,
+		},
+		// error case
+		{
+			Name:     "[email protected]",
+			Password: "",
+			ToEmail:  "185039@[email protected]",
+			Subject:  "test",
+			Content:  "test",
+			wantErr:  true,
 		},
 	}
 	for _, tt := range tests {
 
-		if tt.Password == "" {
-			t.Skip("no passwd skip")
-		}
-
 		rsp, err := toolSet.(*emailToolSet).sendMail(context.Background(), &sendMailRequest{
 			Auth: Auth{
 				Name:     tt.Name,
@@ -58,10 +75,19 @@ func TestMailTool_sendMail(t *testing.T) {
 				},
 			},
 		})
+		t.Logf("rsp: %+v err:%v", rsp, err)
+		if tt.Password == "" {
+			t.Logf("password is empty, skip")
+			continue
+		}
 		if err != nil {
-			t.Errorf("send mail err: %v", err)
+			if tt.wantErr == true {
+				t.Errorf("send mail err: %v", err)
+			}
+		} else if tt.wantErr == false {
+			t.Errorf("should err but not")
 		}
-		t.Logf("rsp: %+v", rsp)
+
 	}
 }
 
@@ -147,6 +173,12 @@ func Test_checkMailBoxType(t *testing.T) {
 			want:    MAIL_GMAIL,
 			wantErr: false,
 		},
+		{
+			name:    "not valid email",
+			args:    args{email: "UserGmAiL.cOm"},
+			want:    MAIL_GMAIL,
+			wantErr: true,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {