-
Notifications
You must be signed in to change notification settings - Fork 157
test: add tests to PauseUnpauseTokenAccountPrecompileSuite #12492
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
Conversation
Signed-off-by: Mustafa Uzun <[email protected]>
Node: HAPI Test (Restart) Results2 tests 2 ✅ 5m 28s ⏱️ Results for commit 9534cd5. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Node Death Reconnect) Results1 tests 1 ✅ 27s ⏱️ For more details on these parsing errors, see this check. Results for commit 9534cd5. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Token) Results207 tests 207 ✅ 24m 6s ⏱️ Results for commit 9534cd5. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Crypto) Results310 tests 310 ✅ 47m 26s ⏱️ Results for commit 9534cd5. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Misc) Results445 tests 435 ✅ 46m 51s ⏱️ Results for commit 9534cd5. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Time Consuming) Results21 tests 21 ✅ 54m 7s ⏱️ Results for commit 9534cd5. ♻️ This comment has been updated with latest results. |
Node: Unit Test Results 2 271 files +2 2 271 suites +2 2h 51m 33s ⏱️ - 1h 31m 43s Results for commit 9534cd5. ± Comparison against base commit dda6b07. This pull request removes 3986 and adds 3763 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Node: HAPI Test (Smart Contract) Results513 tests 513 ✅ 1h 4m 37s ⏱️ Results for commit 9534cd5. ♻️ This comment has been updated with latest results. |
.../hedera/services/bdd/suites/contract/precompile/PauseUnpauseTokenAccountPrecompileSuite.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Mustafa Uzun <[email protected]>
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!
...ce-impl/src/main/java/com/hedera/node/app/service/token/impl/handlers/TokenPauseHandler.java
Show resolved
Hide resolved
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, ty @mustafauzunn !
Signed-off-by: Mustafa Uzun <[email protected]>
Description:
add tests to PauseUnpauseTokenAccountPrecompileSuite
Related issue(s):
Fixes #12490
Notes for reviewer:
Checklist