Skip to content

wrong location in panic message - #98

Merged
haraldmaida merged 3 commits into
mainfrom
bug/wrong-location-in-panic-message
Sep 26, 2026
Merged

haraldmaida merged 3 commits into
mainfrom
bug/wrong-location-in-panic-message

Conversation

@haraldmaida

@haraldmaida haraldmaida commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

When an assertion fails and panics, the panic message in the terminal does not contain the location of the assertion itself but internal location where the panic occurs.

This pull request fixes this issue. Additionally it adds the ability to verify the panic location in plain unit tests.

@haraldmaida haraldmaida self-assigned this Sep 26, 2026
@haraldmaida haraldmaida added the bug Something isn't working label Sep 26, 2026
@codecov

codecov Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.49558% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.78%. Comparing base (5423488) to head (484ff86).

Files with missing lines Patch % Lines
src/panic_assertion/mod.rs 90.00% 7 Missing and 2 partials ⚠️
src/panic/mod.rs 71.42% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #98      +/-   ##
==========================================
- Coverage   87.84%   87.78%   -0.07%     
==========================================
  Files          42       43       +1     
  Lines        7513     7531      +18     
  Branches     7513     7531      +18     
==========================================
+ Hits         6600     6611      +11     
- Misses        843      850       +7     
  Partials       70       70              

☔ View full report in Codecov by Harness.
📢 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.

@haraldmaida
haraldmaida merged commit 2b71c1c into main Sep 26, 2026
19 of 20 checks passed
@haraldmaida
haraldmaida deleted the bug/wrong-location-in-panic-message branch September 26, 2026 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant