diff options
| author | laanwj <126646+laanwj@users.noreply.github.com> | 2022-06-28 15:09:55 +0200 |
|---|---|---|
| committer | laanwj <126646+laanwj@users.noreply.github.com> | 2022-06-28 15:10:00 +0200 |
| commit | 55c9e2d790fa2e137ccd0d91e6cf3e2d0bff4813 (patch) | |
| tree | 9d8d24f90f80954a53134582cb42b0f1bae6cfca /src/test/util | |
| parent | ba29911e21c88f49780c6c87f94ff8ed6e764a9d (diff) | |
| parent | b2733ab6a85b234a88b83bdc77a0d043e18385b3 (diff) | |
| download | bitcoin-55c9e2d790fa2e137ccd0d91e6cf3e2d0bff4813.tar.xz | |
Merge bitcoin/bitcoin#24378: refactor: make bind() and listen() mockable/testable
b2733ab6a85b234a88b83bdc77a0d043e18385b3 net: add new method Sock::Listen() that wraps listen() (Vasil Dimov)
3ad7de225efce3e76530f56bee8a8f7a75ea0f3c net: add new method Sock::Bind() that wraps bind() (Vasil Dimov)
Pull request description:
_This is a piece of #21878, chopped off to ease review._
Add new methods `Sock::Bind()` and `Sock::Listen()` that wrap `bind()` and `listen()`.
This will help to increase `Sock` usage and make more code mockable.
ACKs for top commit:
pk-b2:
ACK b2733ab6a85b234a88b83bdc77a0d043e18385b3
laanwj:
Code review ACK b2733ab6a85b234a88b83bdc77a0d043e18385b3
Tree-SHA512: c6e737606703e2106fe60cc000cfbbae3a7f43deadb25f70531e2cac0457e0b0581440279d14c76c492eb85c12af4adde52c30baf74542c41597e419817488e8
Diffstat (limited to 'src/test/util')
| -rw-r--r-- | src/test/util/net.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/test/util/net.h b/src/test/util/net.h index 03a99648b7..c5dbaeca3e 100644 --- a/src/test/util/net.h +++ b/src/test/util/net.h @@ -122,6 +122,10 @@ public: int Connect(const sockaddr*, socklen_t) const override { return 0; } + int Bind(const sockaddr*, socklen_t) const override { return 0; } + + int Listen(int) const override { return 0; } + std::unique_ptr<Sock> Accept(sockaddr* addr, socklen_t* addr_len) const override { if (addr != nullptr) { |