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 theSiteList
parsing/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
.