test: fix TAV tests for @cucumber/cucumber and @aws-sdk/client-s3

[trentm]

• Switch to 'npm run --ws test-all-versions ...' for running TAV tests,
  instead of 'lerna run test-all-versions ...' because nx sets
  'npm_config_legacy_peer_deps=true' which breaks
  '@cucumber/[email protected]' install and could break other installs by
  ignoring 'peerDependencies'.
• Skip the bad '@aws-sdk/[email protected]' release in TAV tests.

Also:

• Reduce the number of versions of 'aws-sdk' and '@aws-sdk/*' packages
  test in TAV tests from 249, 143, and 132 versions currently, to 7
  each.
• Add a top-level npm run test-all-versions script to run that script
  in all packages that have one. This is the equivalent of the
  "test-all-versions.yml" CI workflow.

Fixes: #1828

---

Update for reviewers: See #1838 (comment) below for why I think this PR should go in, even though TAV tests are still failing.

[codecov]

Codecov Report

Merging #1838 (e664c0c) into main (3cb2802) will increase coverage by 0.04%.
The diff coverage is n/a.

❗ Current head e664c0c differs from pull request most recent head 06427fb. Consider uploading reports for the commit 06427fb to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1838      +/-   ##
==========================================
+ Coverage   91.45%   91.49%   +0.04%     
==========================================
  Files         144      144              
  Lines        7406     7388      -18     
  Branches     1483     1474       -9     
==========================================
- Hits         6773     6760      -13     
+ Misses        633      628       -5     

see 8 files with indirect coverage changes

[trentm]

Note to reviewers: The TAV tests failed, but for a different reason, so I think the PR is good to go.

1. The TAV tests for cucumber passed. Good.
2. The TAV tests for @aws-sdk/[email protected] failed (Bad, but a separate issue, see below), before even getting to the @aws-sdk/client-s3 tests that are being fixed here.

the client-sqs failure

> @opentelemetry/[email protected] test-all-versions
> tav

-- required packages ["@aws-sdk/[email protected]"]
-- installing ["@aws-sdk/[email protected]"]
-- running test "npm run test" with @aws-sdk/client-sqs (env: {})
...
  3 failing

  1) instrumentation-aws-sdk-v3
       custom service behavior
         SQS
           sqs send add messaging attributes:
     SyntaxError: Unexpected token < in JSON at position 0
  Deserialization error: to see the raw response, inspect the hidden field {error}.$response on this object.
      at JSON.parse (<anonymous>)
      at /home/runner/work/opentelemetry-js-contrib/opentelemetry-js-contrib/node_modules/@aws-sdk/client-sqs/dist-cjs/protocols/Aws_json1_0.js:1980:21
      at processTicksAndRejections (internal/process/task_queues.js:95:5)
      at de_SendMessageCommand (/home/runner/work/opentelemetry-js-contrib/opentelemetry-js-contrib/node_modules/@aws-sdk/client-sqs/dist-cjs/protocols/Aws_json1_0.js:1013:18)
      at /home/runner/work/opentelemetry-js-contrib/opentelemetry-js-contrib/node_modules/@smithy/middleware-serde/dist-cjs/deserializerMiddleware.js:7:24
      at /home/runner/work/opentelemetry-js-contrib/opentelemetry-js-contrib/node_modules/@aws-sdk/client-sqs/node_modules/@aws-sdk/middleware-signing/dist-cjs/awsAuthMiddleware.js:30:20
      at /home/runner/work/opentelemetry-js-contrib/opentelemetry-js-contrib/node_modules/@smithy/middleware-retry/dist-cjs/retryMiddleware.js:27:46
      at /home/runner/work/opentelemetry-js-contrib/opentelemetry-js-contrib/node_modules/@aws-sdk/middleware-sdk-sqs/dist-cjs/send-message.js:7:18
      at /home/runner/work/opentelemetry-js-contrib/opentelemetry-js-contrib/node_modules/@aws-sdk/client-sqs/node_modules/@aws-sdk/middleware-logger/dist-cjs/loggerMiddleware.js:7:26
...

While we have #1477 already for client-sqs TAV test failures... this failure is actually a different failure reason than the one currently being discussed there. I can update that issue (or create a separate one).

It would help if this PR went in, because the .tav.yml change means we are testing fewer versions of client-sqs so we should hit the test failures quicker in TAV test runs.

[pichlermarc]

Looks great, thanks for taking care of this 🙂