Lint/EmptyWhen false positive with inline comments.

[marcandre]

0. Proper bug: the following will detect an offense even with 'AllowComments' => true

case foo
when :bar then # do nothing
end

Note that the following is rightfully accepted

case foo
when :bar
  # do nothing
end

The spec file has multiple issues and is in need of love:

1. Explicit testing of AllowComments is minimal, should be expanded
2. Doesn't use except_offense/no_offense so spec failures are next to impossible to locate.
3. The default is AllowComment: true, but the :config spec will not load the defaults unless given there is a cop_config method. Add let(:cop_config) { {} } to the current spec and it fails (related to Improved shared context. Refactor & simplify some specs. #7970)

[Darhazer]

Should

case foo
when :bar # do nothing
end

also be accepted?