Merge pull request #796 GetInsertionIndex should include the entry's name when it throws an InvalidOperationException

wilbaker · web-flow · commit 69d927547770 · 2019-02-19T09:29:10.000-08:00
GetInsertionIndex should include the entry's name when it throws an InvalidOperationException
diff --git a/GVFS/GVFS.Virtualization/Projection/GitIndexProjection.SortedFolderEntries.cs b/GVFS/GVFS.Virtualization/Projection/GitIndexProjection.SortedFolderEntries.cs
@@ -132,7 +132,7 @@ private int GetInsertionIndex(LazyUTF8String name)
                     insertionIndex = this.GetSortedEntriesIndexOfName(name);
                     if (insertionIndex >= 0)
                     {
-                        throw new InvalidOperationException("All entries should be unique");
+                        throw new InvalidOperationException($"All entries should be unique, non-unique entry: {name.GetString()}");
                     }
 
                     // When the name is not found the returned value is the bitwise complement of

Original file line number	Diff line number	Diff line change
`@@ -132,7 +132,7 @@ private int GetInsertionIndex(LazyUTF8String name)`
`132`	`132`	`insertionIndex = this.GetSortedEntriesIndexOfName(name);`
`133`	`133`	`if (insertionIndex >= 0)`
`134`	`134`	`{`
`135`		`- throw new InvalidOperationException("All entries should be unique");`
	`135`	`+ throw new InvalidOperationException($"All entries should be unique, non-unique entry: {name.GetString()}");`
`136`	`136`	`}`
`137`	`137`
`138`	`138`	`// When the name is not found the returned value is the bitwise complement of`