-
Notifications
You must be signed in to change notification settings - Fork 26
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
ZIP 320 manual testing ultimate Issues #1483
Comments
Really nice work! I'd like to work out a road map where eventually all of these states are scripted in an emulator and asserted to be correct. I think detox tests might permit that? @Edicksonjga @juanky201271 @dorianvp Can we meet to discuss the work that remains to get to that point for our test harness? |
I see this as a cross-repo concern. I believe we need to have automated tests running on an emulated app that verify the state of each of these flows. (In zingo-mobile?) I believe that these issues should properly be fixed in zingolib. |
i am sure, based on edicksonz results, that 3 and 4 are due to #1437 |
I'm working on e2e test for those flows... in zingo-mobile. |
about the number 1 issue I noticed something interesting... 2 different scenarios:
I hope this make sense to someone... @zancas @fluidvanadium @AloeareV |
Update:
|
Update: Closing... |
refund
transparent addresses in the Value Transfers List instead asend-to-self
transaction without address.@Edicksonjga Maybe you have more or another issues...
The text was updated successfully, but these errors were encountered: