-
Notifications
You must be signed in to change notification settings - Fork 947
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Track caller's emitted events #3268
Conversation
430d746
to
83a3ef4
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3268 +/- ##
==========================================
+ Coverage 53.79% 53.85% +0.06%
==========================================
Files 314 315 +1
Lines 105784 105987 +203
==========================================
+ Hits 56903 57083 +180
- Misses 48881 48904 +23 ☔ View full report in Codecov by Sentry. |
854703a
to
b5dd972
Compare
ebbed13
to
73e0543
Compare
73e0543
to
11b9d05
Compare
|
||
let trace = event.read_attribute::<EventOrigin>().unwrap(); | ||
|
||
assert!(trace.line > start_line && trace.line < end_line); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: maybe trace.line == start_line + 1 && trace.line == END_LINE - 1
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doesn't matter to be precise, we only care to check if the event was emitted within a certain region of the source code
11b9d05
to
df4b4f7
Compare
* origin/tiago/track-caller-events: Changelog for #3268 Activate `debug` feat when `testing` is active Test event emission origin Track the origin of emitted events
Describe your changes
Track the source code origin of some emitted event in Namada.
Indicate on which release or other PRs this topic is based on
v0.36.1
Checklist before merging to
draft