Merge pull request #1205 from hashicorp/TF-28651_deprecate

TF-28651 deprecate endpoints

Author: sahar-azizighannad

CHANGELOG.md

@@ -3,6 +3,10 @@
 ## Enhancements
 * Exports the StackConfiguration UploadTarGzip receiver function[#1219](https://github.com/hashicorp/go-tfe/pull/1219)
 
+## Deprecations
+
+* The beta `StackDeployments`, `StackPlan`, `StackPlanOperation`, and `StackSource` resources have been removed, by @sahar-azizighannad  [#1205](https://github.com/hashicorp/go-tfe/pull/1205)
+
 # v1.92.0
 
 ## Enhancements

stack.go

@@ -138,21 +138,6 @@ type StackConfiguration struct {
 	Stack *Stack `jsonapi:"relation,stack"`
 }
 
-// StackDeployment represents a stack deployment, specified by configuration
-type StackDeployment struct {
-	// Attributes
-	ID            string    `jsonapi:"primary,stack-deployments"`
-	Name          string    `jsonapi:"attr,name"`
-	Status        string    `jsonapi:"attr,status"`
-	DeployedAt    time.Time `jsonapi:"attr,deployed-at,iso8601"`
-	ErrorsCount   int       `jsonapi:"attr,errors-count"`
-	WarningsCount int       `jsonapi:"attr,warnings-count"`
-	PausedCount   int       `jsonapi:"attr,paused-count"`
-
-	// Relationships
-	CurrentStackState *StackState `jsonapi:"relation,current-stack-state"`
-}
-
 // StackState represents a stack state
 type StackState struct {
 	// Attributes
@@ -307,7 +292,7 @@ func (s stacks) Update(ctx context.Context, stackID string, options StackUpdateO
 
 // Delete deletes a stack.
 func (s stacks) Delete(ctx context.Context, stackID string) error {
-	req, err := s.client.NewRequest("POST", fmt.Sprintf("stacks/%s/delete", url.PathEscape(stackID)), nil)
+	req, err := s.client.NewRequest("DELETE", fmt.Sprintf("stacks/%s", url.PathEscape(stackID)), nil)
 	if err != nil {
 		return err
 	}
@@ -317,7 +302,7 @@ func (s stacks) Delete(ctx context.Context, stackID string) error {
 
 // ForceDelete deletes a stack that still has deployments.
 func (s stacks) ForceDelete(ctx context.Context, stackID string) error {
-	req, err := s.client.NewRequest("POST", fmt.Sprintf("stacks/%s/force-delete", url.PathEscape(stackID)), nil)
+	req, err := s.client.NewRequest("DELETE", fmt.Sprintf("stacks/%s?force=true", url.PathEscape(stackID)), nil)
 	if err != nil {
 		return err
 	}

stack_deployments.go

@@ -353,37 +353,6 @@ func pollStackDeployments(t *testing.T, ctx context.Context, client *Client, sta
 	return
 }
 
-func pollStackDeploymentStatus(t *testing.T, ctx context.Context, client *Client, stackID, deploymentName, status string) {
-	// pollStackDeployments will poll the given stack until it has deployments or the deadline is reached.
-	ctx, cancel := context.WithDeadline(ctx, time.Now().Add(5*time.Minute))
-	defer cancel()
-
-	deadline, _ := ctx.Deadline()
-	t.Logf("Polling stack %q for deployments with deadline of %s", stackID, deadline)
-
-	ticker := time.NewTicker(2 * time.Second)
-	defer ticker.Stop()
-
-	for finished := false; !finished; {
-		t.Log("...")
-		select {
-		case <-ctx.Done():
-			t.Fatalf("Stack deployment %s/%s did not have status %q at deadline", stackID, deploymentName, status)
-		case <-ticker.C:
-			var err error
-			deployment, err := client.StackDeployments.Read(ctx, stackID, deploymentName)
-			if err != nil {
-				t.Fatalf("Failed to read stack deployment %s/%s: %s", stackID, deploymentName, err)
-			}
-
-			t.Logf("Stack deployment %s/%s had status %q", stackID, deploymentName, deployment.Status)
-			if deployment.Status == status {
-				finished = true
-			}
-		}
-	}
-}
-
 func pollStackConfigurationStatus(t *testing.T, ctx context.Context, client *Client, stackConfigID, status string) (stackConfig *StackConfiguration) {
 	// pollStackDeployments will poll the given stack until it has deployments or the deadline is reached.
 	ctx, cancel := context.WithDeadline(ctx, time.Now().Add(5*time.Minute))
@@ -453,9 +422,5 @@ func TestStackConverged(t *testing.T) {
 	require.ElementsMatch(t, deployments, stack.DeploymentNames)
 	require.NotNil(t, stack.LatestStackConfiguration)
 
-	for _, deployment := range deployments {
-		pollStackDeploymentStatus(t, ctx, client, stack.ID, deployment, "paused")
-	}
-
 	pollStackConfigurationStatus(t, ctx, client, stack.LatestStackConfiguration.ID, "converged")
 }