First of all, we move the spec file from `spec` to `spec/helpers`. This is where
it is supposed to be.
Next, we add tests for all the methods that the module defines. During this
process I had to change `module_function` to `extend self`. Rubocop doesn't like
it for some unknown to me reason, so I had to disable the rule. There's no harm
in doing so.
Finally, I refactored some methods (low-hanging fruits only) and discovered that
the `command_error` method is not necessary at all.
All in all, this module is a lot better now but I feel like it shouldn't exist
at all, since almost all methods are very specific to certain Pry commands. It's
hardly a general purpose module for Pry plugins.
This cop makes sense to me, however a lot of our code has offences that are not
fixable on the same level. To avoid them we'd need to redesign API of some
classes, which isn't worth it.
We plan to use SimpleCov and Bundler.require gets in the way. Without it Gemfile
groups are useless, so I am removing them.
I am also removing the silly `Bundler/OrderedGems` cop, one of the most useless
cops you can add.
I realise that some code might be less readable now, but now that we set a good
default limit, we protect the codebase from further mess. It's important to do
this to prevent adding more mess to already messy code that we have. :doctor:
Douglas Eichelberger