Fixed: missing test coverage for applying default page size in includes

Author: bkoelman

test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Pagination/PaginationWithTotalCountTests.cs

@@ -500,14 +500,15 @@ public async Task Uses_default_page_number_and_size()
 
  Blog blog = _fakers.Blog.Generate();
  blog.Posts = _fakers.BlogPost.Generate(3);
+ blog.Posts.ToList().ForEach(post => post.Labels = _fakers.Label.Generate(3).ToHashSet());
 
  await _testContext.RunOnDatabaseAsync(async dbContext =>
  {
  dbContext.Blogs.Add(blog);
  await dbContext.SaveChangesAsync();
  });
 
- string route = $"/blogs/{blog.StringId}/posts";
+ string route = $"/blogs/{blog.StringId}/posts?include=labels";
 
  // Act
  (HttpResponseMessage httpResponse, Document responseDocument) = await _testContext.ExecuteGetAsync<Document>(route);
@@ -519,10 +520,12 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  responseDocument.Data.ManyValue[0].Id.Should().Be(blog.Posts[0].StringId);
  responseDocument.Data.ManyValue[1].Id.Should().Be(blog.Posts[1].StringId);
 
+ responseDocument.Included.ShouldHaveCount(4);
+
  responseDocument.Links.ShouldNotBeNull();
  responseDocument.Links.Self.Should().Be($"{HostPrefix}{route}");
  responseDocument.Links.First.Should().Be(responseDocument.Links.Self);
- responseDocument.Links.Last.Should().Be($"{HostPrefix}{route}?page%5Bnumber%5D=2");
+ responseDocument.Links.Last.Should().Be($"{responseDocument.Links.Self}&page%5Bnumber%5D=2");
  responseDocument.Links.Prev.Should().BeNull();
  responseDocument.Links.Next.Should().Be(responseDocument.Links.Last);
  }