add Copy() method to configstore structs

[mantas-sidlauskas]

What changed?
Helper functions copy* are replaced with struct methods

Why?
Better reusability

How did you test it?
Added unit tests

Potential risks

Release notes

Documentation Changes

[coveralls]

Pull Request Test Coverage Report for Build 01907298-99db-4403-a675-2e4158bcca0f

Details

• 41 of 41 (100.0%) changed or added relevant lines in 2 files are covered.
• 41 unchanged lines in 14 files lost coverage.
• Overall coverage remained the same at 71.522%

---

Files with Coverage Reduction	New Missed Lines	%
service/history/queue/timer_queue_processor_base.go	1	77.66%
common/task/weighted_round_robin_task_scheduler.go	2	89.05%
common/task/parallel_task_processor.go	2	93.06%
service/matching/tasklist/db.go	2	73.23%
common/persistence/sql/sqlplugin/mysql/task.go	2	73.68%
service/matching/tasklist/task_writer.go	2	82.21%
common/persistence/sql/sqlplugin/mysql/db.go	2	79.49%
common/persistence/historyManager.go	2	66.67%
common/log/tag/tags.go	3	49.64%
service/history/task/timer_standby_task_executor.go	3	85.63%

Totals
Change from base Build 0190705f-c927-4d1b-b54d-772b615f64e2:	0.0%
Covered Lines:	105309
Relevant Lines:	147241

---

💛 - Coveralls

[Shaddoll]

Do you mind adding tests verifying that changing the copy doesn't affect the original?

[mantas-sidlauskas]

good idea! Will do