main_test.go

package main

import (
	"strings"
	"testing"

	"github.com/stretchr/testify/assert"
	"github.com/stretchr/testify/require"

	"github.com/macadmins/carafe/cudo"
	"github.com/macadmins/carafe/exec"
	"github.com/macadmins/carafe/shell/testshell"
)

func TestValidateFormulaArg_AllowsValid(t *testing.T) {
	valid := []string{
		"openssl",
		"openssl@1.1",
		"libxml++",
		"qt-5",
		"foo_bar.baz",
		"gcc@12",
		"a",
		strings.Repeat("a", 128),
	}
	for _, v := range valid {
		assert.NoError(t, validateFormulaArg(v))
		assert.True(t, formulaRe.MatchString(v))
	}
}

func TestValidateFormulaArg_RejectsInvalid(t *testing.T) {
	invalid := []string{
		"",
		"homebrew/cask/vlc",
		"foo bar",
		"naïve",
		"foo:bar",
		"abc!",
		"a\nb",
		strings.Repeat("a", 129),
	}
	for _, v := range invalid {
		assert.Error(t, validateFormulaArg(v))
		assert.False(t, formulaRe.MatchString(v))
	}
}

func TestFormulaRe_DoesNotPartiallyMatch(t *testing.T) {
	assert.False(t, formulaRe.MatchString("valid@1.0!")) // '!' not allowed anywhere
	assert.False(t, formulaRe.MatchString(" valid"))     // leading space
	assert.False(t, formulaRe.MatchString("valid "))     // trailing space
}

func newMockConfig() exec.CarafeConfig {
	return exec.CarafeConfig{
		Arch: "arm64",
		CUSudo: &cudo.CUSudo{
			CurrentUser: "testuser",
			Platform:    "darwin",
			OSFunc:      &cudo.MockOSFunc{},
			UserHome:    "/Users/testuser",
			Executor:    testshell.NewExecutor(),
		},
	}
}

func TestCheckCmd_InvalidCacheTTL(t *testing.T) {
	cmd := buildRootCmd(newMockConfig(), "test")
	cmd.SetArgs([]string{"check", "--cache-ttl=banana", "git"})
	err := cmd.Execute()
	require.Error(t, err)
	assert.Contains(t, err.Error(), "invalid argument")
}

func TestCheckCmd_NoCacheFlag(t *testing.T) {
	// --no-cache should be accepted without error (brew call itself will fail
	// with the mock executor, but the flag parsing must succeed).
	cmd := buildRootCmd(newMockConfig(), "test")
	cmd.SetArgs([]string{"check", "--no-cache", "git"})
	// The mock executor has no output configured so brew.Check will error,
	// but the error must NOT be a flag-parsing error.
	err := cmd.Execute()
	if err != nil {
		assert.NotContains(t, err.Error(), "invalid argument")
		assert.NotContains(t, err.Error(), "unknown flag")
	}
}

func TestCheckCmd_CacheTTLFlag(t *testing.T) {
	// Valid duration strings must be accepted.
	for _, ttl := range []string{"30s", "2m", "1h", "0s"} {
		cmd := buildRootCmd(newMockConfig(), "test")
		cmd.SetArgs([]string{"check", "--cache-ttl=" + ttl, "--no-cache", "git"})
		err := cmd.Execute()
		if err != nil {
			assert.NotContains(t, err.Error(), "invalid argument", "TTL %q should be valid", ttl)
		}
	}
}