Merge pull request #70 from sashamelentyev/feat/httptest

feat: add check method in httptest.NewRequest

pkg/analyzer/analyzer.go

@@ -220,6 +220,16 @@ func funArgs(pass *analysis.Pass, x *ast.Ident, fun *ast.SelectorExpr, args []as
 				checkHTTPStatusCode(pass, basicLit)
 			}
 		}
+	case "httptest":
+		if fun.Sel.Name == "NewRequest" {
+			if !lookupFlag(pass, HTTPMethodFlag) {
+				return
+			}
+
+			if basicLit := getBasicLitFromArgs(args, 3, 0, token.STRING); basicLit != nil {
+				checkHTTPMethod(pass, basicLit)
+			}
+		}
 	default:
 		// w.WriteHeader(http.StatusOk)
 		if fun.Sel.Name == "WriteHeader" {

pkg/analyzer/internal/template/test-httpmethod.go.tmpl

@@ -2,7 +2,10 @@
 
 package {{ .PackageName }}
 
-import "net/http"
+import (
+    "net/http"
+    "net/http/httptest"
+)
 
 var (
 {{- range $key, $value := .Mapping }}
@@ -103,3 +106,9 @@ func _() {
     }
 }
 {{ end -}}
+
+func _() {
+{{- range $key, $value := .Mapping }}
+	_ = httptest.NewRequest("{{ $key }}", "", http.NoBody) // want `"{{ quoteMeta $key }}" can be replaced by {{ quoteMeta $value }}`
+{{- end }}
+}