More rubocop

@localshred follow up from the last PR. This gets us nearly all the way to rubocop-completeness, but more importantly, it makes the tests run again, which I broke in the last PR :(
Why spaces inside hashes but not arrays? The style guide just has some really bizarre and conflicting rules (e.g. `->` for one-liners, `lambda` otherwise)