Connection manager consolidation

[everoddandeven]

• Consolidate connection manager logic
• Add DockerWalletRpcManager
• Enable TestMoneroConnectionManager
• Setup another docker wallet rpc instance
• Other minor fixes

[codacy-production]

Not up to standards ⛔

🔴 Issues 1 high · 3 medium · 1 minor

Alerts:
⚠ 5 issues (≤ 0 issues of at least minor severity)

Results:
5 new issues

Category	Results
Security	1 high
CodeStyle	1 minor
Complexity	3 medium

View in Codacy

🟢 Metrics 16 complexity · -9 duplication

Metric	Results
Complexity	16
Duplication	-9

View in Codacy

🟢 Coverage 93.00% diff coverage · +3.13% coverage variation

Metric	Results
Coverage variation	✅ +3.13% coverage variation (-1.00%)
Diff coverage	✅ 93.00% diff coverage

View coverage diff in Codacy

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (ae4658b)	18142	15589	85.93%
Head commit (03586e2)	18314 (+172)	16310 (+721)	89.06% (+3.13%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#79)	657	611	93.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

_{TIP This summary will be updated as you push new changes. Give us feedback}