From 68cc46a50b3069f6142ed1a77799635777dd6c52 Mon Sep 17 00:00:00 2001 From: puneeth_aditya_5656 Date: Sat, 2 May 2026 00:58:59 +0530 Subject: [PATCH] fix: return error from json.Unmarshal in GetTestResult instead of ignoring it Signed-off-by: puneeth_aditya_5656 --- pkg/connectors/microcks_client.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/pkg/connectors/microcks_client.go b/pkg/connectors/microcks_client.go index cabb85f..fefac12 100644 --- a/pkg/connectors/microcks_client.go +++ b/pkg/connectors/microcks_client.go @@ -410,9 +410,11 @@ func (c *microcksClient) GetTestResult(testResultID string) (*TestResultSummary, } result := TestResultSummary{} - json.Unmarshal([]byte(body), &result) + if err := json.Unmarshal(body, &result); err != nil { + return nil, fmt.Errorf("failed to parse test result response: %w", err) + } - return &result, err + return &result, nil } func (c *microcksClient) UploadArtifact(specificationFilePath string, mainArtifact bool) (string, error) {