ruby-pwsh 0.11.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cc99d2fca1cd56446129475226f6912f702826d58f5c7e54a2f33d6dff254d50
-  data.tar.gz: 2e5b9b3867d7c310d2bcbab34587703c66854c712fe40a20446fd1516b24d4a9
+  metadata.gz: dd8ad4546d9fbdc7bf62c17f8132adb9647e311e9e3319b22b5161f924fd94c3
+  data.tar.gz: c607771ae691c4593b839cc1134d3e0fcc15b0a454eef93a3b146301d13b3411
 SHA512:
-  metadata.gz: bdfbe174c34e47676450894956a0902dc17ec7b2b034fb6f41339bbbe76f659f7722e44ea0a0ba452a98ca415b508d04da5a9d4c63cb08572b4d4a012f00734c
-  data.tar.gz: 0cdebec313aa91df0b9b12b7adb133c6e626571c148c686c73a9f2b729759e118df8edfe48febf8b3e9939e5a2abdad64812a428830d07325ff9bcc2f0aeedfd
+  metadata.gz: e8efb9f23a1fe2302369938416d08e463c4c5e088712dac293b10f0e8abc09431f91057039f95f918382e4775b932da4ab270901591f58d834a01c1d784d4fe6
+  data.tar.gz: 788c3c601d46c37478b41abe5a068eac7da7c0912ec71f703841f4d462a3363d2c613bbeb8356b6a38b21a3870eec0d567e4bfa2d2892a40aadd1e4a62dc38e1

data/.rubocop.yml

@@ -17,3 +17,11 @@ AllCops:
 # Disabled
 Style/ClassAndModuleChildren:
   Enabled: false
+
+####################################################
+# Cops below here due for deprecation
+####################################################
+# ``Rspec/FilePath`` is going to be deprecated in the next major release of rubocop >=3.0.0: see <https://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/FilePath>
+# As the new cops are already present, e.g., Rspec/SpecFilePathPathFormat, then disabling this in preparation
+RSpec/FilePath:
+  Enabled: false

data/README.md

@@ -1,5 +1,9 @@
 # ruby-pwsh
 
+[![Code Owners](https://img.shields.io/badge/owners-DevX--team-blue)](https://github.com/puppetlabs/ruby-pwsh/blob/main/CODEOWNERS)
+[![ci](https://github.com/puppetlabs/ruby-pwsh/actions/workflows/ci.yml/badge.svg)](https://github.com/puppetlabs/ruby-pwsh/actions/workflows/ci.yml)
+![GitHub release (latest by date)](https://img.shields.io/github/v/release/puppetlabs/ruby-pwsh)
+
 > _The PowerShell gem._
 
 This gem enables you to execute PowerShell from within ruby without having to instantiate and tear down a PowerShell process for each command called.
@@ -70,58 +74,6 @@ After checking out the repo, run `bin/setup` to install dependencies. Then, run
 
 To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and tags, and push the `.gem` file to [rubygems.org](https://rubygems.org). -->
 
-## Releasing the Gem and Puppet Module
-
-Steps to release an update to the gem and module include:
-
-1. From main, checkout a new working branch for the release prep (where xyz is the appropriate version, sans periods):
-   ```bash
-   git checkout -b maint-release_prep_xyz
-   ```
-
-2. Update the version in `lib/pwsh/version.rb` and `metadata.json` to the appropriate version for the new release.
-
-3. Run the changelog update task (make sure to verify the changelog, correctly tagging PRs as needed):
-   ```bash
-   bundle exec rake changelog
-   ```
-
-   4. Commit your changes with a short, sensible commit message, like:
-   ```bash
-   git add lib/pwsh/version.rb
-   git add metadata.json
-   git add CHANGELOG.md
-   git commit -m '(MAINT) Prep for x.y.z release'
-   ```
-
-5. Push your changes and submit a pull request for review _against main:
-   ```bash
-   git push -u origin maint_release_prep_xyz
-   ```
-
-6. Ensure tests pass and the code is merged to `main`.
-
-7. Once the release_prep PR has been merged, checkout main and pull down the latests changes.
-   ```bash
-   git checkout main
-   git pull
-   ```
-
-8. Assuming that the release_prep merge commit is at the HEAD of main we can simply create and push a tag as follows (replacing xyz with the appropriate version).
-   ```bash
-   git tag -a xyx -m "Release xyz"
-   git push --follow-tags
-   ```
-
-9. Execute the publish workflow. This will:
-   - Create a GitHub release
-   - Build and publish the Gem
-   - Build and publish the Puppet module
-
-10. Finally check that the expected versions are present on rubygems.org and the Forge.
-
-## Known Issues
-
 ## Supported Operating Systems
 
 The following platforms are supported:
@@ -133,3 +85,7 @@ The following platforms are supported:
 - OSX
 - RedHat
 - Ubuntu
+
+## License
+
+This codebase is licensed under Apache 2.0. However, the open source dependencies included in this codebase might be subject to other software licenses such as AGPL, GPL2.0, and MIT.

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

@@ -6,27 +6,25 @@ require 'pathname'
 require 'json'
 
 class Puppet::Provider::DscBaseProvider
-  # Initializes the provider, preparing the class variables which cache:
+  # Initializes the provider, preparing the instance variables which cache:
   # - the canonicalized resources across calls
   # - query results
   # - logon failures
   def initialize
-    @@cached_canonicalized_resource ||= []
-    @@cached_query_results ||= []
-    @@cached_test_results ||= []
-    @@logon_failures ||= []
+    @cached_canonicalized_resource = []
+    @cached_query_results = []
+    @cached_test_results = []
+    @logon_failures = []
     super
   end
 
-  def cached_test_results
-    @@cached_test_results
-  end
+  attr_reader :cached_test_results
 
   # Look through a cache to retrieve the hashes specified, if they have been cached.
   # Does so by seeing if each of the specified hashes is a subset of any of the hashes
   # in the cache, so {foo: 1, bar: 2} would return if {foo: 1} was the search hash.
   #
-  # @param cache [Array] the class variable containing cached hashes to search through
+  # @param cache [Array] the instance variable containing cached hashes to search through
   # @param hashes [Array] the list of hashes to search the cache for
   # @return [Array] an array containing the matching hashes for the search condition, if any
   def fetch_cached_hashes(cache, hashes)
@@ -52,14 +50,14 @@ class Puppet::Provider::DscBaseProvider
       # During RSAPI refresh runs mandatory parameters are stripped and not available;
       # Instead of checking again and failing, search the cache for a namevar match.
       namevarized_r = r.select { |k, _v| namevar_attributes(context).include?(k) }
-      cached_result = fetch_cached_hashes(@@cached_canonicalized_resource, [namevarized_r]).first
+      cached_result = fetch_cached_hashes(@cached_canonicalized_resource, [namevarized_r]).first
       if cached_result.nil?
         # If the resource is meant to be absent, skip canonicalization and rely on the manifest
         # value; there's no reason to compare system state to desired state for casing if the
         # resource is being removed.
         if r[:dsc_ensure] == 'absent'
           canonicalized = r.dup
-          @@cached_canonicalized_resource << r.dup
+          @cached_canonicalized_resource << r.dup
         else
           canonicalized = invoke_get_method(context, r)
           # If the resource could not be found or was returned as absent, skip case munging and
@@ -67,7 +65,7 @@ class Puppet::Provider::DscBaseProvider
           # rubocop:disable Metrics/BlockNesting
           if canonicalized.nil? || canonicalized[:dsc_ensure] == 'absent'
             canonicalized = r.dup
-            @@cached_canonicalized_resource << r.dup
+            @cached_canonicalized_resource << r.dup
           else
             parameters = r.select { |name, _properties| parameter_attributes(context).include?(name) }
             canonicalized.merge!(parameters)
@@ -91,7 +89,7 @@ class Puppet::Provider::DscBaseProvider
               canonicalized.delete(key) unless downcased_resource.key?(key)
             end
             # Cache the actually canonicalized resource separately
-            @@cached_canonicalized_resource << canonicalized.dup
+            @cached_canonicalized_resource << canonicalized.dup
           end
           # rubocop:enable Metrics/BlockNesting
         end
@@ -123,13 +121,13 @@ class Puppet::Provider::DscBaseProvider
     context.debug('Collecting data from the DSC Resource')
 
     # If the resource has already been queried, do not bother querying for it again
-    cached_results = fetch_cached_hashes(@@cached_query_results, names)
+    cached_results = fetch_cached_hashes(@cached_query_results, names)
     return cached_results unless cached_results.empty?
 
-    if @@cached_canonicalized_resource.empty?
+    if @cached_canonicalized_resource.empty?
       mandatory_properties = {}
     else
-      canonicalized_resource = @@cached_canonicalized_resource[0].dup
+      canonicalized_resource = @cached_canonicalized_resource[0].dup
       mandatory_properties = canonicalized_resource.select do |attribute, _value|
         (mandatory_get_attributes(context) - namevar_attributes(context)).include?(attribute)
       end
@@ -266,9 +264,9 @@ class Puppet::Provider::DscBaseProvider
       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
+        @logon_failures << name_hash[:dsc_psdscrunascredential].dup
         # This is a hack to handle the query cache to prevent a second lookup
-        @@cached_query_results << name_hash # if fetch_cached_hashes(@@cached_query_results, [data]).empty?
+        @cached_query_results << name_hash # if fetch_cached_hashes(@cached_query_results, [data]).empty?
       else
         context.err(error)
       end
@@ -292,7 +290,7 @@ class Puppet::Provider::DscBaseProvider
   def insync?(context, name, _property_name, _is_hash, should_hash)
     return nil if should_hash[:validation_mode] != 'resource'
 
-    prior_result = fetch_cached_hashes(@@cached_test_results, [name])
+    prior_result = fetch_cached_hashes(@cached_test_results, [name])
     prior_result.empty? ? invoke_test_method(context, name, should_hash) : prior_result.first[:in_desired_state]
   end
 
@@ -361,7 +359,7 @@ class Puppet::Provider::DscBaseProvider
     data = recursively_sort(data)
 
     # Cache the query to prevent a second lookup
-    @@cached_query_results << data.dup if fetch_cached_hashes(@@cached_query_results, [data]).empty?
+    @cached_query_results << data.dup if fetch_cached_hashes(@cached_query_results, [data]).empty?
     context.debug("Returned to Puppet as #{data}")
     data
   end
@@ -400,7 +398,7 @@ class Puppet::Provider::DscBaseProvider
     return nil if data.nil?
 
     in_desired_state = data['indesiredstate']
-    @@cached_test_results << name.merge({ in_desired_state: in_desired_state })
+    @cached_test_results << name.merge({ in_desired_state: in_desired_state })
 
     return in_desired_state if in_desired_state
 
@@ -451,7 +449,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.find { |path| path.match?(%r{#{puppetize_name(module_name)}/lib}) }
+    root_module_path = load_path.grep(%r{#{puppetize_name(module_name)}/lib}).first
     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
@@ -509,12 +507,12 @@ class Puppet::Provider::DscBaseProvider
   #
   # @return [Hash] containing all instantiated variables and the properties that they define
   def instantiated_variables
-    @@instantiated_variables ||= {}
+    @instantiated_variables ||= {}
   end
 
   # Clear the instantiated variables hash to be ready for the next run
   def clear_instantiated_variables!
-    @@instantiated_variables = {}
+    @instantiated_variables = {}
   end
 
   # Return true if the specified credential hash has already failed to execute a DSC resource due to
@@ -523,7 +521,7 @@ class Puppet::Provider::DscBaseProvider
   # @param [Hash] a credential hash with a user and password keys where the password is a sensitive string
   # @return [Bool] true if the credential_hash has already failed logon, false otherwise
   def logon_failed_already?(credential_hash)
-    @@logon_failures.any? do  |failure_hash|
+    @logon_failures.any? do |failure_hash|
       failure_hash['user'] == credential_hash['user'] && failure_hash['password'].unwrap == credential_hash['password'].unwrap
     end
   end
@@ -658,8 +656,8 @@ class Puppet::Provider::DscBaseProvider
     modified_string
   end
 
-  # Parses a resource definition (as from `invocable_resource`) and, if the resource is implemented
-  # as a PowerShell class, ensures the System environment variable for PSModulePath is munged to
+  # Parses a resource definition (as from `invocable_resource`) and
+  # ensures the System environment variable for PSModulePath is munged to
   # include the vendored PowerShell modules. Due to a bug in PSDesiredStateConfiguration, class-based
   # DSC Resources cannot be called via Invoke-DscResource by path, only by module name, *and* the
   # module must be discoverable in the system-level PSModulePath. The postscript for invocation has
@@ -668,8 +666,6 @@ class Puppet::Provider::DscBaseProvider
   # @param resource [Hash] a hash with the information needed to run `Invoke-DscResource`
   # @return [String] A multi-line string which sets the PSModulePath at the system level
   def munge_psmodulepath(resource)
-    return unless resource[:dscmeta_resource_implementation] == 'Class'
-
     vendor_path = resource[:vendored_modules_path].tr('/', '\\')
     <<~MUNGE_PSMODULEPATH.strip
       $UnmungedPSModulePath = [System.Environment]::GetEnvironmentVariable('PSModulePath','machine')

data/lib/pwsh/version.rb

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

data/spec/unit/puppet/provider/dsc_base_provider/dsc_base_provider_spec.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require 'spec_helper'
-require 'puppet/resource_api'
 require 'puppet/type'
 require 'puppet/provider/dsc_base_provider/dsc_base_provider'
 require 'json'
@@ -16,40 +15,40 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
 
   # Reset the caches after each run
   after do
-    described_class.class_variable_set(:@@cached_canonicalized_resource, nil) # rubocop:disable Style/ClassVars
-    described_class.class_variable_set(:@@cached_query_results, nil) # rubocop:disable Style/ClassVars
-    described_class.class_variable_set(:@@cached_test_results, nil) # rubocop:disable Style/ClassVars
-    described_class.class_variable_set(:@@logon_failures, nil) # rubocop:disable Style/ClassVars
+    described_class.instance_variable_set(:@cached_canonicalized_resource, [])
+    described_class.instance_variable_set(:@cached_query_results, [])
+    described_class.instance_variable_set(:@cached_test_results, [])
+    described_class.instance_variable_set(:@logon_failures, [])
   end
 
   describe '.initialize' do
     before do
-      # Need to initialize the provider to load the class variables
+      # Need to initialize the provider to load the instance variables
       provider
     end
 
-    it 'initializes the cached_canonicalized_resource class variable' do
-      expect(described_class.class_variable_get(:@@cached_canonicalized_resource)).to eq([])
+    it 'initializes the cached_canonicalized_resource instance variable' do
+      expect(described_class.instance_variable_get(:@cached_canonicalized_resource)).to eq([])
     end
 
-    it 'initializes the cached_query_results class variable' do
-      expect(described_class.class_variable_get(:@@cached_query_results)).to eq([])
+    it 'initializes the cached_query_results instance variable' do
+      expect(described_class.instance_variable_get(:@cached_query_results)).to eq([])
     end
 
-    it 'initializes the cached_test_results class variable' do
-      expect(described_class.class_variable_get(:@@cached_test_results)).to eq([])
+    it 'initializes the cached_test_results instance variable' do
+      expect(described_class.instance_variable_get(:@cached_test_results)).to eq([])
     end
 
-    it 'initializes the logon_failures class variable' do
-      expect(described_class.class_variable_get(:@@logon_failures)).to eq([])
+    it 'initializes the logon_failures instance variable' do
+      expect(described_class.instance_variable_get(:@logon_failures)).to eq([])
     end
   end
 
   describe '.cached_test_results' do
     let(:cache_value) { %w[foo bar] }
 
-    it 'returns the value of the @@cached_test_results class variable' do
-      described_class.class_variable_set(:@@cached_test_results, cache_value) # rubocop:disable Style/ClassVars
+    it 'returns the value of the @cached_test_results instance variable' do
+      described_class.instance_variable_set(:@cached_test_results, cache_value)
       expect(provider.cached_test_results).to eq(cache_value)
     end
   end
@@ -238,11 +237,11 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
 
   describe '.get' do
     after do
-      described_class.class_variable_set(:@@cached_canonicalized_resource, []) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@cached_canonicalized_resource, [])
     end
 
     it 'checks the cached results, returning if one exists for the specified names' do
-      described_class.class_variable_set(:@@cached_canonicalized_resource, []) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@cached_canonicalized_resource, [])
       allow(context).to receive(:debug)
       expect(provider).to receive(:fetch_cached_hashes).with([], [{ name: 'foo' }]).and_return([{ name: 'foo', property: 'bar' }])
       expect(provider).not_to receive(:invoke_get_method)
@@ -250,7 +249,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
     end
 
     it 'adds mandatory properties to the name hash when calling invoke_get_method' do
-      described_class.class_variable_set(:@@cached_canonicalized_resource, [{ name: 'foo', property: 'bar', dsc_some_parameter: 'baz' }]) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@cached_canonicalized_resource, [{ name: 'foo', property: 'bar', dsc_some_parameter: 'baz' }])
       allow(context).to receive(:debug)
       expect(provider).to receive(:fetch_cached_hashes).with([], [{ name: 'foo' }]).and_return([])
       expect(provider).to receive(:namevar_attributes).and_return([:name]).exactly(3).times
@@ -531,7 +530,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
     end
 
     after do
-      described_class.class_variable_set(:@@cached_query_results, nil) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@cached_query_results, nil)
     end
 
     context 'when the invocation script returns data without errors' do
@@ -558,7 +557,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
 
       it 'caches the result' do
         expect { result }.not_to raise_error
-        expect(described_class.class_variable_get(:@@cached_query_results)).to eq([result])
+        expect(described_class.instance_variable_get(:@cached_query_results)).to eq([result])
       end
 
       it 'removes unrelated properties from the result' do
@@ -720,7 +719,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
         end
 
         after do
-          described_class.class_variable_set(:@@logon_failures, nil) # rubocop:disable Style/ClassVars
+          described_class.instance_variable_set(:@logon_failures, [])
         end
 
         it 'errors specifically for a logon failure and returns nil' do
@@ -729,12 +728,12 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
 
         it 'caches the logon failure' do
           expect { result }.not_to raise_error
-          expect(described_class.class_variable_get(:@@logon_failures)).to eq([credential_hash])
+          expect(described_class.instance_variable_get(:@logon_failures)).to eq([credential_hash])
         end
 
         it 'caches the query results' do
           expect { result }.not_to raise_error
-          expect(described_class.class_variable_get(:@@cached_query_results)).to eq([name_hash])
+          expect(described_class.instance_variable_get(:@cached_query_results)).to eq([name_hash])
         end
       end
 
@@ -982,11 +981,11 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
   end
 
   describe '.invoke_test_method' do
-    subject(:result) { provider.invoke_test_method(context, name, test_should) }
+    subject(:result) { provider.invoke_test_method(context, name, expect(subject).to) }
 
     let(:name) { { name: 'foo', dsc_name: 'bar' } }
-    let(:test_should) { name.merge(dsc_ensure: 'present') }
-    let(:test_properties) { test_should.reject { |k, _v| k == :name } }
+    let(:should) { name.merge(dsc_ensure: 'present') }
+    let(:test_properties) { expect(subject).to.reject { |k, _v| k == :name } }
     let(:invoke_dsc_resource_data) { nil }
 
     before do
@@ -996,7 +995,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
     end
 
     after do
-      described_class.class_variable_set(:@@cached_test_results, []) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@cached_test_results, [])
     end
 
     context 'when something went wrong calling Invoke-DscResource' do
@@ -1044,28 +1043,28 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
 
   describe '.instantiated_variables' do
     after do
-      described_class.class_variable_set(:@@instantiated_variables, nil) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@instantiated_variables, [])
     end
 
-    it 'sets the instantiated_variables class variable to {} if not initialized' do
+    it 'sets the instantiated_variables instance variable to {} if not initialized' do
       expect(provider.instantiated_variables).to eq({})
     end
 
-    it 'returns the instantiated_variables class variable if already initialized' do
-      described_class.class_variable_set(:@@instantiated_variables, { foo: 'bar' }) # rubocop:disable Style/ClassVars
+    it 'returns the instantiated_variables instance variable if already initialized' do
+      described_class.instance_variable_set(:@instantiated_variables, { foo: 'bar' })
       expect(provider.instantiated_variables).to eq({ foo: 'bar' })
     end
   end
 
   describe '.clear_instantiated_variables!' do
     after do
-      described_class.class_variable_set(:@@instantiated_variables, nil) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@instantiated_variables, [])
     end
 
-    it 'sets the instantiated_variables class variable to {}' do
-      described_class.class_variable_set(:@@instantiated_variables, { foo: 'bar' }) # rubocop:disable Style/ClassVars
+    it 'sets the instantiated_variables instance variable to {}' do
+      described_class.instance_variable_set(:@instantiated_variables, { foo: 'bar' })
       expect { provider.clear_instantiated_variables! }.not_to raise_error
-      expect(described_class.class_variable_get(:@@instantiated_variables)).to eq({})
+      expect(described_class.instance_variable_get(:@instantiated_variables)).to eq({})
     end
   end
 
@@ -1088,16 +1087,16 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
       end
 
       after do
-        described_class.class_variable_set(:@@logon_failures, nil) # rubocop:disable Style/ClassVars
+        described_class.instance_variable_set(:@logon_failures, [])
       end
 
       it 'returns false if there have been no failed logons with the username/password combination' do
-        described_class.class_variable_set(:@@logon_failures, [bad_credential_hash]) # rubocop:disable Style/ClassVars
+        described_class.instance_variable_set(:@logon_failures, [bad_credential_hash])
         expect(provider.logon_failed_already?(good_credential_hash)).to be(false)
       end
 
-      it 'returns true if the username/password specified are found in the logon_failures class variable' do
-        described_class.class_variable_set(:@@logon_failures, [good_credential_hash, bad_credential_hash]) # rubocop:disable Style/ClassVars
+      it 'returns true if the username/password specified are found in the logon_failures instance variable' do
+        described_class.instance_variable_set(:@logon_failures, [good_credential_hash, bad_credential_hash])
         expect(provider.logon_failed_already?(bad_credential_hash)).to be(true)
       end
     end
@@ -1511,7 +1510,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
       end
 
       after do
-        described_class.class_variable_set(:@@instantiated_variables, nil) # rubocop:disable Style/ClassVars
+        described_class.instance_variable_set(:@instantiated_variables, [])
       end
 
       it 'writes the ruby representation of the credentials as the value of a key named for the new variable into the instantiated_variables cache' do
@@ -1544,7 +1543,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
     subject(:result) { provider.prepare_cim_instances(test_resource) }
 
     after do
-      described_class.class_variable_set(:@@instantiated_variables, nil) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@instantiated_variables, [])
     end
 
     context 'when a cim instance is passed without nested cim instances' do
@@ -1653,7 +1652,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
 
   describe '.format_ciminstance' do
     after do
-      described_class.class_variable_set(:@@instantiated_variables, nil) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@instantiated_variables, [])
     end
 
     it 'defines and returns a new cim instance as a PowerShell variable, passing the class name and property hash' do
@@ -1669,7 +1668,7 @@ RSpec.describe Puppet::Provider::DscBaseProvider do
     end
 
     it 'interpolates variables in the case of a cim instance containing a nested instance' do
-      described_class.class_variable_set(:@@instantiated_variables, { 'SomeVariable' => { 'bar' => 'ope' } }) # rubocop:disable Style/ClassVars
+      described_class.instance_variable_set(:@instantiated_variables, { 'SomeVariable' => { 'bar' => 'ope' } })
       property_hash = { 'foo' => { 'bar' => 'ope' } }
       expect(provider.format_ciminstance('foo', 'SomeClass', property_hash)).to match(/@\{'foo' = \$SomeVariable\}/)
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-pwsh
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Puppet, Inc.
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-08-16 00:00:00.000000000 Z
+date: 2023-12-13 00:00:00.000000000 Z
 dependencies: []
 description: PowerShell code manager for ruby.
 email: