Make `LocalCopyPath` a setting instead of a hard-coded path (#1881)

1 files changed, 22 insertions, 0 deletions

diff --git a/models/repo_test.go b/models/repo_test.go
index a538d44c0e..13ac272b6d 100644
--- a/models/repo_test.go
+++ b/models/repo_test.go
@@ -5,11 +5,14 @@
 package models
 
 import (
+	"path"
 	"testing"
 
 	"code.gitea.io/gitea/modules/markdown"
+	"code.gitea.io/gitea/modules/setting"
 
 	"github.com/stretchr/testify/assert"
+	"github.com/Unknwon/com"
 )
 
 func TestRepo(t *testing.T) {
@@ -132,3 +135,22 @@ func TestRepoAPIURL(t *testing.T) {
 
 	assert.Equal(t, "https://try.gitea.io/api/v1/repos/user12/repo10", repo.APIURL())
 }
+
+func TestRepoLocalCopyPath(t *testing.T) {
+	assert.NoError(t, PrepareTestDatabase())
+
+	repo, err := GetRepositoryByID(10)
+	assert.NoError(t, err)
+	assert.NotNil(t, repo)
+
+	// test default
+	repoID := com.ToStr(repo.ID)
+	expected := path.Join(setting.AppDataPath, setting.Repository.Local.LocalCopyPath, repoID)
+	assert.Equal(t, expected, repo.LocalCopyPath())
+
+	// test absolute setting
+	tempPath := "/tmp/gitea/local-copy-path"
+	expected = path.Join(tempPath, repoID)
+	setting.Repository.Local.LocalCopyPath = tempPath
+	assert.Equal(t, expected, repo.LocalCopyPath())
+}