Remove Unspeakable Type Workaround (#1843)

dotnet/aspnetcore#47548 has now been fixed. Removing the workaround that was added for it.

Author: eerhardt

src/BenchmarksApps/TodosApi/DataExtensions.cs

@@ -144,21 +144,6 @@ public static Task<NpgsqlDataReader> QueryAsync(this NpgsqlCommand command, Canc
     public static Task<NpgsqlDataReader> QueryAsync(this NpgsqlCommand command, CommandBehavior commandBehavior, CancellationToken cancellationToken = default)
         => command.ExecuteReaderAsync(commandBehavior, cancellationToken);
 
-    public static Task<List<T>> ToListAsync<T>(this IAsyncEnumerable<T> enumerable, CancellationToken cancellationToken)
-        => ToListAsync(enumerable, null, cancellationToken);
-
-    public static async Task<List<T>> ToListAsync<T>(this IAsyncEnumerable<T> enumerable, int? initialCapacity = null, CancellationToken cancellationToken = default)
-    {
-        var list = initialCapacity.HasValue ? new List<T>(initialCapacity.Value) : new List<T>();
-
-        await foreach (var item in enumerable.WithCancellation(cancellationToken))
-        {
-            list.Add(item);
-        }
-
-        return list;
-    }
-
     public static NpgsqlParameterCollection AddTyped<T>(this NpgsqlParameterCollection parameters, T? value)
     {
         parameters.Add(new NpgsqlParameter<T>

src/BenchmarksApps/TodosApi/TodoApi.cs

@@ -13,18 +13,16 @@ public static RouteGroupBuilder MapTodoApi(this IEndpointRouteBuilder routes)
 
         group.AddValidationFilter();
 
-        // BUG: Having to call ToListAsync() on query results until JSON support for unspeakable types (https://github.com/dotnet/aspnetcore/issues/47548) is resolved
-
         group.MapGet("/", (NpgsqlDataSource db, CancellationToken ct) =>
-            db.QueryAsync<Todo>("SELECT * FROM Todos", ct).ToListAsync(ct))
+            db.QueryAsync<Todo>("SELECT * FROM Todos", ct))
             .WithName("GetAllTodos");
 
         group.MapGet("/complete", (NpgsqlDataSource db, CancellationToken ct) =>
-            db.QueryAsync<Todo>("SELECT * FROM Todos WHERE IsComplete = true", ct).ToListAsync(ct))
+            db.QueryAsync<Todo>("SELECT * FROM Todos WHERE IsComplete = true", ct))
             .WithName("GetCompleteTodos");
 
         group.MapGet("/incomplete", (NpgsqlDataSource db, CancellationToken ct) =>
-            db.QueryAsync<Todo>("SELECT * FROM Todos WHERE IsComplete = false", ct).ToListAsync(ct))
+            db.QueryAsync<Todo>("SELECT * FROM Todos WHERE IsComplete = false", ct))
             .WithName("GetIncompleteTodos");
 
         group.MapGet("/{id:int}", async Task<Results<Ok<Todo>, NotFound>> (int id, NpgsqlDataSource db, CancellationToken ct) =>
@@ -104,7 +102,6 @@ await db.ExecuteAsync(
 }
 
 [JsonSerializable(typeof(Todo))]
-[JsonSerializable(typeof(List<Todo>))]
 [JsonSerializable(typeof(IAsyncEnumerable<Todo>))]
 internal partial class TodoApiJsonSerializerContext : JsonSerializerContext
 {