You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The sole purpose of the RunCommand tester is to, as the name implies, run an arbitrary command. This capability already exists with the command parameter in the RunApp tester. An investigation should be conducted to assess the feasibility of potential actions to reduce code duplication. Options are:
Get ride of RunCommand completely, replace its usage with RunApp.
Modify RunApp so it inherits from RunCommand and does not have to implement its own command parameter.
...
If the RunCommand tester is kept, it should be updated to support all options from the class it is derived from (currently Tester). For instance, Tester has the cli_args parameter, but setting it in a RunCommand test has no effect. If a parent parameter cannot be supported, it should be moved to a more derived class or deleted in the class that can't support it.
The text was updated successfully, but these errors were encountered:
The sole purpose of the
RunCommand
tester is to, as the name implies, run an arbitrary command. This capability already exists with thecommand
parameter in theRunApp
tester. An investigation should be conducted to assess the feasibility of potential actions to reduce code duplication. Options are:RunCommand
completely, replace its usage withRunApp
.RunApp
so it inherits fromRunCommand
and does not have to implement its owncommand
parameter.If the
RunCommand
tester is kept, it should be updated to support all options from the class it is derived from (currentlyTester
). For instance,Tester
has thecli_args
parameter, but setting it in aRunCommand
test has no effect. If a parent parameter cannot be supported, it should be moved to a more derived class or deleted in the class that can't support it.The text was updated successfully, but these errors were encountered: