combining ExecutableIsLocatedAt with Command to create FileName to be…

… executed
chucknorris · Jul 10, 2012 · 8c32217 · 8c32217
1 parent b65f329
commit 8c32217
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/product/dropkick.tests/Tasks/Console/PingTest.cs b/product/dropkick.tests/Tasks/Console/PingTest.cs
@@ -1,9 +1,12 @@
 namespace dropkick.tests.Tasks.Console
 {
+    using System;
     using dropkick.DeploymentModel;
     using dropkick.Tasks.CommandLine;
     using FileSystem;
     using NUnit.Framework;
+    using Path = System.IO.Path;
+    using System.Linq;
 
     [TestFixture]
     public class PingTest
@@ -29,5 +32,19 @@ public void Verify()
 
             //Assert.Contains(vi, r.Results);
         }
+
+        [Test]
+        public void VerifyCanRunWithExecutablePathSet()
+        {
+            var t = new LocalCommandLineTask(new DotNetPath(), "ping")
+            {
+                ExecutableIsLocatedAt =
+                    Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.Windows), "system32"),
+                    Args = "localhost"
+            };
+
+            var result = t.Execute();
+            result.Results.Select(r=> r.Status).ToList().ShouldContain(DeploymentItemStatus.Good);
+        }
     }
 }
diff --git a/product/dropkick/Tasks/CommandLine/LocalCommandLineTask.cs b/product/dropkick/Tasks/CommandLine/LocalCommandLineTask.cs
@@ -75,7 +75,7 @@ public DeploymentResult Execute()
 
             if (!string.IsNullOrEmpty(WorkingDirectory)) psi.WorkingDirectory = WorkingDirectory;
 
-            psi.FileName = _path.Combine(WorkingDirectory, Command);
+            psi.FileName = _path.Combine(ExecutableIsLocatedAt, Command);
 
             string output;
             try