Skip to content

Commit 29918cd

Browse files
abhiyankhanalclaude
andcommitted
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 <noreply@anthropic.com>
1 parent e8e47ef commit 29918cd

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

test_helpers_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ func TestRunTestHandlerFlushesStatusCode(t *testing.T) {
2727
for _, tt := range tests {
2828
t.Run(tt.name, func(t *testing.T) {
2929
w := httptest.NewRecorder()
30-
req := httptest.NewRequest("GET", "/", nil)
30+
req := httptest.NewRequest(http.MethodGet, "/", nil)
3131
code := tt.statusCode
3232

3333
c := RunTestHandler(w, req, func(c *Context) {
@@ -42,7 +42,7 @@ func TestRunTestHandlerFlushesStatusCode(t *testing.T) {
4242

4343
func TestRunTestHandlerMultipleHandlers(t *testing.T) {
4444
w := httptest.NewRecorder()
45-
req := httptest.NewRequest("POST", "/resource", nil)
45+
req := httptest.NewRequest(http.MethodPost, "/resource", nil)
4646

4747
callOrder := []string{}
4848

@@ -65,7 +65,7 @@ func TestRunTestHandlerMultipleHandlers(t *testing.T) {
6565

6666
func TestRunTestHandlerDefaultStatus(t *testing.T) {
6767
w := httptest.NewRecorder()
68-
req := httptest.NewRequest("GET", "/", nil)
68+
req := httptest.NewRequest(http.MethodGet, "/", nil)
6969

7070
RunTestHandler(w, req, func(c *Context) {
7171
// Handler that does not set a status explicitly.
@@ -76,7 +76,7 @@ func TestRunTestHandlerDefaultStatus(t *testing.T) {
7676

7777
func TestRunTestHandlerSetsRequest(t *testing.T) {
7878
w := httptest.NewRecorder()
79-
req := httptest.NewRequest("PUT", "/items/42", nil)
79+
req := httptest.NewRequest(http.MethodPut, "/items/42", nil)
8080

8181
c := RunTestHandler(w, req, func(c *Context) {
8282
c.Status(http.StatusOK)

0 commit comments

Comments
 (0)