Merge pull request #5553 from TimeIncOSS/b-fix-retry-err-bug

helper/resource: Error shouldn't be returned in case of success
hashicorp · Mar 10, 2016 · cadd0c5 · cadd0c5
2 parents 2dc1c02 + 034287f
commit cadd0c5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/helper/resource/wait.go b/helper/resource/wait.go
@@ -21,6 +21,7 @@ func Retry(timeout time.Duration, f RetryFunc) error {
 		Refresh: func() (interface{}, string, error) {
 			rerr := f()
 			if rerr == nil {
+				resultErr = nil
 				return 42, "success", nil
 			}
 

diff --git a/helper/resource/wait_test.go b/helper/resource/wait_test.go
@@ -12,14 +12,14 @@ func TestRetry(t *testing.T) {
 	tries := 0
 	f := func() *RetryError {
 		tries++
-		if tries == 1 {
+		if tries == 3 {
 			return nil
 		}
 
 		return RetryableError(fmt.Errorf("error"))
 	}
 
-	err := Retry(2*time.Second, f)
+	err := Retry(10*time.Second, f)
 	if err != nil {
 		t.Fatalf("err: %s", err)
 	}