Order by timestamp (#35)

* Always order events by timestamp

* Add required class constraint to metadata type

Without this, EF fails with an exception when it tries to cast from
TEventMetadata to IEventMetadata<TStreamId>. I'm still not sure why
EF is happier this way, but hey - it seems to work...

src/RdbmsEventStore.EntityFramework.Tests/Infrastructure/EventStoreFixture.cs

@@ -12,7 +12,7 @@ public class EventStoreFixture<TId, TStreamId, TEvent, TEventMetadata, TPersiste
         where TStreamId : IEquatable<TStreamId>
         where TEvent : class, TEventMetadata, IMutableEvent<TStreamId>, new()
         where TPersistedEvent : class, TEventMetadata, IPersistedEvent<TStreamId>, new()
-        where TEventMetadata : IEventMetadata<TStreamId>
+        where TEventMetadata : class, IEventMetadata<TStreamId>
     {
         public EventStoreFixture()
         {

src/RdbmsEventStore.EntityFramework.Tests/Infrastructure/EventStoreTestBase.cs

@@ -8,7 +8,7 @@ namespace RdbmsEventStore.EntityFramework.Tests.Infrastructure
     public class EventStoreTestBase<TId, TStreamId, TEvent, TEventMetadata, TPersistedEvent> : IClassFixture<EventStoreFixture<TId, TStreamId, TEvent, TEventMetadata, TPersistedEvent>>, IDisposable
         where TId : IEquatable<TId>
         where TStreamId : IEquatable<TStreamId>
-        where TEventMetadata : IEventMetadata<TStreamId>
+        where TEventMetadata : class, IEventMetadata<TStreamId>
         where TEvent : class, TEventMetadata, IMutableEvent<TStreamId>, new()
         where TPersistedEvent : class, TEventMetadata, IPersistedEvent<TStreamId>, new()
     {

src/RdbmsEventStore.EntityFramework/EntityFrameworkEventStore.cs

@@ -13,7 +13,7 @@ public class EntityFrameworkEventStore<TId, TStreamId, TContext, TEvent, TEventM
         where TContext : DbContext, IEventDbContext<TPersistedEvent>
         where TEvent : class, TEventMetadata, IMutableEvent<TStreamId>, new()
         where TPersistedEvent : class, IPersistedEvent<TStreamId>, TEventMetadata, new()
-        where TEventMetadata : IEventMetadata<TStreamId>
+        where TEventMetadata : class, IEventMetadata<TStreamId>
     {
         private readonly TContext context;
         private readonly IEventFactory<TStreamId, TEvent> _eventFactory;
@@ -35,6 +35,7 @@ public async Task<IEnumerable<TEvent>> Events(Func<IQueryable<TEventMetadata>, I
             var storedEvents = await context.Events
                 .AsNoTracking()
                 .Apply(query)
+                .OrderBy(e => e.Timestamp)
                 .ToListAsync();
 
             var events = storedEvents
@@ -52,7 +53,9 @@ public async Task<IEnumerable<TEvent>> Events(TStreamId streamId, Func<IQueryabl
                     .Where(e => e.StreamId.Equals(streamId))
                     .AsNoTracking()
                     .Apply(query)
+                    .OrderBy(e => e.Timestamp)
                     .ToListAsync();
+
             var events = storedEvents
                 .Cast<TPersistedEvent>()
                 .Select(_serializer.Deserialize);