Mixing automate changes with manual ones makes this basically impossible to review. You need to split this up.
Tue, Jul 7
There is a missing lock when invalidating a block - and presumably parking too.
Mon, Jul 6
Good start, but you needlessly complicated yourself for the tests.
Sun, Jul 5
Flatten things such as one multi_index is used and all nodes are in it.
Simplify the Peer struct so that it doesn't contain nodes. Nodes wills be stored in their own datastructure later on.
There is a better way to do this using multi index magic.
Sat, Jul 4
Also do AVALANCHE_TIME_STEP_MILLISECONDS