Merge pull request #107 from chef/mwrock/dir_in_command

prevent shellout from attempting to execute a directory on windows

1 files changed, 31 insertions, 0 deletions

diff --git a/spec/mixlib/shellout/windows_spec.rb b/spec/mixlib/shellout/windows_spec.rb
index d3fb26b..dd8a80b 100644
--- a/spec/mixlib/shellout/windows_spec.rb
+++ b/spec/mixlib/shellout/windows_spec.rb
@@ -159,6 +159,37 @@ describe 'Mixlib::ShellOut::Windows', :windows_only do
         end
       end
 
+      context 'with extensionless executable' do
+        let(:stubbed_shell_out) { shell_out }
+        let(:executable_path) { 'C:\Windows\system32/ping.EXE' }
+        let(:cmd) { 'ping' }
+
+        before do
+          allow(ENV).to receive(:[]).with('PATH').and_return('C:\Windows\system32')
+          allow(ENV).to receive(:[]).with('PATHEXT').and_return('.EXE')
+          allow(ENV).to receive(:[]).with('COMSPEC').and_return('C:\Windows\system32\cmd.exe')
+          allow(File).to receive(:executable?).and_return(false)
+          allow(File).to receive(:executable?).with(executable_path).and_return(true)
+          allow(File).to receive(:directory?).and_return(false)            
+        end
+
+        it 'should return with full path with extension' do
+          is_expected.to eql([executable_path, cmd])
+        end
+
+        context 'there is a directory named after command' do
+          before do
+            # File.executable? returns true for directories
+            allow(File).to receive(:executable?).with(cmd).and_return(true)
+            allow(File).to receive(:directory?).with(cmd).and_return(true)            
+          end
+
+          it 'should return with full path with extension' do
+            is_expected.to eql([executable_path, cmd])
+          end
+        end
+      end
+
       context 'with batch files' do
         let(:stubbed_shell_out) { shell_out.tap(&with_valid_exe_at_location) }
         let(:cmd_invocation) { "cmd /c \"#{cmd}\"" }