NXPY-255: Fix Test cases for moto3

[gitofanindya]

No description provided.

[sourcery-ai]

PR Type: Bug fix

PR Summary: This pull request addresses the update of testing dependencies and the corresponding changes required in the test cases to ensure compatibility with the latest versions. Specifically, it updates the usage of the 'moto' library for mocking AWS services in the S3 upload functionality tests.

Decision: Comment

📝 Type: 'Bug fix' - not supported yet.

• Sourcery currently only approves 'Typo fix' PRs.

✅ Issue addressed: this change correctly addresses the issue or implements the desired feature.

No details provided.

📝 Complexity: the changes are too large or complex for Sourcery to approve.

• Unsupported files: the diff contains files that Sourcery does not currently support during reviews.

General suggestions:

• Ensure that all tests pass with the updated 'moto' library version to confirm that the changes do not introduce any regressions.
• Consider adding a brief description in the PR about why the change from 'mock_s3' to 'mock_aws' was necessary. This could help in understanding the context of the change and its impact on the test cases.
• Review other test cases that might be affected by the update in the 'moto' library to ensure consistency across the project.

Thanks for using Sourcery. We offer it for free for open source projects and would be very grateful if you could help us grow. If you like it, would you consider sharing Sourcery on your favourite social media? ✨

Share Sourcery

• X
• Mastodon
• LinkedIn
• Facebook

---

Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a67f14a) 96.63% compared to head (8bb3b26) 96.63%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #303   +/-   ##
=======================================
  Coverage   96.63%   96.63%           
=======================================
  Files          29       29           
  Lines        1813     1813           
=======================================
  Hits         1752     1752           
  Misses         61       61           

Flag	Coverage Δ
functional	90.01% <ø> (ø)
unit	51.73% <ø> (ø)

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.

[mr-shekhar]

Pl describe the reason for change.
With that, approved

[gitofanindya]

Previously, we imported'mock_s3' from moto to mock the S3 during upload testing. However, since we recently upgraded Moto to v5.x.x, all of the decorators have been replaced with a single decorator -'mock_aws'. So we've replaced "mock_s3" with "mock_aws".

Reference: https://github.com/getmoto/moto/blob/master/CHANGELOG.md.

[mr-shekhar]

Pl add detail