cem_acpt 0.2.6-universal-java-17 → 0.2.9-universal-java-17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0a59b5c56bf7b9c5bf417fc95eb7afe0557b84635098cd5de92e2649d53b4cdd
-  data.tar.gz: 2cb8332995cadfd7bcbdc80b728aae8a10d313441e42fbc1cdafdbbe5ca9b697
+  metadata.gz: d4181b3fae3e4d3f0029314896873f2f604374cc4583473b0b05e689464d346f
+  data.tar.gz: 5e01be5d223962785a18e685f17cc1c3cd393f705cd2b5df174327a8631e1598
 SHA512:
-  metadata.gz: 6fb1eef1dae934942b004e5b615d847079afa0c887aabf5d3df084c960b73fe03a847c38a17b2019fe43ed90da608f345f9e46454e400c36d6e604a7f136a41c
-  data.tar.gz: 2369b28c8d7d3c40692dd4fc6522ad0fbec9e40d4cf33eb1a63c7baada1ec6b28eb87e9a78a79dff8f2474a3767890a02f1c2f5dd848925ac769b9e976ff958e
+  metadata.gz: e9dc631cf11d32f85405358a8bc4e8501afc290a15ed68e80688472de838cfdd81cf09e9a10d346f396de6f1c2505188c443c60e0d4b106b346fd77f079c311c
+  data.tar.gz: 7866ca7c6272369e15c2fbf3f2dd69ce6d6caf6d059b08ad686e7b2fcaf5d63d5d093dbfac076c4c68f3ac7c07f4bcacc2bcd01107667531e069821c00e90313

data/Gemfile

@@ -1,9 +1,8 @@
-source "https://rubygems.org"
-
-ruby '2.6.8', engine: 'jruby', engine_version: '>= 9.3.3.0'
+# frozen_string_literal: true
+source 'https://rubygems.org'
 
 # Specify your gem's dependencies in cem_acpt.gemspec
 gemspec
 
-gem "rake", "~> 12.0"
-gem "rspec", "~> 3.0"
+gem 'rake', '>= 12.0'
+gem 'rspec', '>= 3.0'

data/Gemfile.lock

@@ -1,14 +1,12 @@
 PATH
   remote: .
   specs:
-    cem_acpt (0.2.6-universal-java-17)
-      concurrent-ruby (~> 1.1.9)
-      deep_merge (~> 1.2.2)
+    cem_acpt (0.2.9-universal-java-17)
+      concurrent-ruby (>= 1.1, < 2.0)
+      deep_merge (>= 1.2, < 2.0)
       ed25519 (>= 1.2, < 2.0)
-      net-ssh (~> 7.0.0.beta1)
-      puppet-modulebuilder (> 0.0.1)
-      rake
-      rspec
+      net-ssh (>= 7.0, < 8.0)
+      puppet-modulebuilder (>= 0.0.1)
       serverspec-cem-acpt
 
 GEM
@@ -33,7 +31,7 @@ GEM
       minitar (~> 0.9)
       pathspec (>= 0.2.1, < 2.0.0)
     rainbow (3.1.1)
-    rake (12.3.3)
+    rake (13.0.6)
     regexp_parser (2.5.0)
     rexml (3.2.5)
     rspec (3.11.0)
@@ -82,12 +80,9 @@ PLATFORMS
 
 DEPENDENCIES
   cem_acpt!
-  rake (~> 12.0)
-  rspec (~> 3.0)
+  rake (>= 12.0)
+  rspec (>= 3.0)
   rubocop
 
-RUBY VERSION
-   ruby 2.6.8p0 (jruby 9.3.3.0)
-
 BUNDLED WITH
    2.3.15

data/cem_acpt.gemspec

@@ -27,13 +27,11 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
   spec.platform = Gem::Platform.local
-  spec.add_runtime_dependency 'concurrent-ruby', '~> 1.1.9'
-  spec.add_runtime_dependency 'deep_merge', '~> 1.2.2'
-  spec.add_runtime_dependency 'net-ssh', '~> 7.0.0.beta1'
+  spec.add_runtime_dependency 'concurrent-ruby', '>= 1.1', '< 2.0'
+  spec.add_runtime_dependency 'deep_merge', '>= 1.2', '< 2.0'
   spec.add_runtime_dependency 'ed25519', '>= 1.2', '< 2.0'
-  spec.add_runtime_dependency 'puppet-modulebuilder', '> 0.0.1'
-  spec.add_runtime_dependency 'rake'
-  spec.add_runtime_dependency 'rspec'
+  spec.add_runtime_dependency 'net-ssh', '>= 7.0', '< 8.0'
+  spec.add_runtime_dependency 'puppet-modulebuilder', '>= 0.0.1'
   spec.add_runtime_dependency 'serverspec-cem-acpt'
   spec.add_development_dependency 'rubocop'
 end

data/lib/cem_acpt/rspec_utils.rb

@@ -12,13 +12,17 @@ module CemAcpt
     class BundlerNotFoundError < StandardError; end
     class RSpecNotFoundError < StandardError; end
 
-    # Holds and formats a RSpec command
-    class Command
-      include CemAcpt::LoggingAsync
-      attr_reader :debug, :format, :test_path, :use_bundler, :pty_pid
+    # Holds RSpec options used by Command
+    class Options
+      OPTIONS = %i[test_path bundle rspec use_bundler bundle_install format debug quiet env].freeze
 
+      attr_accessor(*OPTIONS)
+
+      # @param config [CemAcpt::Config] A config object
       # @param opts [Hash] options hash for the RSpec command
       # @option opts [String] :test_path The path (or glob path) to the test file(s) to run. If blank, runs all.
+      # @option opts [String] :bundle The path to the `bundle` binary.
+      # @option opts [String] :rspec The path to the `rspec` binary.
       # @option opts [Hash] :format Format options for rspec where the key is the format (documentation, json, etc)
       #   and the value is the out-file path. If you do not want to save the results of a format to a file, the
       #   value should be `nil`.
@@ -30,56 +34,97 @@ module CemAcpt
       #   Default is `true`.
       # @option opts [Boolean] :bundle_install Whether or not to run `bundle install` before the RSpec command
       #   if `use_bundler` is `true`.
-      # @option opts [Boolean] :use_shell Whether or not to add `$SHELL` as a prefix to the command
       # @option opts [Hash] :env Environment variables to prepend to the command
-      def initialize(opts = {})
-        @test_path = opts[:test_path]&.shellescape
-        @format = opts.fetch(:format, {})
-        @debug = opts.fetch(:debug, false)
-        @quiet = @debug ? false : opts.fetch(:quiet, false)
-        @use_bundler = opts.fetch(:use_bundler, false)
-        @bundle_install = opts.fetch(:bundle_install, false)
-        @env = opts.fetch(:env, {})
-        @pty_pid = nil
-        validate_and_set_bin_paths(opts)
+      def initialize(config, **opts)
+        @config = config
+        @opts = opts
+        define_option_instance_vars
+      end
+
+      # Finds and sets the paths to the `bundle` and `rspec` binaries. The paths can
+      # be either passed in as options in the `opts` Hash or interrogated from the
+      # system.
+      # @raise [CemAcpt::RSpecUtils::BundlerNotFoundError] if `@use_bundler` is true and
+      #   `bundle` binary is not found.
+      # @raise [CemAcpt::RSpecUtils::RSpecNotFoundError] if `rspec` binary is not found.
+      def resolve_bin_paths
+        %i[bundle rspec].each do |bin|
+          bin_path = instance_variable_get("@#{bin}") || `#{ENV['SHELL']} -c 'command -v #{bin}'`.strip
+          bin_not_found(bin, bin_path) unless bin_path && File.exist?(bin_path)
+          instance_variable_set("@#{bin}", bin_path)
+        end
+      end
+
+      # Detects if the current Ruby context is JRuby
+      def jruby?
+        Object.const_defined?('JRUBY_VERSION')
       end
 
-      # Sets debug mode to `true`
-      def set_debug
-        @debug = true
-        if @quiet
-          async_debug('Setting :quiet to false because :debug is now true.')
-          @quiet = false
+      private
+
+      def defaults
+        @defaults ||= { use_bundler: false,
+                        bundle_install: false,
+                        format: { documentation: nil },
+                        debug: false,
+                        quiet: false,
+                        env: {} }
+      end
+
+      def define_option_instance_vars
+        OPTIONS.each do |o|
+          val = @config.get("rspec.#{o}") || @opts[o] || defaults[o] || nil
+          instance_variable_set("@#{o}", val)
         end
       end
 
-      # Sets debug mode to `false`
-      def unset_debug
-        @debug = false
+      # Handles binary paths which are not found
+      # @param bin [Symbol] The binary that was not found, either :bundle or :rspec.
+      # @param bin_path [String] The path to the binary that was checked.
+      # @raise [CemAcpt::RSpecUtils::BundlerNotFoundError] if `@use_bundler` is true and
+      #   `bundle` binary is not found.
+      # @raise [CemAcpt::RSpecUtils::RSpecNotFoundError] if `rspec` binary is not found.
+      # @raise [RuntimeError] if `bin` is not :bundle or :rspec.
+      def bin_not_found(bin, bin_path)
+        msg_base = "#{bin} not found."
+        msg = bin_path.nil? ? "#{msg_base} Path is nil." : "#{msg_base} Path: #{bin_path}"
+        case bin
+        when :bundle
+          raise BundlerNotFoundError, msg if opts.use_bundler
+        when :rspec
+          raise RSpecNotFoundError, msg
+        else
+          raise "bin #{bin} not recognized!"
+        end
       end
+    end
+
+    # Holds and formats a RSpec command
+    class Command
+      include CemAcpt::LoggingAsync
+      attr_reader :opts, :pty_pid
+
+      def initialize(opts = Options.new)
+        raise 'opts must be instance of CemAcpt::RSpecUtils::Options' unless opts.is_a?(CemAcpt::RSpecUtils::Options)
 
-      def quiet
-        @quiet && !debug
+        @opts = opts
+        @opts.resolve_bin_paths
+        @opts.env = @opts.env.merge({ 'RSPEC_DEBUG' => 'true' }) if @opts.debug
+        @pty_pid = nil
       end
 
       # Adds a new format to the RSpec command
       # @param fmt [String] The name of the format (i.e. "documentation", "json", etc.)
       # @param out [String] If specified, saves the specified format to a file at this path
       def with_format(fmt, out: nil)
-        @format[fmt.to_sym] = out
-      end
-
-      # Environment variables that will be used for the RSpec command
-      # @return [Hash] A Hash of environment variables with each key pair being: <var name> => <var value>
-      def env
-        @debug ? @env.merge({ 'RSPEC_DEBUG' => 'true' }) : @env
+        opts.format[fmt.to_sym] = out
       end
 
       # Returns an array representation of the RSpec command
       def to_a
         cmd = cmd_base.dup
-        cmd << test_path if test_path
-        format.each do |fmt, out|
+        cmd << opts.test_path if opts.test_path
+        opts.format.each do |fmt, out|
           cmd += ['--format', fmt.to_s.shellescape]
           cmd += ['--out', out.to_s.shellescape] if out
         end
@@ -112,12 +157,12 @@ module CemAcpt
       # @return [Integer] The exit code of the RSpec command
       def execute_pty(log_prefix: 'RSPEC')
         async_debug("Executing RSpec command '#{self}' in PTY...", log_prefix)
-        PTY.spawn(env, ENV['SHELL']) do |r, w, pid|
+        PTY.spawn(opts.env, ENV['SHELL']) do |r, w, pid|
           @pty_pid = pid
           async_debug("Spawned RSpec PTY with PID #{@pty_pid}", log_prefix)
           export_envs(w)
           w.puts "#{self}; exit $?"
-          quiet ? wait_io(r) : read_io(r, log_prefix: log_prefix)
+          handle_io(r, log_prefix: log_prefix)
         end
         $CHILD_STATUS
       end
@@ -130,9 +175,9 @@ module CemAcpt
       def execute_no_pty(log_prefix: 'RSPEC')
         async_info("Executing RSpec command '#{self}' with Open3.popen2e()...", log_prefix)
         exit_status = nil
-        Open3.popen2e(env, to_s) do |stdin, std_out_err, wait_thr|
+        Open3.popen2e(opts.env, to_s) do |stdin, std_out_err, wait_thr|
           stdin.close
-          quiet ? wait_io(std_out_err) : read_io(std_out_err, log_prefix: log_prefix)
+          handle_io(std_out_err, log_prefix: log_prefix)
           exit_status = wait_thr.value
         end
         exit_status
@@ -147,26 +192,21 @@ module CemAcpt
 
       private
 
-      # Detects if the current Ruby context is JRuby
-      def jruby?
-        File.basename(RbConfig.ruby) == 'jruby'
-      end
-
       # The base RSpec command
       def cmd_base
-        use_bundler ? cmd_base_bundler : cmd_base_rspec
+        opts.use_bundler ? cmd_base_bundler : cmd_base_rspec
       end
 
       # The base RSpec command if `:use_bundler` is `true`.
       def cmd_base_bundler
-        base = [@bundle, 'exec', 'rspec']
-        base.unshift("#{@bundle} install;") if @bundle_install
+        base = [opts.bundle, 'exec', 'rspec']
+        base.unshift("#{opts.bundle} install;") if opts.bundle_install
         base
       end
 
       # The base RSpec command if `:use_bundler` is `false`
       def cmd_base_rspec
-        [@rspec]
+        [opts.rspec]
       end
 
       # Puts export statements for each key-value pair in `env` to the given writer.
@@ -174,46 +214,14 @@ module CemAcpt
       # pass the statements to a shell.
       # @param writer [IO] An IO object that supprts `puts` and can send statements to a shell
       def export_envs(writer)
-        env.each do |key, val|
+        @opts.env.each do |key, val|
           writer.puts "export #{key}=#{val}"
         end
       end
 
-      # Finds and sets the paths to the `bundle` and `rspec` binaries. The paths can
-      # be either passed in as options in the `opts` Hash or interrogated from the
-      # system.
-      # @param opts [Hash] The options hash
-      # @option opts [String] :bundle An absolute path on the system to the `bundle` binary.
-      # @option opts [String] :rspec An absolute path on the system to the `rspec` binary.
-      # @raise [CemAcpt::RSpecUtils::BundlerNotFoundError] if `@use_bundler` is true and
-      #   `bundle` binary is not found.
-      # @raise [CemAcpt::RSpecUtils::RSpecNotFoundError] if `rspec` binary is not found.
-      def validate_and_set_bin_paths(opts = {})
-        %i[bundle rspec].each do |bin|
-          bin_path = opts[bin] || `command -v #{bin}`.strip
-          bin_not_found(bin, bin_path) unless bin_path && File.exist?(bin_path)
-          instance_variable_set("@#{bin}", bin_path)
-        end
-      end
-
-      # Handles binary paths which are not found
-      # @param bin [Symbol] The binary that was not found, either :bundle or :rspec.
-      # @param bin_path [String] The path to the binary that was checked.
-      # @raise [CemAcpt::RSpecUtils::BundlerNotFoundError] if `@use_bundler` is true and
-      #   `bundle` binary is not found.
-      # @raise [CemAcpt::RSpecUtils::RSpecNotFoundError] if `rspec` binary is not found.
-      # @raise [RuntimeError] if `bin` is not :bundle or :rspec.
-      def bin_not_found(bin, bin_path)
-        msg_base = "#{bin} not found."
-        msg = bin_path.nil? ? "#{msg_base} Path is nil." : "#{msg_base} Path: #{bin_path}"
-        case bin
-        when :bundle
-          raise BundlerNotFoundError, msg if @use_bundler
-        when :rspec
-          raise RSpecNotFoundError, msg
-        else
-          raise "bin #{bin} not recognized!"
-        end
+      # Handles IO output
+      def handle_io(io_stream, **kwargs)
+        opts.quiet ? wait_io(io_stream) : read_io(io_stream, log_prefix: kwargs[:log_prefix])
       end
 
       # Blocking wait on an IO stream. Wait stops once the IO stream has reached

data/lib/cem_acpt/test_runner/runner.rb

@@ -9,6 +9,7 @@ require_relative 'runner_result'
 module CemAcpt
   module TestRunner
     class RunnerError < StandardError; end
+
     # Error used to wrap fatal errors raised in Runner steps
     class RunnerStepError < StandardError
       attr_reader :step
@@ -19,6 +20,7 @@ module CemAcpt
         set_backtrace err.backtrace if err.respond_to?(:backtrace)
       end
     end
+
     class RunnerProvisionError < RunnerStepError; end
 
     # Runner is a class that runs a single acceptance test suite on a single node.
@@ -143,12 +145,12 @@ module CemAcpt
         async_info("Running test #{@node.test_data[:test_name]} on node #{@node.node_name}...", log_prefix('RSPEC'))
         @node.run_tests do |cmd_env|
           cmd_opts = rspec_opts
-          cmd_opts[:env].merge!(cmd_env) if cmd_env
+          cmd_opts.env = cmd_opts.env.merge(cmd_env) if cmd_env
           # Documentation format gets logged in real time, JSON file is read after the fact
           begin
             @rspec_cmd = CemAcpt::RSpecUtils::Command.new(cmd_opts)
-            @rspec_cmd.execute(log_prefix: log_prefix('RSPEC'))
-            @run_result.from_json_file(cmd_opts[:format][:json])
+            @rspec_cmd.execute(pty: false, log_prefix: log_prefix('RSPEC'))
+            @run_result.from_json_file(cmd_opts.format[:json])
           rescue Errno::EIO => e
             async_error("failed to run rspec: #{@node.test_data[:test_name]}: #{$ERROR_INFO}", log_prefix('RSPEC'))
             @run_result.from_error(e)
@@ -189,21 +191,21 @@ module CemAcpt
 
       # Options used with RSpec
       def rspec_opts
-        opts = {
-          test_path: @node.test_data[:test_file],
-          use_bundler: false,
-          bundle_install: false,
-          format: {
-            json: "results_#{@node.test_data[:test_name]}.json",
-          },
-          debug: (@debug_mode && @context.config.get('verbose')),
-          quiet: @context.config.get('quiet'),
-          env: {
-            'TARGET_HOST' => @node.node_name,
-          }
-        }
-        opts[:format][:documentation] = nil unless @context.config.get('verbose')
-        opts
+        opts_test_path = @node.test_data[:test_file]
+        opts_env = { 'TARGET_HOST' => @node.node_name }
+        opts_debug = (@debug_mode && @context.config.get('verbose'))
+        opts_quiet = @context.config.get('quiet')
+        opts_format = if @context.config.get('verbose')
+                        { json: "results_#{@node.test_data[:test_name]}.json", documentation: nil }
+                      else
+                        { json: "results_#{@node.test_data[:test_name]}.json" }
+                      end
+        CemAcpt::RSpecUtils::Options.new(@context.config,
+                                         test_path: opts_test_path,
+                                         env: opts_env,
+                                         debug: opts_debug,
+                                         quiet: opts_quiet,
+                                         format: opts_format)
       end
     end
   end

data/lib/cem_acpt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CemAcpt
-  VERSION = '0.2.6'
+  VERSION = '0.2.9'
 end

metadata

@@ -1,57 +1,55 @@
 --- !ruby/object:Gem::Specification
 name: cem_acpt
 version: !ruby/object:Gem::Version
-  version: 0.2.6
+  version: 0.2.9
 platform: universal-java-17
 authors:
 - puppetlabs
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-06-27 00:00:00.000000000 Z
+date: 2022-06-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.1'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.1.9
+        version: '2.0'
   name: concurrent-ruby
   prerelease: false
   type: :runtime
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.1'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.1.9
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.2'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.2.2
+        version: '2.0'
   name: deep_merge
   prerelease: false
   type: :runtime
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.2.2
-- !ruby/object:Gem::Dependency
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 7.0.0.beta1
-  name: net-ssh
-  prerelease: false
-  type: :runtime
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
+        version: '1.2'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 7.0.0.beta1
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
@@ -72,48 +70,40 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
-- !ruby/object:Gem::Dependency
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">"
-      - !ruby/object:Gem::Version
-        version: 0.0.1
-  name: puppet-modulebuilder
-  prerelease: false
-  type: :runtime
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">"
-      - !ruby/object:Gem::Version
-        version: 0.0.1
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
-  name: rake
+        version: '7.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '8.0'
+  name: net-ssh
   prerelease: false
   type: :runtime
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '7.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '8.0'
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
-  name: rspec
+        version: 0.0.1
+  name: puppet-modulebuilder
   prerelease: false
   type: :runtime
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.0.1
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements: