Make sure source repo is cloned BEFORE trying to access its contents

Author: seveneleven

src/Moryx.Cli.Commands/CommandBase.cs

@@ -7,7 +7,12 @@ public class CommandBase
     {
         public static CommandResult Exec(Template template, Func<CommandResult> func)
         {
-            TemplateRepository.Clone(template.Settings);
+            return Exec(template.Settings, func);
+        }
+
+        public static CommandResult Exec(TemplateSettings settings, Func<CommandResult> func)
+        {
+            TemplateRepository.Clone(settings);
             return func();
         }
     }

src/Moryx.Cli.Commands/CreateNew.cs

@@ -35,18 +35,18 @@ public static CommandResult Solution(NewOptions options, [NotNull] Action<string
             var settings = config.AsTemplateSettings(dir, solutionName);
             settings.Pull = options.Pull;
 
-            string errorMessage = "";
-            var configuration = TemplateConfigurationFactory.Load(settings.SourceDirectory, error =>
+            return CommandBase.Exec(settings, () =>
             {
-                errorMessage = error;
-            });
-            if (configuration == null)
-                return CommandResult.WithError(errorMessage);
+                string errorMessage = "";
+                var configuration = TemplateConfigurationFactory.Load(settings.SourceDirectory, error =>
+                {
+                    errorMessage = error;
+                });
+                if (configuration == null)
+                    return CommandResult.WithError(errorMessage);
 
-            var template = Template.Load(settings, configuration);
+                var template = Template.Load(settings, configuration);
 
-            return CommandBase.Exec(template, () =>
-            {
                 Directory.CreateDirectory(solutionName);
                 CreateBareSolution(template);
                 config.Save(dir);