client: Return error from unimplemented test methods

Nil response without an error can lead to the undesired behavior. Since other methods are not expected to be called, unimplemented error fits the best. Signed-off-by: Leonard Lyubich <[email protected]>
nspcc-dev · Nov 19, 2024 · 843085c · 843085c
1 parent bdfaa0e
commit 843085c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/client/netmap_test.go b/client/netmap_test.go
@@ -29,7 +29,7 @@ type serverNetMap struct {
 }
 
 func (x *serverNetMap) createSession(context.Context, session.CreateRequest) (*session.CreateResponse, error) {
-	return nil, nil
+	return nil, errors.New("unimplemented")
 }
 
 func (x *serverNetMap) netMapSnapshot(_ context.Context, req v2netmap.SnapshotRequest) (*v2netmap.SnapshotResponse, error) {

diff --git a/client/session_test.go b/client/session_test.go
@@ -2,6 +2,7 @@ package client
 
 import (
 	"context"
+	"errors"
 	"testing"
 
 	v2netmap "github.com/nspcc-dev/neofs-api-go/v2/netmap"
@@ -17,7 +18,7 @@ type sessionAPIServer struct {
 }
 
 func (m sessionAPIServer) netMapSnapshot(context.Context, v2netmap.SnapshotRequest) (*v2netmap.SnapshotResponse, error) {
-	return nil, nil
+	return nil, errors.New("unimplemented")
 }
 
 func (m sessionAPIServer) createSession(context.Context, session.CreateRequest) (*session.CreateResponse, error) {