From 1b02e36595c50792f7051bc6577c985ff11aa1c4 Mon Sep 17 00:00:00 2001
From: Tomas Lycken <tlycken@users.noreply.github.com>
Date: Wed, 1 Nov 2017 13:02:28 +0100
Subject: [PATCH] Fix build warning (#38)

* Add task extensions to enumerate IEnumerable<T>s

* Fix build warning by checking count as well

* Add NuGet metadata to new package
---
 .../EventStoreTests/ExtraMetaTests.cs         |  6 ++++-
 src/tlycken.Extensions/TaskExtensions.cs      | 22 +++++++++++++++++++
 .../tlycken.Extensions.csproj                 | 13 +++++++++++
 3 files changed, 40 insertions(+), 1 deletion(-)
 create mode 100644 src/tlycken.Extensions/TaskExtensions.cs

diff --git a/src/RdbmsEventStore.EntityFramework.Tests/EventStoreTests/ExtraMetaTests.cs b/src/RdbmsEventStore.EntityFramework.Tests/EventStoreTests/ExtraMetaTests.cs
index f3f97d3..f2ffd1f 100644
--- a/src/RdbmsEventStore.EntityFramework.Tests/EventStoreTests/ExtraMetaTests.cs
+++ b/src/RdbmsEventStore.EntityFramework.Tests/EventStoreTests/ExtraMetaTests.cs
@@ -64,7 +64,11 @@ public async Task ReturnsEventsAccordingToQuery(string streamId, long expectedCo
         public async Task ReturnsEventsWithMetadata(string streamId, long expectedCount)
         {
             var store = _fixture.BuildEventStore(_dbContext) as IEventStore<string, ExtraMetaStringEvent, IExtraMeta>;
-            var events = await store.Events(streamId, es => es.Where(e => e.ExtraMeta.StartsWith("Foo")));
+            var events = await store
+                .Events(streamId, es => es.Where(e => e.ExtraMeta.StartsWith("Foo")))
+                .ToReadOnlyCollection();
+
+            Assert.Equal(expectedCount, events.Count);
             Assert.All(events, @event => Assert.StartsWith("Foo", @event.ExtraMeta));
         }
 
diff --git a/src/tlycken.Extensions/TaskExtensions.cs b/src/tlycken.Extensions/TaskExtensions.cs
new file mode 100644
index 0000000..622a4b4
--- /dev/null
+++ b/src/tlycken.Extensions/TaskExtensions.cs
@@ -0,0 +1,22 @@
+using System.Collections.Generic;
+using System.Linq;
+
+// ReSharper disable once CheckNamespace
+namespace System.Threading.Tasks
+{
+    public static class TaskExtensions
+    {
+        /// <summary>
+        /// Appends a continuation to the task, which iterates the resulting <see cref="IEnumerable{T}"/> into a <see cref="List{T}"/>.
+        /// </summary>
+        public static Task<List<T>> ToList<T>(this Task<IEnumerable<T>> task)
+            => task.ContinueWith(t => t.Result.ToList());
+
+        /// <summary>
+        /// Appends a continuation to the task, which iterates the resulting <see cref="IEnumerable{T}"/> into a <see cref="List{T}"/>
+        /// and returns it as an <see cref="IReadOnlyCollection{T}"/>.
+        /// </summary>
+        public static Task<IReadOnlyCollection<T>> ToReadOnlyCollection<T>(this Task<IEnumerable<T>> task)
+            => task.ContinueWith(t => t.Result.ToList() as IReadOnlyCollection<T>);
+    }
+}
\ No newline at end of file
diff --git a/src/tlycken.Extensions/tlycken.Extensions.csproj b/src/tlycken.Extensions/tlycken.Extensions.csproj
index 9f5c4f4..343dd42 100644
--- a/src/tlycken.Extensions/tlycken.Extensions.csproj
+++ b/src/tlycken.Extensions/tlycken.Extensions.csproj
@@ -2,6 +2,19 @@
 
   <PropertyGroup>
     <TargetFramework>netstandard2.0</TargetFramework>
+
+    <Version>0.0.0</Version>
+    <PackageVersion>0.0.0</PackageVersion>
+
+    <Title>tlycken.Extensions</Title>
+    <Description>Extensions for the .NET Standard Library</Description>
+    <Authors>Tomas Lycken (@tlycken)</Authors>
+    <Copyright>2017 Tomas Lycken</Copyright>
+    <PackageProjectUrl>https://github.com/tlycken/RdbmsEventStore</PackageProjectUrl>
+    <RepositoryUrl>https://github.com/tlycken/RdbmsEventStore</RepositoryUrl>
+    <PackageLicenseUrl>https://github.com/tlycken/RdbmsEventStore/blob/master/LICENSE.md</PackageLicenseUrl>
+    <IncludeSymbols>True</IncludeSymbols>
+    <IncludeSource>True</IncludeSource>
   </PropertyGroup>
 
 </Project>