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
|
package authorizedkeys
import (
"context"
"encoding/json"
"net/http"
"testing"
"github.com/stretchr/testify/require"
gitnet "gitlab.com/gitlab-org/gitaly/v14/gitlabnet"
"gitlab.com/gitlab-org/gitlab-shell/client/testserver"
"gitlab.com/gitlab-org/gitlab-shell/internal/config"
)
var (
requests []testserver.TestRequestHandler
)
func init() {
requests = []testserver.TestRequestHandler{
{
Path: "/api/v4/internal/authorized_keys",
Handler: func(w http.ResponseWriter, r *http.Request) {
if r.URL.Query().Get("key") == "key" {
body := &Response{
Id: 1,
Key: "public-key",
}
json.NewEncoder(w).Encode(body)
} else if r.URL.Query().Get("key") == "broken-message" {
w.WriteHeader(http.StatusForbidden)
body := &gitnet.ErrorResponse{
Message: "Not allowed!",
}
json.NewEncoder(w).Encode(body)
} else if r.URL.Query().Get("key") == "broken-json" {
w.Write([]byte("{ \"message\": \"broken json!\""))
} else if r.URL.Query().Get("key") == "broken-empty" {
w.WriteHeader(http.StatusForbidden)
} else {
w.WriteHeader(http.StatusNotFound)
}
},
},
}
}
func TestGetByKey(t *testing.T) {
client := setup(t)
result, err := client.GetByKey(context.Background(), "key")
require.NoError(t, err)
require.Equal(t, &Response{Id: 1, Key: "public-key"}, result)
}
func TestGetByKeyErrorResponses(t *testing.T) {
client := setup(t)
testCases := []struct {
desc string
key string
expectedError string
}{
{
desc: "A response with an error message",
key: "broken-message",
expectedError: "Not allowed!",
},
{
desc: "A response with bad JSON",
key: "broken-json",
expectedError: "Parsing failed",
},
{
desc: "A forbidden (403) response without message",
key: "broken-empty",
expectedError: "Internal API error (403)",
},
{
desc: "A not found (404) response without message",
key: "not-found",
expectedError: "Internal API error (404)",
},
}
for _, tc := range testCases {
t.Run(tc.desc, func(t *testing.T) {
resp, err := client.GetByKey(context.Background(), tc.key)
require.EqualError(t, err, tc.expectedError)
require.Nil(t, resp)
})
}
}
func setup(t *testing.T) *Client {
url := testserver.StartSocketHttpServer(t, requests)
client, err := NewClient(&config.Config{GitlabUrl: url})
require.NoError(t, err)
return client
}
|
