chore: Test Improvements

[glevco]

• Add ability to pass a HathorSettings object to CliBuilder.create_manager() to help mocking settings on tests.
• Change way filepath is constructed on _load_peer_id_pool(), so it's calculated. Before, with the hardcoded path, running tests from any directory other then the project root would cause errors. This prevented running specific test files from an IDE.

Depends on #520.

EDIT: I have rolled back the first change as I found a better way to do it and it's not necessary anymore.

[codecov]

Codecov Report

Merging #521 (a5b432e) into dev (6a8f51e) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #521      +/-   ##
==========================================
+ Coverage   82.81%   82.83%   +0.02%     
==========================================
  Files         198      198              
  Lines       18142    18142              
  Branches     2564     2564              
==========================================
+ Hits        15024    15028       +4     
+ Misses       2554     2551       -3     
+ Partials      564      563       -1     

Impacted Files	Coverage Δ
hathor/p2p/node_sync.py	84.46% <0.00%> (ø)
hathor/p2p/manager.py	68.22% <0.00%> (+1.33%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[jansegre]

I think this won't work quite as expected. I'll talk to you to see a way around it. Anyway, I think the change proposed on this PR shouldn't be merged.

[glevco]

I think this won't work quite as expected. I'll talk to you to see a way around it. Anyway, I the change proposed on this PR shouldn't be merged.

@jansegre could you please re-review it? I have rolled back the HathorSettings change as I found a better way to do it and it's not necessary anymore.