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
It would be usefull to allow the constructor to pass a test console which could be configured externally, I would like to do integration test with interactive test inputs into console but cant do it externally without having to implement my own CommandAppTester since class is sealed. Perhaps just like with a test register we could pass this onto the constructor or maybe use the config console method could be an alternative.
@scarbon I like this idea, it fits within the general paradigm of our development, it keeps the class sealed, and I see the usefulness of it. Unless another maintainer tells me otherwise in the next few days, I would welcome this as a PR. However, please do note that suitable test coverage is required. Tag me when opened and I can review the PR.
It would be usefull to allow the constructor to pass a test console which could be configured externally, I would like to do integration test with interactive test inputs into console but cant do it externally without having to implement my own CommandAppTester since class is sealed. Perhaps just like with a test register we could pass this onto the constructor or maybe use the config console method could be an alternative.
Here we coud use the _console
spectre.console/src/Spectre.Console.Testing/Cli/CommandAppTester.cs
Line 107 in 58bf89a
spectre.console/src/Spectre.Console.Testing/Cli/CommandAppTester.cs
Line 153 in 58bf89a
Woud be happy to raise a PR (if I'm allowed to)
The text was updated successfully, but these errors were encountered: