DefaultApplyStrategy for AggregateRoot fails when using protobuf

[obahareth]

When you use Protobuf for your events along with AggregateRoot, the AggregateRoot default apply strategy tries to call a handler called apply_proto because it isn't reading the event class (It's reading event.class instead of event.data.class in the case of protobuf). It seems like we need a different apply strategy for protobuf.

The examples below all work if you use the default events and YAML, but when switching to Protobuf they don't.

Sample output

AggregateRoot::MissingHandler (Missing handler method apply_proto on aggregate MyAggregate)

Sample aggregate

class MyAggregate
  include AggregateRoot

  def initialize
    @amount = 0
  end

  def add_amount(amount)
    @amount += amount
  end

  on BalanceAdded do |event|
    add_amount(event.data.fetch(:amount, 0))
  end


  private

  attr_reader :amount
end

[paneq]

@obahareth Nice catch. This might fix it #579 , could you give it a try? It's just PoC.

[obahareth]

#579 does fix it, thanks!

[mostlyobvious]

Continued in #582

[mostlyobvious]

Fix has been released in v0.40.0: https://github.com/RailsEventStore/rails_event_store/releases/tag/v0.40.0