Mocking Shell in unit tests is not possible

[alexeyshockov]

Mocking Shell in unit tests is not possible

[madelson]

Hi @alexeyshockov thanks for your interest in the library.

It’s true that we don’t directly support mocking today and that the API isn’t terribly mock-friendly.

One concern I have with providing mocking at the shell level is that truly faking the behavior of a process with all the async things going on, respecting the various start info flags, etc would be very difficult.

When I’ve tested components that leverage this library, I prefer to mock at a higher level so that I only have to replicate the subset of the behavior that the particular usage cares about. For example, I will often wrap the shell invocation in a virtual function with a more straightforward/less-flexible API and the mock that function.

Does that resonate?