admin_db_clean_cmd test Part II

[bowenxia]

What changed?

• Add more tests for admin_db_clean_cmd
• Refactored initializeInvariantManager into ManagerFactory as well, since it is inside of fixExeuction. Increased testability
• Renamed PersistenceManagerFactory to be ManagerFactory since initializeInvariantManager is not a persistence manager.

Why?
code coverage

How did you test it?
unit test

Potential risks

Release notes

Documentation Changes

[codecov]

Codecov Report

Attention: Patch coverage is 52.50000% with 19 lines in your changes missing coverage. Please review.

Project coverage is 77.76%. Comparing base (5b096d1) to head (aa27130).

Files with missing lines	Patch %	Lines
tools/cli/database.go	0.00%	8 Missing ⚠️
tools/cli/app.go	45.45%	4 Missing and 2 partials ⚠️
tools/cli/admin_db_scan_command.go	0.00%	2 Missing ⚠️
tools/cli/admin_commands.go	0.00%	1 Missing ⚠️
tools/cli/admin_db_clean_command.go	94.11%	1 Missing ⚠️
tools/cli/admin_timers.go	0.00%	1 Missing ⚠️

Additional details and impacted files

Files with missing lines	Coverage Δ
tools/cli/admin_commands.go	10.62% <0.00%> (ø)
tools/cli/admin_db_clean_command.go	85.98% <94.11%> (+56.38%)	⬆️
tools/cli/admin_timers.go	0.00% <0.00%> (ø)
tools/cli/admin_db_scan_command.go	0.00% <0.00%> (ø)
tools/cli/app.go	91.24% <45.45%> (-2.48%)	⬇️
tools/cli/database.go	36.73% <0.00%> (-0.26%)	⬇️

... and 8 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 5b096d1...aa27130. Read the comment docs.