Skip to content

fix path in test#730

Merged
braingram merged 1 commit into
spacetelescope:mainfrom
braingram:fix_test_path
Oct 13, 2025
Merged

fix path in test#730
braingram merged 1 commit into
spacetelescope:mainfrom
braingram:fix_test_path

Conversation

@braingram
Copy link
Copy Markdown
Collaborator

@braingram braingram commented Oct 10, 2025

Closes #714

See issue for description of bug. The test was splitting on "rad/resources" instead of "rad/latest" which this PR fixes.

Confirmed to fix test when run with the rad clone at a path that includes a "-" in the parent path contents.

No changelog or regtests as this is only a unit test change.

Tasks

  • Update or add relevant rad tests.
  • Update relevant docstrings and / or docs/ page.
  • Does this PR change any schema files?
    • Schema changes were discussed at RAD Review Board meeting.
  • Does this PR change any API used downstream? (If not, label with no-changelog-entry-needed.)
News fragment change types:
  • changes/<PR#>.feature.rst: new feature
  • changes/<PR#>.bugfix.rst: fixes an issue
  • changes/<PR#>.doc.rst: documentation change
  • changes/<PR#>.removal.rst: deprecation or removal of public API
  • changes/<PR#>.misc.rst: infrastructure or miscellaneous change

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.59%. Comparing base (c85d4f5) to head (cdc2361).
⚠️ Report is 136 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #730   +/-   ##
=======================================
  Coverage   97.59%   97.59%           
=======================================
  Files           8        8           
  Lines         747      747           
=======================================
  Hits          729      729           
  Misses         18       18           

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

@braingram braingram merged commit 9cc01e9 into spacetelescope:main Oct 13, 2025
17 of 19 checks passed
@braingram braingram deleted the fix_test_path branch October 13, 2025 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unit Tests Breaks with Dashed Path

2 participants