From 983b34291d048069c90dd6f24b32a34894f9bbce Mon Sep 17 00:00:00 2001 From: Paul Hinze Date: Mon, 4 Jan 2016 16:29:31 -0600 Subject: [PATCH] provider/google: fix InstanceGroupManager CheckDestroy in tests Nil check was just backwards. Vetted by comparing to other tests with similar CheckDestroy implementations --- .../google/resource_compute_instance_group_manager_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/providers/google/resource_compute_instance_group_manager_test.go b/builtin/providers/google/resource_compute_instance_group_manager_test.go index 4d5bd7c13..5bdb11651 100644 --- a/builtin/providers/google/resource_compute_instance_group_manager_test.go +++ b/builtin/providers/google/resource_compute_instance_group_manager_test.go @@ -69,7 +69,7 @@ func testAccCheckInstanceGroupManagerDestroy(s *terraform.State) error { } _, err := config.clientCompute.InstanceGroupManagers.Get( config.Project, rs.Primary.Attributes["zone"], rs.Primary.ID).Do() - if err != nil { + if err == nil { return fmt.Errorf("InstanceGroupManager still exists") } }