Introduce StatusOptions.IncludeUnaltered

Author: Therzok

LibGit2Sharp.Tests/StatusFixture.cs

@@ -592,5 +592,26 @@ public void RetrievingTheStatusHonorsAssumedUnchangedMarkedIndexEntries()
                 Assert.Equal("hello.txt", status.Modified.Single().FilePath);
             }
         }
+
+        [Fact]
+        public void CanIncludeStatusOfUnalteredFiles()
+        {
+            var path = SandboxStandardTestRepo();
+            string[] unalteredPaths = {
+                "1.txt",
+                "1" + Path.DirectorySeparatorChar + "branch_file.txt",
+                "branch_file.txt",
+                "new.txt",
+                "README",
+            };
+
+            using (var repo = new Repository(path))
+            {
+                RepositoryStatus status = repo.RetrieveStatus(new StatusOptions() { IncludeUnaltered = true });
+
+                Assert.Equal(unalteredPaths.Length, status.Unaltered.Count());
+                Assert.Equal(unalteredPaths, status.Unaltered.OrderBy(s => s.FilePath).Select(s => s.FilePath).ToArray());
+            }
+        }
     }
 }

LibGit2Sharp/RepositoryStatus.cs

@@ -26,6 +26,7 @@ public class RepositoryStatus : IEnumerable<StatusEntry>
         private readonly List<StatusEntry> ignored = new List<StatusEntry>();
         private readonly List<StatusEntry> renamedInIndex = new List<StatusEntry>();
         private readonly List<StatusEntry> renamedInWorkDir = new List<StatusEntry>();
+        private readonly List<StatusEntry> unaltered = new List<StatusEntry>();
         private readonly bool isDirty;
 
         private readonly IDictionary<FileStatus, Action<RepositoryStatus, StatusEntry>> dispatcher = Build();
@@ -42,7 +43,7 @@ private static IDictionary<FileStatus, Action<RepositoryStatus, StatusEntry>> Bu
                            { FileStatus.Removed, (rs, s) => rs.removed.Add(s) },
                            { FileStatus.RenamedInIndex, (rs, s) => rs.renamedInIndex.Add(s) },
                            { FileStatus.Ignored, (rs, s) => rs.ignored.Add(s) },
-                           { FileStatus.RenamedInWorkDir, (rs, s) => rs.renamedInWorkDir.Add(s) }
+                           { FileStatus.RenamedInWorkDir, (rs, s) => rs.renamedInWorkDir.Add(s) },
                        };
         }
 
@@ -81,7 +82,7 @@ internal RepositoryStatus(Repository repo, StatusOptions options)
                     AddStatusEntryForDelta(entry.Status, deltaHeadToIndex, deltaIndexToWorkDir);
                 }
 
-                isDirty = statusEntries.Any(entry => entry.State != FileStatus.Ignored);
+                isDirty = statusEntries.Any(entry => entry.State != FileStatus.Ignored && entry.State != FileStatus.Unaltered);
             }
         }
 
@@ -134,6 +135,12 @@ private static GitStatusOptions CreateStatusOptions(StatusOptions options)
                     GitStatusOptionFlags.DisablePathspecMatch;
             }
 
+            if (options.IncludeUnaltered)
+            {
+                coreOptions.Flags |=
+                    GitStatusOptionFlags.IncludeUnmodified;
+            }
+
             return coreOptions;
         }
 
@@ -164,14 +171,21 @@ private void AddStatusEntryForDelta(FileStatus gitStatus, GitDiffDelta deltaHead
 
             StatusEntry statusEntry = new StatusEntry(filePath, gitStatus, headToIndexRenameDetails, indexToWorkDirRenameDetails);
 
-            foreach (KeyValuePair<FileStatus, Action<RepositoryStatus, StatusEntry>> kvp in dispatcher)
+            if (gitStatus == FileStatus.Unaltered)
             {
-                if (!gitStatus.HasFlag(kvp.Key))
+                unaltered.Add(statusEntry);
+            }
+            else
+            {
+                foreach (KeyValuePair<FileStatus, Action<RepositoryStatus, StatusEntry>> kvp in dispatcher)
                 {
-                    continue;
-                }
+                    if (!gitStatus.HasFlag(kvp.Key))
+                    {
+                        continue;
+                    }
 
-                kvp.Value(this, statusEntry);
+                    kvp.Value(this, statusEntry);
+                }
             }
 
             statusEntries.Add(statusEntry);
@@ -289,6 +303,14 @@ public virtual IEnumerable<StatusEntry> RenamedInWorkDir
             get { return renamedInWorkDir; }
         }
 
+        /// <summary>
+        /// List of files that were unmodified in the working directory.
+        /// </summary>
+        public virtual IEnumerable<StatusEntry> Unaltered
+        {
+            get { return unaltered; }
+        }
+
         /// <summary>
         /// True if the index or the working directory has been altered since the last commit. False otherwise.
         /// </summary>

LibGit2Sharp/StatusOptions.cs

@@ -76,5 +76,12 @@ public StatusOptions()
         /// as explicit paths, and NOT as pathspecs containing globs.
         /// </summary>
         public bool DisablePathSpecMatch { get; set; }
+
+        /// Include unaltered files when scanning for status
+        /// </summary>
+        /// <remarks>
+        /// Unaltered meaning the file is identical in the working directory, the index and HEAD.
+        /// </remarks>
+        public bool IncludeUnaltered { get; set; }
     }
 }