Skip to content

Commit b6af4bf

Browse files
authored
Fix dead kinesis firehose tests (#254)
* enable dead firehose event tests * fix previously dead kinesis firehose tests
1 parent d89e42d commit b6af4bf

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

events/firehose.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,5 +39,6 @@ type KinesisFirehoseRecordMetadata struct {
3939
ShardID string `json:"shardId"`
4040
PartitionKey string `json:"partitionKey"`
4141
SequenceNumber string `json:"sequenceNumber"`
42+
SubsequenceNumber string `json:"subsequenceNumber"`
4243
ApproximateArrivalTimestamp MilliSecondsEpochTime `json:"approximateArrivalTimestamp"`
4344
}

events/firehose_test.go

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,20 +11,19 @@ import (
1111
"github.com/stretchr/testify/assert"
1212
)
1313

14-
func testFirehoseEventMarshaling(t *testing.T) {
15-
testMarshaling(t, "./testdata/kinesis-firehose-event.json")
14+
func TestFirehoseEventMarshaling(t *testing.T) {
15+
testMarshaling(t, &KinesisFirehoseEvent{}, "./testdata/kinesis-firehose-event.json")
1616
}
1717

18-
func testFirehoseResponseMarshaling(t *testing.T) {
19-
testMarshaling(t, "./testdata/kinesis-firehose-response.json")
18+
func TestFirehoseResponseMarshaling(t *testing.T) {
19+
testMarshaling(t, &KinesisFirehoseResponse{}, "./testdata/kinesis-firehose-response.json")
2020
}
2121

22-
func testMarshaling(t *testing.T, jsonFile string) {
22+
func testMarshaling(t *testing.T, inputEvent interface{}, jsonFile string) {
2323
// 1. read JSON from file
2424
inputJson := test.ReadJSONFromFile(t, jsonFile)
2525

2626
// 2. de-serialize into Go object
27-
var inputEvent KinesisFirehoseEvent
2827
if err := json.Unmarshal(inputJson, &inputEvent); err != nil {
2928
t.Errorf("could not unmarshal event. details: %v", err)
3029
}

events/testdata/kinesis-firehose-event.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"partitionKey": "4d1ad2b9-24f8-4b9d-a088-76e9947c317a",
1414
"approximateArrivalTimestamp": 1507217624302,
1515
"sequenceNumber": "49546986683135544286507457936321625675700192471156785154",
16-
"subsequenceNumber": ""
16+
"subsequenceNumber": "123456"
1717
}
1818
},
1919
{
@@ -25,7 +25,7 @@
2525
"partitionKey": "4d1ad2b9-24f8-4b9d-a088-76e9947c318a",
2626
"approximateArrivalTimestamp": 1507217624302,
2727
"sequenceNumber": "49546986683135544286507457936321625675700192471156785155",
28-
"subsequenceNumber": ""
28+
"subsequenceNumber": "123457"
2929
}
3030
}
3131
]

0 commit comments

Comments
 (0)