sshkit 1.22.2 → 1.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b7db39cc723be7334eecf9f54d9e7a8ca8cd03c22514956d0a5fda5e58764ba
-  data.tar.gz: 431b7368d840461e7bb47e5ae6af21de3995da4a38492bc193fc3a902a0ec945
+  metadata.gz: f40ad7a1382ef707a259094c945692f08c44392e0bba7c37fe4099e748301a5e
+  data.tar.gz: 72f3fea395eaaa0036dd701c04692e75c379867465cffaa4ac6cdf1c4f647813
 SHA512:
-  metadata.gz: 7829a6485761cc5fc79ffb44723c15089004b4267e4e54e028db7a2cbf4cc074d57541391bfa89e0b60c239533dbe240f72f2fddf6df1b3997f301b93c268dd5
-  data.tar.gz: 300efa2a1398ecfd27a420536aa9203a283d116318cf68ff4ecb94ee229c500a9c93134db93504075e95f3add04460fad12c71982a3551a742eacbadd150d10e
+  metadata.gz: ab28074eba7cb9bdbfcbca9857f5e74fc89ed06acdf872148fcb7329b45754a714c658dacd879fead076de3bbd2541d84bcfd8ef66369007a1444b3edc2b3ac3
+  data.tar.gz: a6d0deb01db101ba2a41dcfcd861a10c7604b17ebe5e25981eebe6d74a0b5c9a9ea345d2dcbb45a4ba30952b571b6736f3bdcd7721d7d8fe684db772cc723700

data/.docker/Dockerfile

@@ -0,0 +1,6 @@
+FROM ubuntu:22.04
+WORKDIR /provision
+COPY ./ubuntu_setup.sh ./
+RUN ./ubuntu_setup.sh
+EXPOSE 22
+CMD ["/usr/sbin/sshd", "-D"]

data/.docker/ubuntu_setup.sh

@@ -0,0 +1,22 @@
+#!/bin/bash
+
+set -e
+
+export DEBIAN_FRONTEND=noninteractive
+apt -y update
+
+# Create `deployer` user that can sudo without a password
+apt-get -y install sudo
+adduser --disabled-password deployer < /dev/null
+echo "deployer:topsecret" | chpasswd
+echo "deployer ALL=(ALL) NOPASSWD:ALL" >> /etc/sudoers
+
+# Install and configure sshd
+apt-get -y install openssh-server
+{
+  echo "Port 22"
+  echo "PasswordAuthentication yes"
+  echo "ChallengeResponseAuthentication no"
+} >> /etc/ssh/sshd_config
+mkdir /var/run/sshd
+chmod 0755 /var/run/sshd

data/.github/release-drafter.yml

@@ -1,5 +1,5 @@
-name-template: "$NEXT_PATCH_VERSION"
-tag-template: "v$NEXT_PATCH_VERSION"
+name-template: "$RESOLVED_VERSION"
+tag-template: "v$RESOLVED_VERSION"
 categories:
   - title: "⚠️ Breaking Changes"
     label: "⚠️ Breaking"
@@ -11,7 +11,15 @@ categories:
     label: "📚 Docs"
   - title: "🏠 Housekeeping"
     label: "🏠 Housekeeping"
+version-resolver:
+  minor:
+    labels:
+      - "⚠️ Breaking"
+      - "✨ Feature"
+  default: patch
 change-template: "- $TITLE (#$NUMBER) @$AUTHOR"
 no-changes-template: "- No changes"
 template: |
   $CHANGES
+
+  **Full Changelog:** https://github.com/$OWNER/$REPOSITORY/compare/$PREVIOUS_TAG...v$RESOLVED_VERSION

data/.github/workflows/ci.yml

@@ -69,3 +69,30 @@ jobs:
           bundler-cache: true
       - name: Run rubocop
         run: bundle exec rake lint
+
+  functional:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby: ["2.0", "ruby"]
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - name: Run functional tests
+        run: bundle exec rake test:functional
+
+  functional-all:
+    runs-on: ubuntu-latest
+    needs: [functional]
+    if: always()
+    steps:
+      - name: All tests ok
+        if: ${{ !(contains(needs.*.result, 'failure')) }}
+        run: exit 0
+      - name: Some tests failed
+        if: ${{ contains(needs.*.result, 'failure') }}
+        run: exit 1

data/.gitignore

@@ -3,6 +3,5 @@
 bin/rake
 .bundle
 .yardoc
-.vagrant*
 test/tmp
 Gemfile.lock

data/.rubocop_todo.yml

@@ -102,7 +102,6 @@ Layout/IndentHash:
   Exclude:
     - 'test/functional/backends/test_local.rb'
     - 'test/functional/backends/test_netssh.rb'
-    - 'test/support/vagrant_wrapper.rb'
     - 'test/unit/formatters/test_custom.rb'
     - 'test/unit/formatters/test_pretty.rb'
     - 'test/unit/test_mapping_interaction_handler.rb'
@@ -445,12 +444,6 @@ Style/MethodName:
   Exclude:
     - 'test/unit/test_color.rb'
 
-# Offense count: 1
-# Cop supports --auto-correct.
-Style/MutableConstant:
-  Exclude:
-    - 'Vagrantfile'
-
 # Offense count: 1
 # Cop supports --auto-correct.
 # Configuration parameters: Strict.

data/CONTRIBUTING.md

@@ -24,8 +24,8 @@ using unsupported features.
 
 ## Tests
 
-SSHKit has a unit test suite and a functional test suite. Some functional tests run against
-[Vagrant](https://www.vagrantup.com/) VMs. If possible, you should make sure that the
+SSHKit has a unit test suite and a functional test suite. Some functional tests run using
+[Docker](https://docs.docker.com/get-docker/). If possible, you should make sure that the
 tests pass for each commit by running `rake` in the sshkit directory. This is in case we
 need to cherry pick commits or rebase. You should ensure the tests pass, (preferably on
 the minimum and maximum ruby version), before creating a PR.

data/RELEASING.md

@@ -9,7 +9,7 @@
 ## How to release
 
 1. Run `bundle install` to make sure that you have all the gems necessary for testing and releasing.
-2.  **Ensure the tests are passing by running `rake test`.** If functional tests fail, ensure you have [Vagrant](https://www.vagrantup.com) installed and have started it with `vagrant up`.
+2.  **Ensure the tests are passing by running `rake test`.** If functional tests fail, ensure you have [Docker installed](https://docs.docker.com/get-docker/) and running.
 3. Determine which would be the correct next version number according to [semver](http://semver.org/).
 4. Update the version in `./lib/sshkit/version.rb`.
 5. Commit the `version.rb` change with a message like "Preparing vX.Y.Z"

data/Rakefile

@@ -21,10 +21,6 @@ namespace :test do
 
 end
 
-Rake::Task["test:functional"].enhance do
-  warn "Remember there are still some VMs running, kill them with `vagrant halt` if you are finished using them."
-end
-
 desc 'Run RuboCop lint checks'
 RuboCop::RakeTask.new(:lint) do |task|
   task.options = ['--lint']

data/docker-compose.yml

@@ -0,0 +1,8 @@
+name: sshkit
+
+services:
+  ssh_server:
+    build:
+      context: .docker
+    ports:
+      - "2122:22"

data/lib/sshkit/backends/netssh/known_hosts.rb

@@ -7,12 +7,11 @@ module SSHKit
     class Netssh < Abstract
 
       class KnownHostsKeys
-        include Mutex_m
-
         def initialize(path)
           super()
           @path = File.expand_path(path)
           @hosts_keys = nil
+          @mutex = Mutex.new
         end
 
         def keys_for(hostlist)
@@ -46,7 +45,7 @@ module SSHKit
         end
 
         def parse_file
-          synchronize do
+          @mutex.synchronize do
             return if hosts_keys && hosts_hashes
 
             unless File.readable?(path)
@@ -112,11 +111,10 @@ module SSHKit
       end
 
       class KnownHosts
-        include Mutex_m
-
         def initialize
           super()
           @files = {}
+          @mutex = Mutex.new
         end
 
         def search_for(host, options = {})
@@ -128,13 +126,13 @@ module SSHKit
 
         def add(*args)
           ::Net::SSH::KnownHosts.add(*args)
-          synchronize { @files = {} }
+          @mutex.synchronize { @files = {} }
         end
 
         private
 
         def known_hosts_file(path)
-          @files[path] || synchronize { @files[path] ||= KnownHostsKeys.new(path) }
+          @files[path] || @mutex.synchronize { @files[path] ||= KnownHostsKeys.new(path) }
         end
       end
 

data/lib/sshkit/backends/netssh.rb

@@ -1,6 +1,5 @@
 require 'English'
 require 'strscan'
-require 'mutex_m'
 require 'net/ssh'
 
 module Net

data/lib/sshkit/host.rb

@@ -1,4 +1,5 @@
 require 'ostruct'
+require 'resolv'
 
 module SSHKit
 
@@ -122,6 +123,22 @@ module SSHKit
 
   end
 
+  # @private
+  # :nodoc:
+  class IPv6HostParser < SimpleHostParser
+    def self.suitable?(host_string)
+      host_string.match(Resolv::IPv6::Regex)
+    end
+
+    def port
+
+    end
+
+    def hostname
+      @host_string.match(Resolv::IPv6::Regex)[0]
+    end
+  end
+
   class HostWithPortParser < SimpleHostParser
 
     def self.suitable?(host_string)
@@ -192,6 +209,7 @@ module SSHKit
 
   PARSERS = [
     SimpleHostParser,
+    IPv6HostParser,
     HostWithPortParser,
     HostWithUsernameAndPortParser,
     IPv6HostWithPortParser,

data/lib/sshkit/version.rb

@@ -1,3 +1,3 @@
 module SSHKit
-  VERSION = "1.22.2".freeze
+  VERSION = "1.23.0".freeze
 end

data/sshkit.gemspec

@@ -21,7 +21,6 @@ Gem::Specification.new do |gem|
   gem.version       = SSHKit::VERSION
 
   gem.add_runtime_dependency('base64') if RUBY_VERSION >= "2.4"
-  gem.add_runtime_dependency('mutex_m')
   gem.add_runtime_dependency('net-ssh',  '>= 2.8.0')
   gem.add_runtime_dependency('net-scp',  '>= 1.1.2')
   gem.add_runtime_dependency('net-sftp', '>= 2.1.2')

data/test/functional/backends/netssh_transfer_tests.rb

@@ -11,7 +11,7 @@ module SSHKit
       end
 
       def a_host
-        VagrantWrapper.hosts['one']
+        DockerWrapper.host
       end
 
       def test_upload_and_then_capture_file_contents

data/test/functional/backends/test_netssh.rb

@@ -15,7 +15,7 @@ module SSHKit
       end
 
       def a_host
-        VagrantWrapper.hosts['one']
+        DockerWrapper.host
       end
 
       def test_simple_netssh

data/test/helper.rb

@@ -28,51 +28,13 @@ class UnitTest < Minitest::Test
 end
 
 class FunctionalTest < Minitest::Test
-
   def setup
-    unless VagrantWrapper.running?
-      warn "Vagrant VMs are not running. Please, start it manually with `vagrant up`"
-    end
-  end
-
-  private
-
-  def create_user_with_key(username, password = :secret)
-    username, password = username.to_s, password.to_s
-
-    keys = VagrantWrapper.hosts.collect do |_name, host|
-      Net::SSH.start(host.hostname, host.user, port: host.port, password: host.password) do |ssh|
-
-        # Remove the user, make it again, force-generate a key for him
-        # short keys save us a few microseconds
-        ssh.exec!("sudo userdel -rf #{username}; true") # The `rescue nil` of the shell world
-        ssh.exec!("sudo useradd -m #{username}")
-        ssh.exec!("sudo echo y | ssh-keygen -b 1024 -f #{username} -N ''")
-        ssh.exec!("sudo chown vagrant:vagrant #{username}*")
-        ssh.exec!("sudo echo #{username}:#{password} | chpasswd")
+    require_relative "support/docker_wrapper"
+    return if DockerWrapper.running?
 
-        # Make the .ssh directory, change the ownership and the
-        ssh.exec!("sudo mkdir -p /home/#{username}/.ssh")
-        ssh.exec!("sudo chown #{username}:#{username} /home/#{username}/.ssh")
-        ssh.exec!("sudo chmod 700 /home/#{username}/.ssh")
-
-        # Move the key to authorized keys and chown and chmod it
-        ssh.exec!("sudo cat #{username}.pub > /home/#{username}/.ssh/authorized_keys")
-        ssh.exec!("sudo chown #{username}:#{username} /home/#{username}/.ssh/authorized_keys")
-        ssh.exec!("sudo chmod 600 /home/#{username}/.ssh/authorized_keys")
-
-        key = ssh.exec!("cat /home/vagrant/#{username}")
-
-        # Clean Up Files
-        ssh.exec!("sudo rm #{username} #{username}.pub")
-
-        key
-      end
-    end
-
-    Hash[VagrantWrapper.hosts.collect { |n, _h| n.to_sym }.zip(keys)]
+    DockerWrapper.start
+    DockerWrapper.wait_for_ssh_server
   end
-
 end
 
 #

data/test/support/docker_wrapper.rb

@@ -0,0 +1,71 @@
+require "socket"
+
+Minitest.after_run do
+  DockerWrapper.stop if DockerWrapper.running?
+end
+
+module DockerWrapper
+  SSH_SERVER_PORT = 2122
+
+  class << self
+    def host
+      SSHKit::Host.new(
+        user: "deployer",
+        hostname: "localhost",
+        port: SSH_SERVER_PORT,
+        password: "topsecret",
+        ssh_options: host_verify_options
+      )
+    end
+
+    def running?
+      out, status = run_compose_command("ps --status running", false)
+      status.success? && out.include?("ssh_server")
+    end
+
+    def start
+      run_compose_command("up -d")
+    end
+
+    def stop
+      run_compose_command("down")
+    end
+
+    def wait_for_ssh_server(retries=3)
+      Socket.tcp("localhost", SSH_SERVER_PORT, connect_timeout: 1).close
+      sleep(1)
+    rescue Errno::ECONNREFUSED, Errno::ETIMEDOUT
+      retries -= 1
+      sleep(2) && retry if retries.positive?
+      raise
+    end
+
+    private
+
+    def run_compose_command(command, echo=true)
+      $stderr.puts "[docker compose] #{command}" if echo
+      Open3.popen2e("docker compose #{command}") do |stdin, outerr, wait_thread|
+        stdin.close
+        output = Thread.new { capture_stream(outerr, echo) }
+        [output.value, wait_thread.value]
+      end
+    end
+
+    def capture_stream(stream, echo=true)
+      buffer = String.new
+      while (line = stream.gets)
+        buffer << line
+        $stderr.puts("[docker compose] #{line}") if echo
+      end
+      buffer
+    end
+
+    def host_verify_options
+      if Net::SSH::Version::MAJOR >= 5
+        { verify_host_key: :never }
+      else
+        { paranoid: false }
+      end
+    end
+  end
+end

data/test/unit/test_command_map.rb

@@ -27,26 +27,26 @@ module SSHKit
 
     def test_prefix
       map = CommandMap.new
-      map.prefix[:rake].push("/home/vagrant/.rbenv/bin/rbenv exec")
+      map.prefix[:rake].push("/home/deployer/.rbenv/bin/rbenv exec")
       map.prefix[:rake].push("bundle exec")
 
-      assert_equal map[:rake], "/home/vagrant/.rbenv/bin/rbenv exec bundle exec rake"
+      assert_equal map[:rake], "/home/deployer/.rbenv/bin/rbenv exec bundle exec rake"
     end
 
     def test_prefix_procs
       map = CommandMap.new
-      map.prefix[:rake].push("/home/vagrant/.rbenv/bin/rbenv exec")
+      map.prefix[:rake].push("/home/deployer/.rbenv/bin/rbenv exec")
       map.prefix[:rake].push(proc{ "bundle exec" })
 
-      assert_equal map[:rake], "/home/vagrant/.rbenv/bin/rbenv exec bundle exec rake"
+      assert_equal map[:rake], "/home/deployer/.rbenv/bin/rbenv exec bundle exec rake"
     end
 
     def test_prefix_unshift
       map = CommandMap.new
       map.prefix[:rake].push("bundle exec")
-      map.prefix[:rake].unshift("/home/vagrant/.rbenv/bin/rbenv exec")
+      map.prefix[:rake].unshift("/home/deployer/.rbenv/bin/rbenv exec")
 
-      assert_equal map[:rake], "/home/vagrant/.rbenv/bin/rbenv exec bundle exec rake"
+      assert_equal map[:rake], "/home/deployer/.rbenv/bin/rbenv exec bundle exec rake"
     end
 
     def test_indifferent_setter
@@ -59,10 +59,10 @@ module SSHKit
 
     def test_indifferent_prefix
       map = CommandMap.new
-      map.prefix[:rake].push("/home/vagrant/.rbenv/bin/rbenv exec")
+      map.prefix[:rake].push("/home/deployer/.rbenv/bin/rbenv exec")
       map.prefix["rake"].push("bundle exec")
 
-      assert_equal map[:rake], "/home/vagrant/.rbenv/bin/rbenv exec bundle exec rake"
+      assert_equal map[:rake], "/home/deployer/.rbenv/bin/rbenv exec bundle exec rake"
     end
 
     def test_prefix_initialization_is_thread_safe

data/test/unit/test_host.rb

@@ -28,6 +28,12 @@ module SSHKit
       assert_equal 'example.com', h.hostname
     end
 
+    def test_custom_host_with_port
+      h = Host.new 'db:22'
+      assert_equal 22,   h.port
+      assert_equal 'db', h.hostname
+    end
+
     def test_host_with_username
       h = Host.new 'root@example.com'
       assert_equal 'root',        h.username
@@ -50,6 +56,12 @@ module SSHKit
       assert_equal 'localhost',         h.hostname
     end
 
+    def test_ipv6_without_brackets
+      h = Host.new '1fff:0:a88:85a3::ac1f'
+      assert_nil h.port
+      assert_equal '1fff:0:a88:85a3::ac1f', h.hostname
+    end
+
     def test_does_not_confuse_ipv6_hosts_with_port_specification
       h = Host.new '[1fff:0:a88:85a3::ac1f]:8001'
       assert_equal 8001,                    h.port

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sshkit
 version: !ruby/object:Gem::Version
-  version: 1.22.2
+  version: 1.23.0
 platform: ruby
 authors:
 - Lee Hambley
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-04-30 00:00:00.000000000 Z
+date: 2024-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -25,20 +25,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: mutex_m
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: net-ssh
   requirement: !ruby/object:Gem::Requirement
@@ -222,6 +208,8 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".docker/Dockerfile"
+- ".docker/ubuntu_setup.sh"
 - ".github/dependabot.yml"
 - ".github/release-drafter.yml"
 - ".github/workflows/ci.yml"
@@ -241,7 +229,7 @@ files:
 - README.md
 - RELEASING.md
 - Rakefile
-- Vagrantfile
+- docker-compose.yml
 - examples/images/example_output.png
 - examples/images/logo.png
 - examples/simple_connection.rb
@@ -284,18 +272,16 @@ files:
 - lib/sshkit/runners/sequential.rb
 - lib/sshkit/version.rb
 - sshkit.gemspec
-- test/boxes.json
 - test/functional/backends/netssh_transfer_tests.rb
 - test/functional/backends/test_local.rb
 - test/functional/backends/test_netssh.rb
 - test/functional/backends/test_netssh_scp.rb
 - test/functional/backends/test_netssh_sftp.rb
-- test/functional/test_ssh_server_comes_up_for_functional_tests.rb
 - test/helper.rb
 - test/known_hosts/github
 - test/known_hosts/github_hash
 - test/known_hosts/github_ip
-- test/support/vagrant_wrapper.rb
+- test/support/docker_wrapper.rb
 - test/unit/backends/test_abstract.rb
 - test/unit/backends/test_connection_pool.rb
 - test/unit/backends/test_local.rb
@@ -338,23 +324,21 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.9
+rubygems_version: 3.5.13
 signing_key:
 specification_version: 4
 summary: SSHKit makes it easy to write structured, testable SSH commands in Ruby
 test_files:
-- test/boxes.json
 - test/functional/backends/netssh_transfer_tests.rb
 - test/functional/backends/test_local.rb
 - test/functional/backends/test_netssh.rb
 - test/functional/backends/test_netssh_scp.rb
 - test/functional/backends/test_netssh_sftp.rb
-- test/functional/test_ssh_server_comes_up_for_functional_tests.rb
 - test/helper.rb
 - test/known_hosts/github
 - test/known_hosts/github_hash
 - test/known_hosts/github_ip
-- test/support/vagrant_wrapper.rb
+- test/support/docker_wrapper.rb
 - test/unit/backends/test_abstract.rb
 - test/unit/backends/test_connection_pool.rb
 - test/unit/backends/test_local.rb

data/Vagrantfile

@@ -1,24 +0,0 @@
-VAGRANTFILE_API_VERSION = "2"
-
-Vagrant.configure(VAGRANTFILE_API_VERSION) do |config|
-  config.vm.box = 'bento/ubuntu-22.10'
-
-  config.vm.boot_timeout = 600 # seconds
-  config.ssh.insert_key = false
-  config.vm.provision "shell", inline: <<-SHELL
-  echo 'ClientAliveInterval 3' >> /etc/ssh/sshd_config
-  echo 'ClientAliveCountMax 3' >> /etc/ssh/sshd_config
-  echo 'MaxAuthTries 6' >> /etc/ssh/sshd_config
-  service ssh restart
-  SHELL
-
-  json_config_path = File.join("test", "boxes.json")
-  list = File.open(json_config_path).read
-  list = JSON.parse(list)
-
-  list.each do |vm|
-    config.vm.define vm["name"] do |web|
-      web.vm.network "forwarded_port", guest: 22, host: vm["port"]
-    end
-  end
-end

data/test/boxes.json

@@ -1,17 +0,0 @@
-[
-  {
-    "name": "one",
-    "port": 3001,
-    "user": "vagrant",
-    "password": "vagrant",
-    "hostname": "localhost"
-  },
-  {
-    "name": "two",
-    "port": 3002
-  },
-  {
-    "name": "three",
-    "port": 3003
-  }
-]

data/test/functional/test_ssh_server_comes_up_for_functional_tests.rb

@@ -1,24 +0,0 @@
-require 'helper'
-
-module SSHKit
-
-  class TestHost < FunctionalTest
-
-    def host
-      @_host ||= Host.new('')
-    end
-
-    def test_that_it_works
-      assert true
-    end
-
-    def test_creating_a_user_gives_us_back_his_private_key_as_a_string
-      skip 'It is not safe to create an user for non vagrant envs' unless VagrantWrapper.running?
-      keys = create_user_with_key(:peter)
-      assert_equal [:one, :two, :three], keys.keys
-      assert keys.values.all?
-    end
-
-  end
-
-end

data/test/support/vagrant_wrapper.rb

@@ -1,64 +0,0 @@
-class VagrantWrapper
-  class << self
-    def hosts
-      @vm_hosts ||= begin
-        result = {}
-
-        boxes = boxes_list
-
-        unless running?
-          boxes.map! do |box|
-            box['user'] = ENV['USER']
-            box['port'] = '22'
-            box
-          end
-        end
-
-        boxes.each do |vm|
-          result[vm['name']] = vm_host(vm)
-        end
-
-        result
-      end
-    end
-
-    def running?
-      @running ||= begin
-        status = `#{vagrant_binary} status`
-        status.include?('running')
-      end
-    end
-
-    def boxes_list
-      json_config_path = File.join('test', 'boxes.json')
-      boxes = File.open(json_config_path).read
-      JSON.parse(boxes)
-    end
-
-    def vagrant_binary
-      'vagrant'
-    end
-
-    private
-
-    def vm_host(vm)
-      host_options = {
-          user: vm['user'] || 'vagrant',
-          hostname: vm['hostname'] || 'localhost',
-          port: vm['port'] || '22',
-          password: vm['password'] || 'vagrant',
-          ssh_options: host_verify_options
-      }
-
-      SSHKit::Host.new(host_options)
-    end
-
-    def host_verify_options
-      if Net::SSH::Version::MAJOR >= 5
-        { verify_host_key: :never }
-      else
-        { paranoid: false }
-      end
-    end
-  end
-end