Add Guard.ArgumentIsGitPath to appropriate methods

jcansdale · jcansdale · commit 7d87a4086953 · 2019-01-09T17:00:21.000Z
diff --git a/src/GitHub.App/Services/GitClient.cs b/src/GitHub.App/Services/GitClient.cs
@@ -279,7 +279,7 @@ public Task<string> ExtractFile(IRepository repository, string commitSha, string
         {
             Guard.ArgumentNotNull(repository, nameof(repository));
             Guard.ArgumentNotEmptyString(commitSha, nameof(commitSha));
-            Guard.ArgumentNotEmptyString(fileName, nameof(fileName));
+            Guard.ArgumentIsGitPath(fileName, nameof(fileName));
 
             return Task.Run(() =>
             {
@@ -298,7 +298,7 @@ public Task<byte[]> ExtractFileBinary(IRepository repository, string commitSha,
         {
             Guard.ArgumentNotNull(repository, nameof(repository));
             Guard.ArgumentNotEmptyString(commitSha, nameof(commitSha));
-            Guard.ArgumentNotEmptyString(fileName, nameof(fileName));
+            Guard.ArgumentIsGitPath(fileName, nameof(fileName));
 
             return Task.Run(() =>
             {
@@ -327,7 +327,7 @@ public Task<byte[]> ExtractFileBinary(IRepository repository, string commitSha,
         public Task<bool> IsModified(IRepository repository, string path, byte[] contents)
         {
             Guard.ArgumentNotNull(repository, nameof(repository));
-            Guard.ArgumentNotEmptyString(path, nameof(path));
+            Guard.ArgumentIsGitPath(path, nameof(path));
 
             return Task.Run(() =>
             {

Original file line number	Diff line number	Diff line change
`@@ -279,7 +279,7 @@ public Task<string> ExtractFile(IRepository repository, string commitSha, string`
`279`	`279`	`{`
`280`	`280`	`Guard.ArgumentNotNull(repository, nameof(repository));`
`281`	`281`	`Guard.ArgumentNotEmptyString(commitSha, nameof(commitSha));`
`282`		`- Guard.ArgumentNotEmptyString(fileName, nameof(fileName));`
	`282`	`+ Guard.ArgumentIsGitPath(fileName, nameof(fileName));`
`283`	`283`
`284`	`284`	`return Task.Run(() =>`
`285`	`285`	`{`
`@@ -298,7 +298,7 @@ public Task<byte[]> ExtractFileBinary(IRepository repository, string commitSha,`
`298`	`298`	`{`
`299`	`299`	`Guard.ArgumentNotNull(repository, nameof(repository));`
`300`	`300`	`Guard.ArgumentNotEmptyString(commitSha, nameof(commitSha));`
`301`		`- Guard.ArgumentNotEmptyString(fileName, nameof(fileName));`
	`301`	`+ Guard.ArgumentIsGitPath(fileName, nameof(fileName));`
`302`	`302`
`303`	`303`	`return Task.Run(() =>`
`304`	`304`	`{`
`@@ -327,7 +327,7 @@ public Task<byte[]> ExtractFileBinary(IRepository repository, string commitSha,`
`327`	`327`	`public Task<bool> IsModified(IRepository repository, string path, byte[] contents)`
`328`	`328`	`{`
`329`	`329`	`Guard.ArgumentNotNull(repository, nameof(repository));`
`330`		`- Guard.ArgumentNotEmptyString(path, nameof(path));`
	`330`	`+ Guard.ArgumentIsGitPath(path, nameof(path));`
`331`	`331`
`332`	`332`	`return Task.Run(() =>`
`333`	`333`	`{`