Ensure sourceUrl is not null either

Ruben Verdoes · Ruben Verdoes · commit b24a00cb19a0 · 2014-12-22T16:15:02.000+01:00
diff --git a/LibGit2Sharp.Tests/CloneFixture.cs b/LibGit2Sharp.Tests/CloneFixture.cs
@@ -234,5 +234,13 @@ public void CloningWithoutWorkdirPathThrows(string url)
         {
             Assert.Throws<ArgumentNullException>(() => Repository.Clone(url, null));
         }
+
+        [Fact]
+        public void CloningWithoutUrlThrows()
+        {
+            var scd = BuildSelfCleaningDirectory();
+
+            Assert.Throws<ArgumentNullException>(() => Repository.Clone(null, scd.DirectoryPath));
+        }
     }
 }
diff --git a/LibGit2Sharp/Repository.cs b/LibGit2Sharp/Repository.cs
@@ -538,6 +538,7 @@ public static string Discover(string startingPath)
         public static string Clone(string sourceUrl, string workdirPath,
             CloneOptions options = null)
         {
+            Ensure.ArgumentNotNull(sourceUrl, "sourceUrl");
             Ensure.ArgumentNotNull(workdirPath, "workdirPath");
 
             options = options ?? new CloneOptions();

Original file line number	Diff line number	Diff line change
`@@ -234,5 +234,13 @@ public void CloningWithoutWorkdirPathThrows(string url)`
`234`	`234`	`{`
`235`	`235`	`Assert.Throws<ArgumentNullException>(() => Repository.Clone(url, null));`
`236`	`236`	`}`
	`237`	`+`
	`238`	`+ [Fact]`
	`239`	`+ public void CloningWithoutUrlThrows()`
	`240`	`+ {`
	`241`	`+ var scd = BuildSelfCleaningDirectory();`
	`242`	`+`
	`243`	`+ Assert.Throws<ArgumentNullException>(() => Repository.Clone(null, scd.DirectoryPath));`
	`244`	`+ }`
`237`	`245`	`}`
`238`	`246`	`}`
Original file line number	Diff line number	Diff line change
`@@ -538,6 +538,7 @@ public static string Discover(string startingPath)`
`538`	`538`	`public static string Clone(string sourceUrl, string workdirPath,`
`539`	`539`	`CloneOptions options = null)`
`540`	`540`	`{`
	`541`	`+ Ensure.ArgumentNotNull(sourceUrl, "sourceUrl");`
`541`	`542`	`Ensure.ArgumentNotNull(workdirPath, "workdirPath");`
`542`	`543`
`543`	`544`	`options = options ?? new CloneOptions();`