Make get_all_collections properly recursive

[jsignell]

Related Issue(s):

• closes get_all_collections only goes one level deeper #1360

Description:

The docstring states that this method is meant to traverse subcatalogs recursively.

PR Checklist:

• pre-commit hooks pass locally
• Tests pass (run scripts/test)
• 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.22%. Comparing base (52d59b3) to head (ba1f397).
Report is 41 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1361   +/-   ##
=======================================
  Coverage   91.22%   91.22%           
=======================================
  Files          51       51           
  Lines        7042     7042           
  Branches     1008     1008           
=======================================
  Hits         6424     6424           
  Misses        443      443           
  Partials      175      175           

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

[gadomski]

Just needs a ## Fixed changelog entry