Skip to content

Commit d30e64a

Browse files
test: Update upload_test.go to match latest implementation
Co-Authored-By: [email protected] <[email protected]>
1 parent 40016eb commit d30e64a

File tree

1 file changed

+1
-77
lines changed

1 file changed

+1
-77
lines changed

cmd/sourcemaps/upload_test.go

Lines changed: 1 addition & 77 deletions
Original file line numberDiff line numberDiff line change
@@ -37,56 +37,6 @@ func (m *mockResourcesClient) GetVersion() string {
3737

3838

3939

40-
func TestVerifyApiKey(t *testing.T) {
41-
server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
42-
assert.Equal(t, http.MethodPost, r.Method)
43-
assert.Equal(t, "application/json", r.Header.Get("Content-Type"))
44-
45-
var requestBody map[string]interface{}
46-
err := json.NewDecoder(r.Body).Decode(&requestBody)
47-
assert.NoError(t, err)
48-
49-
variables, ok := requestBody["variables"].(map[string]interface{})
50-
assert.True(t, ok)
51-
assert.Equal(t, "account123", variables["ld_account_id"])
52-
assert.Equal(t, "project123", variables["ld_project_id"])
53-
54-
response := `{"data":{"ld_credential":{"project_id":"project123","api_key":"highlight-key-123"}}}`
55-
w.Header().Set("Content-Type", "application/json")
56-
w.WriteHeader(http.StatusOK)
57-
_, _ = w.Write([]byte(response))
58-
}))
59-
defer server.Close()
60-
61-
highlightKey, projectID, err := verifyApiKey("account123", "project123", server.URL)
62-
assert.NoError(t, err)
63-
assert.Equal(t, "highlight-key-123", highlightKey)
64-
assert.Equal(t, "project123", projectID)
65-
66-
invalidServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
67-
response := `{"data":{"ld_credential":{"project_id":"","api_key":""}}}`
68-
w.Header().Set("Content-Type", "application/json")
69-
w.WriteHeader(http.StatusOK)
70-
_, _ = w.Write([]byte(response))
71-
}))
72-
defer invalidServer.Close()
73-
74-
_, _, err = verifyApiKey("account123", "project123", invalidServer.URL)
75-
assert.Error(t, err)
76-
assert.Contains(t, err.Error(), "invalid API key")
77-
78-
errorServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
79-
response := `{"errors":[{"message":"Invalid credentials"}]}`
80-
w.Header().Set("Content-Type", "application/json")
81-
w.WriteHeader(http.StatusOK)
82-
_, _ = w.Write([]byte(response))
83-
}))
84-
defer errorServer.Close()
85-
86-
_, _, err = verifyApiKey("account123", "project123", errorServer.URL)
87-
assert.Error(t, err)
88-
assert.Contains(t, err.Error(), "failed to verify API key")
89-
}
9040

9141
func TestGetSourceMapUploadUrls(t *testing.T) {
9242
server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
@@ -249,23 +199,6 @@ func TestGetAllSourceMapFiles(t *testing.T) {
249199
assert.Contains(t, err.Error(), "no .js.map files found")
250200
}
251201

252-
func TestVerifyApiKeyErrors(t *testing.T) {
253-
_, _, err := verifyApiKey("account123", "project123", "://invalid-url")
254-
assert.Error(t, err)
255-
256-
_, _, err = verifyApiKey("account123", "project123", "http://non-existent-host.invalid")
257-
assert.Error(t, err)
258-
259-
invalidJSONServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
260-
w.Header().Set("Content-Type", "application/json")
261-
w.WriteHeader(http.StatusOK)
262-
_, _ = w.Write([]byte(`{"data":invalid-json`))
263-
}))
264-
defer invalidJSONServer.Close()
265-
266-
_, _, err = verifyApiKey("account123", "project123", invalidJSONServer.URL)
267-
assert.Error(t, err)
268-
}
269202

270203
func TestGetSourceMapUploadUrlsErrors(t *testing.T) {
271204
_, err := getSourceMapUploadUrls("test-key", []string{"path"}, "://invalid-url")
@@ -289,7 +222,6 @@ func TestRunE(t *testing.T) {
289222
// Create a mock client that returns predefined responses
290223
mockClient := &mockResourcesClient{
291224
responses: map[string][]byte{
292-
"/api/v2/caller-identity": []byte(`{"AccountID":"account123"}`),
293225
"/api/v2/projects/test-project": []byte(`{"_id":"project123"}`),
294226
},
295227
}
@@ -305,14 +237,6 @@ func TestRunE(t *testing.T) {
305237
err = os.WriteFile(testMapFile, []byte("{}"), 0644)
306238
assert.NoError(t, err)
307239

308-
verifyServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
309-
response := `{"data":{"ld_credential":{"project_id":"project123","api_key":"highlight-key-123"}}}`
310-
w.Header().Set("Content-Type", "application/json")
311-
w.WriteHeader(http.StatusOK)
312-
_, _ = w.Write([]byte(response))
313-
}))
314-
defer verifyServer.Close()
315-
316240
urlsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
317241
response := `{"data":{"get_source_map_upload_urls":["https://example.com/upload"]}}`
318242
w.Header().Set("Content-Type", "application/json")
@@ -334,7 +258,7 @@ func TestRunE(t *testing.T) {
334258
assert.NoError(t, err)
335259
err = cmd.Flags().Set(pathFlag, testMapFile)
336260
assert.NoError(t, err)
337-
err = cmd.Flags().Set(backendUrlFlag, verifyServer.URL)
261+
err = cmd.Flags().Set(backendUrlFlag, urlsServer.URL)
338262
assert.NoError(t, err)
339263

340264
err = runFunc(cmd, args)

0 commit comments

Comments
 (0)