ruby-pwsh 0.10.3 → 0.11.0.rc.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: adc4879de83f4b7756c10664ab59c44ab3b34e93e848ebedcefb049c969137f8
-  data.tar.gz: 7ffc7b8c36775a252075fe3c0bffbb60c8b2e287bcbec65d61c1594f6ff4522a
+  metadata.gz: a3357d3c933ddf7b2ecce5d3be1958e9d6aaf6b9966f0b4e4d1f45c40f13963c
+  data.tar.gz: 59e415fc5c1804e09c481bfb6a3c8aa58a7b9767dd04f0a31b3f65c6986ab9c0
 SHA512:
-  metadata.gz: 638fb1d09d7c0ec4fe29ee0473dba74cdbf2aa142a730eed6c43b0e472ec2f3a7452a87e18ff8c36a4dd95f0897ec8388d30357406481aea8772d409087183b7
-  data.tar.gz: d5de2cb597e68c7c911b36c1cd92cbfc50a13916570f744bc8674b138ed62c30cc728ac2dba8f50a2856e6915ceb48ceede851c6f11a91cf8cca1af2d6cdfe9c
+  metadata.gz: dd03304fb4b624296d2770b33296464ca7968e0d1f3b5ee31ddad4490892fcefeca55359447dad2389fb7123bb6b95f42d287f1dd29c50a4ccdd0244338d862e
+  data.tar.gz: 0a4c863c752858ee96975d0278220ca914ba1ea5e02bddd57ab8c798b23c775c2914e81a67482c3bfa5b54af64e0f55946e99af0c2d0fef2b7470c3aec379c86

data/.rubocop.yml

@@ -1,41 +1,19 @@
-Gemspec/RequiredRubyVersion:
-  Enabled: false
-Layout/EndOfLine:
-  Description: Don't enforce CRLF on Windows.
-  Enabled: false
-Metrics/LineLength:
-  Description: People have wide screens, use them.
-  Max: 200
-Metrics/BlockLength:
-  Enabled: false
-Metrics/MethodLength:
-  Enabled: false
-Metrics/ClassLength:
-  Enabled: false
-Metrics/PerceivedComplexity:
-  Enabled: false
-Metrics/CyclomaticComplexity:
-  Enabled: false
-Metrics/AbcSize:
-  Enabled: false
+inherit_from: .rubocop_todo.yml
 
-# requires 2.3's squiggly HEREDOC support, which we can't use, yet
-# see http://www.virtuouscode.com/2016/01/06/about-the-ruby-squiggly-heredoc-syntax/
-Layout/HeredocIndentation:
-  Enabled: false
-# Need to ignore rubocop complaining about the name of the library.
-Naming/FileName:
+require:
+  - rubocop-performance
+  - rubocop-rspec
+
+AllCops:
   Exclude:
-    - lib/ruby-pwsh.rb
-Style/RescueStandardError:
-  Enabled: false
-Style/ExpandPathArguments:
-  Enabled: false
-Style/Documentation:
-  Enabled: false
+    - Gemfile
+    - Rakefile
+    - spec/fixtures/**/*
+    - vendor/bundle/**/*
+  NewCops: enable
+  SuggestExtensions: false
+  TargetRubyVersion: '2.7'
+
+# Disabled
 Style/ClassAndModuleChildren:
-  Exclude:
-    - lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb
-Style/ClassVars:
-  Exclude:
-    - lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb
+  Enabled: false

data/lib/puppet/provider/dsc_base_provider/dsc_base_provider.rb

@@ -88,7 +88,7 @@ class Puppet::Provider::DscBaseProvider
               # - When the values match case insensitively but the attribute is an enum, prefer the casing of the manifest enum.
               # - When the values match case insensitively and the attribute is not an enum, prefer the casing from invoke_get_method
               canonicalized[key] = r[key] unless same?(value, downcased_resource[key]) && !enum_attributes(context).include?(key)
-              canonicalized.delete(key) unless downcased_resource.keys.include?(key)
+              canonicalized.delete(key) unless downcased_resource.key?(key)
             end
             # Cache the actually canonicalized resource separately
             @@cached_canonicalized_resource << canonicalized.dup
@@ -134,7 +134,7 @@ class Puppet::Provider::DscBaseProvider
         (mandatory_get_attributes(context) - namevar_attributes(context)).include?(attribute)
       end
       # If dsc_psdscrunascredential was specified, re-add it here.
-      mandatory_properties[:dsc_psdscrunascredential] = canonicalized_resource[:dsc_psdscrunascredential] if canonicalized_resource.keys.include?(:dsc_psdscrunascredential)
+      mandatory_properties[:dsc_psdscrunascredential] = canonicalized_resource[:dsc_psdscrunascredential] if canonicalized_resource.key?(:dsc_psdscrunascredential)
     end
     names.collect do |name|
       name = { name: name } if name.is_a? String
@@ -253,7 +253,7 @@ class Puppet::Provider::DscBaseProvider
 
     begin
       data = JSON.parse(output)
-    rescue => e
+    rescue StandardError => e
       context.err(e)
       return nil
     end
@@ -263,7 +263,7 @@ class Puppet::Provider::DscBaseProvider
     unless error.nil? || error.empty?
       # NB: We should have a way to stop processing this resource *now* without blowing up the whole Puppet run
       # Raising an error stops processing but blows things up while context.err alerts but continues to process
-      if error =~ /Logon failure: the user has not been granted the requested logon type at this computer/
+      if error.include?('Logon failure: the user has not been granted the requested logon type at this computer')
         logon_error = "PSDscRunAsCredential account specified (#{name_hash[:dsc_psdscrunascredential]['user']}) does not have appropriate logon rights; are they an administrator?"
         name_hash[:name].nil? ? context.err(logon_error) : context.err(name_hash[:name], logon_error)
         @@logon_failures << name_hash[:dsc_psdscrunascredential].dup
@@ -343,19 +343,19 @@ class Puppet::Provider::DscBaseProvider
       end
       # PowerShell does not distinguish between a return of empty array/string
       #  and null but Puppet does; revert to those values if specified.
-      data[type_key] = [] if data[type_key].nil? && query_props.keys.include?(type_key) && query_props[type_key].is_a?(Array)
+      data[type_key] = [] if data[type_key].nil? && query_props.key?(type_key) && query_props[type_key].is_a?(Array)
     end
     # If a resource is found, it's present, so refill this Puppet-only key
-    data.merge!({ name: name_hash[:name] })
+    data[:name] = name_hash[:name]
 
     # Have to check for this to avoid a weird canonicalization warning
     # The Resource API calls canonicalize against the current state which
     # will lead to dsc_ensure being set to absent in the name_hash even if
     # it was set to present in the manifest
-    name_hash_has_nil_keys = name_hash.select { |_k, v| v.nil? }.count.positive?
+    name_hash_has_nil_keys = name_hash.count { |_k, v| v.nil? }.positive?
     # We want to throw away all of the empty keys if and only if the manifest
     # declaration is for an absent resource and the resource is actually absent
-    data.reject! { |_k, v| v.nil? } if data[:dsc_ensure] == 'Absent' && name_hash[:dsc_ensure] == 'Absent' && !name_hash_has_nil_keys
+    data.compact! if data[:dsc_ensure] == 'Absent' && name_hash[:dsc_ensure] == 'Absent' && !name_hash_has_nil_keys
 
     # Sort the return for order-insensitive nested enumerable comparison:
     data = recursively_sort(data)
@@ -451,7 +451,7 @@ class Puppet::Provider::DscBaseProvider
     # path to allow multiple modules to with shared dsc_resources to be installed side by side
     # The old vendored_modules_path: puppet_x/dsc_resources
     # The new vendored_modules_path: puppet_x/<module_name>/dsc_resources
-    root_module_path = load_path.select { |path| path.match?(%r{#{puppetize_name(module_name)}/lib}) }.first
+    root_module_path = load_path.find { |path| path.match?(%r{#{puppetize_name(module_name)}/lib}) }
     vendored_path = if root_module_path.nil?
                       File.expand_path(Pathname.new(__FILE__).dirname + '../../../' + "puppet_x/#{puppetize_name(module_name)}/dsc_resources") # rubocop:disable Style/StringConcatenation
                     else
@@ -501,7 +501,7 @@ class Puppet::Provider::DscBaseProvider
   # @return [String] a uuid with underscores instead of dashes.
   def random_variable_name
     # PowerShell variables can't include dashes
-    SecureRandom.uuid.gsub('-', '_')
+    SecureRandom.uuid.tr('-', '_')
   end
 
   # Return a Hash containing all of the variables defined for instantiation as well as the Ruby hash for their
@@ -640,7 +640,7 @@ class Puppet::Provider::DscBaseProvider
   # @param context [Object] the Puppet runtime context to operate in and send feedback to
   # @return [Array] returns an array of attribute names as symbols which are enums
   def enum_attributes(context)
-    context.type.attributes.select { |_name, properties| properties[:type].match(/Enum\[/) }.keys
+    context.type.attributes.select { |_name, properties| properties[:type].include?('Enum[') }.keys
   end
 
   # Look through a fully formatted string, replacing all instances where a value matches the formatted properties
@@ -670,7 +670,7 @@ class Puppet::Provider::DscBaseProvider
   def munge_psmodulepath(resource)
     return unless resource[:dscmeta_resource_implementation] == 'Class'
 
-    vendor_path = resource[:vendored_modules_path].gsub('/', '\\')
+    vendor_path = resource[:vendored_modules_path].tr('/', '\\')
     <<~MUNGE_PSMODULEPATH.strip
       $UnmungedPSModulePath = [System.Environment]::GetEnvironmentVariable('PSModulePath','machine')
       $MungedPSModulePath = $env:PSModulePath + ';#{vendor_path}'
@@ -731,7 +731,7 @@ class Puppet::Provider::DscBaseProvider
       # name = property_name.to_s.gsub(/^dsc_/, '').to_sym
       # Process nested CIM instances first as those neeed to be passed to higher-order
       # instances and must therefore be declared before they must be referenced
-      cim_instance_hashes = nested_cim_instances(property_hash[:value]).flatten.reject(&:nil?)
+      cim_instance_hashes = nested_cim_instances(property_hash[:value]).flatten.compact
       # Sometimes the instances are an empty array
       unless cim_instance_hashes.count.zero?
         cim_instance_hashes.each do |instance|
@@ -743,7 +743,7 @@ class Puppet::Provider::DscBaseProvider
         end
       end
       # We have to handle arrays of CIM instances slightly differently
-      if property_hash[:mof_type] =~ /\[\]$/
+      if /\[\]$/.match?(property_hash[:mof_type])
         class_name = property_hash[:mof_type].gsub('[]', '')
         property_hash[:value].each do |hash|
           variable_name = random_variable_name
@@ -847,7 +847,7 @@ class Puppet::Provider::DscBaseProvider
       params_block = params_block.gsub("#{param_block_name} = @()", "#{param_block_name} = [#{properties[:mof_type]}]@()")
     end
     # HACK: make CIM instances work:
-    resource[:parameters].select { |_key, hash| hash[:mof_is_embedded] && hash[:mof_type] =~ /\[\]/ }.each do |_property_name, property_hash|
+    resource[:parameters].select { |_key, hash| hash[:mof_is_embedded] && hash[:mof_type].include?('[]') }.each do |_property_name, property_hash|
       formatted_property_hash = interpolate_variables(format(property_hash[:value]))
       params_block = params_block.gsub(formatted_property_hash, "[CimInstance[]]#{formatted_property_hash}")
     end
@@ -861,7 +861,7 @@ class Puppet::Provider::DscBaseProvider
   # @param resource [Hash] a hash with the information needed to run `Invoke-DscResource`
   # @return [String] A string representing the PowerShell script which will invoke the DSC Resource.
   def ps_script_content(resource)
-    template_path = File.expand_path('../', __FILE__)
+    template_path = File.expand_path(__dir__)
     # Defines the helper functions
     functions     = File.new("#{template_path}/invoke_dsc_resource_functions.ps1").read
     # Defines the response hash and the runtime settings
@@ -888,7 +888,7 @@ class Puppet::Provider::DscBaseProvider
   def format(value)
     Pwsh::Util.format_powershell_value(value)
   rescue RuntimeError => e
-    raise unless e.message =~ /Sensitive \[value redacted\]/
+    raise unless e.message.include?('Sensitive [value redacted]')
 
     Pwsh::Util.format_powershell_value(unwrap(value))
   end
@@ -948,12 +948,12 @@ class Puppet::Provider::DscBaseProvider
   def handle_secrets(text, replacement, error_message)
     # Every secret unwrapped in this module will unwrap as "'secret#{SECRET_POSTFIX}'"
     # Currently, no known resources specify a SecureString instead of a PSCredential object.
-    return text unless text.match(/#{Regexp.quote(SECRET_POSTFIX)}/)
+    return text unless /#{Regexp.quote(SECRET_POSTFIX)}/.match?(text)
 
     # In order to reduce time-to-parse, look at each line individually and *only* attempt
     # to substitute if a naive match for the secret postfix is found on the line.
     modified_text = text.split("\n").map do |line|
-      if line.match(/#{Regexp.quote(SECRET_POSTFIX)}/)
+      if /#{Regexp.quote(SECRET_POSTFIX)}/.match?(line)
         line.gsub(SECRET_DATA_REGEX, replacement)
       else
         line
@@ -963,7 +963,7 @@ class Puppet::Provider::DscBaseProvider
     modified_text = modified_text.join("\n")
 
     # Something has gone wrong, error loudly
-    raise error_message if modified_text =~ /#{Regexp.quote(SECRET_POSTFIX)}/
+    raise error_message if /#{Regexp.quote(SECRET_POSTFIX)}/.match?(modified_text)
 
     modified_text
   end

data/lib/pwsh/version.rb

@@ -2,5 +2,5 @@
 
 module Pwsh
   # The version of the ruby-pwsh gem
-  VERSION = '0.10.3'
+  VERSION = '0.11.0.rc.1'
 end

data/lib/pwsh/windows_powershell.rb

@@ -82,7 +82,7 @@ if Pwsh::Util.on_windows?
           HKLM.open(PS_ONE_REG_PATH, ACCESS_TYPE) do |reg|
             version = reg[REG_KEY]
           end
-        rescue
+        rescue StandardError
           version = nil
         end
         version
@@ -98,7 +98,7 @@ if Pwsh::Util.on_windows?
           HKLM.open(PS_THREE_REG_PATH, ACCESS_TYPE) do |reg|
             version = reg[REG_KEY]
           end
-        rescue
+        rescue StandardError
           version = nil
         end
         version

data/lib/pwsh.rb

@@ -55,7 +55,7 @@ module Pwsh
         # ignore any errors trying to tear down this unusable instance
         begin
           manager.exit unless manager.nil? # rubocop:disable Style/SafeNavigation
-        rescue
+        rescue StandardError
           nil
         end
         @@instances[key] = Manager.new(cmd, args, options)
@@ -151,7 +151,7 @@ module Pwsh
                     UNIXSocket.new(pipe_path)
                   end
           break
-        rescue
+        rescue StandardError
           sleep sleep_interval
         end
       end
@@ -208,7 +208,6 @@ module Pwsh
       out[:stderr] = out[:stderr].nil? ? [] : [out[:stderr]]
       out[:stderr] += err unless err.nil?
       out[:native_stdout] = native_stdout
-
       out
     end
 
@@ -224,7 +223,7 @@ module Pwsh
       # rather than expecting the pipe.close to terminate it
       begin
         write_pipe(pipe_command(:exit)) unless @pipe.closed?
-      rescue
+      rescue StandardError
         nil
       end
 
@@ -245,8 +244,8 @@ module Pwsh
     # @return [String] full path to the bootstrap template
     def self.template_path
       # A PowerShell -File compatible path to bootstrap the instance
-      path = File.expand_path('../templates', __FILE__)
-      path = File.join(path, 'init.ps1').gsub('/', '\\')
+      path = File.expand_path('templates', __dir__)
+      path = File.join(path, 'init.ps1').tr('/', '\\')
       "\"#{path}\""
     end
 
@@ -268,7 +267,7 @@ module Pwsh
 
         # Lower bound protection. The polling resolution is only 50ms.
         timeout_ms = 50 if timeout_ms < 50
-      rescue
+      rescue StandardError
         timeout_ms = 300 * 1000
       end
 
@@ -308,17 +307,17 @@ module Pwsh
 
       # PS Side expects Invoke-PowerShellUserCode is always the return value here
       # TODO: Refactor to use <<~ as soon as we can :sob:
-      <<-CODE
-$params = @{
-  Code                     = @'
-#{powershell_code}
-'@
-  TimeoutMilliseconds      = #{timeout_ms}
-  WorkingDirectory         = "#{working_dir}"
-  AdditionalEnvironmentVariables = #{additional_environment_variables}
-}
-
-Invoke-PowerShellUserCode @params
+      <<~CODE
+        $params = @{
+          Code                     = @'
+        #{powershell_code}
+        '@
+          TimeoutMilliseconds      = #{timeout_ms}
+          WorkingDirectory         = "#{working_dir}"
+          AdditionalEnvironmentVariables = #{additional_environment_variables}
+        }
+
+        Invoke-PowerShellUserCode @params
       CODE
     end
 
@@ -336,10 +335,10 @@ Invoke-PowerShellUserCode @params
     #
     # @return [String] the absolute path to the PowerShell executable. Returns 'powershell.exe' if no more specific path found.
     def self.powershell_path
-      if File.exist?("#{ENV['SYSTEMROOT']}\\sysnative\\WindowsPowershell\\v1.0\\powershell.exe")
-        "#{ENV['SYSTEMROOT']}\\sysnative\\WindowsPowershell\\v1.0\\powershell.exe"
-      elsif File.exist?("#{ENV['SYSTEMROOT']}\\system32\\WindowsPowershell\\v1.0\\powershell.exe")
-        "#{ENV['SYSTEMROOT']}\\system32\\WindowsPowershell\\v1.0\\powershell.exe"
+      if File.exist?("#{ENV.fetch('SYSTEMROOT', nil)}\\sysnative\\WindowsPowershell\\v1.0\\powershell.exe")
+        "#{ENV.fetch('SYSTEMROOT', nil)}\\sysnative\\WindowsPowershell\\v1.0\\powershell.exe"
+      elsif File.exist?("#{ENV.fetch('SYSTEMROOT', nil)}\\system32\\WindowsPowershell\\v1.0\\powershell.exe")
+        "#{ENV.fetch('SYSTEMROOT', nil)}\\system32\\WindowsPowershell\\v1.0\\powershell.exe"
       else
         'powershell.exe'
       end
@@ -353,7 +352,7 @@ Invoke-PowerShellUserCode @params
       # Convert all the key names to upcase so we can be sure to find PATH etc.
       # Also while ruby can have difficulty changing the case of some UTF8 characters, we're
       # only going to use plain ASCII names so this is safe.
-      current_path = Pwsh::Util.on_windows? ? ENV.select { |k, _| k.upcase == 'PATH' }.values[0] : ENV['PATH']
+      current_path = Pwsh::Util.on_windows? ? ENV.select { |k, _| k.casecmp('PATH').zero? }.values[0] : ENV.fetch('PATH', nil)
       current_path = '' if current_path.nil?
 
       # Prefer any additional paths
@@ -367,10 +366,10 @@ Invoke-PowerShellUserCode @params
         # TODO: What about PS 8?
         # TODO: Need to check on French/Turkish windows if ENV['PROGRAMFILES'] parses UTF8 names correctly
         # TODO: Need to ensure ENV['PROGRAMFILES'] is case insensitive, i.e. ENV['PROGRAMFiles'] should also resolve on Windows
-        search_paths += ";#{ENV['PROGRAMFILES']}\\PowerShell\\6" \
-                        ";#{ENV['PROGRAMFILES(X86)']}\\PowerShell\\6" \
-                        ";#{ENV['PROGRAMFILES']}\\PowerShell\\7" \
-                        ";#{ENV['PROGRAMFILES(X86)']}\\PowerShell\\7"
+        search_paths += ";#{ENV.fetch('PROGRAMFILES', nil)}\\PowerShell\\6" \
+                        ";#{ENV.fetch('PROGRAMFILES(X86)', nil)}\\PowerShell\\6" \
+                        ";#{ENV.fetch('PROGRAMFILES', nil)}\\PowerShell\\7" \
+                        ";#{ENV.fetch('PROGRAMFILES(X86)', nil)}\\PowerShell\\7"
       end
       raise 'No paths discovered to search for Powershell!' if search_paths.split(File::PATH_SEPARATOR).empty?
 
@@ -433,7 +432,7 @@ Invoke-PowerShellUserCode @params
         # as this resolves to a HANDLE and then calls the Windows API
         !stream.stat.nil?
     # Any exceptions mean the stream is dead
-    rescue
+    rescue StandardError
       false
     end
 
@@ -497,9 +496,7 @@ Invoke-PowerShellUserCode @params
     #
     # @return nil
     def write_pipe(input)
-      # For Compat with Ruby 2.1 and lower, it's important to use syswrite and
-      # not write - otherwise, the pipe breaks after writing 1024 bytes.
-      written = @pipe.syswrite(input)
+      written = @pipe.write(input)
       @pipe.flush
 
       if written != input.length # rubocop:disable Style/GuardClause
@@ -541,7 +538,7 @@ Invoke-PowerShellUserCode @params
       read_from_pipe(pipe, 0) { |s| output << s } while self.class.readable?(pipe)
 
       # String has been binary up to this point, so force UTF-8 now
-      output == [] ? [] : [output.join('').force_encoding(Encoding::UTF_8)]
+      output == [] ? [] : [output.join.force_encoding(Encoding::UTF_8)]
     end
 
     # Open threads and pipes to read stdout and stderr from the PowerShell manager,
@@ -592,8 +589,8 @@ Invoke-PowerShellUserCode @params
 
       [
         output,
-        stdout == [] ? nil : stdout.join(''), # native stdout
-        stderr_reader.value                   # native stderr
+        stdout == [] ? nil : stdout.join, # native stdout
+        stderr_reader.value # native stderr
       ]
     ensure
       # Failsafe if the prior unlock was never reached / Mutex wasn't unlocked

data/spec/acceptance/dsc/basic.rb

@@ -64,6 +64,7 @@ RSpec.describe 'DSC Acceptance: Basic' do
       expect(second_run_result[:exitcode]).to be(0)
     end
   end
+
   context 'Creating' do
     let(:manifest) do
       [
@@ -127,6 +128,7 @@ RSpec.describe 'DSC Acceptance: Basic' do
       expect(second_run_result[:exitcode]).to be(0)
     end
   end
+
   context 'PSDscRunAsCredential' do
     before(:all) do
       prep_command = <<~PREP_USER.strip
@@ -144,6 +146,7 @@ RSpec.describe 'DSC Acceptance: Basic' do
       PREP_USER
       execute_reset_command(prep_command)
     end
+
     after(:all) do
       cleanup_command = <<~CLEANUP_USER.strip
         Remove-LocalUser -Name #{local_user} -ErrorAction Stop
@@ -178,6 +181,7 @@ RSpec.describe 'DSC Acceptance: Basic' do
         expect(second_run_result[:exitcode]).to be(0)
       end
     end
+
     context 'with an invalid credential' do
       let(:manifest) do
         [

data/spec/acceptance/dsc/cim_instances.rb

@@ -1,10 +1,9 @@
 # frozen_string_literal: true
+
 # TODO: Test against mcollera/AccessControlDsc for CIM instance behavior
 # 1. Make sure valid nested CIM instances can be passed to Invoke-DscResource
 # 2. Make sure nested CIM instances can be read back from Invoke-DscResource
 
-# frozen_string_literal: true
-
 require 'spec_helper'
 require 'ruby-pwsh'
 
@@ -26,7 +25,7 @@ RSpec.describe 'DSC Acceptance: Complex' do
   end
 
   context 'Managing the access control list of a folder' do
-    before(:each) do
+    before do
       reset_command = <<~RESET_COMMAND
         $TestFolderPath = Join-Path -Path "#{fixtures_path}" -Childpath access_control
         # Delete the test folder if it exists (to clear access control modifications)
@@ -62,7 +61,7 @@ RSpec.describe 'DSC Acceptance: Complex' do
           ]
         }
       MANIFEST
-      File.open(test_manifest, 'w') { |file| file.write(content) }
+      File.write(test_manifest, content)
       # Apply the test manifest
       first_run_result = powershell.execute(puppet_apply)
       expect(first_run_result[:exitcode]).to be(2)

data/spec/acceptance/dsc/class.rb

@@ -32,7 +32,7 @@ RSpec.describe 'DSC Acceptance: Class-Based Resource' do
       ].join(' ')
     end
 
-    before(:each) do
+    before do
       reset_command = <<~RESET_COMMAND
         $PsrcPath = '#{psrc_path}'
         # Delete the test PSRC fixture if it exists
@@ -68,7 +68,7 @@ RSpec.describe 'DSC Acceptance: Class-Based Resource' do
       ].join(' ')
     end
 
-    before(:each) do
+    before do
       reset_command = <<~RESET_COMMAND
         $PsrcPath = '#{psrc_path}'
         # Delete the test PSRC fixture if it exists
@@ -104,7 +104,7 @@ RSpec.describe 'DSC Acceptance: Class-Based Resource' do
       ].join(' ')
     end
 
-    before(:each) do
+    before do
       reset_command = <<~RESET_COMMAND
         $PsrcPath = '#{psrc_path}'
         # Delete the test PSRC fixture if it exists

data/spec/acceptance/dsc/complex.rb

@@ -21,7 +21,7 @@ RSpec.describe 'DSC Acceptance: Complex' do
   end
 
   context 'Adding a new website' do
-    before(:each) do
+    before do
       reset_command = <<~RESET_COMMAND
         # Ensure IIS is not installed
         $Feature = Get-WindowsFeature -Name 'Web-Asp-Net45'
@@ -108,7 +108,7 @@ RSpec.describe 'DSC Acceptance: Complex' do
             require             => File['WebContentIndex'],
         }
       MANIFEST
-      File.open(test_manifest, 'w') { |file| file.write(content) }
+      File.write(test_manifest, content)
       # Puppet apply the test manifest
       first_run_result = powershell.execute(puppet_apply)
       expect(first_run_result[:exitcode]).to be(2)

data/spec/acceptance/support/setup_winrm.ps1

@@ -0,0 +1,6 @@
+Get-ChildItem WSMan:\localhost\Listener\ -OutVariable Listeners | Format-List * -Force
+$HTTPListener = $Listeners | Where-Object -FilterScript { $_.Keys.Contains('Transport=HTTP') }
+If ($HTTPListener.Count -eq 0) {
+  winrm create winrm/config/Listener?Address=*+Transport=HTTP
+  winrm e winrm/config/listener
+}