Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

contracts: Remove unnecessary (and wrong) test #231

Merged
merged 2 commits into from
Jan 14, 2021
Merged

contracts: Remove unnecessary (and wrong) test #231

merged 2 commits into from
Jan 14, 2021

Commits on Jan 13, 2021

  1. contracts: Remove unnecessary (and wrong) test 

    It was an artifical test to cover the else branch of an assert, but
asserts shouldn’t count for covergage:

sc-forks/solidity-coverage#556
    @bingen
    bingen committed Jan 13, 2021
    Configuration menu
    Copy the full SHA
    1fa2509 View commit details
    Browse the repository at this point in the history

Commits on Jan 14, 2021

  1. Merge branch 'contract-changes' into _removeTroveOwner_tests_2

    @bingen
    bingen authored Jan 14, 2021
    Configuration menu
    Copy the full SHA
    615a511 View commit details
    Browse the repository at this point in the history