merge-script 8d55154655
Merge bitcoin/bitcoin#34602: test: addrman: successive failures in the last week for IsTerrible
6202acd284bc0284fc9b144fdc39774f112fcdf2 test: addrman: successive failures in the last week for IsTerrible (brunoerg)
f611d3bdaf035280f44d5183a8322c0a0c6e3d27 refactor: addrman: move consts to .h (brunoerg)

Pull request description:

  This PR adds test coverage for the case that an address is considered terrible if we had N successive failures in the last week.

  It kills the following mutant (https://corecheck.dev/mutation/src/addrman.cpp#L88):
  ```diff
  diff --git a/src/addrman.cpp b/src/addrman.cpp
  index e3981e6a40..f8045491c1 100644
  --- a/src/addrman.cpp
  +++ b/src/addrman.cpp
  @@ -65,7 +65,7 @@ bool AddrInfo::IsTerrible(NodeSeconds now) const
       }

       if (now - m_last_success > ADDRMAN_MIN_FAIL && nAttempts >= ADDRMAN_MAX_FAILURES) { // N successive failures in the last week
  -        return true;
  +        return false;
       }

       return false;

  ```

ACKs for top commit:
  frankomosh:
    re-ACK 6202acd284bc0284fc9b144fdc39774f112fcdf2
  naiyoma:
    tACK 6202acd284bc0284fc9b144fdc39774f112fcdf2
  danielabrozzoni:
    tACK 6202acd284bc0284fc9b144fdc39774f112fcdf2
  sedited:
    ACK 6202acd284bc0284fc9b144fdc39774f112fcdf2

Tree-SHA512: b4736ef91b75ba4c060dc18a2b796aee94d0d8be5ca58b9b873156248cd0dc6910595595e0f56d75bffff4f94c035319ac8428f7d79f07fe685bdba27a188829
2026-03-16 22:29:38 +01:00
..
2026-02-03 11:19:01 +01:00
2026-02-16 16:09:30 -03:00
2025-05-19 16:40:33 +01:00
2026-02-17 21:40:46 +05:30
2026-02-25 14:36:19 -08:00
2025-09-30 11:06:43 -07:00
2026-01-22 10:35:14 -05:00
2026-03-04 15:06:34 +01:00
2025-11-14 01:17:38 +02:00
2026-02-02 17:22:31 +00:00