fix(#4178): nested versioned events now dispatch to correct projection apply overload

[rd-wm]

Closes #4178

EventNamingStyle.ClassicTypeName (default option) derives the type alias from the simple class name only. Both V1.CustomerCreated and V2.CustomerCreated collapse to "customer_created", so only one CLR type can occupy the _byEventName alias cache in EventGraph.

From EventGraph.cs, line 72:
_byEventName.OnMissing = name => AllEvents().FirstOrDefault(x => x.EventTypeName == name);

EventDocumentStorage.Resolve() / ResolveAsync() only fall back to reading the dotnet_type column (index 2) when the alias lookup returns null. Because the alias always matches something (the cache winner), dotnet_type was never consulted.

--

Really, my project should have used opts.Events.EventNamingStyle = EventNamingStyle.SmarterTypeName; but switching to that now would require modifying the mt_events table, which I don't feel is ideal. I suppose there may be an argument that SmarterTypeName should be default? (https://martendb.io/events/storage#event-type-names).

[jeremydmiller]

I think that's fine, but let CI run on it now

[jeremydmiller]

Yeah, sorry, no. That causes some regression errors.

[rd-wm]

Yeah, sorry, no. That causes some regression errors.

Ah! Leave it with me, the tests passed locally but maybe they weren't all discovered

[rd-wm]

Yeah, sorry, no. That causes some regression errors.

Should be satisfactory now, I hope!