Fix errors on callbacks #486

[jonian]

Fixes #486

[AaronLasseigne]

Can you add a test for this?

[jonian]

Yes, I can write a test for this. I have also updated issue #486 with more info and examples to explain the issue better. Is the test below ok?

context 'with block not called and error added in execute around callback' do
  class CheckExecuteAroundCallbackForFailure
    include ActiveInteraction::ActiveModelable
    include ActiveInteraction::Runnable

    set_callback :execute, :around, -> { errors.add(:base, 'invalid') }

    def execute
      true
    end
  end

  it 'is invalid' do
    outcome = CheckExecuteAroundCallbackForFailure.run
    expect(outcome).to_not be_valid
  end
end

[AaronLasseigne]

Looks good.

[jonian]

Test added. If there anything else I should do, please let me know.

[AaronLasseigne]

Thanks for the contribution!