luciano/kinesis-consumer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: wrap kinesis get records error in ScanShard instead of printing error message

Browse source 
* wrap get records error using %w fmt code
* add unit test for kinesis get records using errors.Is
This commit is contained in:
Andrey Piterkin 2024-12-09 20:38:16 -05:00 committed by Andrey Piterkin
parent baf8258298
commit e062e02f98
2 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
consumer.go
View file

@ -192,7 +192,7 @@ func (c *Consumer) ScanShard(ctx context.Context, shardID string, fn ScanFunc) e
c.logger.Log("[CONSUMER] get records error:", err.Error())
if !isRetriableError(err) {
return fmt.Errorf("get records error: %v", err.Error())
return fmt.Errorf("get records error: %w", err)
}
shardIterator, err = c.getShardIterator(ctx, c.streamName, shardID, lastSeqNum)

8
consumer_test.go
View file

@ -396,6 +396,7 @@ func TestScanShard_ShardIsClosed_WithShardClosedHandler(t *testing.T) {
}
func TestScanShard_GetRecordsError(t *testing.T) {
getRecordsError := &types.InvalidArgumentException{Message: aws.String("aws error message")}
var client = &kinesisClientMock{
getShardIteratorMock: func(ctx context.Context, params *kinesis.GetShardIteratorInput, optFns ...func(*kinesis.Options)) (*kinesis.GetShardIteratorOutput, error) {
return &kinesis.GetShardIteratorOutput{
@ -406,8 +407,7 @@ func TestScanShard_GetRecordsError(t *testing.T) {
return &kinesis.GetRecordsOutput{
NextShardIterator: nil,
Records: nil,
},
&types.InvalidArgumentException{Message: aws.String("aws error message")}
}, getRecordsError
},
}
@ -424,6 +424,10 @@ func TestScanShard_GetRecordsError(t *testing.T) {
if err.Error() != "get records error: InvalidArgumentException: aws error message" {
t.Fatalf("unexpected error: %v", err)
}
if !errors.Is(err, getRecordsError) {
t.Fatalf("unexpected error: %v", err)
}
}
type kinesisClientMock struct {