tests: additional coverage#1261
Conversation
Codecov Report
Flags with carried forward coverage won't be shown. Click here to find out more. |
|
Ah, when seeing this Would be a pity if we miss this opportunity and introduce half-thought-through new options which are hard to revert or change later on. @ryanio maybe you have also got an opinion on this, or @jochem-brouwer might also want to express some final thoughts here since there were still some open questions before. |
holgerd77
left a comment
holgerd77
left a comment
There was a problem hiding this comment.
Nice, looks good, really nice test additions! 😀
| }) | ||
|
|
||
| tape(`${method}: call with no peers`, (t) => { | ||
| const client = createClient({ noPeers: true }) |
There was a problem hiding this comment.
Ah, I actually didn't know how to test this. 😄
2 participants
This PR adds coverage for missed lines of code, mostly gathered from codecov check warnings from #1148.