Capybara/CurrentPathExpectation autocorrect incompatible with Style/TrailingComma autocorrect

[izzergh]

rubocop version: 1.18.3
rubocop-rspec version: 2.4.0

Example code:

# original code
expect(current_path).to eq(
  some_path(
    record_id: record.id
  )
)

If Style/TrailingComma (in vanilla rubocop) is enabled for methods, it adds a comma after record.id and after the closed parenthesis.

Also, Capybara/CurrentPathExpectation adds , ignore_query: true to the have_current_path params. If both are fixed with the same autocorrect, the following is the result:

# after autocorrect
expect(page).to have_current_path(
  some_path(
    record_id: record.id
  ),, ignore_query: true
)

which is a syntax error (the double-comma) and still doesn't have the trailing comma.

A workaround is to run the Style cops first, then the RSpec cops, or vice-versa - just not in the same command

[pirj]

Thanks for reporting. Would you like to submit a PR?
I see two ways to resolving this:

1. check if expectation_last_child contains a trailing comma
2. strip expectation_last_child from its trailing comma with what RangeHelp provides (not sure if it's possible just by glancing at its code)