RedundantBegin cop autocorrect modifies whitespace formatting

[KurtPreston]

I've noticed that the Style/RedundantBegin cop sometimes has two undesirable actions:

1. It sometimes removes blank lines
2. It sometimes causes unusual indentation, i.e. indenting the raise clause by a single space.

The following rspec test demonstrates both of these anomalies. This spec could be added to spec/rubocop/cop/style/redundant_begin_spec.rb:

  it "doesn't modify spacing when auto-correcting" do
    src = """
    def method
      begin
        BlockA do |strategy|
          foo
        end

        BlockB do |portfolio|
          foo
        end

      rescue => e
        bar
      end
    end
    """

    result_src = """
    def method
      BlockA do |strategy|
        foo
      end

      BlockB do |portfolio|
        foo
      end

    rescue => e
      bar
    end
    """
    new_source = autocorrect_source(cop, src)
    expect(new_source).to eq(result_src)
  end

[KurtPreston]

@yous Damn that was fast. Thanks!