Skip to content

Preserve IFDS return traces and actor request lifecycle in #225 - #429

Draft
CaelmBleidd wants to merge 6 commits into
sergey/ifdsfrom
caelmbleidd/review-j225-20261008
Draft

CaelmBleidd wants to merge 6 commits into
sergey/ifdsfrom
caelmbleidd/review-j225-20261008

Conversation

@CaelmBleidd

@CaelmBleidd CaelmBleidd commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

Preserve IFDS return traces and bounded actor lifecycle/resource ownership.

Coverage: full workflow 38328063614 passed. codecov/patch passes at 98.33%; codecov/project remains red (73.12% vs 74.79%). The first IFDS test activates 18 previously absent inherited source files (362 lines); 17 are unchanged. Coverage of the pre-existing file set rises to 75.08%. This is a baseline expansion, with no lost source-file coverage. Thresholds and reported files remain unchanged.

@CaelmBleidd
CaelmBleidd force-pushed the caelmbleidd/review-j225-20261008 branch from 4841ace to b9e0bdd Compare October 12, 2026 22:56
@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Lifecycle test results

48 tests   48 ✅  1m 22s ⏱️
 5 suites   0 💤
 5 files     0 ❌

Results for commit c3383c2.

♻️ This comment has been updated with latest results.

@github-actions

Copy link
Copy Markdown
Contributor

Test results on JDK 19

1 339 tests   1 327 ✅  7m 0s ⏱️
   53 suites     12 💤
   53 files        0 ❌

Results for commit c3383c2.

@github-actions

Copy link
Copy Markdown
Contributor

Test results on JDK 11

1 339 tests   1 329 ✅  6m 23s ⏱️
   53 suites     10 💤
   53 files        0 ❌

Results for commit c3383c2.

@github-actions

Copy link
Copy Markdown
Contributor

Test results on JDK 8

1 339 tests   1 325 ✅  6m 5s ⏱️
   53 suites     14 💤
   53 files        0 ❌

Results for commit c3383c2.

@codecov

codecov Bot commented Oct 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 73.12%. Comparing base (1b5872c) to head (c3383c2).

Files with missing lines Patch % Lines
.../org/jacodb/actors/impl/workers/UserActorWorker.kt 93.33% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##             sergey/ifds     #429      +/-   ##
=================================================
- Coverage          74.79%   73.12%   -1.68%     
- Complexity          1695     1721      +26     
=================================================
  Files                188      206      +18     
  Lines              10218    10619     +401     
  Branches            1838     1893      +55     
=================================================
+ Hits                7643     7765     +122     
- Misses              1834     2105     +271     
- Partials             741      749       +8     

☔ 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.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant