Skip to content

Conversation

@Lipen
Copy link
Member

@Lipen Lipen commented Apr 14, 2025

This PR introduces some changes in ETS IR model.

@Lipen Lipen requested a review from CaelmBleidd April 14, 2025 11:18
@Lipen Lipen marked this pull request as ready for review April 14, 2025 11:23
@github-actions
Copy link
Contributor

github-actions bot commented Apr 14, 2025

Lifecycle test results

78 tests  ±0   78 ✅ ±0   1m 25s ⏱️ -3s
 7 suites ±0    0 💤 ±0 
 7 files   ±0    0 ❌ ±0 

Results for commit fdbbbfa. ± Comparison against base commit 6ec737c.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 14, 2025

Test results on JDK 19

 72 files  ±0   72 suites  ±0   4m 50s ⏱️ +11s
632 tests ±0  622 ✅ ±0  10 💤 ±0  0 ❌ ±0 
704 runs  ±0  694 ✅ ±0  10 💤 ±0  0 ❌ ±0 

Results for commit fdbbbfa. ± Comparison against base commit 6ec737c.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Apr 14, 2025

Codecov Report

Attention: Patch coverage is 59.39716% with 229 lines in your changes missing coverage. Please review.

Project coverage is 70.57%. Comparing base (6ec737c) to head (fdbbbfa).
Report is 1 commits behind head on neo.

Files with missing lines Patch % Lines
...ain/kotlin/org/jacodb/ets/utils/BlockCfgBuilder.kt 54.80% 42 Missing and 5 partials ⚠️
...b-ets/src/main/kotlin/org/jacodb/ets/model/Type.kt 44.64% 27 Missing and 4 partials ⚠️
.../main/kotlin/org/jacodb/ets/utils/EtsFileToText.kt 0.00% 27 Missing ⚠️
.../src/main/kotlin/org/jacodb/ets/model/CfgLinear.kt 37.93% 18 Missing ⚠️
...-ets/src/main/kotlin/org/jacodb/ets/dto/Convert.kt 79.22% 13 Missing and 3 partials ⚠️
...s/src/main/kotlin/org/jacodb/ets/model/CfgBlock.kt 61.29% 7 Missing and 5 partials ⚠️
...b-ets/src/main/kotlin/org/jacodb/ets/model/Expr.kt 76.31% 9 Missing ⚠️
...db-ets/src/main/kotlin/org/jacodb/ets/dto/Types.kt 57.89% 8 Missing ⚠️
.../src/main/kotlin/org/jacodb/ets/model/Namespace.kt 0.00% 6 Missing ⚠️
.../main/kotlin/org/jacodb/ets/utils/BlockCfgToDot.kt 80.64% 2 Missing and 4 partials ⚠️
... and 19 more
Additional details and impacted files
@@             Coverage Diff              @@
##                neo     #313      +/-   ##
============================================
+ Coverage     69.51%   70.57%   +1.06%     
+ Complexity     2476     2450      -26     
============================================
  Files           270      275       +5     
  Lines         15471    15260     -211     
  Branches       2282     2265      -17     
============================================
+ Hits          10754    10770      +16     
+ Misses         3790     3569     -221     
+ Partials        927      921       -6     

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

@github-actions
Copy link
Contributor

github-actions bot commented Apr 14, 2025

Test results on JDK 11

 72 files  ±0   72 suites  ±0   5m 25s ⏱️ +10s
632 tests ±0  624 ✅ ±0  8 💤 ±0  0 ❌ ±0 
704 runs  ±0  696 ✅ ±0  8 💤 ±0  0 ❌ ±0 

Results for commit fdbbbfa. ± Comparison against base commit 6ec737c.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 14, 2025

Test results on JDK 8

 72 files  ±0   72 suites  ±0   4m 47s ⏱️ -3s
632 tests ±0  619 ✅ ±0  13 💤 ±0  0 ❌ ±0 
704 runs  ±0  691 ✅ ±0  13 💤 ±0  0 ❌ ±0 

Results for commit fdbbbfa. ± Comparison against base commit 6ec737c.

♻️ This comment has been updated with latest results.

@Lipen Lipen merged commit 3b6a17f into neo Apr 17, 2025
9 of 10 checks passed
@Lipen Lipen deleted the lipen/ts-model branch April 17, 2025 12:06
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.

3 participants