Merge pull request #66 from paultyng/delete-operations

add delete operations for applications and deployments

api/applications.go

@@ -1,6 +1,7 @@
 package api
 
 import (
+	"fmt"
 	"net/url"
 	"strconv"
 )
@@ -57,3 +58,10 @@ func (c *Client) queryApplications(filters applicationsFilters) ([]Application,
 func (c *Client) ListApplications() ([]Application, error) {
 	return c.queryApplications(applicationsFilters{})
 }
+
+// DeleteApplication deletes a non-reporting application from your account.
+func (c *Client) DeleteApplication(id int) error {
+	u := &url.URL{Path: fmt.Sprintf("/applications/%v.json", id)}
+	_, err := c.Do("DELETE", u.String(), nil, nil)
+	return err
+}

api/applications_test.go

@@ -1,6 +1,7 @@
 package api
 
 import (
+	"fmt"
 	"net/http"
 	"testing"
 )
@@ -35,3 +36,39 @@ func TestQueryApplications_Basic(t *testing.T) {
 		t.Fatal("No applications found")
 	}
 }
+
+func TestDeleteApplication(t *testing.T) {
+	c := newTestAPIClient(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Type", "application/json")
+		w.WriteHeader(http.StatusOK)
+		w.Write([]byte(`
+      {
+		"application": {
+			"id": 123,
+			"name": "test",
+			"language": "go",
+			"health_status": "gray",
+			"reporting": false,
+			"settings": {
+				"app_apdex_threshold": 0.5,
+				"end_user_apdex_threshold": 7,
+				"enable_real_user_monitoring": true,
+				"use_server_side_config": false
+			},
+			"links": {
+				"application_instances": [],
+				"servers": [],
+				"application_hosts": []
+			}
+		}
+      }
+    `))
+	}))
+
+	err := c.DeleteApplication(123)
+	if err != nil {
+		fmt.Printf("Error: %s", err)
+		t.Log(err)
+		t.Fatal("DeleteApplication error")
+	}
+}

api/deployments.go

@@ -51,3 +51,10 @@ func (c *Client) CreateDeployment(applicationID int, deployment Deployment) (*De
 
 	return &resp.Deployment, nil
 }
+
+// DeleteDeployment deletes an application deployment from an application.
+func (c *Client) DeleteDeployment(applicationID, deploymentID int) error {
+	u := &url.URL{Path: fmt.Sprintf("/applications/%v/deployments/%v.json", applicationID, deploymentID)}
+	_, err := c.Do("DELETE", u.String(), nil, nil)
+	return err
+}

api/deployments_test.go

@@ -80,3 +80,37 @@ func TestCreateDeployment(t *testing.T) {
 		t.Fatal("Revision was not parsed correctly")
 	}
 }
+
+func TestDeleteDeployment(t *testing.T) {
+	c := newTestAPIClient(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Type", "application/json")
+		w.WriteHeader(http.StatusOK)
+		w.Write([]byte(`
+		{
+			"deployment": {
+				"id": 456,
+				"revision": "master",
+				"changelog": "Foo Bar",
+				"description": "12345678-1234-1234-1234-1234567890ab",
+				"user": "foo",
+				"timestamp": "2019-10-31T15:25:38-07:00",
+				"links": {
+					"application": 123
+				}
+			},
+			"links": {
+				"deployment.agent": "/v2/applications/{application_id}"
+			}
+		}
+      `))
+	}))
+
+	appID := 123
+	depID := 456
+
+	err := c.DeleteDeployment(appID, depID)
+	if err != nil {
+		t.Log(err)
+		t.Fatal("CreateDeployment error")
+	}
+}

cmd/applications.go

@@ -1,6 +1,8 @@
 package cmd
 
 import (
+	"fmt"
+
 	"github.com/imdario/mergo"
 	"github.com/spf13/cobra"
 )
@@ -24,6 +26,7 @@ var getApplicationsCmd = makeApplicationsCmd(cobra.Command{
 		if err != nil {
 			return err
 		}
+
 		resources, err := client.ListApplications()
 		if err != nil {
 			return err
@@ -33,6 +36,32 @@ var getApplicationsCmd = makeApplicationsCmd(cobra.Command{
 	},
 })
 
+var deleteApplicationCmd = makeApplicationsCmd(cobra.Command{
+	RunE: func(cmd *cobra.Command, args []string) error {
+		client, err := newAPIClient(cmd)
+		if err != nil {
+			return err
+		}
+
+		id, err := cmd.Flags().GetInt("id")
+		if err != nil {
+			return err
+		}
+
+		err = client.DeleteApplication(id)
+		if err != nil {
+			return err
+		}
+
+		fmt.Printf("Application '%v' deleted.\n", id)
+
+		return nil
+	},
+})
+
 func init() {
 	getCmd.AddCommand(getApplicationsCmd)
+
+	deleteCmd.AddCommand(deleteApplicationCmd)
+	deleteApplicationCmd.Flags().Int("id", 0, "ID of the application to delete")
 }

cmd/deployments.go

@@ -89,6 +89,34 @@ var createDeploymentCmd = makeDeploymentsCmd(cobra.Command{
 	},
 })
 
+var deleteDeploymentCmd = makeDeploymentsCmd(cobra.Command{
+	RunE: func(cmd *cobra.Command, args []string) error {
+		client, err := newAPIClient(cmd)
+		if err != nil {
+			return err
+		}
+
+		appId, err := cmd.Flags().GetInt("application-id")
+		if err != nil {
+			return err
+		}
+
+		depId, err := cmd.Flags().GetInt("deployment-id")
+		if err != nil {
+			return err
+		}
+
+		err = client.DeleteDeployment(appId, depId)
+		if err != nil {
+			return err
+		}
+
+		fmt.Printf("Application deployment '%v' deleted.\n", depId)
+
+		return nil
+	},
+})
+
 func init() {
 	getCmd.AddCommand(getDeploymentsCmd)
 	getDeploymentsCmd.Flags().IntP("application-id", "a", 0, "application ID of the deployments to get")
@@ -102,4 +130,10 @@ func init() {
 	createDeploymentCmd.Flags().StringP("description", "d", "", "Description of the deployment")
 	createDeploymentCmd.MarkFlagRequired("application-id")
 	createDeploymentCmd.MarkFlagRequired("revision")
+
+	deleteCmd.AddCommand(deleteDeploymentCmd)
+	deleteDeploymentCmd.Flags().IntP("application-id", "a", 0, "application ID of the deployment to delete")
+	deleteDeploymentCmd.Flags().IntP("deployment-id", "d", 0, "deployment ID of the deployment to delete")
+	deleteDeploymentCmd.MarkFlagRequired("application-id")
+	deleteDeploymentCmd.MarkFlagRequired("deployment-id")
 }