mirror of
				https://github.com/go-gitea/gitea.git
				synced 2025-10-28 22:37:54 -04:00 
			
		
		
		
	Backport #16586 There is a mistake in #16319 and #16487 which means that the first time a wiki page is created a 500 is reported because the `master` branch is not in existence in that wiki yet. This PR simply checks for this error and returns not found. Fix #16584 Signed-off-by: Andrew Thornton <art27@cantab.net>
		
			
				
	
	
		
			293 lines
		
	
	
		
			8.2 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			293 lines
		
	
	
		
			8.2 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // Copyright 2019 The Gitea Authors. All rights reserved.
 | |
| // Use of this source code is governed by a MIT-style
 | |
| // license that can be found in the LICENSE file.
 | |
| 
 | |
| package wiki
 | |
| 
 | |
| import (
 | |
| 	"io/ioutil"
 | |
| 	"os"
 | |
| 	"path/filepath"
 | |
| 	"testing"
 | |
| 
 | |
| 	"code.gitea.io/gitea/models"
 | |
| 	"code.gitea.io/gitea/modules/git"
 | |
| 	"code.gitea.io/gitea/modules/util"
 | |
| 
 | |
| 	"github.com/stretchr/testify/assert"
 | |
| )
 | |
| 
 | |
| func TestMain(m *testing.M) {
 | |
| 	models.MainTest(m, filepath.Join("..", ".."))
 | |
| }
 | |
| 
 | |
| func TestWikiNameToSubURL(t *testing.T) {
 | |
| 	type test struct {
 | |
| 		Expected string
 | |
| 		WikiName string
 | |
| 	}
 | |
| 	for _, test := range []test{
 | |
| 		{"wiki-name", "wiki name"},
 | |
| 		{"wiki-name", "wiki-name"},
 | |
| 		{"name-with%2Fslash", "name with/slash"},
 | |
| 		{"name-with%25percent", "name with%percent"},
 | |
| 	} {
 | |
| 		assert.Equal(t, test.Expected, NameToSubURL(test.WikiName))
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestNormalizeWikiName(t *testing.T) {
 | |
| 	type test struct {
 | |
| 		Expected string
 | |
| 		WikiName string
 | |
| 	}
 | |
| 	for _, test := range []test{
 | |
| 		{"wiki name", "wiki name"},
 | |
| 		{"wiki name", "wiki-name"},
 | |
| 		{"name with/slash", "name with/slash"},
 | |
| 		{"name with%percent", "name-with%percent"},
 | |
| 		{"%2F", "%2F"},
 | |
| 	} {
 | |
| 		assert.Equal(t, test.Expected, NormalizeWikiName(test.WikiName))
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestWikiNameToFilename(t *testing.T) {
 | |
| 	type test struct {
 | |
| 		Expected string
 | |
| 		WikiName string
 | |
| 	}
 | |
| 	for _, test := range []test{
 | |
| 		{"wiki-name.md", "wiki name"},
 | |
| 		{"wiki-name.md", "wiki-name"},
 | |
| 		{"name-with%2Fslash.md", "name with/slash"},
 | |
| 		{"name-with%25percent.md", "name with%percent"},
 | |
| 	} {
 | |
| 		assert.Equal(t, test.Expected, NameToFilename(test.WikiName))
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestWikiFilenameToName(t *testing.T) {
 | |
| 	type test struct {
 | |
| 		Expected string
 | |
| 		Filename string
 | |
| 	}
 | |
| 	for _, test := range []test{
 | |
| 		{"hello world", "hello-world.md"},
 | |
| 		{"symbols/?*", "symbols%2F%3F%2A.md"},
 | |
| 	} {
 | |
| 		name, err := FilenameToName(test.Filename)
 | |
| 		assert.NoError(t, err)
 | |
| 		assert.Equal(t, test.Expected, name)
 | |
| 	}
 | |
| 	for _, badFilename := range []string{
 | |
| 		"nofileextension",
 | |
| 		"wrongfileextension.txt",
 | |
| 	} {
 | |
| 		_, err := FilenameToName(badFilename)
 | |
| 		assert.Error(t, err)
 | |
| 		assert.True(t, models.IsErrWikiInvalidFileName(err))
 | |
| 	}
 | |
| 	_, err := FilenameToName("badescaping%%.md")
 | |
| 	assert.Error(t, err)
 | |
| 	assert.False(t, models.IsErrWikiInvalidFileName(err))
 | |
| }
 | |
| 
 | |
| func TestWikiNameToFilenameToName(t *testing.T) {
 | |
| 	// converting from wiki name to filename, then back to wiki name should
 | |
| 	// return the original (normalized) name
 | |
| 	for _, name := range []string{
 | |
| 		"wiki-name",
 | |
| 		"wiki name",
 | |
| 		"wiki name with/slash",
 | |
| 		"$$$%%%^^&&!@#$(),.<>",
 | |
| 	} {
 | |
| 		filename := NameToFilename(name)
 | |
| 		resultName, err := FilenameToName(filename)
 | |
| 		assert.NoError(t, err)
 | |
| 		assert.Equal(t, NormalizeWikiName(name), resultName)
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestRepository_InitWiki(t *testing.T) {
 | |
| 	models.PrepareTestEnv(t)
 | |
| 	// repo1 already has a wiki
 | |
| 	repo1 := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository)
 | |
| 	assert.NoError(t, InitWiki(repo1))
 | |
| 
 | |
| 	// repo2 does not already have a wiki
 | |
| 	repo2 := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 2}).(*models.Repository)
 | |
| 	assert.NoError(t, InitWiki(repo2))
 | |
| 	assert.True(t, repo2.HasWiki())
 | |
| }
 | |
| 
 | |
| func TestRepository_AddWikiPage(t *testing.T) {
 | |
| 	assert.NoError(t, models.PrepareTestDatabase())
 | |
| 	const wikiContent = "This is the wiki content"
 | |
| 	const commitMsg = "Commit message"
 | |
| 	repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository)
 | |
| 	doer := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User)
 | |
| 	for _, wikiName := range []string{
 | |
| 		"Another page",
 | |
| 		"Here's a <tag> and a/slash",
 | |
| 	} {
 | |
| 		wikiName := wikiName
 | |
| 		t.Run("test wiki exist: "+wikiName, func(t *testing.T) {
 | |
| 			t.Parallel()
 | |
| 			assert.NoError(t, AddWikiPage(doer, repo, wikiName, wikiContent, commitMsg))
 | |
| 			// Now need to show that the page has been added:
 | |
| 			gitRepo, err := git.OpenRepository(repo.WikiPath())
 | |
| 			assert.NoError(t, err)
 | |
| 			defer gitRepo.Close()
 | |
| 			masterTree, err := gitRepo.GetTree("master")
 | |
| 			assert.NoError(t, err)
 | |
| 			wikiPath := NameToFilename(wikiName)
 | |
| 			entry, err := masterTree.GetTreeEntryByPath(wikiPath)
 | |
| 			assert.NoError(t, err)
 | |
| 			assert.Equal(t, wikiPath, entry.Name(), "%s not added correctly", wikiName)
 | |
| 		})
 | |
| 	}
 | |
| 
 | |
| 	t.Run("check wiki already exist", func(t *testing.T) {
 | |
| 		t.Parallel()
 | |
| 		// test for already-existing wiki name
 | |
| 		err := AddWikiPage(doer, repo, "Home", wikiContent, commitMsg)
 | |
| 		assert.Error(t, err)
 | |
| 		assert.True(t, models.IsErrWikiAlreadyExist(err))
 | |
| 	})
 | |
| 
 | |
| 	t.Run("check wiki reserved name", func(t *testing.T) {
 | |
| 		t.Parallel()
 | |
| 		// test for reserved wiki name
 | |
| 		err := AddWikiPage(doer, repo, "_edit", wikiContent, commitMsg)
 | |
| 		assert.Error(t, err)
 | |
| 		assert.True(t, models.IsErrWikiReservedName(err))
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestRepository_EditWikiPage(t *testing.T) {
 | |
| 	assert.NoError(t, models.PrepareTestDatabase())
 | |
| 
 | |
| 	const newWikiContent = "This is the new content"
 | |
| 	const commitMsg = "Commit message"
 | |
| 	repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository)
 | |
| 	doer := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User)
 | |
| 	for _, newWikiName := range []string{
 | |
| 		"Home", // same name as before
 | |
| 		"New home",
 | |
| 		"New/name/with/slashes",
 | |
| 	} {
 | |
| 		models.PrepareTestEnv(t)
 | |
| 		assert.NoError(t, EditWikiPage(doer, repo, "Home", newWikiName, newWikiContent, commitMsg))
 | |
| 
 | |
| 		// Now need to show that the page has been added:
 | |
| 		gitRepo, err := git.OpenRepository(repo.WikiPath())
 | |
| 		assert.NoError(t, err)
 | |
| 		masterTree, err := gitRepo.GetTree("master")
 | |
| 		assert.NoError(t, err)
 | |
| 		wikiPath := NameToFilename(newWikiName)
 | |
| 		entry, err := masterTree.GetTreeEntryByPath(wikiPath)
 | |
| 		assert.NoError(t, err)
 | |
| 		assert.Equal(t, wikiPath, entry.Name(), "%s not editted correctly", newWikiName)
 | |
| 
 | |
| 		if newWikiName != "Home" {
 | |
| 			_, err := masterTree.GetTreeEntryByPath("Home.md")
 | |
| 			assert.Error(t, err)
 | |
| 		}
 | |
| 		gitRepo.Close()
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestRepository_DeleteWikiPage(t *testing.T) {
 | |
| 	models.PrepareTestEnv(t)
 | |
| 	repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository)
 | |
| 	doer := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User)
 | |
| 	assert.NoError(t, DeleteWikiPage(doer, repo, "Home"))
 | |
| 
 | |
| 	// Now need to show that the page has been added:
 | |
| 	gitRepo, err := git.OpenRepository(repo.WikiPath())
 | |
| 	assert.NoError(t, err)
 | |
| 	defer gitRepo.Close()
 | |
| 	masterTree, err := gitRepo.GetTree("master")
 | |
| 	assert.NoError(t, err)
 | |
| 	wikiPath := NameToFilename("Home")
 | |
| 	_, err = masterTree.GetTreeEntryByPath(wikiPath)
 | |
| 	assert.Error(t, err)
 | |
| }
 | |
| 
 | |
| func TestPrepareWikiFileName(t *testing.T) {
 | |
| 	models.PrepareTestEnv(t)
 | |
| 	repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository)
 | |
| 	gitRepo, err := git.OpenRepository(repo.WikiPath())
 | |
| 	defer gitRepo.Close()
 | |
| 	assert.NoError(t, err)
 | |
| 
 | |
| 	tests := []struct {
 | |
| 		name      string
 | |
| 		arg       string
 | |
| 		existence bool
 | |
| 		wikiPath  string
 | |
| 		wantErr   bool
 | |
| 	}{{
 | |
| 		name:      "add suffix",
 | |
| 		arg:       "Home",
 | |
| 		existence: true,
 | |
| 		wikiPath:  "Home.md",
 | |
| 		wantErr:   false,
 | |
| 	}, {
 | |
| 		name:      "test special chars",
 | |
| 		arg:       "home of and & or wiki page!",
 | |
| 		existence: false,
 | |
| 		wikiPath:  "home-of-and-%26-or-wiki-page%21.md",
 | |
| 		wantErr:   false,
 | |
| 	}, {
 | |
| 		name:      "fount unescaped cases",
 | |
| 		arg:       "Unescaped File",
 | |
| 		existence: true,
 | |
| 		wikiPath:  "Unescaped File.md",
 | |
| 		wantErr:   false,
 | |
| 	}}
 | |
| 	for _, tt := range tests {
 | |
| 		t.Run(tt.name, func(t *testing.T) {
 | |
| 			existence, newWikiPath, err := prepareWikiFileName(gitRepo, tt.arg)
 | |
| 			if (err != nil) != tt.wantErr {
 | |
| 				assert.NoError(t, err)
 | |
| 				return
 | |
| 			}
 | |
| 			if existence != tt.existence {
 | |
| 				if existence {
 | |
| 					t.Errorf("expect to find no escaped file but we detect one")
 | |
| 				} else {
 | |
| 					t.Errorf("expect to find an escaped file but we could not detect one")
 | |
| 				}
 | |
| 			}
 | |
| 			assert.Equal(t, tt.wikiPath, newWikiPath)
 | |
| 		})
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestPrepareWikiFileName_FirstPage(t *testing.T) {
 | |
| 	models.PrepareTestEnv(t)
 | |
| 
 | |
| 	// Now create a temporaryDirectory
 | |
| 	tmpDir, err := ioutil.TempDir("", "empty-wiki")
 | |
| 	assert.NoError(t, err)
 | |
| 	defer func() {
 | |
| 		if _, err := os.Stat(tmpDir); !os.IsNotExist(err) {
 | |
| 			_ = util.RemoveAll(tmpDir)
 | |
| 		}
 | |
| 	}()
 | |
| 
 | |
| 	err = git.InitRepository(tmpDir, true)
 | |
| 	assert.NoError(t, err)
 | |
| 
 | |
| 	gitRepo, err := git.OpenRepository(tmpDir)
 | |
| 	defer gitRepo.Close()
 | |
| 	assert.NoError(t, err)
 | |
| 
 | |
| 	existence, newWikiPath, err := prepareWikiFileName(gitRepo, "Home")
 | |
| 	assert.False(t, existence)
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "Home.md", newWikiPath)
 | |
| }
 |