Rename variable from ID -> UUID

buildkite · Aug 28, 2024 · debec2a · debec2a
1 parent a27c4b4
commit debec2a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/api/builds.go b/api/builds.go
@@ -6,12 +6,12 @@ import (
 )
 
 type Build struct {
-	ID string `json:"id"`
+	UUID string `json:"uuid"`
 }
 
-// CancelBuild cancels a build with the given ID
-func (c *Client) CancelBuild(ctx context.Context, id string) (*Build, *Response, error) {
-	u := fmt.Sprintf("builds/%s/cancel", railsPathEscape(id))
+// CancelBuild cancels a build with the given UUID
+func (c *Client) CancelBuild(ctx context.Context, uuid string) (*Build, *Response, error) {
+	u := fmt.Sprintf("builds/%s/cancel", railsPathEscape(uuid))
 
 	req, err := c.newRequest(ctx, "POST", u, nil)
 	if err != nil {

diff --git a/clicommand/build_cancel.go b/clicommand/build_cancel.go
@@ -104,7 +104,7 @@ func cancelBuild(ctx context.Context, cfg BuildCancelConfig, l logger.Logger) er
 			return err
 		}
 
-		l.Info("Successfully cancelled build %s", build.ID)
+		l.Info("Successfully cancelled build %s", build.UUID)
 		return nil
 	}); err != nil {
 		return fmt.Errorf("failed to cancel build: %w", err)

diff --git a/clicommand/build_cancel_test.go b/clicommand/build_cancel_test.go
@@ -18,7 +18,7 @@ func TestBuildCancel(t *testing.T) {
 	t.Run("success", func(t *testing.T) {
 		server := httptest.NewServer(http.HandlerFunc(func(rw http.ResponseWriter, req *http.Request) {
 			rw.WriteHeader(http.StatusOK)
-			rw.Write([]byte(`{"status": "canceled", "id": "1"}`))
+			rw.Write([]byte(`{"status": "canceled", "uuid": "1"}`))
 		}))
 
 		cfg := BuildCancelConfig{