busser 0.9.0 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a59df776e51a951b813e8427048a91915f8a51488899ea85ed38e244704f7fc4
-  data.tar.gz: 4fa1ecc9d303f82f4a26629993229a7a82c46edd1d1f14a77c544689b7e8642a
+  metadata.gz: cdaf83b2875620df49aa36203b6d4f7f2f6af8065ecbaa218705c949fa96fe04
+  data.tar.gz: 40471ed874096d5b55bb8c613a33a7729e761d691f6727e4a848edf0fe398dbd
 SHA512:
-  metadata.gz: 18e4d5ef6b17149137cade75655c0406194ad432fbb14835a0ce05cd8e32bf0b60c8fa00f2dc4a7015990110972284d23b24488324fcd5af1cce5f445b0806e7
-  data.tar.gz: 8bc4b488740a43e4a28a40edf93f0b021fbac496f2edaee2df43f0eb898f708b447b69f14e72498d9d46ffb7b34821d2f1a1f25fae4373c9b62d76cb8e85e84f
+  metadata.gz: 01b77c64592e8bed4f630149be35b89e5b3ac9935119f0c62216b5c16c0ae32ef3feb75c61d68ddf1127ea7bafff7de31f5d11297f2498501d9641b74eab163d
+  data.tar.gz: 8efbe414c2a434a98ff91be2f368bd2811cf8110bfce65c337859fbe3806ddca29751f9969a1fa9718c75b8ced4b92176ee33b137bf93b2046cace29fe77f0f6

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/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.9.0"
+  ".": "0.9.2"
 }

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,24 @@ 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.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)
+
+
+### Bug Fixes
+
+* generate a plugin project that resolves and builds ([#68](https://github.com/test-kitchen/busser/issues/68)) ([0c9a450](https://github.com/test-kitchen/busser/commit/0c9a450af83f086ac3627be938db856c9657208b))
+* report signals and keep bundler out of the busser binstub ([#66](https://github.com/test-kitchen/busser/issues/66)) ([4fc9b35](https://github.com/test-kitchen/busser/commit/4fc9b350c7e3990c82ce4e229927be60a6431ee7))
+
 ## 0.8.0 / 2020-08-20
 
 - Add http_proxy support to `busser plugin install`

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/features/plugin_create_command.feature

@@ -11,7 +11,7 @@ Feature: Plugin create command
     ## 0.1.0 / Unreleased
     """
     And the file "busser-junit/Gemfile" should contain "gemspec"
-    And the file "busser-junit/Rakefile" should contain "task :stats"
+    And the file "busser-junit/Rakefile" should contain "Cucumber::Rake::Task"
     And the file "busser-junit/README.md" should contain:
     """
     Busser::RunnerPlugin::Junit
@@ -28,15 +28,13 @@ Feature: Plugin create command
     """
     Gemfile.lock
     """
-    And the file "busser-junit/.tailor" should contain:
+    And the file "busser-junit/.github/workflows/test.yml" should contain:
     """
-    config.file_set 'lib/**/*.rb'
+    bundle exec rake test
     """
-    And the file "busser-junit/.travis.yml" should contain:
-    """
-    language: ruby
-    """
-    And a file named "busser-junit/.cane" should exist
+    And a file named "busser-junit/.travis.yml" should not exist
+    And a file named "busser-junit/.tailor" should not exist
+    And a file named "busser-junit/.cane" should not exist
     And the file "busser-junit/lib/busser/junit/version.rb" should contain:
     """
     module Busser
@@ -49,7 +47,7 @@ Feature: Plugin create command
     """
     And the file "busser-junit/features/support/env.rb" should contain:
     """
-    require 'busser/cucumber'
+    require "busser/cucumber"
     """
     And the file "busser-junit/features/plugin_install_command.feature" should contain:
     """

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,6 +36,9 @@ 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]
       def create
         self.class.source_root(Busser.source_root.join("templates", "plugin"))
 
@@ -47,6 +50,9 @@ module Busser
 
       private
 
+      # Writes the gemspec, Gemfile, Rakefile, README and licence.
+      #
+      # @return [void]
       def create_core_files
         empty_directory(target_dir)
 
@@ -57,11 +63,12 @@ module Busser
         create_template("gemspec.erb", "#{config[:gem_name]}.gemspec")
         create_template("license_#{config[:license]}.erb", license_filename)
         create_template("gitignore.erb", ".gitignore")
-        create_template("tailor.erb", ".tailor")
-        create_template("travis.yml.erb", ".travis.yml")
-        create_file(File.join(target_dir, ".cane"))
+        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"))
@@ -76,6 +83,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"))
 
@@ -97,6 +107,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")
@@ -104,14 +118,21 @@ 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
       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"
@@ -128,21 +149,25 @@ module Busser
             email: email,
             license: options[:license],
             license_string: license_string,
+            license_spdx: license_spdx,
             year: Time.now.year,
           }
         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"
@@ -154,6 +179,23 @@ module Busser
         end
       end
 
+      # RubyGems validates spec.license against the SPDX list and warns on
+      # anything else, so the gemspec cannot reuse the human readable name
+      # that the README wants. "All rights reserved" has no SPDX identifier,
+      # so those gemspecs declare no license at all.
+      def license_spdx
+        case options[:license]
+        when "mit" then "MIT"
+        when "apachev2" then "Apache-2.0"
+        when "lgplv3" then "LGPL-3.0-only"
+        when "reserved" then nil
+        else
+          raise ArgumentError, "No such license #{options[:license]}"
+        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"
@@ -164,6 +206,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