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

chore: fix IndexOutOfBoundsException when invalid function selector #11373

Closed
wants to merge 1 commit into from

Conversation

Neeharika-Sompalli
Copy link
Member

Fixes #11371

Fix IndexOutOfBoundsException to not throw FAIL_INVALID when invalid function selector is provided

Copy link
Collaborator

@tinker-michaelj tinker-michaelj left a comment

Choose a reason for hiding this comment

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

LGTM, tyvm @Neeharika-Sompalli !

@tinker-michaelj tinker-michaelj changed the title Fix IndexOutOfBoundsException when invalid function selector chore: fix IndexOutOfBoundsException when invalid function selector Feb 5, 2024
Copy link
Member

@lukelee-sl lukelee-sl left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

github-actions bot commented Feb 5, 2024

Node: HAPI Test (Restart) Results

1 tests   1 ✔️  6m 11s ⏱️
1 suites  0 💤
1 files    0

Results for commit a2dc4ff.

Copy link

github-actions bot commented Feb 5, 2024

Node: HAPI Test (Node Death Reconnect) Results

1 tests   1 ✔️  7m 47s ⏱️
1 suites  0 💤
1 files    0

Results for commit a2dc4ff.

Copy link

github-actions bot commented Feb 5, 2024

Node: HAPI Test (Token) Results

189 tests   189 ✔️  20m 51s ⏱️
  13 suites      0 💤
  13 files        0

Results for commit a2dc4ff.

Copy link

github-actions bot commented Feb 5, 2024

Node: E2E Test Results

    1 files      1 suites   23m 36s ⏱️
311 tests 311 ✔️ 0 💤 0
333 runs  333 ✔️ 0 💤 0

Results for commit a2dc4ff.

Copy link

github-actions bot commented Feb 5, 2024

Node: Unit Test Results

    2 285 files  ±0      2 285 suites  ±0   3h 41m 17s ⏱️ - 57m 35s
118 479 tests ±0  118 424 ✔️ ±0  54 💤 ±0  1 ±0 
126 996 runs  ±0  126 941 ✔️ ±0  54 💤 ±0  1 ±0 

For more details on these failures, see this check.

Results for commit a2dc4ff. ± Comparison against base commit 6c654d9.

Copy link

github-actions bot commented Feb 5, 2024

Node: HAPI Test (Crypto) Results

210 tests   210 ✔️  29m 29s ⏱️
  22 suites      0 💤
  22 files        0

Results for commit a2dc4ff.

@codecov-commenter
Copy link

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (6c654d9) 63.02% compared to head (a2dc4ff) 63.02%.
Report is 1 commits behind head on develop.

Files Patch % Lines
...e/contracts/precompile/HTSPrecompiledContract.java 0.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop   #11373      +/-   ##
=============================================
- Coverage      63.02%   63.02%   -0.01%     
- Complexity     31057    31060       +3     
=============================================
  Files           3367     3367              
  Lines         137589   137591       +2     
  Branches       14461    14462       +1     
=============================================
+ Hits           86720    86721       +1     
- Misses         47170    47175       +5     
+ Partials        3699     3695       -4     

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

Copy link

github-actions bot commented Feb 5, 2024

Node: HAPI Test (Misc) Results

430 tests   416 ✔️  45m 11s ⏱️
  74 suites    14 💤
  74 files        0

Results for commit a2dc4ff.

Copy link

github-actions bot commented Feb 5, 2024

Node: HAPI Test (Time Consuming) Results

20 tests   20 ✔️  54m 17s ⏱️
  2 suites    0 💤
  2 files      0

Results for commit a2dc4ff.

Copy link

github-actions bot commented Feb 5, 2024

Node: Integration Test Results

    5 files      5 suites   1h 5m 45s ⏱️
319 tests 319 ✔️ 0 💤 0
331 runs  331 ✔️ 0 💤 0

Results for commit a2dc4ff.

Copy link

github-actions bot commented Feb 5, 2024

Node: HAPI Test (Smart Contract) Results

452 tests   448 ✔️  1h 12m 23s ⏱️
  52 suites      4 💤
  52 files        0

Results for commit a2dc4ff.

@Neeharika-Sompalli
Copy link
Member Author

Will be fixed by #11359

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.

Fix IndexOutOfBoundsException when Invalid function selector is provided
4 participants