Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Update compile and deployment module #265

Merged
merged 17 commits into from
Jan 17, 2019
Merged

Conversation

ccowell
Copy link
Contributor

@ccowell ccowell commented Dec 20, 2018

  • Modularize pretest setup into a helper function
  • Added support for multiple contract filenames
  • Updated tests dependent on previous compile/deploy module

  - abi
  - accounts
  - bytecode
  - contract
  - instance
  - provider
  - sources
  - web3
  - abi
  - accounts
  - bytecode
  - contract
  - instance
  - provider
  - sources
  - web3

Updated undefined.js to use new pretest setUp module.
…core into compile-and-deploy

Updated undefined.js to use compile and deploy module.
Check for file extension when handling contracts
Copy link
Member

@davidmurdoch davidmurdoch left a comment

Choose a reason for hiding this comment

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

Looking great so far @ccowell! Have a couple of things I want changed and a few suggestions too. Thanks!

test/contracts/Call.sol Outdated Show resolved Hide resolved
test/helpers/compile_deploy.js Outdated Show resolved Hide resolved
test/helpers/compile_deploy.js Outdated Show resolved Hide resolved
test/helpers/compile_deploy.js Outdated Show resolved Hide resolved
test/helpers/compile_deploy.js Outdated Show resolved Hide resolved
test/helpers/compile_deploy.js Outdated Show resolved Hide resolved
test/options/gasPrice.js Outdated Show resolved Hide resolved
test/call/undefined.js Outdated Show resolved Hide resolved
test/helpers/compile_deploy.js Outdated Show resolved Hide resolved
test/helpers/compile_deploy.js Outdated Show resolved Hide resolved
@davidmurdoch davidmurdoch merged commit 5674fba into develop Jan 17, 2019
@davidmurdoch davidmurdoch deleted the compile-and-deploy branch January 17, 2019 23:06
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