Skip to content
This repository has been archived by the owner on Jan 24, 2022. It is now read-only.

Adds tests for the vouching app deploy scripts #212

Merged
merged 18 commits into from
Oct 11, 2018

Conversation

eternauta1337
Copy link
Contributor

@eternauta1337 eternauta1337 commented Oct 9, 2018

Addresses #201

Tasklist

  • Deploy script can inject tpl-contracts-zos in local network (was done manually, and so it is needed for automated testing).
  • Validate script needs a new function to validate if the app was pushed correctly.
  • Isolate all tpl configuration validations into a new function instead of at the end of other functions.
  • Export the functions and use them in a test script.
  • Make sure nothing throws in the test (this will be the test itself).
  • (NEED zOS FIX) Resolve bug where ZEPToken json abi needs to be tweaked by hand.

@facuspagnuolo facuspagnuolo self-assigned this Oct 10, 2018
@facuspagnuolo facuspagnuolo added the status:in-progress Under development, do not merge this PR label Oct 10, 2018
@facuspagnuolo facuspagnuolo removed the status:in-progress Under development, do not merge this PR label Oct 11, 2018
@facuspagnuolo facuspagnuolo merged commit 7ed3d1f into master Oct 11, 2018
@facuspagnuolo facuspagnuolo deleted the vouching-deploy-scripts-tests branch October 11, 2018 16:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants