Skip to content

Commit

Permalink
update test for SQS usage of JSON protocol, even though this test is …
Browse files Browse the repository at this point in the history
…skipped; doc the reason for skipping this test
  • Loading branch information
trentm committed Dec 6, 2023
1 parent 9a2bd2e commit 64f8aa3
Showing 1 changed file with 11 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -445,13 +445,24 @@ describe('instrumentation-aws-sdk-v3', () => {
});
});

// Propagating span context to SQS ReceiveMessage promise handler is
// broken with `@aws-sdk/client-sqs` v3.316.0 and later.
// https://github.com/open-telemetry/opentelemetry-js-contrib/issues/1477
it.skip('sqs receive context', done => {
nock(`https://sqs.${region}.amazonaws.com/`)
.matchHeader('content-type', 'application/x-www-form-urlencoded')
.post('/')
.reply(
200,
fs.readFileSync('./test/mock-responses/sqs-receive.xml', 'utf8')
);
nock(`https://sqs.${region}.amazonaws.com/`)
.matchHeader('content-type', 'application/x-amz-json-1.0')
.post('/')
.reply(
200,
fs.readFileSync('./test/mock-responses/sqs-receive.json', 'utf8')
);

const params = {
QueueUrl:
Expand Down

0 comments on commit 64f8aa3

Please sign in to comment.