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
|
package repository
import (
"os"
"path/filepath"
"testing"
"github.com/stretchr/testify/require"
"gitlab.com/gitlab-org/gitaly/v14/internal/git/gittest"
"gitlab.com/gitlab-org/gitaly/v14/internal/helper"
"gitlab.com/gitlab-org/gitaly/v14/internal/testhelper"
"gitlab.com/gitlab-org/gitaly/v14/internal/testhelper/testserver"
"gitlab.com/gitlab-org/gitaly/v14/proto/go/gitalypb"
)
func TestRemoveRepository(t *testing.T) {
t.Parallel()
cfg, client := setupRepositoryServiceWithoutRepo(t)
ctx := testhelper.Context(t)
repo := &gitalypb.Repository{
StorageName: cfg.Storages[0].Name,
RelativePath: gittest.NewRepositoryName(t, true),
}
_, err := client.CreateRepository(ctx, &gitalypb.CreateRepositoryRequest{
Repository: repo,
})
require.NoError(t, err)
_, err = client.RemoveRepository(ctx, &gitalypb.RemoveRepositoryRequest{
Repository: repo,
})
require.NoError(t, err)
require.NoFileExists(t, filepath.Join(cfg.Storages[0].Path, repo.RelativePath))
}
func TestRemoveRepository_doesNotExist(t *testing.T) {
t.Parallel()
ctx := testhelper.Context(t)
// Praefect special-cases repository removals, so we disable Praefect here.
cfg, client := setupRepositoryServiceWithoutRepo(t, testserver.WithDisablePraefect())
_, err := client.RemoveRepository(ctx, &gitalypb.RemoveRepositoryRequest{
Repository: &gitalypb.Repository{StorageName: cfg.Storages[0].Name, RelativePath: "/does/not/exist"},
})
testhelper.RequireGrpcError(t, helper.ErrNotFoundf("repository does not exist"), err)
}
func TestRemoveRepository_locking(t *testing.T) {
t.Parallel()
ctx := testhelper.Context(t)
// Praefect special-cases repository removals, so we disable Praefect here.
_, repo, repoPath, client := setupRepositoryService(ctx, t, testserver.WithDisablePraefect())
// Simulate a concurrent RPC holding the repository lock.
lockPath := repoPath + ".lock"
require.NoError(t, os.WriteFile(lockPath, []byte{}, 0o644))
defer func() { require.NoError(t, os.RemoveAll(lockPath)) }()
_, err := client.RemoveRepository(ctx, &gitalypb.RemoveRepositoryRequest{Repository: repo})
testhelper.RequireGrpcError(t, helper.ErrFailedPreconditionf("repository is already locked"), err)
}
|
