-
Notifications
You must be signed in to change notification settings - Fork 157
fix: tokenAssociate throws IndexOutOfBoundsException #12552
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
fix: tokenAssociate throws IndexOutOfBoundsException #12552
Conversation
Signed-off-by: Zhivko Kelchev <[email protected]>
Signed-off-by: Zhivko Kelchev <[email protected]>
Signed-off-by: Zhivko Kelchev <[email protected]>
Signed-off-by: Zhivko Kelchev <[email protected]>
Signed-off-by: Zhivko Kelchev <[email protected]>
Node: HAPI Test (Restart) Results2 tests 2 ✅ 5m 36s ⏱️ Results for commit 7c61d78. |
Node: HAPI Test (Node Death Reconnect) Results1 tests 1 ✅ 18s ⏱️ For more details on these parsing errors, see this check. Results for commit 7c61d78. |
Node: HAPI Test (Token) Results207 tests 207 ✅ 21m 43s ⏱️ Results for commit 7c61d78. |
Node: HAPI Test (Misc) Results433 tests 422 ✅ 39m 8s ⏱️ For more details on these failures, see this check. Results for commit 7c61d78. |
Node: HAPI Test (Crypto) Results311 tests 311 ✅ 39m 32s ⏱️ Results for commit 7c61d78. |
Node: HAPI Test (Time Consuming) Results21 tests 21 ✅ 54m 27s ⏱️ Results for commit 7c61d78. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @JivkoKelchev
Node: Unit Test Results 2 272 files ±0 2 272 suites ±0 3h 31m 7s ⏱️ - 14m 34s Results for commit 7c61d78. ± Comparison against base commit 58dfd23. This pull request removes 3994 and adds 3764 tests. Note that renamed tests count towards both.
|
Signed-off-by: Zhivko Kelchev <[email protected]> Co-authored-by: Valentin Tronkov <[email protected]>
* develop: feat: Introduce software update admin (#12489) feat: Remove getOtherId (#12465) fix: tokenAssociate throws IndexOutOfBoundsException (#12552) fix: enable HelloWorldEthereumSuite hapi tests (#11602) fix: tokenAssociate throws IndexOutOfBoundsException (#12453) fix: treat an out-of-range allowance amount as an invalid op (#12445) feat: restrict qualified delegates to appropriate transfer functions only (#12358) feat: move the JVM anchor into the wiring model (#12534) feat: Automate scheduler construction for the event deduplicator (#12527) feat: cleaner wiring for complete state nexus (#12502) fix: finalize stake metadata only once per transaction (#12537) feat: platform component builder (#12499) feat: address book serializes text hostnames (#12515) feat: improve bottom half of the wiring diagram (#12511) refactor: Migrate some tests to use event test util (#12471) test: security v2 model - add test for token associate with callcode (#12168) chore: Added extra-checks to prevent unchecked submits (#12436) chore: Change owner of DEV configuration (#12523) feat: construct schedulers automatically (#12504) feat: Cherry-pick: Pass Metrics to created VirtualMaps (#12495) # Conflicts: # hedera-node/hedera-app/src/main/java/com/hedera/node/app/workflows/handle/HandleContextImpl.java
Description:
Addition to #12453
Related issue(s):
Fixes #12440
Notes for reviewer:
Checklist