Skip to content

Commit 5a29128

Browse files
committed
update AWS tests.
1 parent a81d88f commit 5a29128

File tree

2 files changed

+31
-24
lines changed

2 files changed

+31
-24
lines changed

cloud/aws/runtime/gateway/lambda_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -271,8 +271,8 @@ var _ = Describe("Lambda", func() {
271271
"test": "test",
272272
})
273273

274-
message := ep.Message{
275-
Content: &ep.Message_StructPayload{
274+
message := ep.TopicMessage{
275+
Content: &ep.TopicMessage_StructPayload{
276276
StructPayload: content,
277277
},
278278
}

cloud/aws/runtime/queue/sqs_test.go

+29-22
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ import (
3535
mocks_sqs "github.com/nitrictech/nitric/cloud/aws/mocks/sqs"
3636
"github.com/nitrictech/nitric/cloud/aws/runtime/resource"
3737
queuepb "github.com/nitrictech/nitric/core/pkg/proto/queues/v1"
38+
queuespb "github.com/nitrictech/nitric/core/pkg/proto/queues/v1"
3839
)
3940

4041
var _ = Describe("Sqs", func() {
@@ -114,13 +115,15 @@ var _ = Describe("Sqs", func() {
114115
By("Calling SendMessageBatch with the expected batch entries")
115116
sqsMock.EXPECT().SendMessageBatch(gomock.Any(), gomock.Any()).Return(&sqs.SendMessageBatchOutput{}, nil)
116117

117-
_, err := plugin.Send(context.TODO(), &queuepb.QueueSendRequestBatch{
118+
_, err := plugin.Enqueue(context.TODO(), &queuepb.QueueEnqueueRequest{
118119
QueueName: "test-queue",
119-
Requests: []*queuepb.QueueSendRequest{
120+
Messages: []*queuespb.QueueMessage{
120121
{
121-
Payload: &structpb.Struct{
122-
Fields: map[string]*structpb.Value{
123-
"Test": structpb.NewStringValue("Test"),
122+
Content: &queuespb.QueueMessage_StructPayload{
123+
StructPayload: &structpb.Struct{
124+
Fields: map[string]*structpb.Value{
125+
"Test": structpb.NewStringValue("Test"),
126+
},
124127
},
125128
},
126129
},
@@ -162,13 +165,15 @@ var _ = Describe("Sqs", func() {
162165
By("Calling SendMessageBatch with the expected batch entries")
163166
sqsMock.EXPECT().SendMessageBatch(gomock.Any(), gomock.Any()).Return(nil, opErr)
164167

165-
_, err := plugin.Send(context.TODO(), &queuepb.QueueSendRequestBatch{
168+
_, err := plugin.Enqueue(context.TODO(), &queuepb.QueueEnqueueRequest{
166169
QueueName: "test-queue",
167-
Requests: []*queuepb.QueueSendRequest{
170+
Messages: []*queuepb.QueueMessage{
168171
{
169-
Payload: &structpb.Struct{
170-
Fields: map[string]*structpb.Value{
171-
"Test": structpb.NewStringValue("Test"),
172+
Content: &queuepb.QueueMessage_StructPayload{
173+
StructPayload: &structpb.Struct{
174+
Fields: map[string]*structpb.Value{
175+
"Test": structpb.NewStringValue("Test"),
176+
},
172177
},
173178
},
174179
},
@@ -193,13 +198,15 @@ var _ = Describe("Sqs", func() {
193198
By("provider GetResources returning an error")
194199
providerMock.EXPECT().GetResources(gomock.Any(), resource.AwsResource_Queue).Return(nil, fmt.Errorf("mock-error"))
195200

196-
_, err := plugin.Send(context.TODO(), &queuepb.QueueSendRequestBatch{
201+
_, err := plugin.Enqueue(context.TODO(), &queuepb.QueueEnqueueRequest{
197202
QueueName: "test-queue",
198-
Requests: []*queuepb.QueueSendRequest{
203+
Messages: []*queuepb.QueueMessage{
199204
{
200-
Payload: &structpb.Struct{
201-
Fields: map[string]*structpb.Value{
202-
"Test": structpb.NewStringValue("Test"),
205+
Content: &queuepb.QueueMessage_StructPayload{
206+
StructPayload: &structpb.Struct{
207+
Fields: map[string]*structpb.Value{
208+
"Test": structpb.NewStringValue("Test"),
209+
},
203210
},
204211
},
205212
},
@@ -256,15 +263,15 @@ var _ = Describe("Sqs", func() {
256263
}, nil)
257264

258265
By("Returning the task")
259-
response, err := plugin.Receive(context.TODO(), &queuepb.QueueReceiveRequest{
266+
response, err := plugin.Dequeue(context.TODO(), &queuepb.QueueDequeueRequest{
260267
QueueName: "mock-queue",
261268
Depth: 10,
262269
})
263270

264271
Expect(err).ShouldNot(HaveOccurred())
265-
Expect(response.Tasks).To(HaveLen(1))
266-
Expect(response.Tasks[0].LeaseId).To(BeEquivalentTo("mockreceipthandle"))
267-
Expect(response.Tasks[0].Payload.AsMap()).To(BeEquivalentTo(testStruct.AsMap()))
272+
Expect(response.Messages).To(HaveLen(1))
273+
Expect(response.Messages[0].LeaseId).To(BeEquivalentTo("mockreceipthandle"))
274+
Expect(response.Messages[0].Message.GetStructPayload().AsMap()).To(BeEquivalentTo(testStruct.AsMap()))
268275

269276
ctrl.Finish()
270277
})
@@ -302,13 +309,13 @@ var _ = Describe("Sqs", func() {
302309
Messages: []types.Message{},
303310
}, nil)
304311

305-
response, err := plugin.Receive(context.TODO(), &queuepb.QueueReceiveRequest{
312+
response, err := plugin.Dequeue(context.TODO(), &queuepb.QueueDequeueRequest{
306313
QueueName: "mock-queue",
307314
Depth: 10,
308315
})
309316

310317
By("Returning an empty array of tasks")
311-
Expect(response.Tasks).To(HaveLen(0))
318+
Expect(response.Messages).To(HaveLen(0))
312319

313320
By("Not returning an error")
314321
Expect(err).ShouldNot(HaveOccurred())
@@ -352,7 +359,7 @@ var _ = Describe("Sqs", func() {
352359
QueueUrl: queueUrl,
353360
}).Times(1).Return(nil, opErr)
354361

355-
_, err := plugin.Receive(context.TODO(), &queuepb.QueueReceiveRequest{
362+
_, err := plugin.Dequeue(context.TODO(), &queuepb.QueueDequeueRequest{
356363
QueueName: "mock-queue",
357364
Depth: 10,
358365
})

0 commit comments

Comments
 (0)