Defer closing bigtable client in tests.

rileykarson · Oct 23, 2018 · f4afb91 · f4afb91
1 parent 7acbfa3
commit f4afb91
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/provider/terraform/tests/resource_bigtable_instance_test.go b/provider/terraform/tests/resource_bigtable_instance_test.go
@@ -86,12 +86,12 @@ func testAccCheckBigtableInstanceDestroy(s *terraform.State) error {
 			return fmt.Errorf("Error starting instance admin client. %s", err)
 		}
 
+		defer c.Close()
+
 		_, err = c.InstanceInfo(ctx, rs.Primary.Attributes["name"])
 		if err == nil {
 			return fmt.Errorf("Instance %s still exists.", rs.Primary.Attributes["name"])
 		}
-
-		c.Close()
 	}
 
 	return nil
@@ -114,13 +114,13 @@ func testAccBigtableInstanceExists(n string) resource.TestCheckFunc {
 			return fmt.Errorf("Error starting instance admin client. %s", err)
 		}
 
+		defer c.Close()
+
 		_, err = c.InstanceInfo(ctx, rs.Primary.Attributes["name"])
 		if err != nil {
 			return fmt.Errorf("Error retrieving instance %s.", rs.Primary.Attributes["name"])
 		}
 
-		c.Close()
-
 		return nil
 	}
 }