Skip to content

[investigation/do-not-merge] surface legacy instrument error.stack - #1212

Draft
vringar wants to merge 2 commits into
masterfrom
investigate/legacy-error-stack-trace
Draft

vringar wants to merge 2 commits into
masterfrom
investigate/legacy-error-stack-trace

test: demonstrate that popping the top stack frame cannot hide the le…

eba7f78
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 6, 2026 in 1s

61.26% (target 40.00%)

View this Pull Request on Codecov

61.26% (target 40.00%)

Details

Codecov Report

❌ Patch coverage is 8.86076% with 72 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.26%. Comparing base (975b09c) to head (eba7f78).

Files with missing lines Patch % Lines
test/test_js_instrument_stack_pop.py 6.97% 40 Missing ⚠️
test/test_js_instrument_error_stack.py 11.11% 32 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1212      +/-   ##
==========================================
- Coverage   62.31%   61.26%   -1.06%     
==========================================
  Files          40       42       +2     
  Lines        3930     4009      +79     
==========================================
+ Hits         2449     2456       +7     
- Misses       1481     1553      +72     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.