mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2026-05-13 06:20:24 +00:00
This PR is replacing repository based hooks hooks with centralised files, this way the files don't need to be copied into every repository, only one line of config need to be added in the repository. Closes: #3523 Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/10397 Reviewed-by: Gusted <gusted@noreply.codeberg.org>
213 lines
6.6 KiB
Go
213 lines
6.6 KiB
Go
// Copyright 2023 The Gitea Authors. All rights reserved.
|
|
// Copyright 2024 The Forgejo Authors c/o Codeberg e.V.. All rights reserved.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package integration
|
|
|
|
import (
|
|
"fmt"
|
|
"net/http"
|
|
"net/url"
|
|
"testing"
|
|
|
|
repo_model "forgejo.org/models/repo"
|
|
"forgejo.org/models/unittest"
|
|
user_model "forgejo.org/models/user"
|
|
api "forgejo.org/modules/structs"
|
|
"forgejo.org/tests"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
"go.yaml.in/yaml/v3"
|
|
)
|
|
|
|
func createIssueConfigInDirectory(t *testing.T, user *user_model.User, repo *repo_model.Repository, dir string, issueConfig map[string]any) {
|
|
config, err := yaml.Marshal(issueConfig)
|
|
require.NoError(t, err)
|
|
|
|
err = createOrReplaceFileInBranch(user, repo, fmt.Sprintf("%s/ISSUE_TEMPLATE/config.yaml", dir), repo.DefaultBranch, string(config))
|
|
require.NoError(t, err)
|
|
}
|
|
|
|
func createIssueConfig(t *testing.T, user *user_model.User, repo *repo_model.Repository, issueConfig map[string]any) {
|
|
createIssueConfigInDirectory(t, user, repo, ".gitea", issueConfig)
|
|
}
|
|
|
|
func getIssueConfig(t *testing.T, owner, repo string) api.IssueConfig {
|
|
urlStr := fmt.Sprintf("/api/v1/repos/%s/%s/issue_config", owner, repo)
|
|
req := NewRequest(t, "GET", urlStr)
|
|
resp := MakeRequest(t, req, http.StatusOK)
|
|
|
|
var issueConfig api.IssueConfig
|
|
DecodeJSON(t, resp, &issueConfig)
|
|
|
|
return issueConfig
|
|
}
|
|
|
|
func TestAPIRepoGetIssueConfig(t *testing.T) {
|
|
onApplicationRun(t, func(t *testing.T, _ *url.URL) {
|
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 49})
|
|
owner := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: repo.OwnerID})
|
|
|
|
t.Run("Default", func(t *testing.T) {
|
|
defer tests.PrintCurrentTest(t)()
|
|
|
|
issueConfig := getIssueConfig(t, owner.Name, repo.Name)
|
|
|
|
assert.True(t, issueConfig.BlankIssuesEnabled)
|
|
assert.Empty(t, issueConfig.ContactLinks)
|
|
})
|
|
|
|
t.Run("DisableBlankIssues", func(t *testing.T) {
|
|
defer tests.PrintCurrentTest(t)()
|
|
|
|
config := make(map[string]any)
|
|
config["blank_issues_enabled"] = false
|
|
|
|
createIssueConfig(t, owner, repo, config)
|
|
|
|
issueConfig := getIssueConfig(t, owner.Name, repo.Name)
|
|
|
|
assert.False(t, issueConfig.BlankIssuesEnabled)
|
|
assert.Empty(t, issueConfig.ContactLinks)
|
|
})
|
|
|
|
t.Run("ContactLinks", func(t *testing.T) {
|
|
defer tests.PrintCurrentTest(t)()
|
|
|
|
contactLink := make(map[string]string)
|
|
contactLink["name"] = "TestName"
|
|
contactLink["url"] = "https://example.com"
|
|
contactLink["about"] = "TestAbout"
|
|
|
|
config := make(map[string]any)
|
|
config["contact_links"] = []map[string]string{contactLink}
|
|
|
|
createIssueConfig(t, owner, repo, config)
|
|
|
|
issueConfig := getIssueConfig(t, owner.Name, repo.Name)
|
|
|
|
assert.True(t, issueConfig.BlankIssuesEnabled)
|
|
assert.Len(t, issueConfig.ContactLinks, 1)
|
|
|
|
assert.Equal(t, "TestName", issueConfig.ContactLinks[0].Name)
|
|
assert.Equal(t, "https://example.com", issueConfig.ContactLinks[0].URL)
|
|
assert.Equal(t, "TestAbout", issueConfig.ContactLinks[0].About)
|
|
})
|
|
|
|
t.Run("Full", func(t *testing.T) {
|
|
defer tests.PrintCurrentTest(t)()
|
|
|
|
contactLink := make(map[string]string)
|
|
contactLink["name"] = "TestName"
|
|
contactLink["url"] = "https://example.com"
|
|
contactLink["about"] = "TestAbout"
|
|
|
|
config := make(map[string]any)
|
|
config["blank_issues_enabled"] = false
|
|
config["contact_links"] = []map[string]string{contactLink}
|
|
|
|
createIssueConfig(t, owner, repo, config)
|
|
|
|
issueConfig := getIssueConfig(t, owner.Name, repo.Name)
|
|
|
|
assert.False(t, issueConfig.BlankIssuesEnabled)
|
|
assert.Len(t, issueConfig.ContactLinks, 1)
|
|
|
|
assert.Equal(t, "TestName", issueConfig.ContactLinks[0].Name)
|
|
assert.Equal(t, "https://example.com", issueConfig.ContactLinks[0].URL)
|
|
assert.Equal(t, "TestAbout", issueConfig.ContactLinks[0].About)
|
|
})
|
|
})
|
|
}
|
|
|
|
func TestAPIRepoIssueConfigPaths(t *testing.T) {
|
|
onApplicationRun(t, func(t *testing.T, _ *url.URL) {
|
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 49})
|
|
owner := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: repo.OwnerID})
|
|
|
|
templateConfigCandidates := []string{
|
|
".forgejo/ISSUE_TEMPLATE/config",
|
|
".forgejo/issue_template/config",
|
|
".gitea/ISSUE_TEMPLATE/config",
|
|
".gitea/issue_template/config",
|
|
".github/ISSUE_TEMPLATE/config",
|
|
".github/issue_template/config",
|
|
"docs/issue_template/config",
|
|
}
|
|
|
|
for _, candidate := range templateConfigCandidates {
|
|
for _, extension := range []string{".yaml", ".yml"} {
|
|
fullPath := candidate + extension
|
|
t.Run(fullPath, func(t *testing.T) {
|
|
defer tests.PrintCurrentTest(t)()
|
|
|
|
configMap := make(map[string]any)
|
|
configMap["blank_issues_enabled"] = false
|
|
|
|
configData, err := yaml.Marshal(configMap)
|
|
require.NoError(t, err)
|
|
|
|
_, err = createFileInBranch(owner, repo, fullPath, repo.DefaultBranch, string(configData))
|
|
require.NoError(t, err)
|
|
|
|
issueConfig := getIssueConfig(t, owner.Name, repo.Name)
|
|
|
|
assert.False(t, issueConfig.BlankIssuesEnabled)
|
|
assert.Empty(t, issueConfig.ContactLinks)
|
|
|
|
err = deleteFileInBranch(owner, repo, fullPath, repo.DefaultBranch)
|
|
require.NoError(t, err)
|
|
})
|
|
}
|
|
}
|
|
})
|
|
}
|
|
|
|
func TestAPIRepoValidateIssueConfig(t *testing.T) {
|
|
onApplicationRun(t, func(t *testing.T, _ *url.URL) {
|
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 49})
|
|
owner := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: repo.OwnerID})
|
|
|
|
urlStr := fmt.Sprintf("/api/v1/repos/%s/%s/issue_config/validate", owner.Name, repo.Name)
|
|
|
|
t.Run("Valid", func(t *testing.T) {
|
|
defer tests.PrintCurrentTest(t)()
|
|
|
|
req := NewRequest(t, "GET", urlStr)
|
|
resp := MakeRequest(t, req, http.StatusOK)
|
|
|
|
var issueConfigValidation api.IssueConfigValidation
|
|
DecodeJSON(t, resp, &issueConfigValidation)
|
|
|
|
assert.True(t, issueConfigValidation.Valid)
|
|
assert.Empty(t, issueConfigValidation.Message)
|
|
})
|
|
|
|
t.Run("Invalid", func(t *testing.T) {
|
|
dirs := []string{".gitea", ".forgejo", "docs"}
|
|
for _, dir := range dirs {
|
|
t.Run(dir, func(t *testing.T) {
|
|
defer tests.PrintCurrentTest(t)()
|
|
defer func() {
|
|
deleteFileInBranch(owner, repo, fmt.Sprintf("%s/ISSUE_TEMPLATE/config.yaml", dir), repo.DefaultBranch)
|
|
}()
|
|
|
|
config := make(map[string]any)
|
|
config["blank_issues_enabled"] = "Test"
|
|
|
|
createIssueConfigInDirectory(t, owner, repo, dir, config)
|
|
|
|
req := NewRequest(t, "GET", urlStr)
|
|
resp := MakeRequest(t, req, http.StatusOK)
|
|
|
|
var issueConfigValidation api.IssueConfigValidation
|
|
DecodeJSON(t, resp, &issueConfigValidation)
|
|
|
|
assert.False(t, issueConfigValidation.Valid)
|
|
assert.NotEmpty(t, issueConfigValidation.Message)
|
|
})
|
|
}
|
|
})
|
|
})
|
|
}
|