Right, but with cucumber, tests are insta-fail. So if the `berks install` fails, it never checks the CLI code - instead it fails on the next line where we (typically) match the output to some string. And then the error message isn't "command failed", but rather "failed to match string". It think it's more correct to use successfully than test the exit status. We should only test exit status if it's supposed to be non-zero, IMO