Skip to content

Commit

Permalink
Fix swallowed errors in RS and deployment tests
Browse files Browse the repository at this point in the history
  • Loading branch information
soltysh committed Aug 4, 2017
1 parent 68a7221 commit f08f504
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions test/e2e/apimachinery/garbage_collector.go
Original file line number Diff line number Diff line change
Expand Up @@ -413,8 +413,8 @@ var _ = SIGDescribe("Garbage collector", func() {
return len(rsList.Items) > 0, nil

})
if err == wait.ErrWaitTimeout {
err = fmt.Errorf("Failed to wait for the Deployment to create some ReplicaSet: %v", err)
if err != nil {
framework.Failf("Failed to wait for the Deployment to create some ReplicaSet: %v", err)
}

By("delete the deployment")
Expand Down Expand Up @@ -464,8 +464,8 @@ var _ = SIGDescribe("Garbage collector", func() {
return len(rsList.Items) > 0, nil

})
if err == wait.ErrWaitTimeout {
err = fmt.Errorf("Failed to wait for the Deployment to create some ReplicaSet: %v", err)
if err != nil {
framework.Failf("Failed to wait for the Deployment to create some ReplicaSet: %v", err)
}

By("delete the deployment")
Expand Down

0 comments on commit f08f504

Please sign in to comment.