Files
deploy/util/localconfig/localconfig_test.go
41536738c8 refactor(cd): hard-fork - rename CRD API group argoproj.io -> deploy.hanzo.ai (#1)
* feat(api): hard-fork CRD API group argoproj.io -> deploy.hanzo.ai

Hanzo CD hard-fork, phase 1.1. Single source of truth is
pkg/apis/application/register.go's Group constant (Application/AppProject/
ApplicationSet all derive their FullName from it); every other Go call-site
that hardcoded the literal group string is updated to match: the 3 generated
informer GVRs, the app-of-apps nested-Application detector in types.go, the
resource-tree Application-kind cache populator, the 3 NotFound error
constructors in server/application, and the applicationset-controller's
leader-election Lease name/ID (kept in lockstep across Go source + RBAC
resourceNames, or leader election would get RBAC-denied on startup).

CRD YAMLs (application/appproject/applicationset) and every RBAC apiGroups
rule that actually governs our 3 CRDs are repointed to deploy.hanzo.ai, across
both manifests/base/* (namespaced Roles) and manifests/cluster-rbac/* (cluster
scope). Two adjacent things that share the "argoproj.io" string but are NOT
ours are deliberately left untouched: gitops-engine's health-check dispatch
for Argo Workflows' `Workflow` kind, and argocd-server-clusterrole's second
apiGroups block granting read on Argo Workflows' `workflows` resource — both
belong to a separate, unrelated Argoproj-family project we are not forking.

Also untouched, per explicit scope: annotation/label keys (argocd.argoproj.io/*)
and finalizers (*.argoproj.io) that track already-applied live resources, and
argocd-cm.yaml's resource.customizations.ignoreResourceUpdates keys (a narrow
app-of-apps diffing nicety, not build- or RBAC-relevant).

CGO_ENABLED=0 go build ./... verified green.

* ci: wire canonical hanzo.yml + cicd.yml (Hanzo CD image build)

Phase 1.3. Follows the same one-way pattern every hanzo/lux/zoo repo uses:
hanzo.yml declares the image (root Dockerfile -> ghcr.io/hanzoai/deploy) and a
fast go-build sanity test (the multi-stage Dockerfile build is the slow, real
gate); cicd.yml is the ~10-line trigger shim importing hanzoai/ci's reusable
build workflow on the arc pool, secrets from KMS only.

No `deploy:` block yet, matching the hanzoai/cms build-only-first-cutover
pattern: this repo's own live hanzo-cd namespace runs the whole Hanzo fleet's
GitOps, so the prod image/CRD swap is a deliberate separate action (see the
[HELD] universe PR), never an automatic rollout on merge. Every PR/push still
builds and pushes an immutable ghcr.io/hanzoai/deploy:sha-<7>-amd64 image.

* feat(ui): rebrand Argo CD -> Hanzo CD (title, favicon, logo, sidebar)

Phase 1.2. User-facing "Argo CD"/"ArgoCD" text replaced with "Hanzo CD"
across the login page, help modal, settings screens, version-info panel,
sync-options warning, fullscreen-logs title, app.tsx page titles, and
index.html's <title>/noscript text (dropped the now-wrong upstream CLI
doc link rather than invent a hanzo.ai URL that doesn't exist).

Logo/favicon assets replaced with the canonical Hanzo mark (never
hand-drawn, copied from ~/work/hanzo/logo/dist): the two real usages of
Argo's logo collapse onto one Hanzo glyph (mono + white variants) at the
same 5 call sites (login icon + footer, login.scss $logoPath/
$logoNegativePath, sidebar text-logo), preserving the existing CSS
invert() filters so both the light login panel and dark sidebar still
render correctly. Old argologo.svg/argo_o.svg/argo.png deleted (no
longer referenced). favicon-{16x16,32x32}.png added at
ui/src/assets/favicon/ (index.html's links to that path were previously
dead - the directory didn't exist in this checkout). package.json name
argo-cd-ui -> hanzo-cd-ui.

Left alone, deliberately: the two Kubernetes resource-kind icon sets
under ui/src/assets/images/resources/ (unrelated to branding), and
upstream doc/project links (argoproj.io, argo-cd.readthedocs.io) since
we have no equivalent hanzo.ai docs to point to yet - except the login
footer logo's outbound link, which now points to hanzo.ai instead of
argoproj.io (a Hanzo-branded mark linking to the upstream project it
was forked from read as an obvious leftover).

* feat(module): rename Go module github.com/argoproj/argo-cd/v3 -> github.com/hanzoai/deploy

Phase 2. No version suffix on the new path (org policy: never publish our
own Go modules above v1.x.x / with a v2+ suffix) — this fork starts a fresh
v0/v1 module identity rather than carrying the v3 the upstream fork point
had reached.

Mechanical, symmetric rewrite: 704 .go files (import statements only; two
disjoint substitutions applied specific-first so the nested gitops-engine
module path never double-matches), plus go.mod (module line, the
gitops-engine require+replace pair, version reset to v0.0.0-... since a
suffix-less path can't carry a v3 semver), gitops-engine/go.mod, Makefile
(ldflags PACKAGE + 2 TEST_MODULE filters), .golangci.yaml, .goreleaser.yaml,
.mockery.yaml (+ gitops-engine's), Procfile, the codegen/proto generation
scripts (hack/generate-proto.sh, update-codegen.sh, update-openapi.sh), and
20 .proto files' go_package options. Also one OpenTelemetry tracer name
string in gitops-engine/pkg/sync/sync_context.go (a cosmestic label, not an
import, but worth keeping consistent).

CGO_ENABLED=0 go build ./... verified green for both the root module and
the nested gitops-engine module; go vet clean on both; go mod tidy run
(network available) to reconcile go.sum, dependency set unchanged.

Left alone, deliberately: doc-only mentions (docs/snyk/*.html scan reports,
gitops-engine/README.md), a comment-only reference in
.github/workflows/release.yaml, and one Renovate bot config line — none are
build- or codegen-load-bearing. Also left alone: comments citing upstream
GitHub issue URLs (github.com/argoproj/argo-cd/issues/NNNNN) and a handful
of test fixtures using "https://github.com/argoproj/argo-cd.git" as a
representative example repo URL — both are accurate historical/test-data
references, not stale self-references.

---------

Co-authored-by: Hanzo Dev <[email protected]>
2026-07-29 18:27:58 -07:00

368 lines
11 KiB
Go

//go:build !windows
package localconfig
import (
"errors"
"os"
"path"
"path/filepath"
"testing"
"github.com/stretchr/testify/require"
"github.com/hanzoai/deploy/util/config"
)
func TestGetUsername(t *testing.T) {
require.Equal(t, "admin", GetUsername("admin:login"))
require.Equal(t, "admin", GetUsername("admin"))
require.Empty(t, GetUsername(""))
}
func TestFilePermission(t *testing.T) {
dirPath := "testfolder/"
err := os.MkdirAll(path.Dir(dirPath), 0o700)
require.NoError(t, err, "Could not create argocd folder with 0700 permission: %v", err)
t.Cleanup(func() {
err := os.RemoveAll(dirPath)
require.NoError(t, err, "Could not remove directory")
})
for _, c := range []struct {
name string
testfile string
perm os.FileMode
expectedError error
}{
{
name: "Test config file with permission 0700",
testfile: ".config_0700",
perm: 0o700,
expectedError: errors.New("config file has incorrect permission flags -rwx------, change the file permission either to 0400 or 0600"),
},
{
name: "Test config file with permission 0777",
testfile: ".config_0777",
perm: 0o777,
expectedError: errors.New("config file has incorrect permission flags -rwxrwxrwx, change the file permission either to 0400 or 0600"),
},
{
name: "Test config file with permission 0600",
testfile: ".config_0600",
perm: 0o600,
expectedError: nil,
},
{
name: "Test config file with permission 0400",
testfile: ".config_0400",
perm: 0o400,
expectedError: nil,
},
{
name: "Test config file with permission 0300",
testfile: ".config_0300",
perm: 0o300,
expectedError: errors.New("config file has incorrect permission flags --wx------, change the file permission either to 0400 or 0600"),
},
} {
t.Run(c.name, func(t *testing.T) {
filePath := filepath.Join(dirPath, c.testfile)
f, err := os.Create(filePath)
require.NoError(t, err, "Could not write create config file: %v", err)
defer func() {
require.NoError(t, f.Close())
}()
err = f.Chmod(c.perm)
require.NoError(t, err, "Could not change the file permission to %s: %v", c.perm, err)
fi, err := os.Stat(filePath)
require.NoError(t, err, "Could not access the fileinfo: %v", err)
if err := getFilePermission(fi); err != nil {
require.EqualError(t, err, c.expectedError.Error())
} else {
require.NoError(t, c.expectedError)
}
})
}
}
const testConfig = `contexts:
- name: argocd1.example.com:443
server: argocd1.example.com:443
user: argocd1.example.com:443
- name: argocd2.example.com:443
server: argocd2.example.com:443
user: argocd2.example.com:443
- name: localhost:8080
server: localhost:8080
user: localhost:8080
current-context: localhost:8080
servers:
- server: argocd1.example.com:443
- server: argocd2.example.com:443
- plain-text: true
server: localhost:8080
users:
- auth-token: vErrYS3c3tReFRe$hToken
name: argocd1.example.com:443
refresh-token: vErrYS3c3tReFRe$hToken
- auth-token: vErrYS3c3tReFRe$hToken
name: argocd2.example.com:443
refresh-token: vErrYS3c3tReFRe$hToken
- auth-token: vErrYS3c3tReFRe$hToken
name: localhost:8080`
const (
testConfigFilePath = "./testdata/local.config"
testConfigFileName = "local.config"
testWriteFileName = "write-local.config"
)
func loadOpts(t *testing.T, opts string) {
t.Helper()
t.Setenv("ARGOCD_OPTS", opts)
require.NoError(t, config.LoadFlags())
}
func TestGetPromptsEnabled_useCLIOpts_false_localConfigPromptsEnabled_true(t *testing.T) {
// Write the test config file
err := os.WriteFile(testConfigFilePath, []byte(testConfig+"\nprompts-enabled: true"), os.ModePerm)
require.NoError(t, err)
defer os.Remove(testConfigFilePath)
err = os.Chmod(testConfigFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
loadOpts(t, "--config "+testConfigFilePath)
require.True(t, GetPromptsEnabled(false))
}
func TestGetPromptsEnabled_useCLIOpts_false_localConfigPromptsEnabled_false(t *testing.T) {
// Write the test config file
err := os.WriteFile(testConfigFilePath, []byte(testConfig+"\nprompts-enabled: false"), os.ModePerm)
require.NoError(t, err)
defer os.Remove(testConfigFilePath)
err = os.Chmod(testConfigFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
loadOpts(t, "--config "+testConfigFilePath)
require.False(t, GetPromptsEnabled(false))
}
func TestGetPromptsEnabled_useCLIOpts_true_forcePromptsEnabled_default(t *testing.T) {
// Write the test config file
err := os.WriteFile(testConfigFilePath, []byte(testConfig+"\nprompts-enabled: false"), os.ModePerm)
require.NoError(t, err)
defer os.Remove(testConfigFilePath)
err = os.Chmod(testConfigFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
loadOpts(t, "--config "+testConfigFilePath+" --prompts-enabled")
require.True(t, GetPromptsEnabled(true))
}
func TestGetPromptsEnabled_useCLIOpts_true_forcePromptsEnabled_true(t *testing.T) {
// Write the test config file
err := os.WriteFile(testConfigFilePath, []byte(testConfig+"\nprompts-enabled: false"), os.ModePerm)
require.NoError(t, err)
defer os.Remove(testConfigFilePath)
err = os.Chmod(testConfigFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
loadOpts(t, "--config "+testConfigFilePath+" --prompts-enabled=true")
require.True(t, GetPromptsEnabled(true))
}
func TestGetPromptsEnabled_useCLIOpts_true_forcePromptsEnabled_false(t *testing.T) {
// Write the test config file
err := os.WriteFile(testConfigFilePath, []byte(testConfig+"\nprompts-enabled: true"), os.ModePerm)
require.NoError(t, err)
defer os.Remove(testConfigFilePath)
err = os.Chmod(testConfigFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
loadOpts(t, "--config "+testConfigFilePath+" --prompts-enabled=false")
require.False(t, GetPromptsEnabled(true))
}
func TestGetToken_Exist(t *testing.T) {
testFilePath := filepath.Join(t.TempDir(), testConfigFileName)
err := os.WriteFile(testFilePath, []byte(testConfig), os.ModePerm)
require.NoError(t, err)
err = os.Chmod(testFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
localConfig, err := ReadLocalConfig(testFilePath)
require.NoError(t, err)
token := localConfig.GetToken(localConfig.CurrentContext)
require.Equal(t, "vErrYS3c3tReFRe$hToken", token)
}
func TestGetToken_Not_Exist(t *testing.T) {
testFilePath := filepath.Join(t.TempDir(), testConfigFileName)
err := os.WriteFile(testFilePath, []byte(testConfig), os.ModePerm)
require.NoError(t, err)
err = os.Chmod(testFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
localConfig, err := ReadLocalConfig(testFilePath)
require.NoError(t, err)
// serverName does exist in TestConfig
token := localConfig.GetToken("localhost")
require.Empty(t, token)
}
func TestRemoveToken_Exist(t *testing.T) {
testFilePath := filepath.Join(t.TempDir(), testConfigFileName)
err := os.WriteFile(testFilePath, []byte(testConfig), os.ModePerm)
require.NoError(t, err)
err = os.Chmod(testFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
localConfig, err := ReadLocalConfig(testFilePath)
require.NoError(t, err)
removed := localConfig.RemoveToken(localConfig.CurrentContext)
require.True(t, removed)
}
func TestRemoveToken_ClearsBothTokens(t *testing.T) {
testFilePath := filepath.Join(t.TempDir(), testConfigFileName)
err := os.WriteFile(testFilePath, []byte(testConfig), os.ModePerm)
require.NoError(t, err)
err = os.Chmod(testFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
localConfig, err := ReadLocalConfig(testFilePath)
require.NoError(t, err)
// Verify tokens exist before removal (use argocd1 which has both auth-token and refresh-token)
serverName := "argocd1.example.com:443"
require.Equal(t, "vErrYS3c3tReFRe$hToken", localConfig.GetToken(serverName))
removed := localConfig.RemoveToken(serverName)
require.True(t, removed)
// Verify both AuthToken and RefreshToken are cleared
require.Empty(t, localConfig.GetToken(serverName))
for _, u := range localConfig.Users {
if u.Name == serverName {
require.Empty(t, u.AuthToken, "AuthToken should be cleared after RemoveToken")
require.Empty(t, u.RefreshToken, "RefreshToken should be cleared after RemoveToken")
return
}
}
t.Fatal("user entry should still exist after RemoveToken")
}
func TestRemoveToken_Not_Exist(t *testing.T) {
testFilePath := filepath.Join(t.TempDir(), testConfigFileName)
err := os.WriteFile(testFilePath, []byte(testConfig), os.ModePerm)
require.NoError(t, err)
err = os.Chmod(testFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
localConfig, err := ReadLocalConfig(testFilePath)
require.NoError(t, err)
// serverName does exist in TestConfig
removed := localConfig.RemoveToken("localhost")
require.False(t, removed)
}
func TestValidateLocalConfig(t *testing.T) {
testFilePath := filepath.Join(t.TempDir(), testConfigFileName)
err := os.WriteFile(testFilePath, []byte(testConfig), os.ModePerm)
require.NoError(t, err)
err = os.Chmod(testFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
localConfig, err := ReadLocalConfig(testFilePath)
require.NoError(t, err)
err = ValidateLocalConfig(*localConfig)
require.NoError(t, err)
}
func TestWriteLocalConfig(t *testing.T) {
testFilePath := filepath.Join(t.TempDir(), testConfigFileName)
err := os.WriteFile(testFilePath, []byte(testConfig), os.ModePerm)
require.NoError(t, err)
err = os.Chmod(testFilePath, 0o600)
require.NoError(t, err, "Could not change the file permission to 0600 %v", err)
localConfig, err := ReadLocalConfig(testFilePath)
require.NoError(t, err)
testWriteFilePath := filepath.Join(t.TempDir(), testWriteFileName)
err = WriteLocalConfig(*localConfig, testWriteFilePath)
require.NoError(t, err)
}
func TestReadLocalConfig_FileNotExist(t *testing.T) {
config, err := ReadLocalConfig("/nonexistent/path/config")
require.NoError(t, err)
require.Nil(t, config)
}
func TestReadLocalConfig_InvalidYAML(t *testing.T) {
tmpFile := filepath.Join(t.TempDir(), "config")
err := os.WriteFile(tmpFile, []byte("invalid: yaml: content: ["), 0o600)
require.NoError(t, err)
config, err := ReadLocalConfig(tmpFile)
require.Error(t, err)
require.Nil(t, config)
}
func TestReadLocalConfig_EmptyFile(t *testing.T) {
tmpFile := filepath.Join(t.TempDir(), "config")
err := os.WriteFile(tmpFile, []byte(""), 0o600)
require.NoError(t, err)
config, err := ReadLocalConfig(tmpFile)
require.NoError(t, err)
// Empty file results in empty config, not nil
require.NotNil(t, config)
}
func TestReadLocalConfig_ValidConfig(t *testing.T) {
tmpFile := filepath.Join(t.TempDir(), "config")
err := os.WriteFile(tmpFile, []byte(testConfig), 0o600)
require.NoError(t, err)
config, err := ReadLocalConfig(tmpFile)
require.NoError(t, err)
require.NotNil(t, config)
require.Equal(t, "localhost:8080", config.CurrentContext)
require.Len(t, config.Contexts, 3)
}