feat: Add capacity information when applicable to dynamodb spans

[nordfjord]

Which problem is this PR solving?

• DynamoDB spans should contain consumed capacity information (source)
• The current implementation only includes it for "BatchGetItems" requests

Short description of the changes

• Removes the check for command type
• If capacity information was returned it is added to the span
• If the returned capacity information is not an array it's turned into a singleton array

[codecov]

Codecov Report

Merging #1365 (e89d7ad) into main (704f76f) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1365   +/-   ##
=======================================
  Coverage   91.80%   91.80%           
=======================================
  Files         139      139           
  Lines        7124     7124           
  Branches     1432     1432           
=======================================
  Hits         6540     6540           
  Misses        584      584           

Files Changed	Coverage Δ
...y-instrumentation-aws-sdk/src/services/dynamodb.ts	100.00% <100.00%> (ø)

[blumamir]

Thanks!
please add tests for this

[haddasbronfman]

Hi @nordfjord, are you still interested in working on this PR?

[github-actions]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[nordfjord]

Hi @nordfjord, are you still interested in working on this PR?

Hey, sorry for my slow replies here. I'll have some time this week to add the requested tests!

[haddasbronfman]

great. thanks!

[nordfjord]

Hey y'all, sorry again for the tardiness, I've added two test cases now to cover the functionality 🎉

[pichlermarc]

The failing TAV script here is related to #1477
cc @carolabadeer I think this PR is still waiting for an owner review 🙂

[carolabadeer]

Looks good to me, thanks for making this change!

[github-actions]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[carolabadeer]

Any blocker to merging this change? I believe the TAV workflow failure is unrelated to this change, it is documented in #1477 which is being worked on

[blumamir]

Any blocker to merging this change? I believe the TAV workflow failure is unrelated to this change, it is documented in #1477 which is being worked on

LGTM.

Let's merge and fix the tav issue mentioned in #1477 run in a different PR.