Enforce testing of secure password credentials

Therzok · Therzok · commit 43565ce6a362 · 2015-06-24T11:43:16.000+03:00
Fixes libgit2#1114
diff --git a/LibGit2Sharp.Tests/CloneFixture.cs b/LibGit2Sharp.Tests/CloneFixture.cs
@@ -195,19 +195,34 @@ public void CanCloneWithCredentials()
             }
         }
 
+        static Credentials CreateUsernamePasswordCredentials (string user, string pass, bool secure)
+        {
+            if (secure)
+            {
+                return new SecureUsernamePasswordCredentials
+                {
+                    Username = user,
+                    Password = Constants.StringToSecureString(pass),
+                };
+            }
+
+            return new UsernamePasswordCredentials
+            {
+                Username = user,
+                Password = pass,
+            };
+        }
+
         [Theory]
-        [InlineData("https://libgit2@bitbucket.org/libgit2/testgitrepository.git", "libgit3", "libgit3")]
-        public void CanCloneFromBBWithCredentials(string url, string user, string pass)
+        [InlineData("https://libgit2@bitbucket.org/libgit2/testgitrepository.git", "libgit3", "libgit3", true)]
+        [InlineData("https://libgit2@bitbucket.org/libgit2/testgitrepository.git", "libgit3", "libgit3", false)]
+        public void CanCloneFromBBWithCredentials(string url, string user, string pass, bool secure)
         {
             var scd = BuildSelfCleaningDirectory();
 
             string clonedRepoPath = Repository.Clone(url, scd.DirectoryPath, new CloneOptions()
             {
-                CredentialsProvider = (_url, _user, _cred) => new UsernamePasswordCredentials
-                {
-                    Username = user,
-                    Password = pass,
-                }
+                CredentialsProvider = (_url, _user, _cred) => CreateUsernamePasswordCredentials (user, pass, secure)
             });
 
             using (var repo = new Repository(clonedRepoPath))
diff --git a/LibGit2Sharp.Tests/TestHelpers/Constants.cs b/LibGit2Sharp.Tests/TestHelpers/Constants.cs
@@ -94,7 +94,7 @@ private static string UnwrapUnixTempPath()
         }
 
         // To help with creating secure strings to test with.
-        private static SecureString StringToSecureString(string str)
+        internal static SecureString StringToSecureString(string str)
         {
             var chars = str.ToCharArray();
 

Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ private static string UnwrapUnixTempPath()`
`94`	`94`	`}`
`95`	`95`
`96`	`96`	`// To help with creating secure strings to test with.`
`97`		`- private static SecureString StringToSecureString(string str)`
	`97`	`+ internal static SecureString StringToSecureString(string str)`
`98`	`98`	`{`
`99`	`99`	`var chars = str.ToCharArray();`
`100`	`100`