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
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
|
package remote
import (
"testing"
"github.com/stretchr/testify/require"
"gitlab.com/gitlab-org/gitaly/internal/git/gittest"
"gitlab.com/gitlab-org/gitaly/internal/helper/text"
"gitlab.com/gitlab-org/gitaly/internal/testhelper"
"gitlab.com/gitlab-org/gitaly/proto/go/gitalypb"
"google.golang.org/grpc/codes"
"google.golang.org/grpc/status"
)
func TestFindRemoteRootRefSuccess(t *testing.T) {
_, repo, repoPath, client := setupRemoteService(t)
originURL := text.ChompBytes(testhelper.MustRunCommand(t, nil,
"git", "-C", repoPath, "remote", "get-url", "origin"))
for _, tc := range []struct {
desc string
request *gitalypb.FindRemoteRootRefRequest
}{
{
desc: "with remote name",
request: &gitalypb.FindRemoteRootRefRequest{Repository: repo, Remote: "origin"},
},
{
desc: "with remote URL",
request: &gitalypb.FindRemoteRootRefRequest{Repository: repo, RemoteUrl: originURL},
},
{
// Unfortunately, we do not really have a nice way to verify we actually got
// the auth header. So this test case only really verifies that it doesn't
// break the world to set up one.
desc: "with remote URL and auth header",
request: &gitalypb.FindRemoteRootRefRequest{
Repository: repo,
RemoteUrl: originURL,
HttpAuthorizationHeader: "mysecret",
},
},
} {
t.Run(tc.desc, func(t *testing.T) {
ctx, cancel := testhelper.Context()
defer cancel()
response, err := client.FindRemoteRootRef(ctx, tc.request)
require.NoError(t, err)
require.Equal(t, "master", response.Ref)
})
}
}
func TestFindRemoteRootRefWithUnbornRemoteHead(t *testing.T) {
cfg, remoteRepo, remoteRepoPath, client := setupRemoteService(t)
// We're creating an empty repository. Empty repositories do have a HEAD set up, but they
// point to an unborn branch because the default branch hasn't yet been created.
_, clientRepoPath, cleanup := gittest.InitBareRepoAt(t, cfg, cfg.Storages[0])
defer cleanup()
testhelper.MustRunCommand(t, nil, "git", "-C", remoteRepoPath, "remote", "add",
"foo", "file://"+clientRepoPath)
ctx, cancel := testhelper.Context()
defer cancel()
for _, request := range []*gitalypb.FindRemoteRootRefRequest{
&gitalypb.FindRemoteRootRefRequest{Repository: remoteRepo, Remote: "foo"},
&gitalypb.FindRemoteRootRefRequest{Repository: remoteRepo, RemoteUrl: "file://" + clientRepoPath},
} {
response, err := client.FindRemoteRootRef(ctx, request)
require.Equal(t, status.Error(codes.NotFound, "no remote HEAD found"), err)
require.Nil(t, response)
}
}
func TestFindRemoteRootRefFailedDueToValidation(t *testing.T) {
_, repo, _, client := setupRemoteService(t)
invalidRepo := &gitalypb.Repository{StorageName: "fake", RelativePath: "path"}
testCases := []struct {
desc string
request *gitalypb.FindRemoteRootRefRequest
expectedErr []error
}{
{
desc: "Invalid repository",
request: &gitalypb.FindRemoteRootRefRequest{
Repository: invalidRepo,
Remote: "remote-name",
},
expectedErr: []error{
status.Error(codes.InvalidArgument, "GetStorageByName: no such storage: \"fake\""),
status.Error(codes.InvalidArgument, "repo scoped: invalid Repository"),
},
},
{
desc: "Repository is nil",
request: &gitalypb.FindRemoteRootRefRequest{
Remote: "remote-name",
},
expectedErr: []error{
status.Error(codes.InvalidArgument, "missing repository"),
status.Error(codes.InvalidArgument, "repo scoped: empty Repository"),
},
},
{
desc: "Remote name and URL is empty",
request: &gitalypb.FindRemoteRootRefRequest{
Repository: repo,
},
expectedErr: []error{
status.Error(codes.InvalidArgument, "got neither remote name nor URL"),
},
},
{
desc: "Remote name and URL is set",
request: &gitalypb.FindRemoteRootRefRequest{
Repository: repo,
Remote: "remote-name",
RemoteUrl: "remote-url",
},
expectedErr: []error{
status.Error(codes.InvalidArgument, "got remote name and URL"),
},
},
}
for _, testCase := range testCases {
t.Run(testCase.desc, func(t *testing.T) {
ctx, cancel := testhelper.Context()
defer cancel()
_, err := client.FindRemoteRootRef(ctx, testCase.request)
// We cannot test for equality given that some errors depend on whether we
// proxy via Praefect or not. We thus simply assert that the actual error is
// one of the possible errors, which is the same as equality for all the
// other tests.
require.Contains(t, testCase.expectedErr, err)
})
}
}
func TestFindRemoteRootRefFailedDueToInvalidRemote(t *testing.T) {
_, repo, _, client := setupRemoteService(t)
t.Run("invalid remote name", func(t *testing.T) {
request := &gitalypb.FindRemoteRootRefRequest{Repository: repo, Remote: "invalid"}
ctx, cancel := testhelper.Context()
defer cancel()
_, err := client.FindRemoteRootRef(ctx, request)
testhelper.RequireGrpcError(t, err, codes.Internal)
})
t.Run("invalid remote URL", func(t *testing.T) {
fakeRepoDir := testhelper.TempDir(t)
// We're using a nonexistent filepath remote URL so we avoid hitting the internet.
request := &gitalypb.FindRemoteRootRefRequest{
Repository: repo, RemoteUrl: "file://" + fakeRepoDir,
}
ctx, cancel := testhelper.Context()
defer cancel()
_, err := client.FindRemoteRootRef(ctx, request)
testhelper.RequireGrpcError(t, err, codes.Internal)
})
}
|
