993 progress: convert last 4 files outside of extensions/

[teks]

Related Issue(s): #993

Description: Another round of conversion to pytest. In test_common_metadata.py, I assumed the several calls to self.item.clone() were unneeded because the original object was never used.

PR Checklist:

• Pre-commit hooks pass (run pre-commit run --all-files)
• Tests pass (run pytest)
• Documentation has been updated to reflect changes, if applicable
• This PR maintains or improves overall codebase code coverage.
• Changes are added to the CHANGELOG. See the docs for information about adding to the changelog.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.49%. Comparing base (6d0bf65) to head (a9fdde3).
Report is 17 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1530   +/-   ##
=======================================
  Coverage   91.49%   91.49%           
=======================================
  Files          54       54           
  Lines        7571     7571           
  Branches      922      922           
=======================================
  Hits         6927     6927           
  Misses        457      457           
  Partials      187      187           

☔ 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.

[gadomski]

Thanks! Looking good, appreciate this...the tests read much cleaner, at least to my eye, with these changes.