fix: Property handler in IProblemFactory

Author: NelsonBN

samples/PowerUtils.AspNetCore.ErrorHandler.Samples/Controllers/ProblemFactoryController.cs

@@ -42,7 +42,19 @@ public IActionResult CreateProblem()
                     ["Key100"] = new("Error114", "description fake"),
                     ["Key114"] = new("Error11124", "description 1444"),
                     ["me"] = new("ti", "111"),
+                    ["MyKey"] = new("MyCode", "MyDisc")
                 }
             ));
+
+        [HttpGet("null-errors")]
+        public IActionResult NullErrors()
+            => new ObjectResult(_problemFactory.CreateProblem(
+                detail: "fake detail",
+                instance: "fake instance",
+                statusCode: (int)HttpStatusCode.BadRequest,
+                title: "fake title",
+                type: "fake type",
+                errors: null
+            ));
     }
 }

src/ApiProblemDetailsFactory.cs

@@ -54,14 +54,19 @@ public ErrorProblemDetails CreateProblem(
             IDictionary<string, ErrorDetails> errors = null
         )
         {
+            errors ??= new Dictionary<string, ErrorDetails>();
+
             var problemDetails = new ErrorProblemDetails
             {
                 Status = statusCode,
                 Title = title,
                 Type = type,
                 Detail = detail,
                 Instance = instance,
-                Errors = errors
+                Errors = errors.ToDictionary(
+                    k => _errorHandlerOptions.Value.PropertyHandler(k.Key),
+                    v => v.Value
+                )
             };
 
             if(string.IsNullOrWhiteSpace(problemDetails.Detail))

tests/PowerUtils.AspNetCore.ErrorHandler.Tests/Tests/Controllers/ProblemFactoryControllerTests.cs

@@ -52,8 +52,8 @@ public async Task ObjectResult_CreateProblemResult_403()
 
             content.ValidateContent(new Dictionary<string, ErrorDetails>()
             {
-                ["Key4"] = new("Error4", "description 111"),
-                ["Key14"] = new("Error124", "description 423423")
+                ["key4"] = new("Error4", "description 111"),
+                ["key14"] = new("Error124", "description 423423")
             });
         }
 
@@ -91,10 +91,46 @@ public async Task Problem_CreateProblem_429()
 
             content.ValidateContent(new Dictionary<string, ErrorDetails>()
             {
-                ["Key100"] = new("Error114", "description fake"),
-                ["Key114"] = new("Error11124", "description 1444"),
+                ["key100"] = new("Error114", "description fake"),
+                ["key114"] = new("Error11124", "description 1444"),
                 ["me"] = new("ti", "111"),
+                ["my_key"] = new("MyCode", "MyDisc")
             });
         }
+
+        [Fact]
+        public async Task WithoutErrors_CreateProblem_400()
+        {
+            // Arrange
+            var requestUri = "/problem-factory/null-errors";
+
+
+            // Act
+            (var response, var content) = await _testsFixture.Client.SendGetAsync(requestUri);
+
+
+            // Assert
+            response.ValidateResponse(HttpStatusCode.BadRequest);
+
+            content.Status.Should()
+                .Be((int)HttpStatusCode.BadRequest);
+
+            content.Type.Should()
+                .Be("fake type");
+
+            content.Title.Should()
+                .Be("fake title");
+
+            content.Instance.Should()
+                .Be("fake instance");
+
+            content.Detail.Should()
+               .Be("fake detail");
+
+            content.TraceId.Should()
+                .NotBeNullOrWhiteSpace();
+
+            content.ValidateContent(new Dictionary<string, ErrorDetails>());
+        }
     }
 }