Skip to content

chore: Reorganized the test utilities in AWS versioned tests#3895

Open
jsumners-nr wants to merge 1 commit intonewrelic:mainfrom
jsumners-nr:reorg-aws-tests
Open

chore: Reorganized the test utilities in AWS versioned tests#3895
jsumners-nr wants to merge 1 commit intonewrelic:mainfrom
jsumners-nr:reorg-aws-tests

Conversation

@jsumners-nr
Copy link
Copy Markdown
Contributor

While working on the SNS and SQS refactors I added test-utils/ to contain utility functions and what-have-you for the various test suites in the aws-sdk-v3 versioned tests. This PR refactors the utilities that already existed within the common.js file into individual files under the test-utils/ directory. I find that this makes it far easier to learn what utilities are available for use due to the file tree showing each one of them.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.72%. Comparing base (8fc1144) to head (c5aec52).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3895      +/-   ##
==========================================
- Coverage   97.83%   97.72%   -0.12%     
==========================================
  Files         493      493              
  Lines       59840    59842       +2     
  Branches        1        1              
==========================================
- Hits        58546    58479      -67     
- Misses       1294     1363      +69     
Flag Coverage Δ
integration-tests-cjs-20.x 74.24% <100.00%> (+<0.01%) ⬆️
integration-tests-cjs-22.x 74.46% <100.00%> (+<0.01%) ⬆️
integration-tests-cjs-24.x 75.11% <100.00%> (+<0.01%) ⬆️
integration-tests-esm-20.x 53.69% <100.00%> (+<0.01%) ⬆️
integration-tests-esm-22.x 53.73% <100.00%> (+<0.01%) ⬆️
integration-tests-esm-24.x 54.93% <100.00%> (+<0.01%) ⬆️
unit-tests-20.x 89.70% <100.00%> (+<0.01%) ⬆️
unit-tests-22.x 89.73% <100.00%> (+<0.01%) ⬆️
unit-tests-24.x 89.73% <100.00%> (+<0.01%) ⬆️
versioned-tests-20.x 80.88% <100.00%> (-0.29%) ⬇️
versioned-tests-22.x 80.88% <100.00%> (-0.29%) ⬇️
versioned-tests-24.x 80.79% <100.00%> (-0.29%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jsumners-nr jsumners-nr marked this pull request as ready for review April 10, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needs PR Review

Development

Successfully merging this pull request may close these issues.

2 participants