Codecov Report

Merging #6742 (cd58853) into master (e14d583) will increase coverage by 0.01%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

hi @zasweq @dfawley, I have a query about @easwars's comment to modify the tests in listener_wrapper_test.go:
the channels created by NewListenerWrapper() are directed "receive-only" channels, unlike the undirected channel stored in the testutils.Channel{} struct

is the expectation here to replace all the directed channels with an undirected one from testutils.Channel{}?

kindly let me know, thank you

I believe @easwars's comment just talking about how it may be able to be used more generally if the channel was exposed for direct access. Not that we must also update all the existing tests.

I would say for the purposes of the issue, this PR is sufficient. If you want to send follow-up PRs to make other existing tests use the testutils.Channel to simplify them, then that would be nice, too. Replacing a read-only channel with something bidirectional is fine since these are just tests.