fix #125: remove strategy.Infinite

strategy/strategy.go

@@ -18,13 +18,6 @@ import (
 // made. This allows for a pre-action delay, etc.
 type Strategy func(attempt uint, err error) bool
 
-// Infinite creates a Strategy that will never stop repeating.
-func Infinite() Strategy {
-	return func(attempt uint, _ error) bool {
-		return true
-	}
-}
-
 // Limit creates a Strategy that limits the number of attempts that Retry will
 // make.
 func Limit(attemptLimit uint) Strategy {

strategy/strategy_test.go

@@ -1,7 +1,6 @@
 package strategy_test
 
 import (
-	"math"
 	"testing"
 	"time"
 
@@ -12,18 +11,6 @@ import (
 // a time-based (sleep) unit before considering invalid.
 const timeMarginOfError = time.Millisecond
 
-func TestInfinite(t *testing.T) {
-	policy := Infinite()
-
-	if !policy(0, nil) {
-		t.Error("strategy expected to return true")
-	}
-
-	if !policy(math.MaxUint32, nil) {
-		t.Error("strategy expected to return true")
-	}
-}
-
 func TestLimit(t *testing.T) {
 	const attemptLimit = 3