search

OpenZeppelin / openzeppelin-test-environment

[Not actively maintained] One-line setup for blazing-fast smart contracts tests
https://docs.openzeppelin.com/test-environment
MIT License
90 stars 39 forks source link

Add a port number as an optional config value #127

Open chebykin opened 4 years ago

chebykin commented 4 years ago

This will solve #65 if you specify the same port number in eth-gas-reporter and test-environment configs. When you ignore a port number value in your config, it will use a default undefined just like before.

frangio commented 4 years ago

Thanks for the suggestion @chebykin! This is an interesting way to achieve support for gas reporting. As a side effect, though, degining a port number will make parallel tests (with Jest or AVA) stop working. So I'm not sure the solution is acceptable for us.