1 files changed, 50 insertions, 4 deletions
diff --git a/service/metrics/rpc_test.go b/service/metrics/rpc_test.go
index 9668c203..78bf3e20 100644
--- a/service/metrics/rpc_test.go
+++ b/service/metrics/rpc_test.go
@@ -24,7 +24,8 @@ func Test_Set_RPC_Metric(t *testing.T) {
 		"address": "localhost:2112",
 		"collect":{
 			"user_gauge_2":{
-				"type": "gauge"
+				"type": "gauge",
+				"labels": ["type", "section"]
 			}
 		}
 
@@ -47,12 +48,57 @@ func Test_Set_RPC_Metric(t *testing.T) {
 
 	var ok bool
 	assert.NoError(t, client.Call("metrics.Set", Metric{
-		Name:  "user_gauge_2",
-		Value: 100.0,
+		Name:   "user_gauge_2",
+		Value:  100.0,
+		Labels: []string{"core", "first"},
 	}, &ok))
 	assert.True(t, ok)
 
 	out, _, err := get("http://localhost:2112/metrics")
 	assert.NoError(t, err)
-	assert.Contains(t, out, "user_gauge_2 100")
+	assert.Contains(t, out, `user_gauge_2{section="first",type="core"} 100`)
+}
+
+func Test_Set_RPC_MetricError(t *testing.T) {
+	logger, _ := test.NewNullLogger()
+	logger.SetLevel(logrus.DebugLevel)
+
+	c := service.NewContainer(logger)
+	c.Register(rpc.ID, &rpc.Service{})
+	c.Register(ID, &Service{})
+
+	assert.NoError(t, c.Init(&testCfg{
+		rpcCfg: `{"enable":true, "listen":"tcp://:5004"}`,
+		metricsCfg: `{
+		"address": "localhost:2112",
+		"collect":{
+			"user_gauge_3":{
+				"type": "gauge",
+				"labels": ["type", "section"]
+			}
+		}
+
+	}`}))
+
+	s, _ := c.Get(ID)
+	assert.NotNil(t, s)
+
+	s2, _ := c.Get(rpc.ID)
+	rs := s2.(*rpc.Service)
+
+	assert.True(t, s.(*Service).Enabled())
+
+	go func() { c.Serve() }()
+	time.Sleep(time.Millisecond * 100)
+	defer c.Stop()
+
+	client, err := rs.Client()
+	assert.NoError(t, err)
+
+	var ok bool
+	assert.Error(t, client.Call("metrics.Set", Metric{
+		Name:   "user_gauge_3",
+		Value:  100.0,
+		Labels: []string{"missing"},
+	}, &ok))
 }