Add unit tests for GetWorkflowExecution

[Shaddoll]

What changed?
Add unit tests for GetWorkflowExecution

Why?
Improve test coverage

How did you test it?
unit tests

Potential risks

Release notes

Documentation Changes

[codecov]

Codecov Report

Merging #5736 (73afe5b) into master (bc5cc93) will increase coverage by 0.06%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

❗ Current head 73afe5b differs from pull request most recent head 2a87b11. Consider uploading reports for the commit 2a87b11 to get more accurate results

Additional details and impacted files

see 12 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 c41bd68...2a87b11. Read the comment docs.

[shijiesheng]

It might be easier to use a mock sql like https://github.com/DATA-DOG/go-sqlmock rather than a gomocked entity.

[coveralls]

Pull Request Test Coverage Report for Build 018e1699-1424-4592-bcbc-1234516091cb

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 26 unchanged lines in 5 files lost coverage.
• Overall coverage increased (+0.06%) to 63.86%

---

Files with Coverage Reduction	New Missed Lines	%
common/task/weighted_round_robin_task_scheduler.go	1	88.06%
tools/cli/admin_db_decode_thrift.go	1	71.79%
service/history/task/timer_task_executor_base.go	2	78.38%
service/history/task/transfer_active_task_executor.go	2	72.38%
service/history/task/task_util.go	20	70.57%

Totals
Change from base Build 018e1694-625a-41b7-8e9f-05b4c8370a15:	0.06%
Covered Lines:	93618
Relevant Lines:	146599

---

💛 - Coveralls