test: added more test cases for DownloadCopilotMetrics and skip the method in metadata

Author: MasterOogway1466

github/copilot_test.go

@@ -2827,10 +2827,13 @@ func TestCopilotService_DownloadCopilotMetrics(t *testing.T) {
 
 	ctx := t.Context()
 	url := client.BaseURL.String() + "path/to/download"
-	got, _, err := client.Copilot.DownloadCopilotMetrics(ctx, url)
+	got, resp, err := client.Copilot.DownloadCopilotMetrics(ctx, url)
 	if err != nil {
 		t.Errorf("Copilot.DownloadCopilotMetrics returned error: %v", err)
 	}
+	if resp.StatusCode != http.StatusOK {
+		t.Errorf("Copilot.DownloadCopilotMetrics returned status code: %v", resp.StatusCode)
+	}
 
 	want := []*CopilotMetrics{
 		{
@@ -2885,6 +2888,29 @@ func TestCopilotService_DownloadCopilotMetrics(t *testing.T) {
 	urlErr := client.BaseURL.String() + "path/to/download/error"
 	_, _, err = client.Copilot.DownloadCopilotMetrics(ctx, urlErr)
 	if err == nil {
-		t.Errorf("Copilot.DownloadCopilotMetrics expected error but got none")
+		t.Error("Copilot.DownloadCopilotMetrics expected error but got none")
+	}
+
+	// Test invalid URL (fails http.NewRequestWithContext)
+	_, _, err = client.Copilot.DownloadCopilotMetrics(ctx, "\n")
+	if err == nil {
+		t.Error("Copilot.DownloadCopilotMetrics expected error for invalid URL, got none")
+	}
+
+	// Test invalid scheme (fails client.Do)
+	_, _, err = client.Copilot.DownloadCopilotMetrics(ctx, "invalid-scheme://test")
+	if err == nil {
+		t.Error("Copilot.DownloadCopilotMetrics expected error for invalid scheme, got none")
+	}
+
+	// Test json decoding error
+	mux.HandleFunc("/path/to/download/badjson", func(w http.ResponseWriter, r *http.Request) {
+		testMethod(t, r, "GET")
+		fmt.Fprint(w, `[{invalid JSON`)
+	})
+	urlBadJson := client.BaseURL.String() + "path/to/download/badjson"
+	_, _, err = client.Copilot.DownloadCopilotMetrics(ctx, urlBadJson)
+	if err == nil {
+		t.Error("Copilot.DownloadCopilotMetrics expected error for bad JSON, got none")
 	}
 }

tools/metadata/metadata.go

@@ -547,4 +547,5 @@ var skipServiceMethod = map[string]bool{
 	"BillingService.GetOrganizationStorageBilling":  true,
 	"BillingService.GetPackagesBilling":             true,
 	"BillingService.GetStorageBilling":              true,
+	"CopilotService.DownloadCopilotMetrics":         true,
 }