Skip to content
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

feat: 14136 native support #14357

Closed
wants to merge 14 commits into from
Closed

feat: 14136 native support #14357

wants to merge 14 commits into from

Conversation

mxtartaglia-sl
Copy link
Contributor

@mxtartaglia-sl mxtartaglia-sl commented Jul 23, 2024

Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
@mxtartaglia-sl mxtartaglia-sl self-assigned this Jul 23, 2024
@mxtartaglia-sl mxtartaglia-sl added this to the v0.53 milestone Jul 23, 2024
@mxtartaglia-sl mxtartaglia-sl changed the title 14136 native support feat: 14136 native support Jul 23, 2024
Copy link

github-actions bot commented Jul 23, 2024

Node: Unit Test Results

  1 542 files    1 542 suites   5h 20m 48s ⏱️
117 411 tests 117 353 ✅ 58 💤 0 ❌
125 639 runs  125 581 ✅ 58 💤 0 ❌

Results for commit 5dec62d.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 23, 2024

Node: HAPI Test (Node Death Reconnect) Results

3 tests   3 ✅  5m 0s ⏱️
3 suites  0 💤
3 files    0 ❌

Results for commit 5dec62d.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 23, 2024

Node: HAPI Test (Token) Results

 22 files   22 suites   7m 19s ⏱️
280 tests 280 ✅ 0 💤 0 ❌
359 runs  359 ✅ 0 💤 0 ❌

Results for commit 5dec62d.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 23, 2024

Node: HAPI Test (Restart) Results

7 files  7 suites   8m 12s ⏱️
6 tests 6 ✅ 0 💤 0 ❌
7 runs  7 ✅ 0 💤 0 ❌

Results for commit 5dec62d.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 23, 2024

Node: HAPI Test (Crypto) Results

 24 files   24 suites   12m 20s ⏱️
347 tests 347 ✅ 0 💤 0 ❌
372 runs  372 ✅ 0 💤 0 ❌

Results for commit 5dec62d.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 23, 2024

Node: HAPI Test (Misc) Results

 61 files   61 suites   17m 48s ⏱️
318 tests 318 ✅ 0 💤 0 ❌
408 runs  408 ✅ 0 💤 0 ❌

Results for commit 5dec62d.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 23, 2024

Node: HAPI Test (Smart Contract) Results

 81 files   81 suites   24m 18s ⏱️
650 tests 650 ✅ 0 💤 0 ❌
758 runs  758 ✅ 0 💤 0 ❌

Results for commit 5dec62d.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 23, 2024

Node: HAPI Test (Time Consuming) Results

19 tests   19 ✅  22m 50s ⏱️
 4 suites   0 💤
 4 files     0 ❌

Results for commit 5dec62d.

♻️ This comment has been updated with latest results.

/**
* The {@link ResourceLoader} used to load the Helm executable.
*/
private static final ResourceLoader RESOURCE_LOADER = new ResourceLoader(LibraryLoader.class);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This assumes the resources are always located in the jar file containing the LibraryLoader class.

Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Copy link

codecov bot commented Jul 23, 2024

Codecov Report

Attention: Patch coverage is 77.17391% with 21 lines in your changes missing coverage. Please review.

Project coverage is 57.47%. Comparing base (9973fae) to head (5dec62d).
Report is 178 commits behind head on develop.

Files Patch % Lines
...om/hedera/common/nativesupport/ResourceLoader.java 80.00% 6 Missing and 1 partial ⚠️
.../com/hedera/common/nativesupport/Architecture.java 50.00% 5 Missing and 1 partial ⚠️
...m/hedera/common/nativesupport/OperatingSystem.java 50.00% 4 Missing and 2 partials ⚠️
...com/hedera/common/nativesupport/LibraryLoader.java 85.71% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##             develop   #14357       +/-   ##
==============================================
- Coverage      67.58%   57.47%   -10.12%     
+ Complexity     36486    21816    -14670     
==============================================
  Files           3732     2861      -871     
  Lines         149767   109904    -39863     
  Branches       15571    11249     -4322     
==============================================
- Hits          101223    63165    -38058     
+ Misses         44179    43162     -1017     
+ Partials        4365     3577      -788     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
Signed-off-by: mxtartaglia <maxi@swirldslabs.com>
@mxtartaglia-sl
Copy link
Contributor Author

Closing this pull request as hedera-cryptography repository is already available:https://github.com/hashgraph/hedera-cryptography/pull/27

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.

Implement native-support library
2 participants