Improved exception on missing source files in TestFileSystem.Copy()

asbjornu · asbjornu · commit 552c0300f179 · 2016-04-28T11:48:59.000+02:00
diff --git a/src/GitVersionCore.Tests/TestFileSystem.cs b/src/GitVersionCore.Tests/TestFileSystem.cs
@@ -18,7 +18,12 @@ public void Copy(string @from, string to, bool overwrite)
             else
                 throw new IOException("File already exists");
         }
-        fileSystem.Add(to, fileSystem[from]);
+
+        string source;
+        if (!fileSystem.TryGetValue(from, out source))
+            throw new FileNotFoundException(string.Format("The source file '{0}' was not found", from), from);
+
+        fileSystem.Add(to, source);
     }
 
     public void Move(string @from, string to)

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,12 @@ public void Copy(string @from, string to, bool overwrite)`
`18`	`18`	`else`
`19`	`19`	`throw new IOException("File already exists");`
`20`	`20`	`}`
`21`		`- fileSystem.Add(to, fileSystem[from]);`
	`21`	`+`
	`22`	`+ string source;`
	`23`	`+ if (!fileSystem.TryGetValue(from, out source))`
	`24`	`+ throw new FileNotFoundException(string.Format("The source file '{0}' was not found", from), from);`
	`25`	`+`
	`26`	`+ fileSystem.Add(to, source);`
`22`	`27`	`}`
`23`	`28`
`24`	`29`	`public void Move(string @from, string to)`