fix: SQS receive message middleware properly calls next() once

[alexforsyth]

Issue #, if available:
#1394
SQS middleware was calling next({}) in https://github.com/alexforsyth/aws-sdk-js-v3/blob/58e4d4a4d20acfa8e123b87532baeb8b836960e3/packages/middleware-sdk-sqs/src/receive-message.ts#L30 and later on in the return. Fixed so that the return correctly gives back the result of processing in the queue

Description of changes:
SQS middleware no longer calls next({...}) twice and now correctly returns messages from the queue.

(async () => {
    
    const QueueUrl = "https://sqs.us-west-2.amazonaws.com/XXXXXXXXXX";    

    try {
        const client = new SQS({});
        const res = await client.receiveMessage({
            QueueUrl,
            MaxNumberOfMessages: 1,
            WaitTimeSeconds: 10
        });
        console.log("actual returned messages", res.Messages.length);
        

    } catch (e) {
        console.log(e);
    }
})();

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[codecov-io]

Codecov Report

Merging #1680 (58e4d4a) into master (de75f7e) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1680      +/-   ##
==========================================
- Coverage   79.77%   79.71%   -0.07%     
==========================================
  Files         325      328       +3     
  Lines       12087    12573     +486     
  Branches     2553     2670     +117     
==========================================
+ Hits         9643    10022     +379     
- Misses       2444     2551     +107     

Impacted Files	Coverage Δ
...tocol_tests/aws-restxml/commands/XmlMapsCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...ocol_tests/aws-restxml/commands/XmlBlobsCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...ocol_tests/aws-restxml/commands/XmlEnumsCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...ocol_tests/aws-restxml/commands/XmlListsCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...col_tests/aws-restjson/commands/JsonMapsCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...ol_tests/aws-restjson/commands/JsonBlobsCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...ol_tests/aws-restjson/commands/JsonEnumsCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...ol_tests/aws-restjson/commands/JsonListsCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...tests/aws-restxml/commands/XmlAttributesCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
...tests/aws-restxml/commands/XmlNamespacesCommand.ts	95.65% <0.00%> (-4.35%)	⬇️
... and 101 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a3fab9f...58e4d4a. Read the comment docs.

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.