Pushed updates addressing review comments.

Author: mrmarcosmagalhaes

test/JsonApiDotNetCoreTests/IntegrationTests/Meta/OperationsRequestMetaTests.cs

@@ -144,6 +144,7 @@ public async Task Accepts_meta_in_update_resource_operation_with_ToMany_relation
         Dictionary<string, object?> identifierMeta1 = _fakers.IdentifierMeta.GenerateOne();
         Dictionary<string, object?> identifierMeta2 = _fakers.IdentifierMeta.GenerateOne();
 
+        string newFamilyName = _fakers.ProductFamily.GenerateOne().Name;
         ProductFamily existingFamily = _fakers.ProductFamily.GenerateOne();
         SupportTicket existingTicket1 = _fakers.SupportTicket.GenerateOne();
         SupportTicket existingTicket2 = _fakers.SupportTicket.GenerateOne();
@@ -168,7 +169,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
                         id = existingFamily.StringId,
                         attributes = new
                         {
-                            name = existingFamily.Name
+                            name = newFamilyName
                         },
                         relationships = new
                         {

test/JsonApiDotNetCoreTests/IntegrationTests/Meta/RequestMetaTests.cs

@@ -90,10 +90,10 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
         string route = $"/supportTickets/{existingTicket.StringId}";
 
         // Act
-        (HttpResponseMessage response, _) = await _testContext.ExecutePatchAsync<Document>(route, requestBody);
+        (HttpResponseMessage httpResponse, _) = await _testContext.ExecutePatchAsync<Document>(route, requestBody);
 
         // Assert
-        response.ShouldHaveStatusCode(HttpStatusCode.NoContent);
+        httpResponse.ShouldHaveStatusCode(HttpStatusCode.NoContent);
 
         store.Document.Should().NotBeNull();
 
@@ -176,10 +176,10 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
         string route = $"/productFamilies/{existingFamily.StringId}";
 
         // Act
-        (HttpResponseMessage response, _) = await _testContext.ExecutePatchAsync<Document>(route, requestBody);
+        (HttpResponseMessage httpResponse, _) = await _testContext.ExecutePatchAsync<Document>(route, requestBody);
 
         // Assert
-        response.ShouldHaveStatusCode(HttpStatusCode.NoContent);
+        httpResponse.ShouldHaveStatusCode(HttpStatusCode.NoContent);
 
         store.Document.Should().NotBeNull();
 
@@ -189,8 +189,6 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
 
         store.Document.Data.SingleValue.Meta.Should().BeEquivalentToJson(resourceMeta);
 
-        store.Document.Data.SingleValue.Relationships.Should().NotBeNull();
-
         store.Document.Data.SingleValue.Relationships.Should().ContainKey("tickets").WhoseValue.With(value =>
         {
             value.Should().NotBeNull();
@@ -265,8 +263,6 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
 
         store.Document.Data.SingleValue.Meta.Should().BeEquivalentToJson(resourceMeta);
 
-        store.Document.Data.SingleValue.Relationships.Should().NotBeNull();
-
         store.Document.Data.SingleValue.Relationships.Should().ContainKey("productFamily").WhoseValue.With(value =>
         {
             value.Should().NotBeNull();
@@ -352,8 +348,6 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
         store.Document.Data.SingleValue.Should().NotBeNull();
         store.Document.Data.SingleValue.Meta.Should().BeEquivalentToJson(resourceMeta);
 
-        store.Document.Data.SingleValue.Relationships.Should().NotBeNull();
-
         store.Document.Data.SingleValue.Relationships.Should().ContainKey("tickets").WhoseValue.With(value =>
         {
             value.Should().NotBeNull();