improve mapping function

server/v2/api/grpcgateway/interceptor.go

@@ -237,7 +237,7 @@ func createRegexMapping(logger log.Logger, annotationMapping map[string]string)
 	for annotation, queryInputName := range annotationMapping {
 		pattern, wildcardNames := patternToRegex(annotation)
 		if len(wildcardNames) == 0 {
-			if otherAnnotation, ok := annotationMapping[annotation]; ok {
+			if otherAnnotation, ok := seenPatterns[annotation]; ok {
 				// TODO: eventually we want this to error, but there is currently a duplicate in the protobuf.
 				// see: https://github.com/cosmos/cosmos-sdk/issues/23281
 				logger.Warn("duplicate HTTP annotation found", "annotation1", annotation, "annotation2", otherAnnotation, "query_input_name", queryInputName)

server/v2/api/grpcgateway/interceptor_test.go

@@ -20,39 +20,55 @@ import (
 
 func Test_createRegexMapping(t *testing.T) {
 	tests := []struct {
-		name        string
-		annotations map[string]string
-		wantWarn    bool
+		name           string
+		annotations    map[string]string
+		expectedRegex  int
+		expectedSimple int
+		wantWarn       bool
 	}{
 		{
 			name: "no annotations should not warn",
 		},
+		{
+			name: "expected correct amount of regex and simple matchers",
+			annotations: map[string]string{
+				"/foo/bar/baz":   "",
+				"/foo/{bar}/baz": "",
+				"/foo/bar/bell":  "",
+			},
+			expectedRegex:  1,
+			expectedSimple: 2,
+		},
 		{
 			name: "different annotations should not warn",
 			annotations: map[string]string{
 				"/foo/bar/{baz}":     "",
 				"/crypto/{currency}": "",
 			},
+			expectedRegex: 2,
 		},
 		{
 			name: "duplicate annotations should warn",
 			annotations: map[string]string{
 				"/hello/{world}":      "",
 				"/hello/{developers}": "",
 			},
-			wantWarn: true,
+			expectedRegex: 2,
+			wantWarn:      true,
 		},
 	}
 	buf := bytes.NewBuffer(nil)
 	logger := log.NewLogger(buf)
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			createRegexMapping(logger, tt.annotations)
+			regex, simple := createRegexMapping(logger, tt.annotations)
 			if tt.wantWarn {
 				require.NotEmpty(t, buf.String())
 			} else {
 				require.Empty(t, buf.String())
 			}
+			require.Equal(t, tt.expectedRegex, len(regex))
+			require.Equal(t, tt.expectedSimple, len(simple))
 		})
 	}
 }