Individual failures are not checked on deleteMessageBatch

[AntonioJuliano]

Description

In consumer#deleteMessageBatch when batch deleting messages individual failures are not checked. As per the AWS docs "Because the batch request can result in a combination of successful and unsuccessful actions, you should check for batch errors even when the call returns an HTTP status code of 200"

Expected behaviour

The return value of this.sqs.deleteMessageBatch in the callback specifies a list of failed IDs, which should be validated to be empty. If any IDs fail, they should either be retried or the error should be surfaced

Actual behaviour

Individual failures are silent

Steps to reproduce

Have AWS return a non-empty Failed array

[AntonioJuliano]

Also, the same thing with terminateVisabilityTimeoutBatch

[nspragg]

@AntonioJuliano - thanks for raising this issue.

[nicholasgriffintn]

I believe this will be resolved at the same time as this issue:

#245

Which is to return a list of successful messages, which when compared would also give you a list of failed messages.

Feel free to reopen if that doesn't resolve this issue.