feat: return error message in the grpc server (#34)

Author: LinuxSuRen

pkg/server/remote_server.go

@@ -105,6 +105,7 @@ func (s *server) Run(ctx context.Context, task *TestTask) (reply *HelloReply, er
 	}
 
 	buf := new(bytes.Buffer)
+	reply = &HelloReply{}
 
 	for _, testCase := range suite.Items {
 		simpleRunner := runner.NewSimpleTestCaseRunner()
@@ -116,14 +117,14 @@ func (s *server) Run(ctx context.Context, task *TestTask) (reply *HelloReply, er
 			testCase.Request.API = fmt.Sprintf("%s%s", suite.API, testCase.Request.API)
 		}
 
-		var output interface{}
-		if output, err = simpleRunner.RunTestCase(&testCase, dataContext, ctx); err == nil {
+		if output, testErr := simpleRunner.RunTestCase(&testCase, dataContext, ctx); testErr == nil {
 			dataContext[testCase.Name] = output
 		} else {
+			reply.Error = testErr.Error()
 			break
 		}
 	}
-	reply = &HelloReply{Message: buf.String()}
+	reply.Message = buf.String()
 	return
 }
 

pkg/server/server.pb.go

@@ -19,6 +19,7 @@ message TestTask {
 
 message HelloReply {
   string message = 1;
+  string error = 2;
 }
 
 message Empty {

pkg/server/server.proto

@@ -19,6 +19,7 @@ func TestUnimplement(t *testing.T) {
 
 	var reply *server.HelloReply
 	assert.Empty(t, reply.GetMessage())
+	assert.Empty(t, reply.GetError())
 	assert.Empty(t, &server.Empty{})
 
 	var task *server.TestTask
@@ -47,6 +48,7 @@ func TestServer(t *testing.T) {
 	reply, err := client.GetVersion(context.Background(), &server.Empty{})
 	assert.NotNil(t, reply)
 	assert.Equal(t, "version", reply.GetMessage())
+	assert.Empty(t, reply.GetError())
 	assert.Nil(t, err)
 
 	reply, err = client.Run(context.Background(), &server.TestTask{})