Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

swap: pass depositAmount to testDeploy #2061

Merged
merged 2 commits into from
Dec 23, 2019
Merged

Conversation

ralph-pichler
Copy link
Member

@ralph-pichler ralph-pichler commented Dec 20, 2019

This PR removes the magical deployment deposit constant and lets each call of testDeploy decide the value individually. Deposits where chosen to the minimum amount required in each test.

To make things easier this PR also changes the way test deposits are made. Previously they were funded by the swap owner address (requiring token prefunding of that account also). Now they are always minted on demand by the token owner (the global ownerKey).

closes #1928

Copy link
Contributor

@mortelli mortelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fully behind this change. just left a few questions

swap/simulations_test.go Outdated Show resolved Hide resolved
swap/swap_test.go Outdated Show resolved Hide resolved
@ralph-pichler ralph-pichler merged commit 9cdd812 into master Dec 23, 2019
@ralph-pichler ralph-pichler deleted the swap_testdeploy branch January 10, 2020 15:57
@acud acud added this to the 0.5.5 milestone Jan 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TestDeploy magical constant for initial deposit
4 participants