refactor pinot_visibility_store_test

[bowenxia]

What changed?
refactor pinot_visibility_store_test.
factored out mocks and pass those in as table test params.

Why?
refactor pinot_visibility_store_test to follow a better practice just like this one: #5777

How did you test it?
unit test

Potential risks

Release notes

Documentation Changes

[codecov]

Codecov Report

Merging #5780 (65265bf) into master (c6a10dc) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files

see 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 c6a10dc...65265bf. Read the comment docs.

[coveralls]

Pull Request Test Coverage Report for Build 018e3544-0501-42f8-be95-9f1ebe44886f

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 80 unchanged lines in 19 files lost coverage.
• Overall coverage increased (+0.02%) to 64.88%

---

Files with Coverage Reduction	New Missed Lines	%
common/task/weighted_round_robin_task_scheduler.go	1	88.06%
service/history/task/transfer_standby_task_executor.go	2	86.19%
common/persistence/sql/sqlplugin/postgres/task.go	2	73.4%
common/persistence/sql/sqlplugin/postgres/db.go	2	80.0%
client/history/client.go	2	35.78%
service/history/handler/handler.go	2	49.59%
service/matching/db.go	2	73.23%
service/history/task/transfer_active_task_executor.go	2	72.38%
common/util.go	2	91.69%
common/membership/hashring.go	2	82.23%

Totals
Change from base Build 018e341f-2cdc-43b3-a7ef-6d5189e3d4cd:	0.02%
Covered Lines:	94589
Relevant Lines:	145791

---

💛 - Coveralls