enhancements: Create internal extension to properties naming

Author: NelsonBN

CHANGELOG.md

@@ -3,6 +3,17 @@
 
 
 
+## [1.0.1] - 2022-05-28
+[Full Changelog](https://github.com/TechNobre/PowerUtils.Text/compare/v1.0.0...v1.0.1)
+
+
+### Fixes
+
+- Fix `System.IndexOutOfRangeException` when format the properties name to camel case;
+
+
+
+
 ## [1.0.0] - 2022-03-15
 
 - Kickoff;

src/PowerUtils.AspNetCore.ErrorHandler.csproj

@@ -15,7 +15,7 @@
     <PackageId>PowerUtils.AspNetCore.ErrorHandler</PackageId>
     <title>PowerUtils.AspNetCore.ErrorHandler</title>
     <Product>PowerUtils.AspNetCore.ErrorHandler</Product>
-    <Version>1.0.0</Version>
+    <Version>1.0.1</Version>
 
     <Authors>Nelson Nobre</Authors>
     <Company>TechNobre</Company>

src/ProblemDetailsFactory.cs

@@ -7,7 +7,6 @@
 using Microsoft.AspNetCore.WebUtilities;
 using Microsoft.Extensions.Options;
 using PowerUtils.Net.Constants;
-using PowerUtils.Text;
 
 namespace PowerUtils.AspNetCore.ErrorHandler
 {
@@ -22,16 +21,14 @@ internal static ProblemDetailsResponse Create(HttpContext httpContext)
         {
             var result = new ProblemDetailsResponse();
 
-            result.Status = httpContext.GetStatusCode() ?? 0;
+            result.Status = httpContext.GetStatusCode() ?? 0; // Default value is 0
             result.Type = result.Status.GetStatusCodeLinkOrDefault();
             result.Title = result.Status == 0 ? null : ReasonPhrases.GetReasonPhrase(result.Status);
 
             result.Instance = httpContext.GetRequestEndpoint();
 
             result.TraceID = httpContext.GetCorrelationId();
 
-            result.Errors = new Dictionary<string, string>();
-
             return result;
         }
 
@@ -111,50 +108,9 @@ private string _formatPropertyName(string propertyName)
             => _options.Value.PropertyNamingPolicy switch
             {
                 PropertyNamingPolicy.Original => propertyName,
-                PropertyNamingPolicy.SnakeCase => _formatPropertyToSnakeCase(propertyName),
-                _ => _formatPropertyToCamelCase(propertyName),
+                PropertyNamingPolicy.SnakeCase => propertyName.FormatToSnakeCase(),
+                _ => propertyName.FormatToCamelCase(),
             };
 
-        private static string _formatPropertyToCamelCase(string propertyName)
-        {
-            if(string.IsNullOrWhiteSpace(propertyName))
-            {
-                return "";
-            }
-
-            var propertyParts = propertyName.Split('.');
-            if(propertyParts.Length == 1)
-            {
-                return char.ToLowerInvariant(propertyName[0]) + propertyName[1..];
-            }
-
-
-            for(var count = 0; count < propertyParts.Length; count++)
-            {
-                // Prevent System.IndexOutOfRangeException: 'Index was outside the bounds of the array.' for cases "Hello."
-                if(propertyParts[count].Length == 0)
-                {
-                    continue;
-                }
-
-                propertyParts[count] = char.ToLowerInvariant(propertyParts[count][0]) + propertyParts[count][1..];
-            }
-
-
-            return string.Join(".", propertyParts);
-        }
-
-        private static string _formatPropertyToSnakeCase(string propertyName)
-        {
-            var propertyParts = propertyName.Split('.');
-
-            for(var count = 0; count < propertyParts.Length; count++)
-            {
-                propertyParts[count] = propertyParts[count].ToSnakeCase();
-            }
-
-
-            return string.Join(".", propertyParts);
-        }
     }
 }

src/ProblemDetailsResponse.cs

@@ -5,7 +5,6 @@ namespace PowerUtils.AspNetCore.ErrorHandler
 {
     public class ProblemDetailsResponse
     {
-        #region PROPERTIES
         /// <summary>
         /// The HTTP status code([RFC7231], Section 6) generated
         /// </summary>
@@ -38,16 +37,17 @@ public class ProblemDetailsResponse
         /// { "Property": "Error" }
         /// </example>
         public IDictionary<string, string> Errors { get; set; }
-        #endregion
 
         /// <summary>
         /// Serialize problem details to JSON
         /// </summary>
-        /// <returns></returns>
         public override string ToString()
             => JsonSerializer.Serialize(this);
 
         public static implicit operator string(ProblemDetailsResponse problemDetailsResponse)
             => problemDetailsResponse.ToString();
+
+        public ProblemDetailsResponse()
+            => Errors = new Dictionary<string, string>();
     }
 }

src/PropertiesNamingExtensions.cs

@@ -0,0 +1,48 @@
+using PowerUtils.Text;
+
+namespace PowerUtils.AspNetCore.ErrorHandler
+{
+    internal static class PropertiesNamingExtensions
+    {
+        public static string FormatToCamelCase(this string propertyName)
+        {
+            if(string.IsNullOrWhiteSpace(propertyName))
+            {
+                return "";
+            }
+
+            var propertyParts = propertyName.Split('.');
+            for(var count = 0; count < propertyParts.Length; count++)
+            {
+                // Prevent System.IndexOutOfRangeException: 'Index was outside the bounds of the array.' for cases "Hello."
+                if(propertyParts[count].Length == 0)
+                {
+                    continue;
+                }
+
+                propertyParts[count] = char.ToLowerInvariant(propertyParts[count][0]) + propertyParts[count][1..];
+            }
+
+
+            return string.Join(".", propertyParts);
+        }
+
+        public static string FormatToSnakeCase(this string propertyName)
+        {
+            if(string.IsNullOrWhiteSpace(propertyName))
+            {
+                return "";
+            }
+
+            var propertyParts = propertyName.Split('.');
+
+            for(var count = 0; count < propertyParts.Length; count++)
+            {
+                propertyParts[count] = propertyParts[count].ToSnakeCase();
+            }
+
+
+            return string.Join(".", propertyParts);
+        }
+    }
+}

tests/PowerUtils.AspNetCore.ErrorHandler.Tests/ProblemDetailsFactoryTests.cs

@@ -110,96 +110,4 @@ public void PropertyNamingPolicy_SnakeCase_FormattedCamelCase()
         // Assert
         act.Should().Be("prop_name.prop_value");
     }
-
-    [Fact(DisplayName = "Format to camel case null value - Should return empty")]
-    public void FormatPropertyToCamelCase_Null_Empty()
-    {
-        // Arrange
-        string propertyName = null;
-
-
-        // Act
-        var act = ObjectInvoker.Invoke<string>(typeof(ProblemDetailsFactory), "_formatPropertyToCamelCase", propertyName);
-
-
-        // Assert
-        act.Should().BeEmpty();
-    }
-
-    [Fact(DisplayName = "Format to camel case empty value - Should return empty")]
-    public void FormatPropertyToCamelCase_Empty_Empty()
-    {
-        // Arrange
-        var propertyName = "";
-
-
-        // Act
-        var act = ObjectInvoker.Invoke<string>(typeof(ProblemDetailsFactory), "_formatPropertyToCamelCase", propertyName);
-
-
-        // Assert
-        act.Should().BeEmpty();
-    }
-
-    [Fact(DisplayName = "Format to camel case white spaces value - Should return empty")]
-    public void FormatPropertyToCamelCase_WhiteSpaces_Empty()
-    {
-        // Arrange
-        var propertyName = "  ";
-
-
-        // Act
-        var act = ObjectInvoker.Invoke<string>(typeof(ProblemDetailsFactory), "_formatPropertyToCamelCase", propertyName);
-
-
-        // Assert
-        act.Should().BeEmpty();
-    }
-
-    [Fact(DisplayName = "Format to camel case white one character - Should return the character lowercase")]
-    public void FormatPropertyToCamelCase_OneCharacter_Lower()
-    {
-        // Arrange
-        var propertyName = "G";
-
-
-        // Act
-        var act = ObjectInvoker.Invoke<string>(typeof(ProblemDetailsFactory), "_formatPropertyToCamelCase", propertyName);
-
-
-        // Assert
-        act.Should().Be("g");
-    }
-
-    [Fact(DisplayName = "Format to camel case a text with only one dot - Should return a dot")]
-    public void FormatPropertyToCamelCase_OnlyOneDot_Dot()
-    {
-        // Arrange
-        var propertyName = ".";
-
-
-        // Act
-        var act = ObjectInvoker.Invoke<string>(typeof(ProblemDetailsFactory), "_formatPropertyToCamelCase", propertyName);
-
-
-        // Assert
-        act.Should().Be(".");
-    }
-
-
-
-    [Fact(DisplayName = "Format to camel case a text ended with an dot - Should format")]
-    public void FormatPropertyToCamelCase_EndWitDot_Format()
-    {
-        // Arrange
-        var propertyName = "Hello.";
-
-
-        // Act
-        var act = ObjectInvoker.Invoke<string>(typeof(ProblemDetailsFactory), "_formatPropertyToCamelCase", propertyName);
-
-
-        // Assert
-        act.Should().Be("hello.");
-    }
 }