Closed horsefacts closed 1 year ago
Coverage after merging horsefacts/coverage-fixups into main will be
98.62% |
---|
File | Stmts | Branches | Funcs | Lines | Uncovered Lines |
---|---|---|---|---|---|
src | |||||
Bundler.sol | 100% | 100% | 100% | 100% | |
FnameResolver.sol | 100% | 100% | 100% | 100% | |
IdRegistry.sol | 100% | 100% | 100% | 100% | |
KeyRegistry.sol | 100% | 100% | 100% | 100% | |
StorageRent.sol | 100% | 100% | 100% | 100% | |
src/lib | |||||
TransferHelper.sol | 0% | 0% | 0% | 0% | 15, 15, 15, 25–26, 26, 26 |
Motivation
Recent coverage reports show a couple uncovered lines in
StorageRent
andFnameResolver
.Change Summary
Add a test for the selector check in
FnameResolver
and the initial price update inStorageRent
.Merge Checklist
Choose all relevant options below by adding an
x
now or at any time before submitting for reviewPR-Codex overview
Focus of the PR:
This PR focuses on adding a new test function
testFuzzResolveRevertsNonAddrFunction
and updating an existing test functiontestInitialPriceUpdate
.Detailed summary:
testFuzzResolveRevertsNonAddrFunction
in theFnameResolver
contract.testInitialPriceUpdate
function in theStorageRent
contract.