Skip to content

Fix smart info File Not Found Error on Initial Install#806

Open
MattToast wants to merge 2 commits into
CrayLabs:developfrom
MattToast:check-for-missing-bin-dir
Open

Fix smart info File Not Found Error on Initial Install#806
MattToast wants to merge 2 commits into
CrayLabs:developfrom
MattToast:check-for-missing-bin-dir

Conversation

@MattToast
Copy link
Copy Markdown
Member

Remove spurious error from smart info complaining about a missing build directory when smart build has not been run before hand on initial install. Does this by suppressing errors raised out of get_db_path without changing the original intent of the function. Adds better parameterization for testing and tests for the function.

Fixes #745

@MattToast MattToast requested a review from ashao May 15, 2026 22:31
@MattToast MattToast self-assigned this May 15, 2026
@MattToast MattToast added the release:052026 To be addressed as part of the May 2026 maintenance release label May 15, 2026
@MattToast MattToast marked this pull request as ready for review May 15, 2026 22:37
@codecov
Copy link
Copy Markdown

codecov Bot commented May 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.17%. Comparing base (d7d979e) to head (0d905de).
⚠️ Report is 28 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #806      +/-   ##
===========================================
- Coverage    83.91%   80.17%   -3.74%     
===========================================
  Files           83       78       -5     
  Lines         6284     6093     -191     
===========================================
- Hits          5273     4885     -388     
- Misses        1011     1208     +197     

see 42 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:052026 To be addressed as part of the May 2026 maintenance release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

smart info raises on fresh install

1 participant