refactor: Improve rollback on peer P2P collection error

[fredcarle]

Relevant issue(s)

Resolves #1389

Description

This PR improves the rollback of P2P collection manipulation errors. It ensures that a rollback is applied if the transaction commit fails.

Tasks

• I made sure the code is well commented, particularly hard-to-understand areas.
• I made sure the repository-held documentation is changed accordingly.
• I made sure the pull request title adheres to the conventional commit style (the subset used in the project can be found in tools/configs/chglog/config.yml).
• I made sure to discuss its limitations such as threats to validity, vulnerability to mistake and misuse, robustness to invalidation of assumptions, resource requirements, ...

How has this been tested?

Unit test coming with net package code coverage improvements.

[codecov]

Codecov Report

Merging #1461 (ca18f6e) into develop (b6bd97a) will increase coverage by 0.22%.
The diff coverage is 40.47%.

❗ Current head ca18f6e differs from pull request most recent head 4ae9d3e. Consider uploading reports for the commit 4ae9d3e to get more accurate results

@@             Coverage Diff             @@
##           develop    #1461      +/-   ##
===========================================
+ Coverage    72.03%   72.25%   +0.22%     
===========================================
  Files          185      185              
  Lines        18224    18226       +2     
===========================================
+ Hits         13127    13169      +42     
+ Misses        4052     4021      -31     
+ Partials      1045     1036       -9     

Impacted Files	Coverage Δ
net/peer.go	50.32% <40.47%> (+2.46%)	⬆️

... and 9 files with indirect coverage changes

[AndrewSisley]

Looks better - cheer Fred, I think there are still a couple of holes here, but they are harder to plug and can perhaps be done later :)