mirror of
https://github.com/edgelesssys/constellation.git
synced 2024-12-26 07:59:37 -05:00
c743398a23
Generalize retrier: * Generalize Do to use a supplied 'retriable' function * Make clock an optional argument in NewIntervalRetrier * Move grpc/retrier to interal package * Update existing unittests to not use retry feature Add retryDownloadToTempDir: * Wrap downloadToTempDir with retrier. * Retry if TCP connection is reset. * Abort by canceling the context. * Use a mock server in the unit test that serves responses depending on the state received through a state channel. Co-authored-by: katexochen <49727155+katexochen@users.noreply.github.com>
99 lines
1.7 KiB
Go
99 lines
1.7 KiB
Go
package retry
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
testclock "k8s.io/utils/clock/testing"
|
|
)
|
|
|
|
func TestDo(t *testing.T) {
|
|
testCases := map[string]struct {
|
|
cancel bool
|
|
errors []error
|
|
wantErr error
|
|
}{
|
|
"no error": {
|
|
errors: []error{
|
|
nil,
|
|
},
|
|
},
|
|
"permanent error": {
|
|
errors: []error{
|
|
errors.New("error"),
|
|
},
|
|
wantErr: errors.New("error"),
|
|
},
|
|
"service unavailable then success": {
|
|
errors: []error{
|
|
errors.New("retry me"),
|
|
nil,
|
|
},
|
|
},
|
|
"service unavailable then permanent error": {
|
|
errors: []error{
|
|
errors.New("retry me"),
|
|
errors.New("error"),
|
|
},
|
|
wantErr: errors.New("error"),
|
|
},
|
|
"cancellation works": {
|
|
cancel: true,
|
|
errors: []error{
|
|
errors.New("retry me"),
|
|
},
|
|
wantErr: context.Canceled,
|
|
},
|
|
}
|
|
|
|
for name, tc := range testCases {
|
|
t.Run(name, func(t *testing.T) {
|
|
assert := assert.New(t)
|
|
doer := newStubDoer()
|
|
clock := testclock.NewFakeClock(time.Now())
|
|
retrier := IntervalRetrier{
|
|
doer: doer,
|
|
clock: clock,
|
|
retriable: isRetriable,
|
|
}
|
|
retrierResult := make(chan error, 1)
|
|
ctx, cancel := context.WithCancel(context.Background())
|
|
defer cancel()
|
|
|
|
go func() { retrierResult <- retrier.Do(ctx) }()
|
|
|
|
if tc.cancel {
|
|
cancel()
|
|
}
|
|
|
|
for _, err := range tc.errors {
|
|
doer.errC <- err
|
|
clock.Step(retrier.interval)
|
|
}
|
|
|
|
assert.Equal(tc.wantErr, <-retrierResult)
|
|
})
|
|
}
|
|
}
|
|
|
|
type stubDoer struct {
|
|
errC chan error
|
|
}
|
|
|
|
func newStubDoer() *stubDoer {
|
|
return &stubDoer{
|
|
errC: make(chan error),
|
|
}
|
|
}
|
|
|
|
func (d *stubDoer) Do(_ context.Context) error {
|
|
return <-d.errC
|
|
}
|
|
|
|
func isRetriable(err error) bool {
|
|
return err.Error() == "retry me"
|
|
}
|