From d79524f78a4ba3483285dd5f68b5a2e1daaa3ee1 Mon Sep 17 00:00:00 2001 From: cgewecke Date: Mon, 1 May 2017 13:52:50 -0700 Subject: [PATCH] Edit readme to reflect repo name, add options, contributors, contrib guidelines --- README.md | 70 ++++++++++++++++++++++++++++++++++++++----------------- 1 file changed, 48 insertions(+), 22 deletions(-) diff --git a/README.md b/README.md index 4241a417..2e644040 100644 --- a/README.md +++ b/README.md @@ -1,7 +1,7 @@ -# SolCover +# solidity-coverage -![CircleCI Status](https://circleci.com/gh/JoinColony/solcover.svg?style=shield&circle-token=53d5360d290ef593c7bdce505b86ae8b9414e684) -[![codecov](https://codecov.io/gh/JoinColony/solcover/branch/master/graph/badge.svg)](https://codecov.io/gh/JoinColony/solcover) +[![CircleCI](https://circleci.com/gh/sc-forks/solidity-coverage.svg?style=svg)](https://circleci.com/gh/sc-forks/solidity-coverage) +[![codecov](https://codecov.io/gh/sc-forks/solidity-coverage/branch/master/graph/badge.svg)](https://codecov.io/gh/sc-forks/solidity-coverage) ### Code coverage for Solidity testing ![coverage example](https://cdn-images-1.medium.com/max/800/1*uum8t-31bUaa6dTRVVhj6w.png) @@ -9,14 +9,16 @@ For more details about what this is, how it works and potential limitations, see [the accompanying article](https://blog.colony.io/code-coverage-for-solidity-eecfa88668c2). +(solidity-coverage is a stand-alone fork of [Solcover](https://github.com/JoinColony/solcover)) + ### Install ``` -$ npm install --save-dev https://github.com/JoinColony/solcover.git#truffle3 +$ npm install --save-dev https://github.com/sc-forks/solidity-coverage.git ``` ### Run ``` -$ ./node_modules/solcover/exec.js +$ ./node_modules/solidity-coverage/exec.js ``` Tests run signficantly slower while coverage is being generated. A 1 to 2 minute delay @@ -25,13 +27,13 @@ test suite is large. Large solidity files can also take a while to instrument. ### Configuration -By default, Solcover generates a stub `truffle.js` that accomodates its special gas needs and +By default, solidity-coverage generates a stub `truffle.js` that accomodates its special gas needs and connects to a modified version of testrpc on port 8555. If your tests will run on the development network using a standard `truffle.js` and a testrpc instance with no special options, you shouldn't have to do any configuration. If your tests depend on logic added to `truffle.js` - for example: [zeppelin-solidity](https://github.com/OpenZeppelin/zeppelin-solidity/blob/master/truffle.js) uses the file to expose a babel polyfill that its suite requires - you can override the -default behavior by declaring a coverage network in `truffle.js`. Solcover will use your 'truffle.js' +default behavior by declaring a coverage network in `truffle.js`. solidity-coverage will use your 'truffle.js' instead of a dynamically generated one. **Example coverage network config** @@ -56,16 +58,22 @@ module.exports = { You can also create a `.solcover.js` config file in the root directory of your project and specify some additional options: -+ **port**: The port you want Solcover to run testrpc on / have truffle connect to. ++ **port**: The port you want solidity-coverage to run testrpc on / have truffle connect to. + **testrpcOptions**: A string of options to be appended to a command line invocation of testrpc. + Example: `--account="0x89a...b1f',10000" --port 8777`". + Note: you should specify the port in your `testrpcOptions` string AND as a `port` option. -+ **testCommand**: By default Solcover runs `truffle test` or `truffle test --network coverage`. ++ **testCommand**: By default solidity-coverage runs `truffle test` or `truffle test --network coverage`. This option lets you run tests some other way: ex: `mocha --timeout 5000`. You -will probably also need to make sure the web3 provider for your tests explicitly connects to the port Solcover's +will probably also need to make sure the web3 provider for your tests explicitly connects to the port solidity-coverage's testrpc is set to run on, e.g: - + `var web3 = new Web3(new Web3.providers.HttpProvider("http://localhost:8555"))` + + `var web3 = new Web3(new Web3.providers.HttpProvider("http://localhost:8555"))` ++ **norpc**: : By default, false. When true, solidity-coverage will not launch its own testrpc instance. This +can be useful if you are running tests using a different vm like the +[`sc-forks` version of `pyethereum`](https://github.com/sc-forks/pyethereum) ++ **dir**: : By default, solidity-coverage looks for a `contracts` folder in your root +directory. `dir` allows you to define a relative path from the root directory to the contracts +folder. A `dir` of `./secretDirectory` would tell solidity-coverage to look for `./secretDirectory/contracts` **Example .solcover.js config file** ``` @@ -73,46 +81,64 @@ module.exports = { port: 6545, testrpcOptions: '-p 6545 -u 0x54fd80d6ae7584d8e9a19fe1df43f04e5282cc43', testCommand: 'mocha --timeout 5000' + norpc: 'true' + dir: './secretDirectory' }; ``` ### Known Issues -**Hardcoded gas costs**: If you have hardcoded gas costs into your tests some of them may fail when using SolCover. +**Hardcoded gas costs**: If you have hardcoded gas costs into your tests some of them may fail when using solidity-coverage. This is because the instrumentation process increases the gas costs for using the contracts, due to the extra events. If this is the case, then the coverage may be incomplete. To avoid this, using `estimateGas` to estimate your gas costs should be more resilient in most cases. -**Events testing**: Because Solcover injects events into your contracts to log which lines your tests reach, +**Events testing**: Because solidity-coverage injects events into your contracts to log which lines your tests reach, any tests that ask how many events are fired or where the event sits in the logs array will probably error while coverage is being generated. **Using `require` in `migrations.js` files**: Truffle overloads Node's `require` function but implements a simplified search algorithm for node_modules packages ([see issue #383 at Truffle](https://github.com/trufflesuite/truffle/issues/383)). -Because Solcover copies an instrumented version of your project into a temporary folder, `require` +Because solidity-coverage copies an instrumented version of your project into a temporary folder, `require` statements handled by Truffle internally won't resolve correctly. -**Coveralls / CodeCov**: These CI services take the Istanbul reports generated by Solcover and display +**Coveralls / CodeCov**: These CI services take the Istanbul reports generated by solidity-coverage and display line coverage. Istanbul's own html report publishes significantly more detail and can show whether your tests actually reach all the conditional branches in your code. It can be found inside the `coverage` folder at `index.html` after you run the tool. ### Examples -+ **Metacoin**: The default truffle project +**WARNING**: This utility is in development and its accuracy is unknown. If you +find discrepancies between the coverage report and your suite's behavior, please open an +[issue](https://github.com/sc-forks/solidity-coverage/issues). The purpose of +the following examples is to help you install solidity-coverage in your own project and evaluate the +coverage of your own tests. The reports below are **not** meaningful analyses of the +the past or present state of any project's testing regime. + ++ **metacoin**: The default truffle project + [HTML reports](https://sc-forks.github.io/metacoin/) - + [Metacoin with Solcover installed](https://github.com/sc-forks/metacoin) (simple, without configuration) + + [Metacoin with solidity-coverage installed](https://github.com/sc-forks/metacoin) (simple, without configuration) + **zeppelin-solidity** at commit 453a19825013a586751b87c67bebd551a252fb50 + [HTML reports]( https://sc-forks.github.io/zeppelin-solidity/) - + [Zeppelin with Solcover installed](https://github.com/sc-forks/zeppelin-solidity) (declares own coverage network in truffle.js) + + [Zeppelin with solidity-coverage installed](https://github.com/sc-forks/zeppelin-solidity) (declares own coverage network in truffle.js) + **numeraire** at commit 5ac3fa432c6b4192468c95a66e52ca086c804c95 + [HTML reports](https://sc-forks.github.io/contract/) - + [Numeraire with Solcover installed](https://github.com/sc-forks/contract) (uses .solcover.js) + + [Numeraire with solidity-coverage installed](https://github.com/sc-forks/contract) (uses .solcover.js) + +### Contribution Guidelines + +Contributions are welcome! If you're opening a PR that adds features please consider writing some +[unit tests](https://github.com/sc-forks/solidity-coverage/tree/master/test) for them. You could +also lint your submission by running 'npm run lint'. Bugs can be reported in the +[issues](https://github.com/sc-forks/solidity-coverage/issues) + +### Contributors ++ [@area](https://github.com/area) ++ [@cgewecke](https://github.com/cgewecke) ++ [@adriamb](https://github.com/adriamb) ### TODO -- [ ] Turn into a true command line tool, rather than just a hacked-together script - [ ] Release on NPM -- [ ] Support for arbitrary testing commands -- [ ] [You tell me](http://github.com/JoinColony/solcover/issues)