Skip to content

Commit

Permalink
Updated avodiing common mistakes to include coverage report
Browse files Browse the repository at this point in the history
  • Loading branch information
gjackson12 committed Aug 23, 2018
1 parent 10f7dc3 commit 0880755
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion avoiding_common_mistakes.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,9 @@ I have mitigated against this risk by:

* Not using tx.origin and instead msg.sender

## 8.
## 8. Solid Code Coverage
The blockchain is immutable in the sense that a once approved transaction is going to stay. Since smart contract deployment happens through a transaction, this results in the inability to fix issues quickly. That is why having a reliable set of tests is so crucial.

I have added a tool called Solidity-Coverage in order to generate a report to understand how extensive my test coverage is for the MediaGallery contract.

The report for my test coverage can be found here: [a relative link](coverage/contracts/index.html)

0 comments on commit 0880755

Please sign in to comment.