fix: make test case logs collapsed by default

cmd/oci/download.go

@@ -9,6 +9,7 @@ import (
 	"time"
 
 	"github.com/konflux-ci/qe-tools/pkg/oci"
+	"github.com/konflux-ci/qe-tools/pkg/utils"
 	"github.com/spf13/cobra"
 )
 
@@ -119,7 +120,7 @@ Examples:
 
 		// If repo is specified, call helper function to download from a single repository
 		if opts.repo != "" {
-			repo, tag, err := parseRepoAndTag(opts.repo)
+			repo, tag, err := utils.ParseRepoAndTag(opts.repo)
 			if err != nil {
 				return err
 			}
@@ -192,24 +193,6 @@ Examples:
 	},
 }
 
-// parseRepoAndTag extracts the repository and tag from the given repo flag.
-func parseRepoAndTag(repoFlag string) (string, string, error) {
-	// Ensure the repoFlag starts with 'quay.io/'
-	if !strings.HasPrefix(repoFlag, "quay.io/") {
-		return "", "", fmt.Errorf("the repository must start with 'quay.io/'")
-	}
-
-	// Remove 'quay.io/' prefix and split the repo and tag using the ':' character
-	repoFlag = strings.TrimPrefix(repoFlag, "quay.io/")
-	parts := strings.SplitN(repoFlag, ":", 2)
-
-	if len(parts) != 2 {
-		return "", "", fmt.Errorf("tag is missing in the repo flag")
-	}
-
-	return parts[0], parts[1], nil
-}
-
 // parseDuration handles the custom duration format
 func parseDuration(since string) (time.Duration, error) {
 	if len(since) > 1 && since[len(since)-1] == 'd' {

pkg/oci/artifact_scanner.go

@@ -4,10 +4,12 @@ import (
 	"fmt"
 	"io"
 	"os"
-	"os/exec"
 	"path/filepath"
 	"regexp"
 	"slices"
+	"time"
+
+	"github.com/konflux-ci/qe-tools/pkg/utils"
 )
 
 // NewArtifactScanner creates a new instance of ArtifactScanner.
@@ -40,11 +42,19 @@ func (as *ArtifactScanner) Run() error {
 }
 
 func (as *ArtifactScanner) pullAndExtractOciArtifact() error {
-	app := "oras"
-	args := []string{"pull", as.config.OciArtifactReference, "--output", as.artifactDirPath}
-	// #nosec G204
-	cmd := exec.Command(app, args...)
-	if err := cmd.Run(); err != nil {
+	cacheDir, err := os.MkdirTemp("", "")
+	if err != nil {
+		return err
+	}
+	ctrl, err := NewController(as.artifactDirPath, cacheDir)
+	if err != nil {
+		return err
+	}
+	repo, tag, err := utils.ParseRepoAndTag(as.config.OciArtifactReference)
+	if err != nil {
+		return err
+	}
+	if err := ctrl.ProcessTag(repo, tag, time.Now().Format(time.RFC1123)); err != nil {
 		return err
 	}
 	return nil

pkg/testresults/formatter.go

@@ -25,12 +25,12 @@ func extractFailedTestCasesBody(f FailedTestCasesReport) (failedTestCasesBody []
 		case tc.Status == "timedout":
 			tcMessage = returnContentWrappedInDropdown(dropdownSummaryString, tc.SystemErr)
 		case tc.Failure != nil:
-			tcMessage = "```\n" + tc.Failure.Message + "\n```"
+			tcMessage = returnContentWrappedInDropdown(dropdownSummaryString, tc.Failure.Message)
 		default:
-			tcMessage = "```\n" + tc.Error.Message + "\n```"
+			tcMessage = returnContentWrappedInDropdown(dropdownSummaryString, tc.Error.Message)
 		}
 
-		testCaseEntry := "* :arrow_right: " + "[**`" + tc.Status + "`**] " + tc.Name + "\n" + tcMessage
+		testCaseEntry := ":arrow_right: " + "[**`" + tc.Status + "`**] " + tc.Name + tcMessage
 		failedTestCasesBody = append(failedTestCasesBody, testCaseEntry)
 	}
 	return
@@ -64,5 +64,5 @@ func GetFormattedReport(report FailedTestCasesReport) (formattedReport string) {
 }
 
 func returnContentWrappedInDropdown(summary, content string) string {
-	return "<details><summary>" + summary + "</summary><br><pre>" + content + "</pre></details>"
+	return "<details><summary>" + summary + "</summary><br><pre>" + content + "</pre></details>\n\n---"
 }

pkg/utils/utils.go

@@ -0,0 +1,24 @@
+package utils
+
+import (
+	"fmt"
+	"strings"
+)
+
+// ParseRepoAndTag extracts the quay.io repository and tag from the given repo flag.
+func ParseRepoAndTag(repoFlag string) (string, string, error) {
+	// Ensure the repoFlag starts with 'quay.io/'
+	if !strings.HasPrefix(repoFlag, "quay.io/") {
+		return "", "", fmt.Errorf("the repository must start with 'quay.io/'")
+	}
+
+	// Remove 'quay.io/' prefix and split the repo and tag using the ':' character
+	repoFlag = strings.TrimPrefix(repoFlag, "quay.io/")
+	parts := strings.SplitN(repoFlag, ":", 2)
+
+	if len(parts) != 2 {
+		return "", "", fmt.Errorf("tag is missing in the repo flag")
+	}
+
+	return parts[0], parts[1], nil
+}