fix: rename unused request parameters to _ in tests

Fixes code quality check failure for unused parameters in
http.HandlerFunc callbacks that don't need the request object.

Author: zdennis

pkg/jira/search_test.go

@@ -169,7 +169,7 @@ func TestSearchPagination(t *testing.T) {
 	}
 
 	t.Run("offset skipping within single page", func(t *testing.T) {
-		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 			w.Header().Set("Content-Type", "application/json")
 			w.WriteHeader(200)
 			// Return 5 issues, client requests from=2, limit=2
@@ -239,7 +239,7 @@ func TestSearchPagination(t *testing.T) {
 
 	t.Run("limit stops fetching early", func(t *testing.T) {
 		requestCount := 0
-		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 			requestCount++
 			w.Header().Set("Content-Type", "application/json")
 			w.WriteHeader(200)
@@ -311,7 +311,7 @@ func TestSearchPagination(t *testing.T) {
 	})
 
 	t.Run("offset beyond available items returns empty", func(t *testing.T) {
-		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 			w.Header().Set("Content-Type", "application/json")
 			w.WriteHeader(200)
 			resp := makeResponse([]*Issue{
@@ -332,7 +332,7 @@ func TestSearchPagination(t *testing.T) {
 	})
 
 	t.Run("empty result set", func(t *testing.T) {
-		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 			w.Header().Set("Content-Type", "application/json")
 			w.WriteHeader(200)
 			resp := makeResponse([]*Issue{}, "", true)
@@ -350,7 +350,7 @@ func TestSearchPagination(t *testing.T) {
 
 	t.Run("error mid-pagination returns error", func(t *testing.T) {
 		requestCount := 0
-		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 			requestCount++
 			w.Header().Set("Content-Type", "application/json")
 
@@ -408,7 +408,7 @@ func TestSearchPagination(t *testing.T) {
 	})
 
 	t.Run("IsLast true when all pages exhausted", func(t *testing.T) {
-		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 			w.Header().Set("Content-Type", "application/json")
 			w.WriteHeader(200)
 			// Single page with all results, isLast=true
@@ -429,7 +429,7 @@ func TestSearchPagination(t *testing.T) {
 	})
 
 	t.Run("IsLast false when limit reached before end", func(t *testing.T) {
-		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 			w.Header().Set("Content-Type", "application/json")
 			w.WriteHeader(200)
 			// Page has more items than requested limit