(FM-7691) Fix tests to not rely on previously registered transports

puppetlabs · Jan 23, 2019 · 7d51510 · 7d51510
1 parent 383028a
commit 7d51510
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/spec/puppet/resource_api/transport_spec.rb b/spec/puppet/resource_api/transport_spec.rb
@@ -11,6 +11,11 @@
     Puppet.debug = true
   end
 
+  after(:each) do
+    # reset registered transports between tests to reduce cross-test poisoning
+    described_class.instance_variable_set(:@transports, nil)
+  end
+
   context 'when registering a schema with missing keys' do
     it { expect { described_class.register([]) }.to raise_error(Puppet::DevError, %r{requires a hash as schema}) }
     it { expect { described_class.register({}) }.to raise_error(Puppet::DevError, %r{requires a `:name`}) }
@@ -25,7 +30,10 @@
     it { expect { described_class.register(schema) }.not_to raise_error }
 
     context 'when re-registering a transport' do
-      it { expect { described_class.register(schema) }.to raise_error(Puppet::DevError, %r{`minimal` is already registered}) }
+      it {
+        described_class.register(schema)
+        expect { described_class.register(schema) }.to raise_error(Puppet::DevError, %r{`minimal` is already registered})
+      }
     end
   end