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]>
406 lines
9.7 KiB
Go
406 lines
9.7 KiB
Go
package pull_request
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
argoprojiov1alpha1 "github.com/hanzoai/cd/pkg/apis/application/v1alpha1"
|
|
)
|
|
|
|
func TestFilterBranchMatchBadRegexp(t *testing.T) {
|
|
t.Parallel()
|
|
provider, _ := NewFakeService(
|
|
t.Context(),
|
|
[]*PullRequest{
|
|
{
|
|
Number: 1,
|
|
Title: "PR branch1",
|
|
Branch: "branch1",
|
|
TargetBranch: "master",
|
|
HeadSHA: "089d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name1",
|
|
},
|
|
},
|
|
nil,
|
|
)
|
|
filters := []argoprojiov1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
BranchMatch: new("("),
|
|
},
|
|
}
|
|
_, err := ListPullRequests(t.Context(), provider, filters)
|
|
require.Error(t, err)
|
|
}
|
|
|
|
func TestFilterBranchMatch(t *testing.T) {
|
|
t.Parallel()
|
|
provider, _ := NewFakeService(
|
|
t.Context(),
|
|
[]*PullRequest{
|
|
{
|
|
Number: 1,
|
|
Title: "PR one",
|
|
Branch: "one",
|
|
TargetBranch: "master",
|
|
HeadSHA: "189d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name1",
|
|
},
|
|
{
|
|
Number: 2,
|
|
Title: "PR two",
|
|
Branch: "two",
|
|
TargetBranch: "master",
|
|
HeadSHA: "289d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name2",
|
|
},
|
|
{
|
|
Number: 3,
|
|
Title: "PR three",
|
|
Branch: "three",
|
|
TargetBranch: "master",
|
|
HeadSHA: "389d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name3",
|
|
},
|
|
{
|
|
Number: 4,
|
|
Title: "PR four",
|
|
Branch: "four",
|
|
TargetBranch: "master",
|
|
HeadSHA: "489d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name4",
|
|
},
|
|
},
|
|
nil,
|
|
)
|
|
filters := []argoprojiov1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
BranchMatch: new("w"),
|
|
},
|
|
}
|
|
pullRequests, err := ListPullRequests(t.Context(), provider, filters)
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 1)
|
|
assert.Equal(t, "two", pullRequests[0].Branch)
|
|
}
|
|
|
|
func TestFilterTargetBranchMatch(t *testing.T) {
|
|
t.Parallel()
|
|
provider, _ := NewFakeService(
|
|
t.Context(),
|
|
[]*PullRequest{
|
|
{
|
|
Number: 1,
|
|
Title: "PR one",
|
|
Branch: "one",
|
|
TargetBranch: "master",
|
|
HeadSHA: "189d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name1",
|
|
},
|
|
{
|
|
Number: 2,
|
|
Title: "PR two",
|
|
Branch: "two",
|
|
TargetBranch: "branch1",
|
|
HeadSHA: "289d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name2",
|
|
},
|
|
{
|
|
Number: 3,
|
|
Title: "PR three",
|
|
Branch: "three",
|
|
TargetBranch: "branch2",
|
|
HeadSHA: "389d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name3",
|
|
},
|
|
{
|
|
Number: 4,
|
|
Title: "PR four",
|
|
Branch: "four",
|
|
TargetBranch: "branch3",
|
|
HeadSHA: "489d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name4",
|
|
},
|
|
},
|
|
nil,
|
|
)
|
|
filters := []argoprojiov1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
TargetBranchMatch: new("1"),
|
|
},
|
|
}
|
|
pullRequests, err := ListPullRequests(t.Context(), provider, filters)
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 1)
|
|
assert.Equal(t, "two", pullRequests[0].Branch)
|
|
}
|
|
|
|
func TestFilterTitleMatch(t *testing.T) {
|
|
t.Parallel()
|
|
provider, _ := NewFakeService(
|
|
t.Context(),
|
|
[]*PullRequest{
|
|
{
|
|
Number: 1,
|
|
Title: "PR one - filter",
|
|
Branch: "one",
|
|
TargetBranch: "master",
|
|
HeadSHA: "189d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name1",
|
|
},
|
|
{
|
|
Number: 2,
|
|
Title: "PR two - ignore",
|
|
Branch: "two",
|
|
TargetBranch: "branch1",
|
|
HeadSHA: "289d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name2",
|
|
},
|
|
{
|
|
Number: 3,
|
|
Title: "[filter] PR three",
|
|
Branch: "three",
|
|
TargetBranch: "branch2",
|
|
HeadSHA: "389d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name3",
|
|
},
|
|
{
|
|
Number: 4,
|
|
Title: "[ignore] PR four",
|
|
Branch: "four",
|
|
TargetBranch: "branch3",
|
|
HeadSHA: "489d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name4",
|
|
},
|
|
},
|
|
nil,
|
|
)
|
|
filters := []argoprojiov1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
TitleMatch: new("\\[filter]"),
|
|
},
|
|
}
|
|
pullRequests, err := ListPullRequests(t.Context(), provider, filters)
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 1)
|
|
assert.Equal(t, "three", pullRequests[0].Branch)
|
|
}
|
|
|
|
func TestMultiFilterOrWithTitle(t *testing.T) {
|
|
t.Parallel()
|
|
provider, _ := NewFakeService(
|
|
t.Context(),
|
|
[]*PullRequest{
|
|
{
|
|
Number: 1,
|
|
Title: "PR one - filter",
|
|
Branch: "one",
|
|
TargetBranch: "master",
|
|
HeadSHA: "189d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name1",
|
|
},
|
|
{
|
|
Number: 2,
|
|
Title: "PR two - ignore",
|
|
Branch: "two",
|
|
TargetBranch: "branch1",
|
|
HeadSHA: "289d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name2",
|
|
},
|
|
{
|
|
Number: 3,
|
|
Title: "[filter] PR three",
|
|
Branch: "three",
|
|
TargetBranch: "branch2",
|
|
HeadSHA: "389d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name3",
|
|
},
|
|
{
|
|
Number: 4,
|
|
Title: "[ignore] PR four",
|
|
Branch: "four",
|
|
TargetBranch: "branch3",
|
|
HeadSHA: "489d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name4",
|
|
},
|
|
},
|
|
nil,
|
|
)
|
|
filters := []argoprojiov1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
TitleMatch: new("\\[filter]"),
|
|
},
|
|
{
|
|
TitleMatch: new("- filter"),
|
|
},
|
|
}
|
|
pullRequests, err := ListPullRequests(t.Context(), provider, filters)
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 2)
|
|
assert.Equal(t, "one", pullRequests[0].Branch)
|
|
assert.Equal(t, "three", pullRequests[1].Branch)
|
|
}
|
|
|
|
func TestMultiFilterOr(t *testing.T) {
|
|
t.Parallel()
|
|
provider, _ := NewFakeService(
|
|
t.Context(),
|
|
[]*PullRequest{
|
|
{
|
|
Number: 1,
|
|
Title: "PR one",
|
|
Branch: "one",
|
|
TargetBranch: "master",
|
|
HeadSHA: "189d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name1",
|
|
},
|
|
{
|
|
Number: 2,
|
|
Title: "PR two",
|
|
Branch: "two",
|
|
TargetBranch: "master",
|
|
HeadSHA: "289d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name2",
|
|
},
|
|
{
|
|
Number: 3,
|
|
Title: "PR three",
|
|
Branch: "three",
|
|
TargetBranch: "master",
|
|
HeadSHA: "389d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name3",
|
|
},
|
|
{
|
|
Number: 4,
|
|
Title: "PR four",
|
|
Branch: "four",
|
|
TargetBranch: "master",
|
|
HeadSHA: "489d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name4",
|
|
},
|
|
},
|
|
nil,
|
|
)
|
|
filters := []argoprojiov1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
BranchMatch: new("w"),
|
|
},
|
|
{
|
|
BranchMatch: new("r"),
|
|
},
|
|
}
|
|
pullRequests, err := ListPullRequests(t.Context(), provider, filters)
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 3)
|
|
assert.Equal(t, "two", pullRequests[0].Branch)
|
|
assert.Equal(t, "three", pullRequests[1].Branch)
|
|
assert.Equal(t, "four", pullRequests[2].Branch)
|
|
}
|
|
|
|
func TestMultiFilterOrWithTargetBranchFilterOrWithTitleFilter(t *testing.T) {
|
|
t.Parallel()
|
|
provider, _ := NewFakeService(
|
|
t.Context(),
|
|
[]*PullRequest{
|
|
{
|
|
Number: 1,
|
|
Title: "PR one",
|
|
Branch: "one",
|
|
TargetBranch: "master",
|
|
HeadSHA: "189d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name1",
|
|
},
|
|
{
|
|
Number: 2,
|
|
Title: "PR two",
|
|
Branch: "two",
|
|
TargetBranch: "branch1",
|
|
HeadSHA: "289d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name2",
|
|
},
|
|
{
|
|
Number: 3,
|
|
Title: "PR three",
|
|
Branch: "three",
|
|
TargetBranch: "branch2",
|
|
HeadSHA: "389d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name3",
|
|
},
|
|
{
|
|
Number: 4,
|
|
Title: "PR four",
|
|
Branch: "four",
|
|
TargetBranch: "branch3",
|
|
HeadSHA: "489d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name4",
|
|
},
|
|
{
|
|
Number: 5,
|
|
Title: "PR title is different than branch name",
|
|
Branch: "five",
|
|
TargetBranch: "branch3",
|
|
HeadSHA: "489d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name5",
|
|
},
|
|
},
|
|
nil,
|
|
)
|
|
filters := []argoprojiov1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
BranchMatch: new("w"),
|
|
TargetBranchMatch: new("1"),
|
|
},
|
|
{
|
|
BranchMatch: new("r"),
|
|
TargetBranchMatch: new("3"),
|
|
},
|
|
{
|
|
TitleMatch: new("two"),
|
|
},
|
|
{
|
|
BranchMatch: new("five"),
|
|
TitleMatch: new("PR title is different than branch name"),
|
|
},
|
|
}
|
|
pullRequests, err := ListPullRequests(t.Context(), provider, filters)
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 3)
|
|
assert.Equal(t, "two", pullRequests[0].Branch)
|
|
assert.Equal(t, "four", pullRequests[1].Branch)
|
|
assert.Equal(t, "five", pullRequests[2].Branch)
|
|
assert.Equal(t, "PR title is different than branch name", pullRequests[2].Title)
|
|
}
|
|
|
|
func TestNoFilters(t *testing.T) {
|
|
t.Parallel()
|
|
provider, _ := NewFakeService(
|
|
t.Context(),
|
|
[]*PullRequest{
|
|
{
|
|
Number: 1,
|
|
Title: "PR one",
|
|
Branch: "one",
|
|
TargetBranch: "master",
|
|
HeadSHA: "189d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name1",
|
|
},
|
|
{
|
|
Number: 2,
|
|
Title: "PR two",
|
|
Branch: "two",
|
|
TargetBranch: "master",
|
|
HeadSHA: "289d92cbf9ff857a39e6feccd32798ca700fb958",
|
|
Author: "name2",
|
|
},
|
|
},
|
|
nil,
|
|
)
|
|
filters := []argoprojiov1alpha1.PullRequestGeneratorFilter{}
|
|
repos, err := ListPullRequests(t.Context(), provider, filters)
|
|
require.NoError(t, err)
|
|
assert.Len(t, repos, 2)
|
|
assert.Equal(t, "one", repos[0].Branch)
|
|
assert.Equal(t, "two", repos[1].Branch)
|
|
}
|