Tests for the #[export] macro#6691
Conversation
|
@poljar is it OK for me to set |
I think you can also just remove that line, yes it's fine. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6691 +/- ##
==========================================
+ Coverage 89.92% 89.94% +0.01%
==========================================
Files 398 398
Lines 111154 111154
Branches 111154 111154
==========================================
+ Hits 99952 99972 +20
+ Misses 7403 7390 -13
+ Partials 3799 3792 -7 ☔ View full report in Codecov by Harness. |
Merging this PR will improve performance by 48.63%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | Simulation | Restore session [memory store] |
168.5 ms | 113.4 ms | +48.63% |
Tip
Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.
Comparing andybalaam/test-export-macro (cdc3d6b) with main (4c15c79)1
Footnotes
48b1235 to
e138043
Compare
e138043 to
2ab4801
Compare
2ab4801 to
cdc3d6b
Compare
I thought there was a bug in the
#[export]macro so I wrote some tests. It seems to be fine, but the tests are hopefully still useful.