334 lines
7.7 KiB
Go
334 lines
7.7 KiB
Go
package common_test
|
|
|
|
import (
|
|
"reflect"
|
|
"testing"
|
|
|
|
"src.opensuse.org/autogits/common"
|
|
"src.opensuse.org/autogits/common/gitea-generated/models"
|
|
)
|
|
|
|
func TestGitUrlParse(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
inputUrl string
|
|
url common.GitUrl
|
|
|
|
error bool
|
|
}{
|
|
{
|
|
name: "Empty string",
|
|
error: true,
|
|
},
|
|
{
|
|
name: "OpenSUSE HTTPS Url",
|
|
url: common.GitUrl{
|
|
Org: "foo",
|
|
Repo: "b",
|
|
},
|
|
inputUrl: "https://src.opensuse.org/foo/b.git",
|
|
},
|
|
{
|
|
name: "OpenSUSE HTTPS Url",
|
|
url: common.GitUrl{
|
|
Org: "a",
|
|
Repo: "b",
|
|
},
|
|
inputUrl: "https://src.opensuse.org/a/b",
|
|
},
|
|
{
|
|
name: "OpenSUSE HTTPS Url",
|
|
url: common.GitUrl{
|
|
Org: "foo",
|
|
Repo: "bar",
|
|
Commit: "main",
|
|
},
|
|
inputUrl: "https://src.opensuse.org/foo/bar.git#main",
|
|
},
|
|
{
|
|
name: "invalid OpenSUSE HTTPS Url",
|
|
inputUrl: "https://src.opensuse.org/bar.git#main",
|
|
error: true,
|
|
},
|
|
{
|
|
name: "OpenSUSE SSH Url",
|
|
url: common.GitUrl{
|
|
Org: "foo",
|
|
Repo: "bar",
|
|
Commit: "main",
|
|
},
|
|
inputUrl: "ssh://src.opensuse.org/foo/bar.git#main",
|
|
},
|
|
{
|
|
name: "SSH native OpenSUSE Url",
|
|
inputUrl: "gitea@src.opensuse.org:foo/bar.git#main",
|
|
url: common.GitUrl{
|
|
Org: "foo",
|
|
Repo: "bar",
|
|
Commit: "main",
|
|
},
|
|
},
|
|
{
|
|
name: "SSH native OpenSUSE Url without user",
|
|
inputUrl: "src.opensuse.org:foo/bar.git#main",
|
|
url: common.GitUrl{
|
|
Org: "foo",
|
|
Repo: "bar",
|
|
Commit: "main",
|
|
},
|
|
},
|
|
{
|
|
name: "invalid SSH native OpenSUSE Url without user",
|
|
inputUrl: "src.opensuse.org:/br.it",
|
|
error: true,
|
|
},
|
|
{
|
|
name: "SSH native OpenSUSE Url without user",
|
|
inputUrl: "src.opensuse.org:foo/bar#main",
|
|
url: common.GitUrl{
|
|
Org: "foo",
|
|
Repo: "bar",
|
|
Commit: "main",
|
|
},
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
t.Run(test.name, func(t *testing.T) {
|
|
url, err := common.ParseGitRemoteUrl(test.inputUrl)
|
|
if test.error && err != nil {
|
|
return
|
|
}
|
|
|
|
if test.error && err == nil {
|
|
t.Fatal("Expected an error but received", *url)
|
|
} else if !test.error && err != nil {
|
|
t.Error(err)
|
|
}
|
|
|
|
if url == nil {
|
|
t.Fatal("Recieved nil. Expected", test.url)
|
|
} else if *url != test.url {
|
|
t.Fatalf("Expected %v but received %v", test.url, *url)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestRemoteName(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
giturl *common.GitUrl
|
|
remotename string
|
|
}{
|
|
{
|
|
name: "empty",
|
|
remotename: "origin",
|
|
},
|
|
{
|
|
name: "regular repo",
|
|
giturl: &common.GitUrl{
|
|
Org: "org1",
|
|
Repo: "repo2",
|
|
Commit: "main",
|
|
},
|
|
remotename: "org1_repo2",
|
|
},
|
|
{
|
|
name: "regular repo with upper case",
|
|
giturl: &common.GitUrl{
|
|
Org: "Org1",
|
|
Repo: "REPO2",
|
|
},
|
|
remotename: "org1_repo2",
|
|
},
|
|
{
|
|
name: "no org",
|
|
giturl: &common.GitUrl{
|
|
Repo: "REPO2",
|
|
},
|
|
remotename: "origin",
|
|
},
|
|
{
|
|
name: "no repo",
|
|
giturl: &common.GitUrl{
|
|
Org: "ORG2",
|
|
},
|
|
remotename: "origin",
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
t.Run(test.name, func(t *testing.T) {
|
|
n := test.giturl.RemoteName()
|
|
if n != test.remotename {
|
|
t.Errorf("Expected '%s' but received '%s'", test.remotename, n)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestRemovedBranchName(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
branchName string
|
|
isRemoved bool
|
|
regularName string
|
|
}{
|
|
{
|
|
name: "Empty branch",
|
|
},
|
|
{
|
|
name: "Removed suffix only",
|
|
branchName: "-rm",
|
|
isRemoved: false,
|
|
regularName: "-rm",
|
|
},
|
|
{
|
|
name: "Capital suffix",
|
|
branchName: "Foo-Rm",
|
|
isRemoved: true,
|
|
regularName: "Foo",
|
|
},
|
|
{
|
|
name: "Other suffixes",
|
|
isRemoved: true,
|
|
branchName: "Goo-Rm-DeleteD",
|
|
regularName: "Goo-Rm",
|
|
},
|
|
{
|
|
name: "Other suffixes",
|
|
isRemoved: true,
|
|
branchName: "main-REMOVED",
|
|
regularName: "main",
|
|
},
|
|
{
|
|
name: "Not removed separator",
|
|
isRemoved: false,
|
|
branchName: "main;REMOVED",
|
|
regularName: "main;REMOVED",
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
t.Run(test.name, func(t *testing.T) {
|
|
if r := common.IsRemovedBranch(test.branchName); r != test.isRemoved {
|
|
t.Error("Expecting isRemoved:", test.isRemoved, "but received", r)
|
|
}
|
|
|
|
if tn := common.TrimRemovedBranchSuffix(test.branchName); tn != test.regularName {
|
|
t.Error("Expected stripped branch name to be:", test.regularName, "but have:", tn)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestNewPackageIssueParsing(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
input string
|
|
issues *common.NewRepos
|
|
}{
|
|
{
|
|
name: "Nothing",
|
|
},
|
|
{
|
|
name: "Basic repo",
|
|
input: "org/repo#branch",
|
|
issues: &common.NewRepos{
|
|
Repos: []struct{ Organization, Repository, Branch, PackageName string }{
|
|
{Organization: "org", Repository: "repo", Branch: "branch", PackageName: "repo"},
|
|
},
|
|
},
|
|
},
|
|
{
|
|
name: "Default branch and junk lines and approval for maintainership",
|
|
input: "\n\nsome comments\n\norg1/repo2\n\nmaintainership: yes",
|
|
issues: &common.NewRepos{
|
|
Repos: []struct{ Organization, Repository, Branch, PackageName string }{
|
|
{Organization: "org1", Repository: "repo2", Branch: "", PackageName: "repo2"},
|
|
},
|
|
IsMaintainer: true,
|
|
},
|
|
},
|
|
{
|
|
name: "Default branch and junk lines and no maintainership",
|
|
input: "\n\nsome comments\n\norg1/repo2\n\nmaintainership: NEVER",
|
|
issues: &common.NewRepos{
|
|
Repos: []struct{ Organization, Repository, Branch, PackageName string }{
|
|
{Organization: "org1", Repository: "repo2", Branch: "", PackageName: "repo2"},
|
|
},
|
|
},
|
|
},
|
|
{
|
|
name: "3 repos with comments and maintainership",
|
|
input: "\n\nsome comments for org1/repo2 are here and more\n\norg1/repo2#master\n org2/repo3#master\n some/repo3#m\nMaintainer ok",
|
|
issues: &common.NewRepos{
|
|
Repos: []struct{ Organization, Repository, Branch, PackageName string }{
|
|
{Organization: "org1", Repository: "repo2", Branch: "master", PackageName: "repo2"},
|
|
{Organization: "org2", Repository: "repo3", Branch: "master", PackageName: "repo3"},
|
|
{Organization: "some", Repository: "repo3", Branch: "m", PackageName: "repo3"},
|
|
},
|
|
IsMaintainer: true,
|
|
},
|
|
},
|
|
{
|
|
name: "Invalid repos with spaces",
|
|
input: "or g/repo#branch\norg/r epo#branch\norg/repo#br anch\norg/repo#branch As foo ++",
|
|
},
|
|
{
|
|
name: "Valid repos with spaces",
|
|
input: " org / repo # branch",
|
|
issues: &common.NewRepos{
|
|
Repos: []struct{ Organization, Repository, Branch, PackageName string }{
|
|
{Organization: "org", Repository: "repo", Branch: "branch", PackageName: "repo"},
|
|
},
|
|
},
|
|
},
|
|
{
|
|
name: "Package name is not repo name",
|
|
input: " org / repo # branch as repo++ \nmaintainer true",
|
|
issues: &common.NewRepos{
|
|
Repos: []struct{ Organization, Repository, Branch, PackageName string }{
|
|
{Organization: "org", Repository: "repo", Branch: "branch", PackageName: "repo++"},
|
|
},
|
|
IsMaintainer: true,
|
|
},
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
t.Run(test.name, func(t *testing.T) {
|
|
issue := common.FindNewReposInIssueBody(test.input)
|
|
if !reflect.DeepEqual(test.issues, issue) {
|
|
t.Error("Expected", test.issues, "but have", issue)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestIssueToString(t *testing.T) {
|
|
t.Run("nil issue", func(t *testing.T) {
|
|
if got := common.IssueToString(nil); got != "(nil)" {
|
|
t.Errorf("expected (nil), got %s", got)
|
|
}
|
|
})
|
|
t.Run("nil repository", func(t *testing.T) {
|
|
// This test currently panics, which is a bug.
|
|
common.IssueToString(&models.Issue{Index: 1})
|
|
})
|
|
}
|
|
|
|
func TestPRtoString(t *testing.T) {
|
|
t.Run("nil pr", func(t *testing.T) {
|
|
if got := common.PRtoString(nil); got != "(null)" {
|
|
t.Errorf("expected (null), got %s", got)
|
|
}
|
|
})
|
|
t.Run("nil Base", func(t *testing.T) {
|
|
// This test currently panics, which is a bug.
|
|
common.PRtoString(&models.PullRequest{Index: 1})
|
|
})
|
|
}
|
|
|