From e2fc0b518fd8728ec822afb6cbd15b184aa10e28 Mon Sep 17 00:00:00 2001 From: Mitchell Hashimoto Date: Sat, 8 Oct 2016 16:58:00 +0800 Subject: [PATCH] terraform: ShadowError returns errors, not the close operation --- terraform/shadow_resource_provider_test.go | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/terraform/shadow_resource_provider_test.go b/terraform/shadow_resource_provider_test.go index 1b04184de..be2fba0ea 100644 --- a/terraform/shadow_resource_provider_test.go +++ b/terraform/shadow_resource_provider_test.go @@ -114,8 +114,11 @@ func TestShadowResourceProviderInput_badInput(t *testing.T) { } // Verify we have an error - if err := shadow.CloseShadow(); err == nil { - t.Fatal("should have error") + if err := shadow.CloseShadow(); err != nil { + t.Fatalf("bad: %s", err) + } + if err := shadow.ShadowError(); err == nil { + t.Fatal("should error") } } @@ -194,8 +197,11 @@ func TestShadowResourceProviderValidate_badInput(t *testing.T) { shadow.Validate(configBad) // Verify we have an error - if err := shadow.CloseShadow(); err == nil { - t.Fatal("should have error") + if err := shadow.CloseShadow(); err != nil { + t.Fatalf("bad: %s", err) + } + if err := shadow.ShadowError(); err == nil { + t.Fatal("should error") } } @@ -265,8 +271,11 @@ func TestShadowResourceProviderConfigure_badInput(t *testing.T) { shadow.Configure(configBad) // Verify we have an error - if err := shadow.CloseShadow(); err == nil { - t.Fatal("should have error") + if err := shadow.CloseShadow(); err != nil { + t.Fatalf("bad: %s", err) + } + if err := shadow.ShadowError(); err == nil { + t.Fatal("should error") } }