From 59894442277af502a4949eac8d2a1b5de30717d9 Mon Sep 17 00:00:00 2001
From: kevin <wanjunfeng@gmail.com>
Date: Sun, 23 Aug 2020 22:33:20 +0800
Subject: [PATCH] add more tests

---
 .codecov.yml                                  |  5 ++-
 .../breakerinterceptor_test.go                | 30 +++++++++++++++
 .../durationinterceptor_test.go               | 37 +++++++++++++++++++
 .../prommetricinterceptor_test.go             | 37 +++++++++++++++++++
 4 files changed, 107 insertions(+), 2 deletions(-)
 create mode 100644 rpcx/internal/clientinterceptors/durationinterceptor_test.go
 create mode 100644 rpcx/internal/clientinterceptors/prommetricinterceptor_test.go

diff --git a/.codecov.yml b/.codecov.yml
index d7cf44de..0b3fb52b 100644
--- a/.codecov.yml
+++ b/.codecov.yml
@@ -1,3 +1,4 @@
 ignore:
-  - "example/*"
-  - "tools/*"
\ No newline at end of file
+  - "doc"
+  - "example"
+  - "tools"
\ No newline at end of file
diff --git a/rpcx/internal/clientinterceptors/breakerinterceptor_test.go b/rpcx/internal/clientinterceptors/breakerinterceptor_test.go
index 8f58650a..63f30b41 100644
--- a/rpcx/internal/clientinterceptors/breakerinterceptor_test.go
+++ b/rpcx/internal/clientinterceptors/breakerinterceptor_test.go
@@ -1,12 +1,15 @@
 package clientinterceptors
 
 import (
+	"context"
+	"errors"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
 	"github.com/tal-tech/go-zero/core/breaker"
 	"github.com/tal-tech/go-zero/core/stat"
 	rcodes "github.com/tal-tech/go-zero/rpcx/internal/codes"
+	"google.golang.org/grpc"
 	"google.golang.org/grpc/codes"
 	"google.golang.org/grpc/status"
 )
@@ -49,3 +52,30 @@ func TestBreakerInterceptorDeadlineExceeded(t *testing.T) {
 	assert.True(t, errs[err] > 0)
 	assert.True(t, errs[breaker.ErrServiceUnavailable] > 0)
 }
+
+func TestBreakerInterceptor(t *testing.T) {
+	tests := []struct {
+		name string
+		err  error
+	}{
+		{
+			name: "nil",
+			err:  nil,
+		},
+		{
+			name: "with error",
+			err:  errors.New("mock"),
+		},
+	}
+	for _, test := range tests {
+		t.Run(test.name, func(t *testing.T) {
+			cc := new(grpc.ClientConn)
+			err := BreakerInterceptor(context.Background(), "/foo", nil, nil, cc,
+				func(ctx context.Context, method string, req, reply interface{}, cc *grpc.ClientConn,
+					opts ...grpc.CallOption) error {
+					return test.err
+				})
+			assert.Equal(t, test.err, err)
+		})
+	}
+}
diff --git a/rpcx/internal/clientinterceptors/durationinterceptor_test.go b/rpcx/internal/clientinterceptors/durationinterceptor_test.go
new file mode 100644
index 00000000..0d88302c
--- /dev/null
+++ b/rpcx/internal/clientinterceptors/durationinterceptor_test.go
@@ -0,0 +1,37 @@
+package clientinterceptors
+
+import (
+	"context"
+	"errors"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"google.golang.org/grpc"
+)
+
+func TestDurationInterceptor(t *testing.T) {
+	tests := []struct {
+		name string
+		err  error
+	}{
+		{
+			name: "nil",
+			err:  nil,
+		},
+		{
+			name: "with error",
+			err:  errors.New("mock"),
+		},
+	}
+	for _, test := range tests {
+		t.Run(test.name, func(t *testing.T) {
+			cc := new(grpc.ClientConn)
+			err := DurationInterceptor(context.Background(), "/foo", nil, nil, cc,
+				func(ctx context.Context, method string, req, reply interface{}, cc *grpc.ClientConn,
+					opts ...grpc.CallOption) error {
+					return test.err
+				})
+			assert.Equal(t, test.err, err)
+		})
+	}
+}
diff --git a/rpcx/internal/clientinterceptors/prommetricinterceptor_test.go b/rpcx/internal/clientinterceptors/prommetricinterceptor_test.go
new file mode 100644
index 00000000..c299e89f
--- /dev/null
+++ b/rpcx/internal/clientinterceptors/prommetricinterceptor_test.go
@@ -0,0 +1,37 @@
+package clientinterceptors
+
+import (
+	"context"
+	"errors"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"google.golang.org/grpc"
+)
+
+func TestPromMetricInterceptor(t *testing.T) {
+	tests := []struct {
+		name string
+		err  error
+	}{
+		{
+			name: "nil",
+			err:  nil,
+		},
+		{
+			name: "with error",
+			err:  errors.New("mock"),
+		},
+	}
+	for _, test := range tests {
+		t.Run(test.name, func(t *testing.T) {
+			cc := new(grpc.ClientConn)
+			err := PromMetricInterceptor(context.Background(), "/foo", nil, nil, cc,
+				func(ctx context.Context, method string, req, reply interface{}, cc *grpc.ClientConn,
+					opts ...grpc.CallOption) error {
+					return test.err
+				})
+			assert.Equal(t, test.err, err)
+		})
+	}
+}