From 6cf4835714f9c6f17b504924a99bdf576bceaae7 Mon Sep 17 00:00:00 2001
From: Martin Atkins <mart@degeneration.co.uk>
Date: Thu, 24 May 2018 18:20:02 -0700
Subject: [PATCH] core: Fix output from context input tests

The previous test output made it hard to understand what was going on when
a test failed. We'll now produce more verbose output.
---
 terraform/context_input_test.go | 126 ++++++++++++++++----------------
 1 file changed, 63 insertions(+), 63 deletions(-)

diff --git a/terraform/context_input_test.go b/terraform/context_input_test.go
index b1bb9c0d6..54f3f6b12 100644
--- a/terraform/context_input_test.go
+++ b/terraform/context_input_test.go
@@ -76,8 +76,8 @@ func TestContext2Input_moduleComputedOutputElement(t *testing.T) {
 		return c, nil
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 }
 
@@ -100,8 +100,8 @@ func TestContext2Input_badVarDefault(t *testing.T) {
 		return c, nil
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 }
 
@@ -132,20 +132,20 @@ func TestContext2Input_provider(t *testing.T) {
 		return nil, c.CheckSet([]string{"foo"})
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Apply(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Apply(); diags.HasErrors() {
+		t.Fatalf("apply errors: %s", diags.Err())
 	}
 
 	if !reflect.DeepEqual(actual, "bar") {
-		t.Fatalf("bad: %#v", actual)
+		t.Fatalf("wrong result\ngot:  %#v\nwant: %#v", actual, "bar")
 	}
 }
 
@@ -179,21 +179,21 @@ func TestContext2Input_providerMulti(t *testing.T) {
 		return nil, c.CheckSet([]string{"foo"})
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Apply(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Apply(); diags.HasErrors() {
+		t.Fatalf("apply errors: %s", diags.Err())
 	}
 
 	expected := []interface{}{"bar", "bar"}
 	if !reflect.DeepEqual(actual, expected) {
-		t.Fatalf("bad: %#v", actual)
+		t.Fatalf("wrong result\ngot:  %#v\nwant: %#v", actual, expected)
 	}
 }
 
@@ -230,8 +230,8 @@ func TestContext2Input_providerOnce(t *testing.T) {
 		return c, nil
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 }
 
@@ -270,20 +270,20 @@ func TestContext2Input_providerId(t *testing.T) {
 		"provider.aws.foo": "bar",
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Apply(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Apply(); diags.HasErrors() {
+		t.Fatalf("apply errors: %s", diags.Err())
 	}
 
 	if !reflect.DeepEqual(actual, "bar") {
-		t.Fatalf("bad: %#v", actual)
+		t.Fatalf("wrong result\ngot:  %#v\nwant: %#v", actual, "bar")
 	}
 }
 
@@ -327,8 +327,8 @@ func TestContext2Input_providerOnly(t *testing.T) {
 		t.Fatalf("err: %s", err)
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
 	state, err := ctx.Apply()
@@ -337,13 +337,13 @@ func TestContext2Input_providerOnly(t *testing.T) {
 	}
 
 	if !reflect.DeepEqual(actual, "bar") {
-		t.Fatalf("bad: %#v", actual)
+		t.Fatalf("wrong result\ngot:  %#v\nwant: %#v", actual, "bar")
 	}
 
 	actualStr := strings.TrimSpace(state.String())
 	expectedStr := strings.TrimSpace(testTerraformInputProviderOnlyStr)
 	if actualStr != expectedStr {
-		t.Fatalf("bad: \n%s", actualStr)
+		t.Fatalf("wrong result\n\ngot:\n%s\n\nwant:\n%s", actualStr, expectedStr)
 	}
 }
 
@@ -383,16 +383,16 @@ func TestContext2Input_providerVars(t *testing.T) {
 		return nil
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Apply(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Apply(); diags.HasErrors() {
+		t.Fatalf("apply errors: %s", diags.Err())
 	}
 
 	if !reflect.DeepEqual(actual, "bar") {
@@ -426,8 +426,8 @@ func TestContext2Input_providerVarsModuleInherit(t *testing.T) {
 		return nil
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 }
 
@@ -471,8 +471,8 @@ func TestContext2Input_varOnly(t *testing.T) {
 		t.Fatalf("err: %s", err)
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
 	state, err := ctx.Apply()
@@ -487,7 +487,7 @@ func TestContext2Input_varOnly(t *testing.T) {
 	actualStr := strings.TrimSpace(state.String())
 	expectedStr := strings.TrimSpace(testTerraformInputVarOnlyStr)
 	if actualStr != expectedStr {
-		t.Fatalf("bad: \n%s", actualStr)
+		t.Fatalf("wrong result\n\ngot:\n%s\n\nwant:\n%s", actualStr, expectedStr)
 	}
 }
 
@@ -522,8 +522,8 @@ func TestContext2Input_varOnlyUnset(t *testing.T) {
 		t.Fatalf("err: %s", err)
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
 	state, err := ctx.Apply()
@@ -534,7 +534,7 @@ func TestContext2Input_varOnlyUnset(t *testing.T) {
 	actualStr := strings.TrimSpace(state.String())
 	expectedStr := strings.TrimSpace(testTerraformInputVarOnlyUnsetStr)
 	if actualStr != expectedStr {
-		t.Fatalf("bad: \n%s", actualStr)
+		t.Fatalf("wrong result\n\ngot:\n%s\n\nwant:\n%s", actualStr, expectedStr)
 	}
 }
 
@@ -565,8 +565,8 @@ func TestContext2Input_varWithDefault(t *testing.T) {
 		t.Fatalf("err: %s", err)
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
 	state, err := ctx.Apply()
@@ -642,12 +642,12 @@ func TestContext2Input_varPartiallyComputed(t *testing.T) {
 		},
 	})
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 }
 
@@ -671,8 +671,8 @@ func TestContext2Input_interpolateVar(t *testing.T) {
 		UIInput: input,
 	})
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 }
 
@@ -702,8 +702,8 @@ func TestContext2Input_hcl(t *testing.T) {
 		t.Fatalf("err: %s", err)
 	}
 
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 
 	state, err := ctx.Apply()
@@ -743,8 +743,8 @@ func TestContext2Input_submoduleTriggersInvalidCount(t *testing.T) {
 		return nil
 	}
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 }
 
@@ -789,15 +789,15 @@ func TestContext2Input_dataSourceRequiresRefresh(t *testing.T) {
 		UIInput: input,
 	})
 
-	if err := ctx.Input(InputModeStd); err != nil {
-		t.Fatalf("err: %s", err)
+	if diags := ctx.Input(InputModeStd); diags.HasErrors() {
+		t.Fatalf("input errors: %s", diags.Err())
 	}
 
 	// ensure that plan works after Refresh
-	if _, err := ctx.Refresh(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Refresh(); diags.HasErrors() {
+		t.Fatalf("refresh errors: %s", diags.Err())
 	}
-	if _, err := ctx.Plan(); err != nil {
-		t.Fatalf("err: %s", err)
+	if _, diags := ctx.Plan(); diags.HasErrors() {
+		t.Fatalf("plan errors: %s", diags.Err())
 	}
 }