From 3f607b8662710b3602b03817cfc2f171416f6591 Mon Sep 17 00:00:00 2001 From: Abhiyan Khanal Date: Sun, 8 Mar 2026 18:56:15 +0545 Subject: [PATCH] fix(test): use stdlib constants for HTTP methods in test helpers Replace string literals with http.Method* constants to satisfy the usestdlibvars linter. Co-Authored-By: Claude Opus 4.6 --- test_helpers_test.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test_helpers_test.go b/test_helpers_test.go index dcdf76da..fe800811 100644 --- a/test_helpers_test.go +++ b/test_helpers_test.go @@ -27,7 +27,7 @@ func TestRunTestHandlerFlushesStatusCode(t *testing.T) { for _, tt := range tests { t.Run(tt.name, func(t *testing.T) { w := httptest.NewRecorder() - req := httptest.NewRequest("GET", "/", nil) + req := httptest.NewRequest(http.MethodGet, "/", nil) code := tt.statusCode c := RunTestHandler(w, req, func(c *Context) { @@ -42,7 +42,7 @@ func TestRunTestHandlerFlushesStatusCode(t *testing.T) { func TestRunTestHandlerMultipleHandlers(t *testing.T) { w := httptest.NewRecorder() - req := httptest.NewRequest("POST", "/resource", nil) + req := httptest.NewRequest(http.MethodPost, "/resource", nil) callOrder := []string{} @@ -65,7 +65,7 @@ func TestRunTestHandlerMultipleHandlers(t *testing.T) { func TestRunTestHandlerDefaultStatus(t *testing.T) { w := httptest.NewRecorder() - req := httptest.NewRequest("GET", "/", nil) + req := httptest.NewRequest(http.MethodGet, "/", nil) RunTestHandler(w, req, func(c *Context) { // Handler that does not set a status explicitly. @@ -76,7 +76,7 @@ func TestRunTestHandlerDefaultStatus(t *testing.T) { func TestRunTestHandlerSetsRequest(t *testing.T) { w := httptest.NewRecorder() - req := httptest.NewRequest("PUT", "/items/42", nil) + req := httptest.NewRequest(http.MethodPut, "/items/42", nil) c := RunTestHandler(w, req, func(c *Context) { c.Status(http.StatusOK)