Add test for multiple spaces, remove unneeded regex

Humanizr · Mar 5, 2019 · 3740384 · 3740384
1 parent 1e7c1b7
commit 3740384
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Humanizer.Tests.Shared/InflectorTests.cs b/src/Humanizer.Tests.Shared/InflectorTests.cs
@@ -102,6 +102,7 @@ public void Hyphenate(string input, string expectedOutput)
         [InlineData("customer_first_name", "CustomerFirstName")]
         [InlineData("customer_first_name goes here", "CustomerFirstNameGoesHere")]
         [InlineData("customer name", "CustomerName")]
+        [InlineData("customer   name", "CustomerName")]
         public void Pascalize(string input, string expectedOutput)
         {
             Assert.Equal(expectedOutput, input.Pascalize());
@@ -116,6 +117,7 @@ public void Pascalize(string input, string expectedOutput)
         [InlineData("customer_first_name", "customerFirstName")]
         [InlineData("customer_first_name goes here", "customerFirstNameGoesHere")]
         [InlineData("customer name", "customerName")]
+        [InlineData("customer   name", "customerName")]
         [InlineData("", "")]
         public void Camelize(string input, string expectedOutput)
         {

diff --git a/src/Humanizer/InflectorExtensions.cs b/src/Humanizer/InflectorExtensions.cs
@@ -72,9 +72,9 @@ public static string Titleize(this string input)
         public static string Pascalize(this string input)
         {
 
-            input = Regex.Replace(input, "(?:^|_| )(.)", match => match.Groups[1].Value.ToUpper());
+            return Regex.Replace(input, "(?:^|_| )(.)", match => match.Groups[1].Value.ToUpper());
 
-            return Regex.Replace(input, @"\s+", "");
+            //return Regex.Replace(input, @"\s+", "");
         }
 
         /// <summary>