Skip to content

Commit

Permalink
Merge #6121: test: disable mocktime in p2p_eviction.py
Browse files Browse the repository at this point in the history
764b3a3 test: disable mocktime in p2p_eviction.py (UdjinM6)

Pull request description:

  ## Issue being fixed or feature implemented
  No idea why CI has no issues but `p2p_eviction.py` fails locally after #6103 (my guess is that it's because P2PInterface can't work with mocktime properly).

  ## What was done?
  Disable mocktime in `p2p_eviction.py`

  ## How Has This Been Tested?
  Run tests locally

  ## Breaking Changes
  n/a

  ## Checklist:
  - [x] I have performed a self-review of my own code
  - [ ] I have commented my code, particularly in hard-to-understand areas
  - [ ] I have added or updated relevant unit/integration/functional/e2e tests
  - [ ] I have made corresponding changes to the documentation
  - [ ] I have assigned this pull request to a milestone _(for repository code-owners and collaborators only)_

Top commit has no ACKs.

Tree-SHA512: a9be9032c7697ff47b2256395f0fb126deeccd9bee6f101a71a1f88e1f25b08fa039ed5eb4cd4b1b308e8136d64510a544b7019ed9147ea2e80f8cb83ff25412
  • Loading branch information
PastaPastaPasta committed Aug 1, 2024
2 parents 84179f7 + 764b3a3 commit bf24a2b
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions test/functional/p2p_eviction.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,10 @@ def set_test_params(self):
# 4 by netgroup, 4 that sent us blocks, 4 that sent us transactions and 8 via lowest ping time
self.extra_args = [['-maxconnections=32']]

def setup_network(self):
self.disable_mocktime()
super().setup_network()

def run_test(self):
protected_peers = set() # peers that we expect to be protected from eviction
current_peer = -1
Expand Down

0 comments on commit bf24a2b

Please sign in to comment.