NullMapper no longer makes sense without serilizaion in mappers

Author: mostlyobvious

ruby_event_store-active_record/spec/skip_ar_serialization_spec.rb

@@ -13,7 +13,7 @@ module ActiveRecord
       let(:repository) { mk_repository.call }
       let(:specification) do
         RubyEventStore::Specification.new(
-          RubyEventStore::SpecificationReader.new(repository, RubyEventStore::Mappers::NullMapper.new)
+          RubyEventStore::SpecificationReader.new(repository, RubyEventStore::Mappers::Default.new)
         )
       end
 

ruby_event_store/lib/ruby_event_store.rb

@@ -34,7 +34,6 @@
 require_relative "ruby_event_store/mappers/forgotten_data"
 require_relative "ruby_event_store/mappers/encryption_mapper"
 require_relative "ruby_event_store/mappers/instrumented_mapper"
-require_relative "ruby_event_store/mappers/null_mapper"
 require_relative "ruby_event_store/serializers/yaml"
 require_relative "ruby_event_store/batch_enumerator"
 require_relative "ruby_event_store/correlated_commands"
@@ -44,4 +43,4 @@
 require_relative "ruby_event_store/version"
 require_relative "ruby_event_store/instrumented_repository"
 require_relative "ruby_event_store/instrumented_dispatcher"
-require_relative "ruby_event_store/instrumented_subscriptions"
+require_relative "ruby_event_store/instrumented_subscriptions"

ruby_event_store/lib/ruby_event_store/mappers/null_mapper.rb

@@ -917,8 +917,7 @@ def self.event_type
           end
 
         subscriptions = Subscriptions.new(event_type_resolver: ->(klass) { klass.event_type })
-        client =
-          Client.new(subscriptions: subscriptions)
+        client = Client.new(subscriptions: subscriptions)
         client.subscribe(handler = Proc.new {}, to: [event_klass])
 
         expect(client.subscribers_for(event_klass)).to eq [handler]