fix: mockReporter does not implement Reporter

apache · Jun 10, 2023 · dab599e · dab599e
1 parent 23b17ca
commit dab599e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/common/extension/metrics_test.go b/common/extension/metrics_test.go
@@ -40,5 +40,8 @@ func TestGetMetricReporter(t *testing.T) {
 type mockReporter struct{}
 
 // implement the interface of Reporter
-func (m mockReporter) ReportAfterInvocation(ctx context.Context, invoker protocol.Invoker, invocation protocol.Invocation, cost time.Duration, res protocol.Result) {
+func (m *mockReporter) ReportAfterInvocation(ctx context.Context, invoker protocol.Invoker, invocation protocol.Invocation, cost time.Duration, res protocol.Result) {
+}
+
+func (m *mockReporter) ReportBeforeInvocation(ctx context.Context, invoker protocol.Invoker, invocation protocol.Invocation) {
 }
diff --git a/filter/metrics/filter_test.go b/filter/metrics/filter_test.go
@@ -76,3 +76,8 @@ func (m *mockReporter) ReportAfterInvocation(ctx context.Context, invoker protoc
  m.Called(ctx, invoker, invocation)
  m.wg.Done()
 }
+
+func (m *mockReporter) ReportBeforeInvocation(ctx context.Context, invoker protocol.Invoker, invocation protocol.Invocation) {
+ m.Called(ctx, invoker, invocation)
+ m.wg.Done()
+}