Move text processing methods to separate class

Author: sandrofigo

src/Assets/SimpleCodeGenerator/Core/TextProcessing.cs

@@ -0,0 +1,27 @@
+using System.Text.RegularExpressions;
+
+namespace SimpleCodeGenerator.Core
+{
+    public static class TextProcessing
+    {
+        public static string SanitizeStringForVariableName(string input)
+        {
+            if (string.IsNullOrWhiteSpace(input))
+                return "";
+
+            string variableName = Regex.Replace(input, "[^0-9A-Za-z_]", string.Empty);
+
+            if (int.TryParse(variableName[0].ToString(), out int _))
+            {
+                variableName = $"_{variableName}";
+            }
+
+            return variableName;
+        }
+
+        public static string EscapeSummaryText(string text)
+        {
+            return $"<![CDATA[{text}]]>";
+        }
+    }
+}

src/Assets/SimpleCodeGenerator/Core/TextProcessing.cs.meta

@@ -51,8 +51,8 @@ public static void GenerateStringDictionary(IEnumerable<StringDictionaryItem> va
 
             for (int i = 0; i < valueArray.Length; i++)
             {
-                valueArray[i].Key = SanitizeStringForVariableName(valueArray[i].Key);
-                valueArray[i].Summary = EscapeSummaryText(valueArray[i].Summary);
+                valueArray[i].Key = TextProcessing.SanitizeStringForVariableName(valueArray[i].Key);
+                valueArray[i].Summary = TextProcessing.EscapeSummaryText(valueArray[i].Summary);
             }
 
             var template = Template.FindBuiltInTemplate("StringDictionary");
@@ -81,7 +81,7 @@ public static void GenerateEnum(IEnumerable<string> values, string namespaceName
 
             for (int i = 0; i < valueArray.Length; i++)
             {
-                valueArray[i] = SanitizeStringForVariableName(valueArray[i]);
+                valueArray[i] = TextProcessing.SanitizeStringForVariableName(valueArray[i]);
             }
 
             var template = Template.FindBuiltInTemplate("Enum");
@@ -149,25 +149,5 @@ private static string GetRelativePathToBuiltInTemplate(string templateName)
         {
             return Template.GetAbsolutePathToBuiltInTemplate(templateName) - Application.dataPath;
         }
-
-        public static string SanitizeStringForVariableName(string input)
-        {
-            if (string.IsNullOrWhiteSpace(input))
-                return "";
-
-            string variableName = Regex.Replace(input, "[^0-9A-Za-z_]", string.Empty);
-
-            if (int.TryParse(variableName[0].ToString(), out int _))
-            {
-                variableName = $"_{variableName}";
-            }
-
-            return variableName;
-        }
-
-        public static string EscapeSummaryText(string text)
-        {
-            return $"<![CDATA[{text}]]>";
-        }
     }
 }

src/Assets/SimpleCodeGenerator/Editor/CodeGenerator.cs

@@ -1,4 +1,5 @@
 using FluentAssertions;
+using SimpleCodeGenerator.Core;
 using SimpleCodeGenerator.Editor;
 
 namespace tests;
@@ -82,24 +83,4 @@ public Task GenerateFromTemplate_RenderFromValidTemplateInstance_TemplateIsRende
         settings.UseDirectory("Verify");
         return Verify(result, settings);
     }
-
-    [Theory]
-    [InlineData("test", "test")]
-    [InlineData("test123", "test123")]
-    [InlineData("1test", "_1test")]
-    [InlineData("1 test", "_1test")]
-    [InlineData("hello world", "helloworld")]
-    [InlineData(" hello world", "helloworld")]
-    [InlineData(" hello   world", "helloworld")]
-    [InlineData(" hello 1 world", "hello1world")]
-    [InlineData(" ", "")]
-    [InlineData("   ", "")]
-    [InlineData("/test", "test")]
-    [InlineData("test%$&§!?`+-#", "test")]
-    public void SanitizeStringForVariableName_SanitizeStrings_StringAreSanitizedCorrectly(string input, string expected)
-    {
-        string result = CodeGenerator.SanitizeStringForVariableName(input);
-
-        result.Should().Be(expected);
-    }
 }

tests/CodeGeneratorTests.cs

@@ -0,0 +1,28 @@
+using FluentAssertions;
+using SimpleCodeGenerator.Core;
+using SimpleCodeGenerator.Editor;
+
+namespace tests;
+
+public class TextProcessingTests
+{
+    [Theory]
+    [InlineData("test", "test")]
+    [InlineData("test123", "test123")]
+    [InlineData("1test", "_1test")]
+    [InlineData("1 test", "_1test")]
+    [InlineData("hello world", "helloworld")]
+    [InlineData(" hello world", "helloworld")]
+    [InlineData(" hello   world", "helloworld")]
+    [InlineData(" hello 1 world", "hello1world")]
+    [InlineData(" ", "")]
+    [InlineData("   ", "")]
+    [InlineData("/test", "test")]
+    [InlineData("test%$&§!?`+-#", "test")]
+    public void SanitizeStringForVariableName_SanitizeStrings_StringAreSanitizedCorrectly(string input, string expected)
+    {
+        string result = TextProcessing.SanitizeStringForVariableName(input);
+
+        result.Should().Be(expected);
+    }
+}