Make restart() inherit timeout and max_retries from run()

Summary

  • When restart() is called without explicit timeout or max_retries parameters, it now inherits the values used in the last run() call
  • This ensures consistent behavior between initial startup and restart

Problem

Previously, restart() defaulted to timeout=30s and max_retries=1, while run() might have been called with different values (e.g., timeout=60s, max_retries=3).

This caused flaky test failures when restart() was used after time jumps that required longer processing time (e.g., escrow tests jumping 4 weeks forward).

Solution

Store _last_timeout and _last_max_retries from run() and use them as defaults in restart() when parameters are None.

Testing

Verified with hive pipeline - escrow_tests and fork_tests all pass.

Merge request reports

Loading