don’t call whoami so many times

This patch just uses the `USER` environment variable instead of shelling out to whoami all the time.

* We're getting sporadic out of memory issues on our server. While I don't think calling `whoami` a ton of times is causing this issue, it is aggravating it.
* This no longer fails tests when `whoami` fails.
* This no longer prints blank lines when `whoami` fails
* This leverages `ENV["USER"]` instead of calling `whoami`.
* The performance increases I got here were less than the margin for error on timing.

Thanks

---

If you would prefer not to cache this value in `@user`, then I can do that, too.