Reuse mock.EndpointPattern constants instead of hardcoded paths

- Add mock import to code_scanning_test.go and git_test.go
- Replace hardcoded paths with mock.GetRepos*.Pattern references
- Ensures consistency with existing test patterns and easier maintenance

Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>

Author: Copilot

pkg/github/code_scanning_test.go

@@ -10,6 +10,7 @@ import (
 	"github.com/github/github-mcp-server/pkg/translations"
 	"github.com/google/go-github/v79/github"
 	"github.com/google/jsonschema-go/jsonschema"
+	"github.com/migueleliasweb/go-github-mock/src/mock"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
@@ -50,7 +51,7 @@ func Test_GetCodeScanningAlert(t *testing.T) {
 		{
 			name: "successful alert fetch",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
-				"GET /repos/{owner}/{repo}/code-scanning/alerts/{alert_number}": mockResponse(t, http.StatusOK, mockAlert),
+				mock.GetReposCodeScanningAlertsByOwnerByRepoByAlertNumber.Method + " " + mock.GetReposCodeScanningAlertsByOwnerByRepoByAlertNumber.Pattern: mockResponse(t, http.StatusOK, mockAlert),
 			}),
 			requestArgs: map[string]interface{}{
 				"owner":       "owner",
@@ -63,7 +64,7 @@ func Test_GetCodeScanningAlert(t *testing.T) {
 		{
 			name: "alert fetch fails",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
-				"GET /repos/{owner}/{repo}/code-scanning/alerts/{alert_number}": http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
+				mock.GetReposCodeScanningAlertsByOwnerByRepoByAlertNumber.Method + " " + mock.GetReposCodeScanningAlertsByOwnerByRepoByAlertNumber.Pattern: http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 					w.WriteHeader(http.StatusNotFound)
 					_, _ = w.Write([]byte(`{"message": "Not Found"}`))
 				}),
@@ -165,7 +166,7 @@ func Test_ListCodeScanningAlerts(t *testing.T) {
 		{
 			name: "successful alerts listing",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
-				"GET /repos/{owner}/{repo}/code-scanning/alerts": expectQueryParams(t, map[string]string{
+				mock.GetReposCodeScanningAlertsByOwnerByRepo.Method + " " + mock.GetReposCodeScanningAlertsByOwnerByRepo.Pattern: expectQueryParams(t, map[string]string{
 					"ref":       "main",
 					"state":     "open",
 					"severity":  "high",
@@ -188,7 +189,7 @@ func Test_ListCodeScanningAlerts(t *testing.T) {
 		{
 			name: "alerts listing fails",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
-				"GET /repos/{owner}/{repo}/code-scanning/alerts": http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
+				mock.GetReposCodeScanningAlertsByOwnerByRepo.Method + " " + mock.GetReposCodeScanningAlertsByOwnerByRepo.Pattern: http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 					w.WriteHeader(http.StatusUnauthorized)
 					_, _ = w.Write([]byte(`{"message": "Unauthorized access"}`))
 				}),

pkg/github/git_test.go

@@ -11,6 +11,7 @@ import (
 	"github.com/github/github-mcp-server/pkg/translations"
 	"github.com/google/go-github/v79/github"
 	"github.com/google/jsonschema-go/jsonschema"
+	"github.com/migueleliasweb/go-github-mock/src/mock"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
@@ -71,8 +72,8 @@ func Test_GetRepositoryTree(t *testing.T) {
 		{
 			name: "successfully get repository tree",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
-				"GET /repos/{owner}/{repo}":                  mockResponse(t, http.StatusOK, mockRepo),
-				"GET /repos/{owner}/{repo}/git/trees/{tree}": mockResponse(t, http.StatusOK, mockTree),
+				mock.GetReposByOwnerByRepo.Method + " " + mock.GetReposByOwnerByRepo.Pattern:                                   mockResponse(t, http.StatusOK, mockRepo),
+				mock.GetReposGitTreesByOwnerByRepoByTreeSha.Method + " " + mock.GetReposGitTreesByOwnerByRepoByTreeSha.Pattern: mockResponse(t, http.StatusOK, mockTree),
 			}),
 			requestArgs: map[string]interface{}{
 				"owner": "owner",
@@ -82,8 +83,8 @@ func Test_GetRepositoryTree(t *testing.T) {
 		{
 			name: "successfully get repository tree with path filter",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
-				"GET /repos/{owner}/{repo}":                  mockResponse(t, http.StatusOK, mockRepo),
-				"GET /repos/{owner}/{repo}/git/trees/{tree}": mockResponse(t, http.StatusOK, mockTree),
+				mock.GetReposByOwnerByRepo.Method + " " + mock.GetReposByOwnerByRepo.Pattern:                                   mockResponse(t, http.StatusOK, mockRepo),
+				mock.GetReposGitTreesByOwnerByRepoByTreeSha.Method + " " + mock.GetReposGitTreesByOwnerByRepoByTreeSha.Pattern: mockResponse(t, http.StatusOK, mockTree),
 			}),
 			requestArgs: map[string]interface{}{
 				"owner":       "owner",
@@ -94,7 +95,7 @@ func Test_GetRepositoryTree(t *testing.T) {
 		{
 			name: "repository not found",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
-				"GET /repos/{owner}/{repo}": http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
+				mock.GetReposByOwnerByRepo.Method + " " + mock.GetReposByOwnerByRepo.Pattern: http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 					w.WriteHeader(http.StatusNotFound)
 					_, _ = w.Write([]byte(`{"message": "Not Found"}`))
 				}),
@@ -109,8 +110,8 @@ func Test_GetRepositoryTree(t *testing.T) {
 		{
 			name: "tree not found",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
-				"GET /repos/{owner}/{repo}": mockResponse(t, http.StatusOK, mockRepo),
-				"GET /repos/{owner}/{repo}/git/trees/{tree}": http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
+				mock.GetReposByOwnerByRepo.Method + " " + mock.GetReposByOwnerByRepo.Pattern: mockResponse(t, http.StatusOK, mockRepo),
+				mock.GetReposGitTreesByOwnerByRepoByTreeSha.Method + " " + mock.GetReposGitTreesByOwnerByRepoByTreeSha.Pattern: http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 					w.WriteHeader(http.StatusNotFound)
 					_, _ = w.Write([]byte(`{"message": "Not Found"}`))
 				}),