package app

import (
	"os"
	"path/filepath"
	"testing"
)

func TestExtractArchivePath(t *testing.T) {
	cases := map[string]string{
		"/appdata/bee/export/bee-sat/gpu-nvidia-20260706-174722":                    "/appdata/bee/export/bee-sat/gpu-nvidia-20260706-174722",
		"Archive written to /appdata/bee/export/bee-sat/gpu-nvidia-20260706-174722": "/appdata/bee/export/bee-sat/gpu-nvidia-20260706-174722",
		"Archive written to /path/with spaces/foo.tar.gz":                           "/path/with spaces/foo.tar.gz",
		"  Archive written to /path/foo.tar.gz  ":                                   "/path/foo.tar.gz",
	}
	for in, want := range cases {
		if got := ExtractArchivePath(in); got != want {
			t.Errorf("ExtractArchivePath(%q) = %q, want %q", in, got, want)
		}
	}
}

func TestReadSATOverallStatus_HandlesActionResultPrefix(t *testing.T) {
	runDir := t.TempDir()
	summary := "run_at_utc=2026-07-06T17:47:22Z\noverall_status=FAILED\n"
	if err := os.WriteFile(filepath.Join(runDir, "summary.txt"), []byte(summary), 0644); err != nil {
		t.Fatal(err)
	}

	// Regression: RunNvidiaAcceptancePackWithOptions wraps the bare run dir as
	// "Archive written to <dir>" before it reaches ReadSATOverallStatus. If the
	// prefix isn't stripped, the summary.txt lookup silently fails and a FAILED
	// sub-job never surfaces as a task failure.
	wrapped := "Archive written to " + runDir
	if got := ReadSATOverallStatus(ExtractArchivePath(wrapped)); got != "FAILED" {
		t.Errorf("ReadSATOverallStatus(wrapped) = %q, want FAILED", got)
	}

	if got := ReadSATOverallStatus(ExtractArchivePath(runDir)); got != "FAILED" {
		t.Errorf("ReadSATOverallStatus(bare) = %q, want FAILED", got)
	}
}