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 test CryptoCreateSuite.syntaxChecksAreAsExpected #9677
Fix test CryptoCreateSuite.syntaxChecksAreAsExpected #9677
Conversation
Update to comments in NetworkUncheckedSubmitHandler Signed-off-by: Kim Rader <kim.rader@swirldslabs.com>
Signed-off-by: Kim Rader <kim.rader@swirldslabs.com>
Signed-off-by: Kim Rader <kim.rader@swirldslabs.com>
Signed-off-by: Kim Rader <kim.rader@swirldslabs.com>
Signed-off-by: Kim Rader <kim.rader@swirldslabs.com>
Signed-off-by: Kim Rader <kim.rader@swirldslabs.com>
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.
I see one HAPI test failing, Is it flaky ?
Not flaky, but it's also failing in develop. |
Ah I did not see that fail on my PR 🤔 . If its failing on develop also how was this enabled? |
It looks like it was fixed in your PR #9535 that just merged. I grabbed latest develop and now it passes locally for me. I merged develop here and tests are re-running, I think they will pass this time. |
...va/com/hedera/node/app/service/networkadmin/impl/handlers/NetworkUncheckedSubmitHandler.java
Outdated
Show resolved
Hide resolved
...a-app/src/main/java/com/hedera/node/app/workflows/handle/validation/ExpiryValidatorImpl.java
Show resolved
Hide resolved
...ra-node/hedera-app-spi/src/main/java/com/hedera/node/app/spi/validation/ExpiryValidator.java
Outdated
Show resolved
Hide resolved
...a-app/src/main/java/com/hedera/node/app/workflows/handle/validation/ExpiryValidatorImpl.java
Show resolved
Hide resolved
Signed-off-by: Kim Rader <kim.rader@swirldslabs.com>
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!
Signed-off-by: Kim Rader <kim.rader@swirldslabs.com> Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Kim Rader <kim.rader@swirldslabs.com> Signed-off-by: Nick Poorman <nick@swirldslabs.com>
Fix test CryptoCreateSuite.syntaxChecksAreAsExpected
Update to comments in NetworkUncheckedSubmitHandler
Related issue(s):
Fixes #9515