Adding unit tests for client/matching/client.go

[sankari165]

What changed?
Unit tests for client/matching/client.go
Renamed peerResolver file
Added mocks for peer resolver and load balancer

Why?
Improve unit tests for the code

How did you test it?
Unit tests

Potential risks

Release notes

Documentation Changes

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.18%. Comparing base (4b2a899) to head (f9a5a04).

Additional details and impacted files

Files	Coverage Δ
client/matching/peer_resolver.go	85.18% <100.00%> (ø)

... and 5 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b2a899...f9a5a04. Read the comment docs.

[jakobht]

LGTM - added a nit comment

[coveralls]

Pull Request Test Coverage Report for Build 018f2eaa-3361-460a-ad32-0c055556257b

Details

• 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
• 51 unchanged lines in 9 files lost coverage.
• Overall coverage increased (+0.02%) to 67.896%

---

Files with Coverage Reduction	New Missed Lines	%
common/task/weighted_round_robin_task_scheduler.go	1	89.05%
service/matching/taskReader.go	2	83.55%
service/history/shard/context.go	2	68.91%
service/matching/taskListManager.go	2	81.16%
common/task/fifo_task_scheduler.go	3	82.47%
service/history/queue/timer_gate.go	3	95.83%
common/persistence/statsComputer.go	3	96.07%
service/history/execution/mutable_state_task_refresher.go	15	59.81%
service/history/task/task_util.go	20	70.57%

Totals
Change from base Build 018f2bf2-19ab-4010-97e7-e281961bcb92:	0.02%
Covered Lines:	99692
Relevant Lines:	146831

---

💛 - Coveralls