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 :(
Same thing here, `!` negation with a space is used all over in this lib (personal style). Why change it?