Don't use `runner.invoke` in tests without checking the result

**Motivation**
`CliRunner.invoke` can fail silently, so we should include likes like this:

```
result = runner.invoke(app, ...)
assert not result.exit_code, result.stdout
```

**Summary of desired enhancement**
Add these checks where relevant.