beaker 4.36.0 → 4.37.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 156fe839f18012ce2b3dcdc385123d328158e21f33e36b6e11debc5a67f51cee
-  data.tar.gz: 0dac80928255da4606d5f5710ffb3f569c93afa20d52039318150acafb883ed5
+  metadata.gz: f0d409c6cd07fc46ac641d68560cfe5ab40515df89f1a15bf6521b9f3bc044d9
+  data.tar.gz: e4d0909c8e1cb8190d35e740ee5c954f9e50241b4e763f416ed2d1374c92d2c2
 SHA512:
-  metadata.gz: '005694a4e2e756da6b1221833476bb0789398b0a78e9923ec0bb16a0081b21fbb1a33a8a7f56811da58f3225a0fb4f3e26f2a00dac91745197e20cefa64fde12'
-  data.tar.gz: 443607f9a33182feef6360a6af2d1956457a581a15156e5c9c2498a132427971332a19ffaf9b18218af9f9f9031c2420a7a2b9a8713b42ceecab50d0a5e44fff
+  metadata.gz: 45402a7f027808b4a3b3b7e797cf6fcdfb620d3dd7af45e0e07771fb289c21f7bb65265698a1225eeac15629542738cd0e1f67ae5dfd29bb1e1abeef9f5db554
+  data.tar.gz: 48f6623ca4b0f61dd737f1ef16d1258558aa238ba57bbf8596cd69b3403d5b3883d8d89977d73fd637c526b5665fe12d03af3fd68d38c37d646031cffac506d0

data/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## [4.37.1](https://github.com/voxpupuli/beaker/tree/4.37.1) (2022-07-27)
+
+## Fixed
+
+- macOS PE tarballs include arch now ([#1747](https://github.com/voxpupuli/beaker/pull/1747))
+
+## [4.37.0](https://github.com/voxpupuli/beaker/tree/4.37.0) (2022-06-27)
+
+## Added
+
+- Add support for Win32-OpenSSH ([#1744](https://github.com/voxpupuli/beaker/pull/1744))
+
+## Fixed
+
+- Create ~/.ssh on Windows if it doesn't exist ([#1745](https://github.com/voxpupuli/beaker/pull/1745))
+
+## [4.36.1](https://github.com/voxpupuli/beaker/tree/4.36.1) (2022-06-16)
+
+## Fixed
+
+- (maint) Remove /etc/environment file for ubuntu2204 ([#1742](https://github.com/voxpupuli/beaker/pull/1742))
+
 ## [4.36.0](https://github.com/voxpupuli/beaker/tree/4.36.0) (2022-05-30)
 
 ## Added

data/lib/beaker/host/mac/pkg.rb

@@ -118,7 +118,15 @@ module Mac::Pkg
 
     variant, version, arch, codename = self['platform'].to_array
     release_file = "/repos/apple/#{version}/#{puppet_collection}/#{arch}/puppet-agent-*"
-    download_file = "puppet-agent-#{variant}-#{version}.tar.gz"
+
+    # macOS puppet-agent tarballs haven't always included arch
+    agent_version = opts[:puppet_agent_version]
+    download_file = if agent_version && (agent_version.to_f < 6.28 || agent_version.to_f < 7.18)
+                      "puppet-agent-#{variant}-#{version}.tar.gz"
+                    else
+                      "puppet-agent-#{variant}-#{version}-#{arch}.tar.gz"
+                    end
+
     return '', release_file, download_file
   end
 

data/lib/beaker/host/unix/exec.rb

@@ -306,7 +306,7 @@ module Unix::Exec
       directory = tmpdir()
       exec(Beaker::Command.new("echo 'PermitUserEnvironment yes' | cat - /etc/ssh/sshd_config > #{directory}/sshd_config.permit"))
       exec(Beaker::Command.new("mv #{directory}/sshd_config.permit /etc/ssh/sshd_config"))
-      exec(Beaker::Command.new("echo '' >/etc/environment")) if self['platform'] =~ /ubuntu-20.04/
+      exec(Beaker::Command.new("echo '' >/etc/environment")) if self['platform'] =~ /ubuntu-2(0|2).04/
     when /(free|open)bsd/
       exec(Beaker::Command.new("sudo perl -pi -e 's/^#?PermitUserEnvironment no/PermitUserEnvironment yes/' /etc/ssh/sshd_config"), {:pty => true} )
     else

data/lib/beaker/host/windows/exec.rb

@@ -120,10 +120,16 @@ module Windows::Exec
   end
 
   # Create the provided directory structure on the host
-  # @param [String] dir The directory structure to create on the host
+  # @param [String,Pathname] dir The directory structure to create on the host
   # @return [Boolean] True, if directory construction succeeded, otherwise False
-  def mkdir_p dir
-    cmd = "mkdir -p \"#{dir}\""
+  def mkdir_p(dir)
+    # single or double quotes will disable ~ expansion, so only quote if we have to
+    str = dir.to_s
+    cmd = if str.start_with?('~') && !str.include?(' ')
+            "mkdir -p #{str}"
+          else
+            "mkdir -p \"#{str}\""
+          end
     result = exec(Beaker::Command.new(cmd), :acceptable_exit_codes => [0, 1])
     result.exit_code == 0
   end

data/lib/beaker/host/windows/file.rb

@@ -59,11 +59,13 @@ module Windows::File
     case determine_ssh_server
     when :bitvise
       # swap out separators
-      network_path = path.gsub('\\', scp_separator)
+      path.gsub('\\', scp_separator)
     when :openssh
       path
+    when :win32_openssh
+      path.gsub('\\', '/')
     else
-      raise ArgumentError("windows/file.rb:scp_path: ssh server not recognized: '#{determine_ssh_server}'")
+      raise ArgumentError, "windows/file.rb:scp_path: ssh server not recognized: '#{determine_ssh_server}'"
     end
   end
 

data/lib/beaker/host/windows.rb

@@ -38,8 +38,14 @@ module Windows
       return @ssh_server if @ssh_server
       @ssh_server = :openssh
       status = execute('cmd.exe /c sc query BvSshServer', :accept_all_exit_codes => true)
-      @ssh_server = :bitvise if status =~ /4  RUNNING/
-      logger.debug("windows.rb:determine_ssh_server: determined ssh server: '#{@ssh_server}'")
+      if status =~ /4  RUNNING/
+        @ssh_server = :bitvise
+      else
+        status = execute('cmd.exe /c sc qc sshd', :accept_all_exit_codes => true)
+        if status =~ /C:\\Windows\\System32\\OpenSSH\\sshd\.exe/
+          @ssh_server = :win32_openssh
+        end
+      end
       @ssh_server
     end
 

data/lib/beaker/version.rb

@@ -1,5 +1,5 @@
 module Beaker
   module Version
-    STRING = '4.36.0'
+    STRING = '4.37.1'
   end
 end

data/spec/beaker/host/windows/exec_spec.rb

@@ -96,5 +96,35 @@ module Beaker
          expect( instance.mv(origin, destination, false) ).to be === 0
       end
     end
+
+    context 'mkdir_p' do
+      let(:path) { '~/.ssh' }
+      let(:result) do
+        result = Beaker::Result.new(nil, '')
+        result.exit_code = 0
+        result
+      end
+
+      it 'accepts Pathname arguments' do
+        expect( instance ).to receive(:exec).with(having_attributes(command: "mkdir -p #{path}"), anything).and_return(result)
+        expect(instance.mkdir_p(Pathname.new(path))).to eq(true)
+      end
+
+      it 'omits quotes if the path starts with ~ and does not contain spaces' do
+        expect( instance ).to receive(:exec).with(having_attributes(command: "mkdir -p #{path}"), anything).and_return(result)
+        expect(instance.mkdir_p(path)).to eq(true)
+      end
+
+      it 'double quotes paths containing spaces' do
+        expect( instance ).to receive(:exec).with(having_attributes(command: "mkdir -p \"~/.foo bar\""), anything).and_return(result)
+        expect(instance.mkdir_p("~/.foo bar")).to eq(true)
+      end
+
+      it 'returns false if the command failed' do
+        result.exit_code = 1
+        expect( instance ).to receive(:exec).with(having_attributes(command: "mkdir -p #{path}"), anything).and_return(result)
+        expect(instance.mkdir_p(path)).to eq(false)
+      end
+    end
   end
 end

data/spec/beaker/host/windows/file_spec.rb

@@ -51,5 +51,37 @@ module Beaker
         host.ls_ld( path )
       end
     end
+
+    describe "#scp_to" do
+      let(:path) { 'C:\Windows' }
+
+      it 'replaces backslashes with ??? when using BitVise and cmd' do
+        allow(host).to receive(:determine_ssh_server).and_return(:bitvise)
+        expect(host.scp_path(path)).to eq('C:\Windows')
+      end
+
+      it 'replaces backslashes with forward slashes when using BitVise and powershell' do
+        host = make_host('name', platform: 'windows', is_cygwin: false)
+        allow(host).to receive(:determine_ssh_server).and_return(:bitvise)
+        expect(host.scp_path(path)).to eq('C:/Windows')
+      end
+
+      it 'leaves backslashes as is when using cygwin' do
+        allow(host).to receive(:determine_ssh_server).and_return(:openssh)
+        expect(host.scp_path(path)).to eq('C:\Windows')
+      end
+
+      it 'replace backslashes with forward slashes when using Win32-OpenSSH' do
+        allow(host).to receive(:determine_ssh_server).and_return(:win32_openssh)
+        expect(host.scp_path(path)).to eq('C:/Windows')
+      end
+
+      it 'raises if the server can not be recognized' do
+        allow(host).to receive(:determine_ssh_server).and_return(:unknown)
+        expect {
+          host.scp_path(path)
+        }.to raise_error(ArgumentError, "windows/file.rb:scp_path: ssh server not recognized: 'unknown'")
+      end
+    end
   end
 end

data/spec/beaker/host/windows_spec.rb

@@ -37,7 +37,7 @@ module Windows
 
     describe '#determine_ssh_server' do
       it 'does not care about return codes from the execute call' do
-        expect( host ).to receive( :execute ).with( anything, :accept_all_exit_codes => true )
+        expect( host ).to receive( :execute ).with( anything, :accept_all_exit_codes => true ).twice
         host.determine_ssh_server
       end
 
@@ -53,6 +53,28 @@ module Windows
         expect( host.determine_ssh_server ).to be === :bitvise
       end
 
+      it 'reads Windows OpenSSH status correctly' do
+        allow(host).to receive(:execute)
+          .with('cmd.exe /c sc query BvSshServer', anything).and_return(bitvise_check_output(:failure))
+        allow(host).to receive(:execute)
+          .with('cmd.exe /c sc qc sshd', anything).and_return(<<~END)
+        [SC] QueryServiceConfig SUCCESS
+
+        SERVICE_NAME: sshd
+                TYPE               : 10  WIN32_OWN_PROCESS 
+                START_TYPE         : 2   AUTO_START
+                ERROR_CONTROL      : 1   NORMAL
+                BINARY_PATH_NAME   : C:\\Windows\\System32\\OpenSSH\\sshd.exe
+                LOAD_ORDER_GROUP   : 
+                TAG                : 0
+                DISPLAY_NAME       : OpenSSH SSH Server
+                DEPENDENCIES       : 
+                SERVICE_START_NAME : LocalSystem
+        END
+
+        expect(host.determine_ssh_server).to eq :win32_openssh
+      end
+
       it 'returns old value if it has already determined before' do
         ssh_server_before = host.instance_variable_get( :@ssh_server )
         test_value = :test916

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: beaker
 version: !ruby/object:Gem::Version
-  version: 4.36.0
+  version: 4.37.1
 platform: ruby
 authors:
 - Puppet
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-05-30 00:00:00.000000000 Z
+date: 2022-07-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec