Closed ellnix closed 1 month ago
I don't know if the
unable_to_run
andget_result
functions are particularly useful as they can only ever be called from one place but it's whatever.
The purpose was more to make each function easier to skim + group and document what those lines of code were doing.
The behavior change in
--timeout
needs fixing though.
Done, sorry about that.
run
function tolazy_run
lazy_run
App
may need to be in the library and that these tests cannot run in CI because they depend on the clash dir, which inspired #72SuiteRun
and replaced it with a call tomap()
:facepalm:App.run
slightlyRegarding
App
I suggest
App
to libApp
App
and just having them as regular functions inmain.rs
(or as part of aCli
type)App
constructor to take a singlepackage_name
parameter:This would then separate the responsibility between calling the commands and the helper functions which we could use in tests. It would also prepare for #61 by allowing configuration to be stored in
App
and be used by users of the lib.