Use URL representation to store template repositories

Template repositories were stored in / cloned into

`~/<localAppData>/<profile-name>/<branch>`

This could lead to problems as profiles could share the same name across
projects while pointing to different repositories. This is solved now by making
the repository URL, i.e. a file system friendly representation of it, the path
identifier to clone templates:

`~/<localAppData>/<url>/<branch>`

Author: seveneleven

src/Moryx.Cli.Templates/Extensions/StringExtensions.cs

@@ -1,15 +1,21 @@
-namespace Moryx.Cli.Templates.Extensions
+using System.Text;
+using System.Text.RegularExpressions;
+
+namespace Moryx.Cli.Templates.Extensions
 {
     public static class StringExtensions
     {
-        public static List<string> ToCleanList(this string str) {  
+        public static List<string> ToCleanList(this string str)
+        {
             return str
                 .Split(',')
                 .Select(s => s.Trim())
                 .Where(s => s.Length > 0)
-                .ToList(); }
+                .ToList();
+        }
 
-        public static string ReplaceFileExtension(this string str, string oldStr, string newStr) {
+        public static string ReplaceFileExtension(this string str, string oldStr, string newStr)
+        {
             var position = str.LastIndexOf(oldStr);
             if (position == -1)
             {
@@ -19,5 +25,22 @@ public static string ReplaceFileExtension(this string str, string oldStr, string
         }
 
         public static string StateBase(this string str) => str + "StateBase";
+
+        public static string FileSystemCompatible(this string url)
+        {
+            var pattern = @"[\:\/\?\#\[\]\!\$\&\'\(\)\*\+\,\;\=]";
+            url = Regex.Replace(url, pattern, "_");
+            var result = new StringBuilder();
+            var previous = '\0';
+            foreach (char current in url)
+            {
+                if (current != '_' || (current == '_' && current != previous))
+                {
+                    result.Append(current);
+                    previous = current;
+                }
+            }
+            return result.ToString();
+        }
     }
 }

src/Moryx.Cli.Templates/Models/TemplateSettings.cs

@@ -1,4 +1,6 @@
-namespace Moryx.Cli.Templates.Models
+using Moryx.Cli.Templates.Extensions;
+
+namespace Moryx.Cli.Templates.Models
 {
     public class TemplateSettings
     {
@@ -8,7 +10,7 @@ public class TemplateSettings
         public required string AppName { get; set; }
         public string ProfileName { get; set; } = "default";
         public required string TargetDirectory { get; set; }
-        public string SourceDirectory { get => Path.Combine(TemplatesRoot(), ProfileName, Branch); }
+        public string SourceDirectory { get => Path.Combine(TemplatesRoot(), Repository.FileSystemCompatible(), Branch); }
 
 
         public string TemplatesRoot()

src/Moryx.Cli.Templates/TemplateRepository.cs

@@ -22,7 +22,7 @@ public static int Clone(TemplateSettings settings, Action<string>? onStatus = nu
             {
                 return ExecCommanLine($"git -C {targetDir} pull", _ => onStatus?.Invoke(_));
             }
-            return -1;
+            return 0;
         }
 
         public static int ExecCommanLine(string command, Action<string> onStatus)

src/Tests/Moryx.Cli.Tests/FileSystemCompatibleUrlTests.cs

@@ -0,0 +1,22 @@
+using Moryx.Cli.Templates.Extensions;
+
+namespace Moryx.Cli.Tests
+{
+    public class FileSystemCompatibleUrlTests
+    {
+
+        [SetUp]
+        public void Setup()
+        {
+        }
+
+        [TestCase("https://github.com/microsoft/vscode.git", "https_github.com_microsoft_vscode.git")]
+        [TestCase("ssh://[email protected]:microsoft/vscode.git", "[email protected]_microsoft_vscode.git")]
+        public void CheckUrlConversion(string input, string expected)
+        {
+            var result = input.FileSystemCompatible();
+
+            Assert.That(result, Is.EqualTo(expected));
+        }
+    }
+}