Mutation Tests testnet/stacks-node/src/neon_node.rs > process_new_tenures, can_run_microblock_tenure

Related #4587

The mutation testing highlighted there are missing unit tests for process_new_tenures.

The steps for making sure the process_new_tenures function has its possible outcomes caught are:

• remove the skip header #[cfg_attr(test, mutants::skip)]
• check if mutation tests are caught for these cases:
  • run

  RUST_BACKTRACE=1 BITCOIND_TEST=1 cargo mutants -vV -F "replace RelayerThread::process_new_tenures" -E ": replace .{1,2} with .{1,2} in " --baseline=skip --test-tool=nextest -- --run-ignored all --no-fail-fast --test-threads 1
  

  • case returning true
  • returned returning false
• create tests if they aren't caught

Same steps for the can_run_microblock_tenure function

RUST_BACKTRACE=1 BITCOIND_TEST=1 cargo mutants -vV -F "replace RelayerThread::can_run_microblock_tenure" -E ": replace .{1,2} with .{1,2} in " --baseline=skip --test-tool=nextest -- --run-ignored all --no-fail-fast --test-threads 1

Start in testnet/stacks-node/src/neon_node.rs and inspect RelayerThread::process_new_tenures and can_run_microblock_tenure. Run the two cargo mutants commands from the issue to check whether true and false outcomes are caught. Remove the test skip attribute and add tests where surviving mutations remain; done means the relevant mutations are caught.