Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Test deposit / withdraw on the mixer #3

Closed
4 of 5 tasks
Tracked by #1
, #17
...
dutterbutter opened this issue Feb 11, 2022 · 0 comments · Fixed by #46
Closed
4 of 5 tasks
Tracked by #1
, #17
...

Test deposit / withdraw on the mixer #3

dutterbutter opened this issue Feb 11, 2022 · 0 comments · Fixed by #46
Assignees
@dharjeezy
Labels
p2 🟡 Issue should be resolved soon testing 🛠️

Comments

@dutterbutter
Copy link
Contributor

dutterbutter commented Feb 11, 2022 •
edited by dharjeezy
Loading

Overview

Tests with multiple contracts in Ink are only possible with Typescript tests. Our system uses multiple contracts for the hasher/verifiers. Therefore, you will want to write tests in Typescript

  • Use webb.js to generate notes for depositing
  • Add a test for depositing the leaf from the note.
  • Use webb.js to generate withdrawal proofs for withdrawing.
  • Add a test for withdrawing the deposit using this proof.
  • Add a test for tampering with the proof and ensure it fails.

Implementations

We have many implementations of using webb.js to generate notes/proofs that use the Rust backend.
@dutterbutter dutterbutter mentioned this issue Feb 11, 2022
Closed
6 tasks
@dutterbutter dutterbutter added the p2 🟡 Issue should be resolved soon label Feb 11, 2022
@drewstone drewstone mentioned this issue Feb 11, 2022
Closed
4 tasks
@drewstone drewstone changed the title Test withdraw on the mixer Test deposit / withdraw on the mixer May 5, 2022
salman01zp added a commit that referenced this issue May 16, 2022
@salman01zp
use webb-sdk for Note generation #3
749609f
salman01zp added a commit that referenced this issue May 30, 2022
@salman01zp
updated test case #3
5303e9d
drewstone added a commit that referenced this issue Jun 6, 2022
@dharjeezy @salman01zp @drewstone
Implement Governed Token Wrapper Contract (#34)
c8c1060 
* start vanchor implementation

* start vanchor implementation

* implement vanchor verifier

* implement merkle tree
implement linkable merkle tree

* ensures build works successfully
impl encode decode

* impl encode like trait manually

* implement 2 verifiers and instatiation of contracts

* address PR issues

* use webb-sdk for Note generation #3

* update edge contract message
vanchor config contract message

* change zeroes

* started validate proof function

* update version of arkwork-setups

* implement proof validation on vanchor
keccak implementation
field ops implementation

* Implement transact_deposit contract function

* transact withdraw started

* transact withdraw partially finished

* merge main
cargo fmt

* make v anchor verifier a reference contract call in vanchor

* started governed token wrapper implementation

* implement warp contract function with helper functions

* doc

* implement unwrap functionality

* implement unwrap functionality

* review changes

* review changes

* review changes

* saturating sub

* fix is_valid_wrapping() check

* implementation of wrap_for, unwrap_for, wrap_for_and_send_to, unwrap_for_and_send_to contract functions

* implementation for add token address and remove token address contract functions

* cargo

* doc

* token address already exists error

* update config contract function

* change cargo author

* updated test case #3

* started testing

* started testing

* revert and update package json

* remove vanchor related stuffs
ready for merge to main

* Update contracts/governed_token_wrapper/lib.rs

* Update contracts/governed_token_wrapper/lib.rs

Co-authored-by: drewstone <[email protected]>

* address PR comments

* address PR comments

* address PR comments

* fix comment/doc issues

Co-authored-by: salman01z <[email protected]>
Co-authored-by: drewstone <[email protected]>
@dharjeezy dharjeezy mentioned this issue Jun 27, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dharjeezy dharjeezy

Labels
p2 🟡 Issue should be resolved soon testing 🛠️
Projects
Webb Universe
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Dami/test for mixer tangle-network/protocol-ink
3 participants
@dharjeezy @OAyomide @dutterbutter