Electric-Coin-Company / zashi-android

The Zashi Zcash Wallet, Android Edition
MIT License
22 stars 15 forks source link

Failed to parse checkpoint file #1180

Open HonzaR opened 8 months ago

HonzaR commented 8 months ago

Describe the issue

Sometimes, there is an occasional bug reported by our screenshot test running on CI (exclusively). It refers to the birthday checkpoint parsing

Can you reliably reproduce the issue?

If so, please list the steps to reproduce below:

Expected behavior

Actual behavior + errors

cash.z.ecc.android.sdk.exception.BirthdayException$MalformattedBirthdayFilesException: Failed to parse file co.electriccoin.zcash/checkpoint/mainnet/419200.json verify that it is formatted as #####.json, where the first portion is an Int representing the height of the tree contained in the file
at cash.z.ecc.android.sdk.tool.CheckpointTool.getFirstValidWalletBirthday$zcash_android_sdk_2_0_4_release(CheckpointTool.kt:164)
at cash.z.ecc.android.sdk.tool.CheckpointTool$getFirstValidWalletBirthday$1.invokeSuspend(Unknown Source:18)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:234)
at android.os.Handler.handleCallback(Handler.java:790)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:164)
at android.app.ActivityThread.main(ActivityThread.java:6494)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)
Suppressed: kotlinx.coroutines.internal.DiagnosticCoroutineContextException: [StandaloneCoroutine{Cancelling}@6e198a6, Dispatchers.Main]
Caused by: kotlinx.coroutines.flow.internal.ChildCancelledException: Child of the scoped flow was cancelled

Any extra information that might be useful in the debugging process.

It usually succeeds when the CI is rerun.

HonzaR commented 5 months ago

Closing this as resolved, possibly because all dependencies update on both SDK and App repositories, and it was not seen in the last PRs' check on CI.

HonzaR commented 4 months ago

Reopening this as it occurred in https://github.com/Electric-Coin-Company/zashi-android/pull/1408 in test_android_modules_wtf_no_coverage CI check again. Note that we are experiencing this only in the CI check.