Skip to content

Conversation

@tieneupin
Copy link
Contributor

@tieneupin tieneupin commented Jan 26, 2026

The search for an atlas XML file currently fails if no search results were returned for the atlas:

src/murfey/client/context.py, L97-99

atlas_xml_path = list(
    (source_visit_dir / partial_path).parent.glob("Atlas_*.xml")
)[0]

This PR adjusts that logic so that it proceeds with the post to the backend server with the minimum amount of informaiton present and safe-ish placeholder name for the atlas.

@tieneupin tieneupin self-assigned this Jan 26, 2026
@tieneupin tieneupin added bug Something isn't working client Relates to the client component labels Jan 26, 2026
@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

❌ Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.65%. Comparing base (29519b8) to head (5c4b519).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #734      +/-   ##
==========================================
- Coverage   46.66%   46.65%   -0.01%     
==========================================
  Files          91       91              
  Lines        9605     9607       +2     
  Branches     1254     1255       +1     
==========================================
  Hits         4482     4482              
- Misses       4911     4912       +1     
- Partials      212      213       +1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stephen-riggs stephen-riggs merged commit 8fb34a2 into main Feb 10, 2026
17 checks passed
@stephen-riggs stephen-riggs deleted the safe-atlas-xml-search branch February 10, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working client Relates to the client component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants