fix: avoid some nil refs and add verbose mode to individual requests

examples/routines/basic/request-1.yml

@@ -2,7 +2,7 @@ kind: request
 name: Cat Breeds - Assertion/Capture Testing
 path: https://catfact.ninja/breeds
 postRequestScript: |
-  console.log('Hello World!');
+  nap.env.set("thirdBreed", nap.env.get("firstBreed"))
 asserts:
   - status == 200
   - duration < 1000

examples/routines/basic/routine.yml

@@ -2,6 +2,4 @@ kind: routine
 name: main routine
 steps:
   - run: request-1.yml
-  - run: request-2.yml
-  - run: subroutine-1.yml
-  - run: subroutine-2.yml
+  - run: script.js

examples/routines/basic/script.js

@@ -1,3 +1 @@
-console.log("my status code: " + nap.env.get("myStatus"))
-console.log("my duration: " + nap.env.get("myDuration"))
-console.log("Here's a second console.log")
+console.log("thirdBreed: " + nap.env.get("thirdBreed"))

napquery/query.go

@@ -28,6 +28,11 @@ import (
 )
 
 func Eval(query string, vmData *napscript.VmHttpData) (string, error) {
+	if vmData == nil || vmData.Response == nil {
+		// return empty here instead of erroring in case this assert is testing for absence of a value
+		return "", nil
+	}
+
 	jsonExpression, isJsonPath := strings.CutPrefix(query, "jsonpath ")
 	if isJsonPath {
 		body := vmData.Response.JsonBody
@@ -42,6 +47,10 @@ func Eval(query string, vmData *napscript.VmHttpData) (string, error) {
 
 	header, isHeader := strings.CutPrefix(query, "header ")
 	if isHeader {
+		if vmData.Response.Headers == nil {
+			return "", nil
+		}
+
 		value := vmData.Response.Headers[header]
 
 		return strings.Join(value, ","), nil

naprequest/request.go

@@ -41,6 +41,7 @@ type Request struct {
 	TimeoutSeconds        int    `yaml:"timeoutSeconds"`
 	Captures              map[string]string
 	Asserts               []string
+	Verbose               bool
 }
 
 func parse(data []byte) (*Request, error) {

naprunner/runner.go

@@ -23,6 +23,7 @@ import (
 	"fmt"
 	"io"
 	"net/http"
+	"net/http/httputil"
 	"os"
 	"path/filepath"
 	"strings"
@@ -102,10 +103,6 @@ func runRequest(ctx *napcontext.Context, request *naprequest.Request) *napreques
 
 	result.HttpResponse, result.Error = executeHttp(request)
 
-	if result.HttpResponse != nil && result.HttpResponse.StatusCode >= 400 {
-		result.Error = errors.New(fmt.Sprintf("HTTP %s", result.HttpResponse.Status))
-	}
-
 	result.EndTime = time.Now()
 
 	vmData, err := napscript.SetVmHttpData(ctx, result)
@@ -187,13 +184,34 @@ func executeHttp(r *naprequest.Request) (*http.Response, error) {
 		request.Header.Add(k, v)
 	}
 
-	resp, err := client.Do(request)
+	response, err := client.Do(request)
+
+	if r.Verbose {
+		fmt.Println("REQUEST:")
+		dump, err := httputil.DumpRequestOut(request, true)
+		if err == nil {
+
+			fmt.Println(string(dump))
+		} else {
+			fmt.Println(err)
+		}
+	}
+
+	if r.Verbose {
+		fmt.Println("RESPONSE:")
+		dump, err := httputil.DumpResponse(response, true)
+		if err == nil {
+			fmt.Println(string(dump))
+		} else {
+			fmt.Println(err)
+		}
+	}
 
 	if err != nil {
 		return nil, err
 	}
 
-	return resp, nil
+	return response, nil
 }
 
 func fileExists(path string) bool {