Thanks. Personally, I would prefer
```erlang
    case rebar_utils:sh(Cmd ++ Output, [{env,[{"TESTDIR", TestDir}]},
                                        return_on_error]) of
```