busser 0.9.1 → 0.9.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c0be7ee63d2b1d111438e2039187a5283308daae897a27d6028c08d9cefa13d
-  data.tar.gz: 49e9d09408a9cc284cfb9b43f8daeac1f55eda961ad66697dbefe8741ce2976a
+  metadata.gz: 22ea0046cc1c5cb2f4473f9315c506ee3ebd4b22d7cf35de392060ae02816ae9
+  data.tar.gz: a42365f1541af1364e6eb82d06049a2910fc5fbceb63cf137e6aaad5243f1d64
 SHA512:
-  metadata.gz: 71b525573141abe2bf35be2559189a811328e608eaef9a1ead8390595b7d500e76344ef7d29c624028c42d3e7a288a23c41e2e4a09d288734101ee9de2975d35
-  data.tar.gz: 4d769038c5032cfefdc0b50217f77e6950d2121162c4a07bdbd7d3a7a4b1b794d35a7d501b94ad5621c7628e942cd8d4f56c90d87fa01d0e7aab722e0961fd54
+  metadata.gz: 4fe454f0801dff2a7dac3677a73f98055bf2368ccaa503a1fb641997cf0210482bfd9ff456f853e4e4a8a7e40c4291e8c67f4f9dfc65ce574596fd8be1123cbd
+  data.tar.gz: 3138b112b0ac5a405361553b0a20e9ea43fbbf5df16fbeb6be0b7cb325811e6c494e8d44998af2dc15c6fc163b7568ec681c3a8edd7dddff79018ade13eb46c1

data/.github/workflows/integration.yml

@@ -0,0 +1,55 @@
+---
+name: "Test Kitchen"
+
+"on":
+  push:
+    branches: [main]
+  pull_request:
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: ${{ github.event_name == 'pull_request' }}
+
+jobs:
+  kitchen:
+    name: "kitchen verify on Ruby ${{ matrix.ruby }}"
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        # The ends of the supported range. The unit and feature suites already
+        # cover everything in between; this job is about the Test Kitchen
+        # integration, not Ruby compatibility.
+        ruby: ["3.2", "4.0"]
+    env:
+      # Outside the checkout on purpose. The runners require "bundler/setup",
+      # which walks up from the suite looking for a Gemfile -- so a Busser root
+      # inside the workspace finds this project's Gemfile and tries to
+      # materialize that bundle inside the isolated gem home, which fails. A
+      # real Busser root is /opt/busser, never inside the project.
+      BUSSER_ROOT: /tmp/busser-kitchen
+    steps:
+      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1  # v7.0.1
+
+      # Deliberately no bundler-cache, and nothing below runs under `bundle
+      # exec`. The verifier shells out to `gem list` to decide what to install,
+      # and bundler in the environment makes that report the bundle's gems
+      # rather than the Busser root's -- so it skips installing busser and the
+      # run dies on a path that was never created. Test Kitchen is not run from
+      # inside a project's bundle in real use either.
+      - uses: ruby/setup-ruby@95ef2b042f9d7a56d8268cba8559e2842e2ad01b  # v1.321.0
+        with:
+          ruby-version: ${{ matrix.ruby }}
+
+      - name: Install Test Kitchen
+        run: gem install test-kitchen --no-document --version "~> 4.1"
+
+      - name: Install this working tree into the Busser root
+        run: ./kitchen-preinstall.sh
+
+      - name: kitchen verify
+        run: kitchen verify

data/.github/workflows/publish.yml

@@ -43,3 +43,25 @@ jobs:
         uses: actionshub/publish-gem-to-rubygems@f9126f7a2d36a4fd31a13e78fde5fbdcc4b7b251  # v2.0.6
         with:
           token: ${{ secrets.RUBYGEMS_API_KEY }}
+
+      # actionshub/publish-gem-to-rubygems logs a rejected push and still exits
+      # 0, so a release that never reached RubyGems shows up as a green job.
+      # Ask RubyGems directly instead of trusting the step. The v2 endpoint is
+      # 404 until the exact version is indexed, so -f turns "not there" into a
+      # non-zero exit rather than a substring match on `gem list` output.
+      - name: Verify the release reached RubyGems
+        if: ${{ steps.release.outputs.release_created }}
+        env:
+          VERSION: ${{ steps.release.outputs.version }}
+        run: |
+          set -euo pipefail
+          name="$(basename "$(ls ./*.gemspec)" .gemspec)"
+          for _ in $(seq 1 12); do
+            if curl -fsS -o /dev/null "https://rubygems.org/api/v2/rubygems/${name}/versions/${VERSION}.json"; then
+              echo "${name} ${VERSION} is on RubyGems"
+              exit 0
+            fi
+            sleep 15
+          done
+          echo "::error::${name} ${VERSION} never reached RubyGems. A permissions failure in the push step above does not fail that step -- read its log."
+          exit 1

data/.gitignore

@@ -14,3 +14,4 @@ spec/reports
 test/tmp
 test/version_tmp
 tmp
+.kitchen/

data/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.9.1"
+  ".": "0.9.3"
 }

data/.rubocop.yml

@@ -3,7 +3,4 @@ require:
   - cookstyle/chefstyle
 
 AllCops:
-  TargetRubyVersion: 3.1
-  Exclude:
-    - "vendor/**/*"
-    - "spec/**/*"
+  TargetRubyVersion: 3.2

data/CHANGELOG.md

@@ -33,6 +33,25 @@ main reason to take this version.
 * Rewritten README, and development documentation moved to
   CONTRIBUTING.md ([#63](https://github.com/test-kitchen/busser/pull/63))
 
+## [0.9.3](https://github.com/test-kitchen/busser/compare/v0.9.2...v0.9.3) (2026-08-29)
+
+
+### Bug Fixes
+
+* bring the generated plugin scaffold up to date ([#81](https://github.com/test-kitchen/busser/issues/81)) ([f67b8d0](https://github.com/test-kitchen/busser/commit/f67b8d0979d924b2cb653dace74a80fec03a3da5))
+* reject plugin names that cannot be a Ruby constant ([#80](https://github.com/test-kitchen/busser/issues/80)) ([bd7ac17](https://github.com/test-kitchen/busser/commit/bd7ac1749afe458e35328ebb7b57c64634ad1acc))
+* use the Windows path separator in the bat binstub ([#79](https://github.com/test-kitchen/busser/issues/79)) ([1eb0cbf](https://github.com/test-kitchen/busser/commit/1eb0cbf744ce68fc3c4126bf2eec73f784f422e5))
+
+## [0.9.2](https://github.com/test-kitchen/busser/compare/v0.9.1...v0.9.2) (2026-08-28)
+
+
+### Bug Fixes
+
+* quote the prepare.sh path before handing it to a shell ([#73](https://github.com/test-kitchen/busser/issues/73)) ([00a24a4](https://github.com/test-kitchen/busser/commit/00a24a4d4c286f50ac698b2cda9e9600b1c54322))
+* restore SSL verification if a plugin postinstall raises ([#72](https://github.com/test-kitchen/busser/issues/72)) ([7f26adb](https://github.com/test-kitchen/busser/commit/7f26adbca1682bd152afb8fc15d90feb55c15d8e))
+* stop `busser plugin list` printing git errors and a phantom plugin ([#77](https://github.com/test-kitchen/busser/issues/77)) ([7a99ebd](https://github.com/test-kitchen/busser/commit/7a99ebd8453ddc836bc0abf0af0f639f07b8d554))
+* tell Thor to exit non-zero when a command fails ([#76](https://github.com/test-kitchen/busser/issues/76)) ([218ba31](https://github.com/test-kitchen/busser/commit/218ba31b812db55cba2479ba6345d305743cb0c1))
+
 ## [0.9.1](https://github.com/test-kitchen/busser/compare/v0.9.0...v0.9.1) (2026-08-28)
 
 

data/Gemfile

@@ -1,13 +1,20 @@
 source "https://rubygems.org"
 
-gemspec development_group: :test
+gemspec
+
 group :cookstyle do
-  gem "cookstyle"
+  gem "cookstyle", ">= 9.0"
 end
 
 group :test do
-  gem "minitest", ">= 6.0"
-  gem "base64" # cucumber needs it; not a default gem on Ruby 4.0
+  gem "aruba", ">= 2.4"
+  gem "base64", ">= 0.3" # cucumber needs it; not a default gem on Ruby 4.0
   gem "cucumber", ">= 11.1"
-  gem "rake"
+  gem "minitest", ">= 6.0"
+  gem "mocha", ">= 2.7"
+  gem "rake", ">= 13.4"
+end
+
+group :development do
+  gem "yard", ">= 0.9.37"
 end

data/README.md

@@ -87,6 +87,88 @@ busser test
 busser test bash minitest
 ```
 
+## Using Busser with Test Kitchen
+
+This is how most people meet Busser, and it needs no `busser` commands of your
+own. Select the verifier in `kitchen.yml`:
+
+```yaml
+verifier:
+  name: busser
+
+suites:
+  - name: default
+```
+
+Then put tests in a directory named after the plugin that should run them,
+inside the suite:
+
+```text
+test/integration/default/bash/smoke_test.sh
+```
+
+`kitchen verify` installs Busser and the matching plugin on the instance, then
+runs the suite. Which plugin runs is decided by that directory name alone --
+`bash/` is picked up by busser-bash, `minitest/` by busser-minitest, and so on.
+There is nothing else to configure.
+
+## A worked example, without Test Kitchen
+
+To see Busser on its own, set a `BUSSER_ROOT` you can write to:
+
+```bash
+export BUSSER_ROOT=/tmp/busser
+busser setup
+busser plugin install busser-bash
+mkdir -p "$BUSSER_ROOT/suites/bash"
+
+cat > "$BUSSER_ROOT/suites/bash/smoke_test.sh" <<'SH'
+#!/usr/bin/env bash
+set -euo pipefail
+echo hello
+SH
+
+busser test bash
+```
+
+A passing run looks like this, and exits `0`:
+
+```text
+-----> Running bash test suite
+-----> [bash] smoke_test.sh
+hello
+```
+
+A failing one names the command and exits non-zero:
+
+```text
+-----> Running bash test suite
+-----> [bash] smoke_test.sh
+!!!!!! Command [bash /tmp/busser/suites/bash/smoke_test.sh] exit code was 1
+```
+
+## When nothing runs
+
+A suite whose files do not match what the plugin looks for produces this, and
+**exits `0`**:
+
+```text
+-----> Running bash test suite
+```
+
+No tests ran, and nothing said so. If a suite looks like it is being skipped,
+work through these in order:
+
+1. **Is the plugin installed?** `busser plugin list` shows what is available.
+   Without the plugin, `busser test` has no runner for that directory.
+2. **Is the directory named after the plugin?** Tests for busser-bash live in
+   `bash/`, not `tests/` or `scripts/`.
+3. **Do the filenames match?** Each plugin globs for its own pattern -- for
+   example busser-bash takes `*_test.sh` and `*_spec.bash` but ignores
+   `mytest.sh`. Each plugin's README states its pattern.
+4. **Is `BUSSER_ROOT` what you think?** `busser suite path` prints where suites
+   are actually being looked for.
+
 ## Contributing
 
 Bug reports and pull requests are welcome. See

data/Rakefile

@@ -9,7 +9,25 @@ Rake::TestTask.new(:unit) do |t|
 end
 
 Cucumber::Rake::Task.new(:features) do |t|
-  t.cucumber_opts = ["features", "-x", "--format progress", "--no-color", "-b"]
+  t.cucumber_opts = ["features", "--format progress", "--fail-fast"]
+end
+
+# yard lives in the :development group, which CI omits when it runs the tests.
+# Requiring it unconditionally would make `rake test` fail there, so the real
+# task is only defined when yard is installed and a stub explains its absence
+# otherwise. Nothing in CI gates on documentation.
+begin
+  require "yard"
+
+  YARD::Rake::YardocTask.new(:doc) do |t|
+    t.files = ["lib/**/*.rb"]
+    t.options = ["--output-dir", "doc", "--markup", "markdown"]
+  end
+rescue LoadError
+  desc "Generate YARD documentation (install the development group first)"
+  task :doc do
+    abort "yard is not available. Run `bundle install --with development` first."
+  end
 end
 
 desc "Run all test suites"

data/busser.gemspec

@@ -7,25 +7,28 @@ Gem::Specification.new do |spec|
   spec.version       = Busser::VERSION
   spec.authors       = ["Fletcher Nichol"]
   spec.email         = ["fnichol@nichol.ca"]
-  spec.description   = %q{Busser - Runs tests for projects in Test Kitchen}
+  spec.description   = "Busser - Runs tests for projects in Test Kitchen"
   spec.summary       = spec.description
   spec.homepage      = "https://github.com/test-kitchen/busser"
   spec.license       = "Apache-2.0"
 
-  spec.files         = `git ls-files`.split($/)
+  spec.required_ruby_version = ">= 3.2"
+
+  spec.files         = `git ls-files -z`.split("\x0")
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.required_ruby_version = ">= 3.2"
+  spec.metadata = {
+    "bug_tracker_uri" => "#{spec.homepage}/issues",
+    "changelog_uri" => "#{spec.homepage}/blob/main/CHANGELOG.md",
+    "documentation_uri" => "#{spec.homepage}/blob/main/README.md",
+    "homepage_uri" => spec.homepage,
+    "source_code_uri" => spec.homepage,
+  }
 
   # thor 1.1.0 references DidYouMean::SPELL_CHECKERS, removed in Ruby 3.1,
   # so the CLI crashed on startup on every supported Ruby
   spec.add_dependency "thor", ">= 1.1"
   # base64 is no longer a default gem; deserialize needs it at runtime
   spec.add_dependency "base64"
-
-  spec.add_development_dependency "aruba", ">= 2.0"
-  spec.add_development_dependency "minitest"
-  spec.add_development_dependency "mocha"
-  spec.add_development_dependency "rake"
 end

data/kitchen-preinstall.sh

@@ -0,0 +1,55 @@
+#!/usr/bin/env bash
+# Installs this working tree into the Busser root the Test Kitchen verifier
+# will use, so `kitchen verify` exercises the code on this branch rather than
+# the last release from RubyGems.
+#
+# The verifier's install script skips its own install when `gem list` already
+# shows what it wants, so putting things in place first is all that is needed.
+# Two consequences to handle:
+#
+#   * It also has to install busser itself. That check is `grep "^busser"` with
+#     no anchor at the end, so a plugin named busser-* satisfies it and busser
+#     would otherwise never be installed at all.
+#   * Skipping `busser plugin install` also skips the plugin's postinstall,
+#     which is where a plugin installs the test framework it drives. So run the
+#     postinstall here, which is what the verifier would have done.
+set -euo pipefail
+
+BUSSER_ROOT="${BUSSER_ROOT:-/tmp/busser-kitchen}"
+gemspec="$(ls ./*.gemspec)"
+gem_name="$(basename "${gemspec}" .gemspec)"
+
+install_opts=(
+  --install-dir "${BUSSER_ROOT}/gems"
+  --bindir "${BUSSER_ROOT}/bin"
+  --no-document
+)
+
+mkdir -p "${BUSSER_ROOT}/gems"
+
+gem build "${gemspec}" --output "${BUSSER_ROOT}/${gem_name}.gem" >/dev/null
+
+if [ "${gem_name}" = "busser" ]; then
+  # No --local: a plugin's runtime dependencies are things its runner needs on
+  # the machine under test, and they have to be fetched into the Busser root
+  # like everything else.
+  gem install "${BUSSER_ROOT}/${gem_name}.gem" "${install_opts[@]}" >/dev/null
+else
+  gem install busser "${install_opts[@]}" >/dev/null
+  # Not --ignore-dependencies: a plugin's runtime dependencies are things its
+  # runner needs on the machine under test, and they have to be in the Busser
+  # root like everything else.
+  # No --local: a plugin's runtime dependencies are things its runner needs on
+  # the machine under test, and they have to be fetched into the Busser root
+  # like everything else.
+  gem install "${BUSSER_ROOT}/${gem_name}.gem" "${install_opts[@]}" >/dev/null
+
+  # The same environment the verifier exports, so the framework this plugin
+  # installs lands in the Busser root rather than the ambient gem home.
+  BUSSER_ROOT="${BUSSER_ROOT}" \
+    GEM_HOME="${BUSSER_ROOT}/gems" \
+    GEM_PATH="${BUSSER_ROOT}/gems" \
+    "${BUSSER_ROOT}/bin/busser" plugin install "${gem_name}" --force-postinstall
+fi
+
+echo "pre-installed ${gem_name} from the working tree into ${BUSSER_ROOT}"

data/kitchen.yml

@@ -0,0 +1,27 @@
+---
+# Runs Busser the way Test Kitchen actually runs it -- real verifier,
+# real busser, real suite transfer -- but against the machine running the tests
+# rather than a VM or container, so it works unchanged in CI.
+driver:
+  name: exec
+
+provisioner:
+  name: dummy
+
+verifier:
+  name: busser
+  # The verifier defaults to Chef's omnibus Ruby, which is not what is running
+  # here, and to sudo, which is not needed for a path under the workspace.
+  ruby_bindir: <%= RbConfig::CONFIG["bindir"] %>
+  sudo: false
+  root_path: <%= ENV.fetch("BUSSER_ROOT", "/tmp/busser-kitchen") %>
+
+platforms:
+  - name: local
+
+suites:
+  # busser-bash is the exercise: a real plugin, installed by the verifier from
+  # RubyGems, driven by the busser built from this working tree. The internal
+  # dummy runner would prove the chain runs but can never fail, so it would not
+  # notice a regression.
+  - name: default

data/lib/busser/command/deserialize.rb

@@ -38,6 +38,10 @@ module Busser
 
       class_option :perms, desc: "Unix permissions on destination file"
 
+      # Writes a file that was streamed in over stdin, checking it arrived
+      # intact before applying its permissions.
+      #
+      # @return [void]
       def perform
         file = File.expand_path(options[:destination])
         contents = Base64.decode64(STDIN.read)

data/lib/busser/command/plugin_create.rb

@@ -36,7 +36,25 @@ module Busser
       class_option :license, aliases: "-l", default: "apachev2",
         desc: "License type for gem (apachev2, mit, lgplv3, reserved)"
 
+      # Generates a new runner plugin project.
+      #
+      # @return [void]
+      # A plugin name becomes three things: a require path, a directory name and
+      # a Ruby constant. Only the last is fussy, and Thor's camel_case only
+      # folds underscores -- so "my-junit" produced `module My-junit`, which is
+      # not parseable Ruby. The generator ran to completion and wrote a project
+      # that could not be loaded at all.
+      NAME_PATTERN = /\A[a-z][a-z0-9_]*\z/
+
       def create
+        unless NAME_PATTERN.match?(name)
+          raise ::Thor::Error,
+            "'#{name}' is not a usable plugin name. A name becomes a Ruby " \
+            "constant, so it must start with a lowercase letter and contain " \
+            "only lowercase letters, digits and underscores. Try " \
+            "'#{suggested_name}'."
+        end
+
         self.class.source_root(Busser.source_root.join("templates", "plugin"))
 
         create_core_files
@@ -47,6 +65,9 @@ module Busser
 
       private
 
+      # Writes the gemspec, Gemfile, Rakefile, README and licence.
+      #
+      # @return [void]
       def create_core_files
         empty_directory(target_dir)
 
@@ -60,6 +81,9 @@ module Busser
         create_template("github_workflow.yml.erb", ".github/workflows/test.yml")
       end
 
+      # Writes lib/, the version file and the runner plugin class.
+      #
+      # @return [void]
       def create_source_files
         empty_directory(File.join(target_dir, "lib/busser", name))
         empty_directory(File.join(target_dir, "lib/busser/runner_plugin"))
@@ -74,6 +98,9 @@ module Busser
         )
       end
 
+      # Writes a starter cucumber suite for the new plugin.
+      #
+      # @return [void]
       def create_features_files
         empty_directory(File.join(target_dir, "features/support"))
 
@@ -95,6 +122,10 @@ module Busser
         )
       end
 
+      # Runs git init in the generated project, so the gemspec's
+      # `git ls-files` has something to read.
+      #
+      # @return [void]
       def initialize_git
         inside(target_dir) do
           run("git init")
@@ -102,14 +133,27 @@ module Busser
         end
       end
 
+      # @param erb [String] template path, relative to the templates directory
+      # @param dest [String] destination path, relative to the target directory
+      # @return [void]
       def create_template(erb, dest)
         template(erb, File.join(target_dir, dest), config)
       end
 
+      # @return [String] directory the new plugin is generated into
+      # @return [String] the given name reshaped into something usable, for the
+      #   error message
+      def suggested_name
+        name.to_s.downcase.gsub(/[^a-z0-9_]+/, "_").sub(/\A[^a-z]+/, "").squeeze("_")
+      end
+
       def target_dir
         File.join(Dir.pwd, "busser-#{name}")
       end
 
+      # Values the templates interpolate.
+      #
+      # @return [Hash] the template binding
       def config
         @config ||= begin
           type_klass_name = "#{::Thor::Util.camel_case(options[:type])}Plugin"
@@ -132,16 +176,19 @@ module Busser
         end
       end
 
+      # @return [String] the author name, from git config where available
       def author
         git_user_name = `git config user.name`.chomp
         git_user_name.empty? ? "TODO: Write your name" : git_user_name
       end
 
+      # @return [String] the author email, from git config where available
       def email
         git_user_email = `git config user.email`.chomp
         git_user_email.empty? ? "TODO: Write your email" : git_user_email
       end
 
+      # @return [String] full licence text for the chosen licence
       def license_string
         case options[:license]
         when "mit" then "MIT"
@@ -168,6 +215,8 @@ module Busser
         end
       end
 
+      # @return [String] the filename the licence is written to, which differs
+      #   by licence
       def license_filename
         case options[:license]
         when "mit" then "LICENSE.txt"
@@ -178,6 +227,7 @@ module Busser
         end
       end
 
+      # @return [String] the licence header comment prepended to source files
       def license_comment
         @license_comment ||= IO.read(File.join(target_dir, license_filename))
           .gsub(/^/, "# ").gsub(/\s+$/, "")

data/lib/busser/command/plugin_install.rb

@@ -40,6 +40,9 @@ module Busser
       class_option :verbose, type: :boolean, default: false,
         desc: "Set a more verbose output"
 
+      # Installs each requested plugin.
+      #
+      # @return [void]
       def install_all
         if options[:verbose]
           Gem.configuration.verbose = 2 if options[:verbose]
@@ -53,6 +56,10 @@ module Busser
 
       private
 
+      # Installs one plugin and runs its postinstall.
+      #
+      # @param plugin [String] gem name, optionally suffixed with @version
+      # @return [void]
       def install(plugin)
         gem_name, version = plugin.split("@")
         name = gem_name.sub(/^busser-/, "")
@@ -65,6 +72,12 @@ module Busser
         end
       end
 
+      # Installs the plugin gem unless it is already available.
+      #
+      # @param gem [String] the gem name
+      # @param version [String, nil] a version requirement, or nil for any
+      # @param name [String] short plugin name, for the message
+      # @return [Boolean] true if the gem was newly installed
       def install_plugin_gem(gem, version, name)
         if internal_plugin?(name) || gem_installed?(gem, version)
           info "Plugin #{name} already installed"
@@ -76,10 +89,17 @@ module Busser
         end
       end
 
+      # @param name [String] short plugin name
+      # @return [void]
       def load_plugin(name)
         Busser::Plugin.require!(Busser::Plugin.runner_plugin(name))
       end
 
+      # Runs the plugin's postinstall block, which is where a plugin installs
+      # the test framework it drives.
+      #
+      # @param name [String] short plugin name
+      # @return [void]
       def run_postinstall(name)
         klass = Busser::Plugin.runner_class(::Thor::Util.camel_case(name))
         if klass.respond_to?(:run_postinstall)
@@ -100,13 +120,29 @@ module Busser
       #
       # Please use with extreme caution.
       #
+      # The restore is in an ensure block. Without one, a postinstall that
+      # raised left VERIFY_PEER set to VERIFY_NONE for the rest of the
+      # process -- and `busser plugin install` takes a list, so every gem
+      # downloaded for every later plugin in that same run would have had its
+      # certificate unchecked.
+      #
+      # @yield the block to run with peer verification dropped
+      # @return [void]
       def drop_ssl_verify_peer
         before = OpenSSL::SSL::VERIFY_PEER
+        set_ssl_verify_peer(OpenSSL::SSL::VERIFY_NONE)
+        begin
+          yield
+        ensure
+          set_ssl_verify_peer(before)
+        end
+      end
+
+      # @param value [Integer] the OpenSSL verify mode to install
+      # @return [void]
+      def set_ssl_verify_peer(value)
         OpenSSL::SSL.send(:remove_const, "VERIFY_PEER")
-        OpenSSL::SSL.const_set("VERIFY_PEER", OpenSSL::SSL::VERIFY_NONE)
-        yield
-        OpenSSL::SSL.send(:remove_const, "VERIFY_PEER")
-        OpenSSL::SSL.const_set("VERIFY_PEER", before)
+        OpenSSL::SSL.const_set("VERIFY_PEER", value)
       end
     end
   end

data/lib/busser/command/plugin_list.rb

@@ -28,6 +28,9 @@ module Busser
     #
     class PluginList < Busser::Thor::BaseGroup
 
+      # Prints the installed plugins and their versions.
+      #
+      # @return [void]
       def list
         if plugin_data.empty?
           say "No plugins installed yet"
@@ -38,11 +41,20 @@ module Busser
 
       private
 
+      # The dummy runner is a fixture shipped inside busser for its own tests,
+      # not something anyone installed. `busser test` already passes over it,
+      # so listing it as an installed plugin was inconsistent as well as
+      # confusing.
+      #
+      # @return [Array<Array(String, String)>] each installed plugin's short
+      #   name and version
       def plugin_data
-        @plugin_data ||= Busser::Plugin.runner_plugins.map do |path|
-          spec = Busser::Plugin.gem_from_path(path)
-          [File.basename(path), (spec && spec.version)]
-        end
+        @plugin_data ||= Busser::Plugin.runner_plugins
+          .reject { |path| File.basename(path) == "dummy" }
+          .map do |path|
+            spec = Busser::Plugin.gem_from_path(path)
+            [File.basename(path), (spec && spec.version)]
+          end
       end
     end
   end