We should remove the mocks for stdout and stderr entirely. These sorta made sense when we were using popen4 because popen4 yields IO objects (though I personally prefer to use StringIO objects for that case because the API is consistent with other IO objects automatically), but now that we've switched to shell_out, stdout and stderr are just strings, and it's pretty silly to use mocks for strings.