TrailingComma still registers an offense when it shouldn't

[jfelchner]

I noticed a bug from the fix that @jonas054 did in #1075. Unfortunately this is still registering an offense:

scout.throttle!(status:     'realtime',
                over_limit: true)

Even when I have the config set like so:

TrailingComma:
  EnforcedStyleForMultiline: 'comma'

I may be wrong, but it seems like params to a method call may need to be handled differently to just a regular hash since:

scout.throttle! status:     'realtime',
                over_limit: true

Cannot have a trailing ,, even though there's no ) on that last line, otherwise it results in a parse error.

Here's the failing spec to help you out:

    context 'when EnforcedStyleForMultiline is comma' do
      it 'accepts a method call with Hash as last parameter split on multiple lines' do
        inspect_source(cop, ['some_method(a: "b",',
                             '            c: "d")'])
        expect(cop.offenses).to be_empty
      end
    end

[jonas054]

Confirmed. There's a bug in the handling of hash parameters without braces. I'll fix it.

[jfelchner]

@jonas054 you're amazing! ❤️