chore: call t.Helper in helper methods, and use require.NoError inste…

…ad of panicking

lib/ecosystems/enrich_spdx_test.go

@@ -19,7 +19,6 @@ package ecosystems
 import (
 	"bytes"
 	"encoding/json"
-	"fmt"
 	"net/http"
 	"testing"
 
@@ -33,32 +32,29 @@ import (
 	"github.com/snyk/parlay/lib/sbom"
 )
 
-func parseJson(jsonStr string) map[string]any {
+func parseJson(t *testing.T, jsonStr string) map[string]any {
+	t.Helper()
 	var result map[string]any
-
-	err := json.Unmarshal([]byte(jsonStr), &result)
-	if err != nil {
-		panic(fmt.Errorf("failed to parse JSON: %w", err))
-	}
-
+	require.NoError(t, json.Unmarshal([]byte(jsonStr), &result))
 	return result
 }
 
-func setupHttpmock(packageVersionsResponse, packageResponse *string) {
+func setupHttpmock(t *testing.T, packageVersionsResponse, packageResponse *string) {
+	t.Helper()
 	httpmock.Activate()
 
 	if packageVersionsResponse != nil {
 		httpmock.RegisterResponder("GET", `=~^https://packages.ecosyste.ms/api/v1/registries/.*/packages/.*/versions`,
 			func(r *http.Request) (*http.Response, error) {
-				return httpmock.NewJsonResponse(200, parseJson(*packageVersionsResponse))
+				return httpmock.NewJsonResponse(200, parseJson(t, *packageVersionsResponse))
 			},
 		)
 	}
 
 	if packageResponse != nil {
 		httpmock.RegisterResponder("GET", `=~^https://packages.ecosyste.ms/api/v1/registries`,
 			func(req *http.Request) (*http.Response, error) {
-				return httpmock.NewJsonResponse(200, parseJson(*packageResponse))
+				return httpmock.NewJsonResponse(200, parseJson(t, *packageResponse))
 			})
 	}
 }