test: add TestDifferentMethodNamersWithClientHandler to validate method name formatting with various configurations

Author: thiagodeev

method_formatter_test.go

@@ -3,11 +3,12 @@ package jsonrpc
 import (
 	"context"
 	"fmt"
-	"github.com/stretchr/testify/require"
 	"net/http"
 	"net/http/httptest"
 	"strings"
 	"testing"
+
+	"github.com/stretchr/testify/require"
 )
 
 func TestDifferentMethodNamers(t *testing.T) {
@@ -123,3 +124,53 @@ func TestDifferentMethodNamersWithClient(t *testing.T) {
 		})
 	}
 }
+
+func TestDifferentMethodNamersWithClientHandler(t *testing.T) {
+	tests := map[string]struct {
+		namer MethodNameFormatter
+	}{
+		"default namer & ws": {
+			namer: DefaultMethodNameFormatter,
+		},
+		"lower first char namer & ws": {
+			namer: NewMethodNameFormatter(true, LowerFirstCharCase),
+		},
+		"no namespace namer & ws": {
+			namer: NewMethodNameFormatter(false, OriginalCase),
+		},
+		"no namespace & lower first char & ws": {
+			namer: NewMethodNameFormatter(false, LowerFirstCharCase),
+		},
+	}
+	for name, test := range tests {
+		t.Run(name, func(t *testing.T) {
+			rpcServer := NewServer(WithReverseClient[RevCallTestClientProxy]("Client"), WithServerMethodNameFormatter(test.namer))
+			rpcServer.Register("Server", &RevCallTestServerHandler{})
+
+			// httptest stuff
+			testServ := httptest.NewServer(rpcServer)
+			defer testServ.Close()
+			// setup client
+
+			var client struct {
+				Call func() error
+			}
+
+			closer, err := NewMergeClient(
+				context.Background(),
+				"ws://"+testServ.Listener.Addr().String(),
+				"Server",
+				[]any{&client},
+				nil,
+				WithMethodNameFormatter(test.namer),
+				WithClientHandler("Client", &RevCallTestClientHandler{}),
+				WithClientHandlerFormatter(test.namer),
+			)
+			require.NoError(t, err)
+			defer closer()
+
+			e := client.Call()
+			require.NoError(t, e)
+		})
+	}
+}