Fix broken tests

bkoelman · bkoelman · commit 528f110970d1 · 2023-09-26T03:27:41.000+02:00
diff --git a/src/Examples/NoEntityFrameworkExample/Data/InMemoryModel.cs b/src/Examples/NoEntityFrameworkExample/Data/InMemoryModel.cs
@@ -19,7 +19,7 @@ private static void SetEntityProperties(RuntimeEntityType entityType, ResourceTy
     {
         foreach (PropertyInfo property in resourceType.ClrType.GetProperties())
         {
-            entityType.AddProperty(property.Name, property.PropertyType, property);
+            entityType.AddProperty(property.Name, property.PropertyType, propertyInfo: property);
         }
     }
 }
diff --git a/test/NoEntityFrameworkTests/PersonTests.cs b/test/NoEntityFrameworkTests/PersonTests.cs
@@ -149,7 +149,7 @@ public async Task Can_select_fields_in_primary_resources()
         responseDocument.Data.ManyValue.Should().AllSatisfy(resource => resource.Attributes.ShouldOnlyContainKeys("lastName", "displayName"));
     }
 
-    [Fact(Skip = "TODO: Investigate why this fails on .NET 8")]
+    [Fact]
     public async Task Can_include_in_primary_resources()
     {
         // Arrange
diff --git a/test/NoEntityFrameworkTests/TodoItemTests.cs b/test/NoEntityFrameworkTests/TodoItemTests.cs
@@ -149,7 +149,7 @@ public async Task Can_select_fields_in_primary_resources()
         responseDocument.Data.ManyValue.Should().AllSatisfy(resource => resource.Attributes.ShouldOnlyContainKeys("description", "priority"));
     }
 
-    [Fact(Skip = "TODO: Investigate why this fails on .NET 8")]
+    [Fact]
     public async Task Can_include_in_primary_resources()
     {
         // Arrange

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ private static void SetEntityProperties(RuntimeEntityType entityType, ResourceTy`
`19`	`19`	`{`
`20`	`20`	`foreach (PropertyInfo property in resourceType.ClrType.GetProperties())`
`21`	`21`	`{`
`22`		`- entityType.AddProperty(property.Name, property.PropertyType, property);`
	`22`	`+ entityType.AddProperty(property.Name, property.PropertyType, propertyInfo: property);`
`23`	`23`	`}`
`24`	`24`	`}`
`25`	`25`	`}`
Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ public async Task Can_select_fields_in_primary_resources()`
`149`	`149`	`responseDocument.Data.ManyValue.Should().AllSatisfy(resource => resource.Attributes.ShouldOnlyContainKeys("lastName", "displayName"));`
`150`	`150`	`}`
`151`	`151`
`152`		`- [Fact(Skip = "TODO: Investigate why this fails on .NET 8")]`
	`152`	`+ [Fact]`
`153`	`153`	`public async Task Can_include_in_primary_resources()`
`154`	`154`	`{`
`155`	`155`	`// Arrange`