1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
|
package repository
import (
"os"
"path/filepath"
"testing"
"github.com/stretchr/testify/require"
"gitlab.com/gitlab-org/gitaly/internal/git/gittest"
"gitlab.com/gitlab-org/gitaly/internal/storage"
"gitlab.com/gitlab-org/gitaly/internal/testhelper"
"gitlab.com/gitlab-org/gitaly/proto/go/gitalypb"
"google.golang.org/grpc/codes"
)
func TestRenameRepositorySuccess(t *testing.T) {
cfg, repo, _, client := setupRepositoryService(t)
req := &gitalypb.RenameRepositoryRequest{Repository: repo, RelativePath: "a-new-location"}
ctx, cancel := testhelper.Context()
defer cancel()
_, err := client.RenameRepository(ctx, req)
require.NoError(t, err)
newDirectory := filepath.Join(cfg.Storages[0].Path, req.RelativePath)
require.DirExists(t, newDirectory)
defer func() { require.NoError(t, os.RemoveAll(newDirectory)) }()
require.True(t, storage.IsGitDirectory(newDirectory), "moved Git repository has been corrupted")
// ensure the git directory that got renamed contains a sha in the seed repo
gittest.GitObjectMustExist(t, cfg.Git.BinPath, newDirectory, "913c66a37b4a45b9769037c55c2d238bd0942d2e")
}
func TestRenameRepositoryDestinationExists(t *testing.T) {
cfg, repo, _, client := setupRepositoryService(t)
destinationRepo, destinationRepoPath, cleanupDestinationRepo := gittest.CloneRepoAtStorage(t, cfg, cfg.Storages[0], "dst")
t.Cleanup(cleanupDestinationRepo)
sha := gittest.WriteCommit(t, cfg, destinationRepoPath)
req := &gitalypb.RenameRepositoryRequest{Repository: repo, RelativePath: destinationRepo.GetRelativePath()}
ctx, cancel := testhelper.Context()
defer cancel()
_, err := client.RenameRepository(ctx, req)
testhelper.RequireGrpcError(t, err, codes.FailedPrecondition)
// ensure the git directory that already existed didn't get overwritten
gittest.GitObjectMustExist(t, cfg.Git.BinPath, destinationRepoPath, sha.String())
}
func TestRenameRepositoryInvalidRequest(t *testing.T) {
_, repo, _, client := setupRepositoryService(t)
ctx, cancel := testhelper.Context()
defer cancel()
testCases := []struct {
desc string
req *gitalypb.RenameRepositoryRequest
}{
{
desc: "empty repository",
req: &gitalypb.RenameRepositoryRequest{Repository: nil, RelativePath: "/tmp/abc"},
},
{
desc: "empty destination relative path",
req: &gitalypb.RenameRepositoryRequest{Repository: repo, RelativePath: ""},
},
{
desc: "destination relative path contains path traversal",
req: &gitalypb.RenameRepositoryRequest{Repository: repo, RelativePath: "../usr/bin"},
},
}
for _, tc := range testCases {
t.Run(tc.desc, func(t *testing.T) {
_, err := client.RenameRepository(ctx, tc.req)
testhelper.RequireGrpcError(t, err, codes.InvalidArgument)
})
}
}
|
