mirror of
https://codeberg.org/forgejo/forgejo
synced 2024-11-24 18:56:11 +01:00
e019eb33a2
When trying to find a `README.md` in a `.profile` repo, do so case insensitively. This change does not make it possible to render readmes in formats other than Markdown, it just removes the hard-coded "README.md". Also adds a few tests to make sure the change works. Fixes #1494. Signed-off-by: Gergely Nagy <forgejo@gergo.csillger.hu> (cherry picked from commitedd219d8e9) (cherry picked from commit2c0105ef17) (cherry picked from commit3975a9f3aa) (cherry picked from commitdee4a18423) (cherry picked from commit60aee6370f)
121 lines
3.2 KiB
Go
121 lines
3.2 KiB
Go
// Copyright 2024 The Forgejo Authors c/o Codeberg e.V.. All rights reserved.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package integration
|
|
|
|
import (
|
|
"net/http"
|
|
"net/url"
|
|
"strings"
|
|
"testing"
|
|
"time"
|
|
|
|
"code.gitea.io/gitea/models/db"
|
|
"code.gitea.io/gitea/models/unittest"
|
|
user_model "code.gitea.io/gitea/models/user"
|
|
"code.gitea.io/gitea/modules/git"
|
|
repo_service "code.gitea.io/gitea/services/repository"
|
|
files_service "code.gitea.io/gitea/services/repository/files"
|
|
"code.gitea.io/gitea/tests"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
func createProfileRepo(t *testing.T, readmeName string) func() {
|
|
user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
|
|
|
// Create a new repository
|
|
repo, err := repo_service.CreateRepository(db.DefaultContext, user2, user2, repo_service.CreateRepoOptions{
|
|
Name: ".profile",
|
|
DefaultBranch: "main",
|
|
IsPrivate: false,
|
|
AutoInit: true,
|
|
Readme: "Default",
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.NotEmpty(t, repo)
|
|
|
|
deleteInitialReadmeResp, err := files_service.ChangeRepoFiles(git.DefaultContext, repo, user2,
|
|
&files_service.ChangeRepoFilesOptions{
|
|
Files: []*files_service.ChangeRepoFile{
|
|
{
|
|
Operation: "delete",
|
|
TreePath: "README.md",
|
|
},
|
|
},
|
|
Message: "Delete the initial readme",
|
|
Author: &files_service.IdentityOptions{
|
|
Name: user2.Name,
|
|
Email: user2.Email,
|
|
},
|
|
Committer: &files_service.IdentityOptions{
|
|
Name: user2.Name,
|
|
Email: user2.Email,
|
|
},
|
|
Dates: &files_service.CommitDateOptions{
|
|
Author: time.Now(),
|
|
Committer: time.Now(),
|
|
},
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.NotEmpty(t, deleteInitialReadmeResp)
|
|
|
|
if readmeName != "" {
|
|
addReadmeResp, err := files_service.ChangeRepoFiles(git.DefaultContext, repo, user2,
|
|
&files_service.ChangeRepoFilesOptions{
|
|
Files: []*files_service.ChangeRepoFile{
|
|
{
|
|
Operation: "create",
|
|
TreePath: readmeName,
|
|
ContentReader: strings.NewReader("# Hi!\n"),
|
|
},
|
|
},
|
|
Message: "Add a readme",
|
|
Author: &files_service.IdentityOptions{
|
|
Name: user2.Name,
|
|
Email: user2.Email,
|
|
},
|
|
Committer: &files_service.IdentityOptions{
|
|
Name: user2.Name,
|
|
Email: user2.Email,
|
|
},
|
|
Dates: &files_service.CommitDateOptions{
|
|
Author: time.Now(),
|
|
Committer: time.Now(),
|
|
},
|
|
})
|
|
|
|
assert.NoError(t, err)
|
|
assert.NotEmpty(t, addReadmeResp)
|
|
}
|
|
|
|
return func() {
|
|
repo_service.DeleteRepository(db.DefaultContext, user2, repo, false)
|
|
}
|
|
}
|
|
|
|
func TestUserProfile(t *testing.T) {
|
|
onGiteaRun(t, func(t *testing.T, u *url.URL) {
|
|
checkReadme := func(t *testing.T, title, readmeFilename string, expectedCount int) {
|
|
t.Run(title, func(t *testing.T) {
|
|
defer tests.PrintCurrentTest(t)()
|
|
defer createProfileRepo(t, readmeFilename)()
|
|
|
|
req := NewRequest(t, "GET", "/user2")
|
|
resp := MakeRequest(t, req, http.StatusOK)
|
|
|
|
doc := NewHTMLParser(t, resp.Body)
|
|
readmeCount := doc.Find("#readme_profile").Length()
|
|
|
|
assert.Equal(t, expectedCount, readmeCount)
|
|
})
|
|
}
|
|
|
|
checkReadme(t, "No readme", "", 0)
|
|
checkReadme(t, "README.md", "README.md", 1)
|
|
checkReadme(t, "readme.md", "readme.md", 1)
|
|
checkReadme(t, "ReadMe.mD", "ReadMe.mD", 1)
|
|
checkReadme(t, "readme.org does not render", "README.org", 0)
|
|
})
|
|
}
|