mirror of
https://github.com/hanzoai/deploy.git
synced 2026-10-07 08:44:50 +00:00
zeekayandHanzo Dev
1ab6e799c1
go.mod still declared `module github.com/hanzoai/deploy` after the repo was renamed. A Go module path is not cosmetic — it is the resolvable location. So `go get github.com/hanzoai/cd` could not work, and anything importing this module had to spell a name that no longer exists at that address. The build log of v3.6.0 is where it surfaced: the compiler was still emitting github.com/hanzoai/deploy/cmd/cd/commands. 3541 import paths across 690 .go, 19 .proto, 4 .yaml, 3 .sh, 2 .mod, Procfile and Makefile. Plus the 62 ghcr.io/hanzoai/deploy image references, moved to ghcr.io/hanzoai/cd to match hanzo.yml's build target — a distinct string from the module path, replaced separately so neither could clobber the other. go mod tidy clean, CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build ./... clean. Co-authored-by: Hanzo Dev <[email protected]>
110 lines
3.4 KiB
Go
110 lines
3.4 KiB
Go
package hook
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
|
|
|
|
"github.com/hanzoai/cd/gitops-engine/pkg/sync/common"
|
|
testingutils "github.com/hanzoai/cd/gitops-engine/pkg/utils/testing"
|
|
)
|
|
|
|
func TestNoHooks(t *testing.T) {
|
|
t.Parallel()
|
|
obj := &unstructured.Unstructured{}
|
|
assert.False(t, IsHook(obj))
|
|
assert.False(t, Skip(obj))
|
|
assert.Nil(t, Types(obj))
|
|
}
|
|
|
|
func TestOneHook(t *testing.T) {
|
|
t.Parallel()
|
|
hookTypesString := []string{"PreSync", "Sync", "PostSync", "SyncFail"}
|
|
hookTypes := []common.HookType{common.HookTypePreSync, common.HookTypeSync, common.HookTypePostSync, common.HookTypeSyncFail}
|
|
for i, hook := range hookTypesString {
|
|
obj := example(hook)
|
|
assert.True(t, IsHook(obj))
|
|
assert.False(t, Skip(obj))
|
|
assert.Equal(t, []common.HookType{hookTypes[i]}, Types(obj))
|
|
}
|
|
}
|
|
|
|
// peculiar case of something marked with "Skip" cannot, by definition, be a hook
|
|
// IMHO this is bad design as it conflates a flag on something that can never be a hook, with something that is
|
|
// always a hook, creating a nasty exception we always need to check for, and a bunch of horrible edge cases
|
|
func TestSkipHook(t *testing.T) {
|
|
t.Parallel()
|
|
obj := example("Skip")
|
|
assert.False(t, IsHook(obj))
|
|
assert.True(t, Skip(obj))
|
|
assert.Equal(t, []common.HookType{common.HookTypeSkip}, Types(obj))
|
|
}
|
|
|
|
// we treat garbage as the user intended you to be a hook, but spelled it wrong, so you are a hook, but we don't
|
|
// know what phase you're a part of
|
|
func TestGarbageHook(t *testing.T) {
|
|
t.Parallel()
|
|
obj := example("Garbage")
|
|
assert.True(t, IsHook(obj))
|
|
assert.False(t, Skip(obj))
|
|
assert.Nil(t, Types(obj))
|
|
}
|
|
|
|
func TestTwoHooks(t *testing.T) {
|
|
t.Parallel()
|
|
obj := example("PreSync,PostSync")
|
|
assert.True(t, IsHook(obj))
|
|
assert.False(t, Skip(obj))
|
|
assert.ElementsMatch(t, []common.HookType{common.HookTypePreSync, common.HookTypePostSync}, Types(obj))
|
|
}
|
|
|
|
func TestDupHookTypes(t *testing.T) {
|
|
t.Parallel()
|
|
assert.Equal(t, []common.HookType{common.HookTypeSync}, Types(example("Sync,Sync")))
|
|
}
|
|
|
|
// horrible edge case
|
|
func TestSkipAndHook(t *testing.T) {
|
|
t.Parallel()
|
|
obj := example("Skip,PreSync,PostSync")
|
|
assert.True(t, IsHook(obj))
|
|
assert.False(t, Skip(obj))
|
|
assert.ElementsMatch(t, []common.HookType{common.HookTypeSkip, common.HookTypePreSync, common.HookTypePostSync}, Types(obj))
|
|
}
|
|
|
|
func TestGarbageAndHook(t *testing.T) {
|
|
t.Parallel()
|
|
obj := example("Sync,Garbage")
|
|
assert.True(t, IsHook(obj))
|
|
assert.False(t, Skip(obj))
|
|
assert.Equal(t, []common.HookType{common.HookTypeSync}, Types(obj))
|
|
}
|
|
|
|
func TestHelmHook(t *testing.T) {
|
|
t.Parallel()
|
|
obj := testingutils.Annotate(testingutils.NewPod(), "helm.sh/hook", "pre-install")
|
|
assert.True(t, IsHook(obj))
|
|
assert.False(t, Skip(obj))
|
|
assert.Equal(t, []common.HookType{common.HookTypePreSync}, Types(obj))
|
|
}
|
|
|
|
func TestGarbageHelmHook(t *testing.T) {
|
|
t.Parallel()
|
|
obj := testingutils.Annotate(testingutils.NewPod(), "helm.sh/hook", "garbage")
|
|
assert.True(t, IsHook(obj))
|
|
assert.False(t, Skip(obj))
|
|
assert.Nil(t, Types(obj))
|
|
}
|
|
|
|
// we should ignore Helm hooks if we have an Argo CD hook
|
|
func TestBothHooks(t *testing.T) {
|
|
t.Parallel()
|
|
obj := testingutils.Annotate(example("Sync"), "helm.sh/hook", "pre-install")
|
|
assert.Equal(t, []common.HookType{common.HookTypeSync}, Types(obj))
|
|
}
|
|
|
|
func example(hook string) *unstructured.Unstructured {
|
|
return testingutils.Annotate(testingutils.NewPod(), "cd.hanzo.ai/hook", hook)
|
|
}
|