Removing unused code in integration tests

v2/cmd/integration-test/dns.go

@@ -16,78 +16,53 @@ type dnsBasic struct{}
 
 // Execute executes a test case and returns an error if occurred
 func (h *dnsBasic) Execute(filePath string) error {
-	var routerErr error
-
 	results, err := testutils.RunNucleiTemplateAndGetResults(filePath, "one.one.one.one", debug)
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 	return expectResultsCount(results, 1)
 }
 
 type dnsPtr struct{}
 
 // Execute executes a test case and returns an error if occurred
 func (h *dnsPtr) Execute(filePath string) error {
-	var routerErr error
-
 	results, err := testutils.RunNucleiTemplateAndGetResults(filePath, "1.1.1.1", debug)
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 	return expectResultsCount(results, 1)
 }
 
 type dnsCAA struct{}
 
 // Execute executes a test case and returns an error if occurred
 func (h *dnsCAA) Execute(filePath string) error {
-	var routerErr error
-
 	results, err := testutils.RunNucleiTemplateAndGetResults(filePath, "google.com", debug)
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 	return expectResultsCount(results, 1)
 }
 
 type dnsTLSA struct{}
 
 // Execute executes a test case and returns an error if occurred
 func (h *dnsTLSA) Execute(filePath string) error {
-	var routerErr error
-
 	results, err := testutils.RunNucleiTemplateAndGetResults(filePath, "scanme.sh", debug)
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 	return expectResultsCount(results, 0)
 }
 
 type dnsVariables struct{}
 
 // Execute executes a test case and returns an error if occurred
 func (h *dnsVariables) Execute(filePath string) error {
-	var routerErr error
-
 	results, err := testutils.RunNucleiTemplateAndGetResults(filePath, "one.one.one.one", debug)
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 	return expectResultsCount(results, 1)
 }

v2/cmd/integration-test/http.go

@@ -460,8 +460,6 @@ type httpRawGetQuery struct{}
 // Execute executes a test case and returns an error if occurred
 func (h *httpRawGetQuery) Execute(filePath string) error {
 	router := httprouter.New()
-	var routerErr error
-
 	router.GET("/", func(w http.ResponseWriter, r *http.Request, _ httprouter.Params) {
 		if strings.EqualFold(r.URL.Query().Get("test"), "nuclei") {
 			fmt.Fprintf(w, "Test is test raw-get-query-matcher text")
@@ -474,9 +472,6 @@ func (h *httpRawGetQuery) Execute(filePath string) error {
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 
 	return expectResultsCount(results, 1)
 }
@@ -486,8 +481,6 @@ type httpRawGet struct{}
 // Execute executes a test case and returns an error if occurred
 func (h *httpRawGet) Execute(filePath string) error {
 	router := httprouter.New()
-	var routerErr error
-
 	router.GET("/", func(w http.ResponseWriter, r *http.Request, _ httprouter.Params) {
 		fmt.Fprintf(w, "Test is test raw-get-matcher text")
 	})
@@ -498,9 +491,6 @@ func (h *httpRawGet) Execute(filePath string) error {
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 
 	return expectResultsCount(results, 1)
 }
@@ -613,6 +603,7 @@ func (h *httpRawCookieReuse) Execute(filePath string) error {
 	return expectResultsCount(results, 1)
 }
 
+// TODO: excluded due to parsing errors with console
 // type httpRawUnsafeRequest struct{
 // Execute executes a test case and returns an error if occurred
 // func (h *httpRawUnsafeRequest) Execute(filePath string) error {
@@ -640,13 +631,12 @@ type httpRequestCondition struct{}
 // Execute executes a test case and returns an error if occurred
 func (h *httpRequestCondition) Execute(filePath string) error {
 	router := httprouter.New()
-	var routerErr error
 
 	router.GET("/200", func(w http.ResponseWriter, r *http.Request, _ httprouter.Params) {
-		w.WriteHeader(200)
+		w.WriteHeader(http.StatusOK)
 	})
 	router.GET("/400", func(w http.ResponseWriter, r *http.Request, _ httprouter.Params) {
-		w.WriteHeader(400)
+		w.WriteHeader(http.StatusBadRequest)
 	})
 	ts := httptest.NewServer(router)
 	defer ts.Close()
@@ -655,9 +645,6 @@ func (h *httpRequestCondition) Execute(filePath string) error {
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 
 	return expectResultsCount(results, 1)
 }
@@ -667,8 +654,6 @@ type httpRequestSelContained struct{}
 // Execute executes a test case and returns an error if occurred
 func (h *httpRequestSelContained) Execute(filePath string) error {
 	router := httprouter.New()
-	var routerErr error
-
 	router.GET("/", func(w http.ResponseWriter, r *http.Request, _ httprouter.Params) {
 		_, _ = w.Write([]byte("This is self-contained response"))
 	})
@@ -685,9 +670,6 @@ func (h *httpRequestSelContained) Execute(filePath string) error {
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 
 	return expectResultsCount(results, 1)
 }

v2/cmd/integration-test/network.go

@@ -99,8 +99,6 @@ type networkRequestSelContained struct{}
 
 // Execute executes a test case and returns an error if occurred
 func (h *networkRequestSelContained) Execute(filePath string) error {
-	var routerErr error
-
 	ts := testutils.NewTCPServer(nil, defaultStaticPort, func(conn net.Conn) {
 		defer conn.Close()
 
@@ -111,9 +109,6 @@ func (h *networkRequestSelContained) Execute(filePath string) error {
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
 
 	return expectResultsCount(results, 1)
 }

v2/cmd/integration-test/template-dir.go

@@ -21,14 +21,10 @@ type templateDirWithTargetTest struct{}
 func (h *templateDirWithTargetTest) Execute(filePath string) error {
 	defer os.RemoveAll(getTemplatesDir())
 
-	var routerErr error
-
 	results, err := testutils.RunNucleiTemplateAndGetResults(filePath, "8x8exch02.8x8.com", debug, "-ud", getTemplatesDir())
 	if err != nil {
 		return err
 	}
-	if routerErr != nil {
-		return routerErr
-	}
+
 	return expectResultsCount(results, 1)
 }