From 3da04ef9fc397fff274c239a700be4a12e7b7f47 Mon Sep 17 00:00:00 2001 From: Kristin Laemmert Date: Fri, 28 Sep 2018 16:35:34 -0700 Subject: [PATCH] backend/local: adding some informative comments to commented-out tests --- backend/local/backend_apply.go | 1 - backend/local/backend_plan_test.go | 4 ++++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/backend/local/backend_apply.go b/backend/local/backend_apply.go index c0465aee1..1b479c752 100644 --- a/backend/local/backend_apply.go +++ b/backend/local/backend_apply.go @@ -84,7 +84,6 @@ func (b *Local) opApply( } trivialPlan := plan.Changes.Empty() - hasUI := op.UIOut != nil && op.UIIn != nil mustConfirm := hasUI && ((op.Destroy && (!op.DestroyForce && !op.AutoApprove)) || (!op.Destroy && !op.AutoApprove && !trivialPlan)) if mustConfirm { diff --git a/backend/local/backend_plan_test.go b/backend/local/backend_plan_test.go index 219b680ae..9c454718a 100644 --- a/backend/local/backend_plan_test.go +++ b/backend/local/backend_plan_test.go @@ -189,6 +189,8 @@ func TestLocal_planDestroy(t *testing.T) { } plan := testReadPlan(t, planPath) + // This statement can be removed when the test is fixed and replaced with the + // commented-out test below. if plan == nil { t.Fatalf("plan is nil") } @@ -223,6 +225,8 @@ func TestLocal_planOutPathNoChange(t *testing.T) { } plan := testReadPlan(t, planPath) + // This statement can be removed when the test is fixed and replaced with the + // commented-out test below. if plan == nil { t.Fatalf("plan is nil") }