Move rb-site SiteList managmeent into the command classes.
Review Request #11363 — Created Jan. 8, 2021 and submitted — Latest diff uploaded
As part of a cleanup effort leading toward modern argument parsing
support, this change moves some of theSiteListparsing/management
code out ofmain()and into each individual command class. That avoids
the mixed logic we had before, simplifyingmain(), and enables
testability.Unit tests were added, as part of an effort to begin building a test
suite forrb-site.
Unit tests pass.
Tested each command with valid and invalid site pathhs.
Tested the upgrade command with
--all-sites.