Merge pull request 'Directory option' (#4) from dirOption into master

Reviewed-on: #4

BLL/CommitDetail.cs

@@ -5,6 +5,10 @@ public class CommitDetail
     private List<string>? _authors;
     private SortedList<string, int>? _commitDetails;
     private string _currentBranch;
+    private string? _outputOption;
+    private string? _branchOption;
+    private string? _tagOption;
+    private string? _selectedDir;
 
     public List<string>? Authors
     {
@@ -24,16 +28,28 @@ public class CommitDetail
         set { _currentBranch = value; }
     }
 
-    public CommitDetail()
+    public string? SelectedDir
     {
+        get { return _selectedDir; }
+        set { _selectedDir = value; }
+    }
+
+    public CommitDetail(string? dir)
+    {
+        _selectedDir = !String.IsNullOrEmpty(dir) ? dir : Directory.GetCurrentDirectory();
         _authors = new List<string>();
         _commitDetails = new SortedList<string, int>();
         _currentBranch = GetCurrentBranch();
+
+        // unused, might be used later for a refactor
+        _outputOption = null;
+        _branchOption = null;
+        _tagOption = null;
     }
 
     public void GetCurrentCommitsByName()
     {
-        using (var repo = new Repository(Directory.GetCurrentDirectory()))
+        using (var repo = new Repository(_selectedDir))
         {
             foreach (var c in repo.Commits)
             {
@@ -53,7 +69,7 @@ public class CommitDetail
 
     public void GetCurrentCommitsByEmail()
     {
-        using (var repo = new Repository(Directory.GetCurrentDirectory()))
+        using (var repo = new Repository(_selectedDir))
         {
             foreach (var c in repo.Commits)
             {
@@ -73,7 +89,7 @@ public class CommitDetail
 
     public int GetCommitTotal()
     {
-        using (var repo = new Repository(Directory.GetCurrentDirectory()))
+        using (var repo = new Repository(_selectedDir))
         {
             return repo.Commits.Count();
         }
@@ -81,7 +97,7 @@ public class CommitDetail
 
     public string GetCurrentBranch()
     {
-        using (var repo = new Repository(Directory.GetCurrentDirectory()))
+        using (var repo = new Repository(_selectedDir))
         {
             return repo.Head.Reference.TargetIdentifier;
         }
@@ -89,7 +105,7 @@ public class CommitDetail
 
     public void GetCommitsByBranch(string branchName)
     {
-        using (var repo = new Repository(Directory.GetCurrentDirectory()))
+        using (var repo = new Repository(_selectedDir))
         {
             var branchResult = repo.Branches[branchName];
 
@@ -126,7 +142,7 @@ public class CommitDetail
 
     public void GetCommitsByTag(string tagName)
     {
-        using (var repo = new Repository(Directory.GetCurrentDirectory()))
+        using (var repo = new Repository(_selectedDir))
         {
             try
             {

DrillSergeant.csproj

@@ -7,7 +7,7 @@
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
     <AssemblyName>drillsergeant</AssemblyName>
-    <Version>1.1.0</Version>
+    <Version>1.2.0</Version>
   </PropertyGroup>
 
   <ItemGroup>

Program.cs

@@ -22,119 +22,141 @@ static class Program
                 );
         tagOption.AddAlias("-t");
 
+        var dirOption = new Option<string>(
+                "--file",
+                "Specify a git directory"
+                );
+        dirOption.AddAlias("-f");
+
         var rootCommand = new RootCommand("Get a tally of contributors' commits")
         {
             outputOption,
             branchOption,
             tagOption,
+            dirOption,
         };
 
-        rootCommand.SetHandler((outputOptionValue, branchOptionValue, tagOptionValue) => {
+        rootCommand.SetHandler((outputOptionValue, branchOptionValue, tagOptionValue, dirOptionValue) =>
-                    CommitDetail commits = new CommitDetail();
+        {
+            CommitDetail commits = new CommitDetail(dirOptionValue);
 
-                    switch (outputOptionValue) 
+            switch (outputOptionValue)
+            {
+                case "stdout":
+                    StdOutDataService outDataService = new StdOutDataService();
+                    DataAccess dataAccess = new DataAccess(outDataService);
+
+                    if (branchOptionValue != null && tagOptionValue != null)
                     {
-                        case "stdout":
+                        Console.WriteLine("Please specify either a branch or a tag");
-                            StdOutDataService outDataService = new StdOutDataService();
+                        Environment.Exit(2);
-                            DataAccess dataAccess = new DataAccess(outDataService);
+                    }
-
+                    else if (branchOptionValue != null && tagOptionValue == null)
-                            if (branchOptionValue != null && tagOptionValue != null) {
+                    {
-                                Console.WriteLine("Please specify either a branch or a tag");
+                        switch (branchOptionValue)
-                                Environment.Exit(2);
+                        {
-                            } else if (branchOptionValue != null && tagOptionValue == null) {
+                            case null:
-                                switch (branchOptionValue)
-                                {
-                                    case null:
-                                        commits.GetCurrentCommitsByName();
-                                        dataAccess.WriteData(commits.CommitDetails);
-                                        break;
-                                    default:
-                                        commits.GetCommitsByBranch(branchOptionValue);
-                                        dataAccess.WriteData(commits.CommitDetails);
-                                        break;
-                                }
-                                break;
-                            } else if (branchOptionValue == null && tagOptionValue != null) {
-                                commits.GetCommitsByTag(tagOptionValue);
-                                dataAccess.WriteData(commits.CommitDetails);
-                            }
-                            break;
-                        case "xlsx":
-                            ExcelDataService excelDataService = new ExcelDataService();
-                            DataAccess dataAccessExcelCase = new DataAccess(excelDataService);
-
-                            if (branchOptionValue != null && tagOptionValue != null) {
-                                Console.WriteLine("Please specify either a branch or a tag.");
-                                Environment.Exit(2);
-                            } else if (branchOptionValue != null && tagOptionValue == null) {
-                                switch (branchOptionValue)
-                                {
-                                    case null:
-                                        commits.GetCurrentCommitsByName();
-                                        dataAccessExcelCase.WriteData(commits.CommitDetails);
-                                        break;
-                                    default:
-                                        commits.GetCommitsByBranch(branchOptionValue);
-                                        dataAccessExcelCase.WriteData(commits.CommitDetails);
-                                        break;
-                                }
-                                break;
-                            } else if (branchOptionValue == null && tagOptionValue != null) {
-                                commits.GetCommitsByTag(tagOptionValue);
-                                dataAccessExcelCase.WriteData(commits.CommitDetails);
-                            }
-                            break;
-                        case "pdf":
-                            PdfDataService pdfDataService = new PdfDataService();
-                            DataAccess dataAccessPdfCase = new DataAccess(pdfDataService);
-
-                            switch (branchOptionValue) 
-                            {
-                                case null:
-                                    commits.GetCurrentCommitsByName();
-                                    dataAccessPdfCase.WriteData(commits.CommitDetails);
-                                    break;
-                                default:
-                                    commits.GetCommitsByBranch(branchOptionValue);
-                                    dataAccessPdfCase.WriteData(commits.CommitDetails);
-                                    break;
-                            }
-                            break;
-                        case null:
-                            StdOutDataService stdOutDataService = new StdOutDataService();
-                            DataAccess dataAccessNullCase = new DataAccess(stdOutDataService);
-
-                            if (branchOptionValue != null && tagOptionValue != null) {
-                                Console.WriteLine("Please specify either a branch or a tag.");
-                                Environment.Exit(2);
-                            } else if (branchOptionValue != null && tagOptionValue == null) {
-                                switch (branchOptionValue)
-                                {
-                                    case null:
-                                        commits.GetCurrentCommitsByName();
-                                        dataAccessNullCase.WriteData(commits.CommitDetails);
-                                        break;
-                                    default:
-                                        commits.GetCommitsByBranch(branchOptionValue);
-                                        dataAccessNullCase.WriteData(commits.CommitDetails);
-                                        break;
-                                }
-                                break;
-                            } else if (branchOptionValue == null && tagOptionValue != null) {
-                                commits.GetCommitsByTag(tagOptionValue);
-                                dataAccessNullCase.WriteData(commits.CommitDetails);
-                            } else {
                                 commits.GetCurrentCommitsByName();
-                                dataAccessNullCase.WriteData(commits.CommitDetails);
+                                dataAccess.WriteData(commits.CommitDetails);
-                            }
+                                break;
+                            default:
+                                commits.GetCommitsByBranch(branchOptionValue);
+                                dataAccess.WriteData(commits.CommitDetails);
+                                break;
+                        }
+                    }
+                    else if (branchOptionValue == null && tagOptionValue != null)
+                    {
+                        commits.GetCommitsByTag(tagOptionValue);
+                        dataAccess.WriteData(commits.CommitDetails);
+                    }
+                    break;
+                case "xlsx":
+                    ExcelDataService excelDataService = new ExcelDataService();
+                    DataAccess dataAccessExcelCase = new DataAccess(excelDataService);
+
+                    if (branchOptionValue != null && tagOptionValue != null)
+                    {
+                        Console.WriteLine("Please specify either a branch or a tag.");
+                        Environment.Exit(2);
+                    }
+                    else if (branchOptionValue != null && tagOptionValue == null)
+                    {
+                        switch (branchOptionValue)
+                        {
+                            case null:
+                                commits.GetCurrentCommitsByName();
+                                dataAccessExcelCase.WriteData(commits.CommitDetails);
+                                break;
+                            default:
+                                commits.GetCommitsByBranch(branchOptionValue);
+                                dataAccessExcelCase.WriteData(commits.CommitDetails);
+                                break;
+                        }
+                    }
+                    else if (branchOptionValue == null && tagOptionValue != null)
+                    {
+                        commits.GetCommitsByTag(tagOptionValue);
+                        dataAccessExcelCase.WriteData(commits.CommitDetails);
+                    }
+                    break;
+                case "pdf":
+                    PdfDataService pdfDataService = new PdfDataService();
+                    DataAccess dataAccessPdfCase = new DataAccess(pdfDataService);
+
+                    switch (branchOptionValue)
+                    {
+                        case null:
+                            commits.GetCurrentCommitsByName();
+                            dataAccessPdfCase.WriteData(commits.CommitDetails);
                             break;
                         default:
-                            System.Console.WriteLine("This should not happen...");
+                            commits.GetCommitsByBranch(branchOptionValue);
-                            Environment.Exit(4);
+                            dataAccessPdfCase.WriteData(commits.CommitDetails);
                             break;
                     }
-                },
+                    break;
-                outputOption, branchOption, tagOption);
+                case null:
+                    StdOutDataService stdOutDataService = new StdOutDataService();
+                    DataAccess dataAccessNullCase = new DataAccess(stdOutDataService);
+
+                    if (branchOptionValue != null && tagOptionValue != null)
+                    {
+                        Console.WriteLine("Please specify either a branch or a tag.");
+                        Environment.Exit(2);
+                    }
+                    else if (branchOptionValue != null && tagOptionValue == null)
+                    {
+                        switch (branchOptionValue)
+                        {
+                            case null:
+                                commits.GetCurrentCommitsByName();
+                                dataAccessNullCase.WriteData(commits.CommitDetails);
+                                break;
+                            default:
+                                commits.GetCommitsByBranch(branchOptionValue);
+                                dataAccessNullCase.WriteData(commits.CommitDetails);
+                                break;
+                        }
+                    }
+                    else if (branchOptionValue == null && tagOptionValue != null)
+                    {
+                        commits.GetCommitsByTag(tagOptionValue);
+                        dataAccessNullCase.WriteData(commits.CommitDetails);
+                    }
+                    else
+                    {
+                        commits.GetCurrentCommitsByName();
+                        dataAccessNullCase.WriteData(commits.CommitDetails);
+                    }
+                    break;
+                default:
+                    System.Console.WriteLine("This should not happen...");
+                    Environment.Exit(4);
+                    break;
+            }
+        },
+                outputOption, branchOption, tagOption, dirOption);
 
         rootCommand.Invoke(args);
     }