Refactor unwrapping of temp repository on unix

Fix #1040

Author: bording

LibGit2Sharp.Tests/BlobFixture.cs

@@ -222,7 +222,7 @@ public void CanTellIfTheBlobContentLooksLikeBinary()
 
         private static void SkipIfNotSupported(string autocrlf)
         {
-            InconclusiveIf(() => autocrlf == "true" && IsRunningOnUnix(), "Non-Windows does not support core.autocrlf = true");
+            InconclusiveIf(() => autocrlf == "true" && Constants.IsRunningOnUnix, "Non-Windows does not support core.autocrlf = true");
         }
     }
 }

LibGit2Sharp.Tests/ShadowCopyFixture.cs

@@ -57,7 +57,7 @@ public void CanProbeForNativeBinariesFromAShadowCopiedAssembly()
             string cachedAssembliesPath = Path.Combine(setup.CachePath, setup.ApplicationName);
             Assert.True(cachedAssemblyLocation.StartsWith(cachedAssembliesPath));
 
-            if (!IsRunningOnUnix())
+            if (!Constants.IsRunningOnUnix)
             {
                 // ...that this cache doesn't contain the `NativeBinaries` folder
                 string cachedAssemblyParentPath = Path.GetDirectoryName(cachedAssemblyLocation);

LibGit2Sharp.Tests/TestHelpers/BaseFixture.cs

@@ -106,14 +106,6 @@ private static bool IsFileSystemCaseSensitiveInternal()
             return !isInsensitive;
         }
 
-        // Should match LibGit2Sharp.Core.NativeMethods.IsRunningOnUnix()
-        protected static bool IsRunningOnUnix()
-        {
-            // see http://mono-project.com/FAQ%3a_Technical#Mono_Platforms
-            var p = (int)Environment.OSVersion.Platform;
-            return (p == 4) || (p == 6) || (p == 128);
-        }
-
         protected void CreateCorruptedDeadBeefHead(string repoPath)
         {
             const string deadbeef = "deadbeef";

LibGit2Sharp.Tests/TestHelpers/Constants.cs

@@ -1,16 +1,25 @@
 using System;
 using System.Diagnostics;
 using System.IO;
+using System.Reflection;
 
 namespace LibGit2Sharp.Tests.TestHelpers
 {
     public static class Constants
     {
+        public static readonly bool IsRunningOnUnix = IsUnixPlatform();
         public static readonly string TemporaryReposPath = BuildPath();
         public const string UnknownSha = "deadbeefdeadbeefdeadbeefdeadbeefdeadbeef";
         public static readonly Identity Identity = new Identity("A. U. Thor", "thor@valhalla.asgard.com");
         public static readonly Signature Signature = new Signature(Identity, new DateTimeOffset(2011, 06, 16, 10, 58, 27, TimeSpan.FromHours(2)));
 
+        private static bool IsUnixPlatform()
+        {
+            // see http://mono-project.com/FAQ%3a_Technical#Mono_Platforms
+            var p = (int)Environment.OSVersion.Platform;
+            return (p == 4) || (p == 6) || (p == 128);
+        }
+
         // Populate these to turn on live credential tests:  set the
         // PrivateRepoUrl to the URL of a repository that requires
         // authentication. Define PrivateRepoCredentials to return an instance of
@@ -40,19 +49,15 @@ public static string BuildPath()
         {
             string tempPath = null;
 
-            var unixPath = Type.GetType("Mono.Unix.UnixPath, Mono.Posix, Version=2.0.0.0, Culture=neutral, PublicKeyToken=0738eb9f132ed756");
-
-            if (unixPath != null)
+            if (IsRunningOnUnix)
             {
                 // We're running on Mono/*nix. Let's unwrap the path
-                tempPath = (string)unixPath.InvokeMember("GetCompleteRealPath",
-                    System.Reflection.BindingFlags.Static | System.Reflection.BindingFlags.FlattenHierarchy |
-                    System.Reflection.BindingFlags.InvokeMethod | System.Reflection.BindingFlags.Public,
-                    null, unixPath, new object[] { Path.GetTempPath() });
+                tempPath = UnwrapUnixTempPath();
             }
             else
             {
                 const string LibGit2TestPath = "LibGit2TestPath";
+
                 // We're running on .Net/Windows
                 if (Environment.GetEnvironmentVariables().Contains(LibGit2TestPath))
                 {
@@ -72,6 +77,16 @@ public static string BuildPath()
             return testWorkingDirectory;
         }
 
+        private static string UnwrapUnixTempPath()
+        {
+            var type = Type.GetType("Mono.Unix.UnixPath, Mono.Posix, Version=2.0.0.0, Culture=neutral, PublicKeyToken=0738eb9f132ed756");
+
+            return (string)type.InvokeMember("GetCompleteRealPath",
+                BindingFlags.Static | BindingFlags.FlattenHierarchy |
+                BindingFlags.InvokeMethod | BindingFlags.Public,
+                null, type, new object[] { Path.GetTempPath() });
+        }
+
         // To help with creating secure strings to test with.
         private static System.Security.SecureString StringToSecureString(string str)
         {