puppet 6.4.5-x86-mingw32 → 6.5.0-x86-mingw32

data/spec/unit/provider/package/pip_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-osfamilies = { 'windows' => ['pip.exe'], 'other' => ['pip', 'pip-python', 'pip2', 'pip-2'] }
+osfamilies = { 'windows' => ['pip.exe'], 'other' => ['pip', 'pip-python'] }
 
 describe Puppet::Type.type(:package).provider(:pip) do
 
@@ -29,14 +29,6 @@ describe Puppet::Type.type(:package).provider(:pip) do
       })
     end
 
-    it "should correctly parse arbitrary equality" do
-      expect(described_class.parse("real_package===1.2.5")).to eq({
-        :ensure   => "1.2.5",
-        :name     => "real_package",
-        :provider => :pip,
-      })
-    end
-
     it "should return nil on invalid input" do
       expect(described_class.parse("foo")).to eq(nil)
     end
@@ -153,7 +145,7 @@ describe Puppet::Type.type(:package).provider(:pip) do
   context "latest" do
     context "with pip version < 1.5.4" do
       before :each do
-        allow(described_class).to receive(:pip_version).with("/fake/bin/pip").and_return('1.0.1')
+        allow(@provider).to receive(:pip_version).with("/fake/bin/pip").and_return('1.0.1')
         allow(described_class).to receive(:which).with('pip').and_return("/fake/bin/pip")
         allow(described_class).to receive(:which).with('pip-python').and_return("/fake/bin/pip")
         allow(described_class).to receive(:which).with('pip.exe').and_return("/fake/bin/pip")
@@ -205,25 +197,18 @@ describe Puppet::Type.type(:package).provider(:pip) do
         @resource[:name] = "fake_package"
         expect(@provider.latest).to eq(nil)
       end
-
-      it "should use 'install_options' when specified" do
-        expect(Puppet::Util::Execution).to receive(:execpipe).with(array_including([["--index=https://fake.example.com"]])).once
-        @resource[:name] = "fake_package"
-        @resource[:install_options] = ['--index' => 'https://fake.example.com']
-        expect(@provider.latest).to eq(nil)
-      end
     end
 
     context "with pip version >= 1.5.4" do
       # For Pip 1.5.4 and above, you can get a version list from CLI - which allows for native pip behavior
       # with regards to custom repositories, proxies and the like
       before :each do
-        allow(described_class).to receive(:pip_version).with("/fake/bin/pip").and_return('1.5.4')
         allow(described_class).to receive(:which).with('pip').and_return("/fake/bin/pip")
         allow(described_class).to receive(:which).with('pip-python').and_return("/fake/bin/pip")
         allow(described_class).to receive(:which).with('pip.exe').and_return("/fake/bin/pip")
         allow(described_class).to receive(:provider_command).and_return('/fake/bin/pip')
         allow(described_class).to receive(:validate_command).with('/fake/bin/pip')
+        allow(@provider).to receive(:pip_version).with("/fake/bin/pip").and_return('1.5.4')
       end
 
       it "should find a version number for real_package" do
@@ -266,13 +251,6 @@ describe Puppet::Type.type(:package).provider(:pip) do
         latest = @provider.latest
         expect(latest).to eq('15.0.2')
       end
-
-      it "should use 'install_options' when specified" do
-        expect(Puppet::Util::Execution).to receive(:execpipe).with(array_including([["--index=https://fake.example.com"]])).once
-        @resource[:name] = "fake_package"
-        @resource[:install_options] = ['--index' => 'https://fake.example.com']
-        expect(@provider.latest).to eq(nil)
-      end
     end
   end
 
@@ -363,43 +341,12 @@ describe Puppet::Type.type(:package).provider(:pip) do
   end
 
   context "pip_version" do
-    let(:pip) { '/fake/bin/pip' }
-
     it "should look up version if pip is present" do
-      allow(described_class).to receive(:pip_cmd).and_return(pip)
-      process = ['pip 8.0.2 from /usr/local/lib/python2.7/dist-packages (python 2.7)']
-      allow(described_class).to receive(:execpipe).with([pip, '--version']).and_yield(process)
-
-      expect(described_class.pip_version(pip)).to eq('8.0.2')
-    end
-
-    it "parses multiple lines of output" do
-      allow(described_class).to receive(:pip_cmd).and_return(pip)
-      process = [
-        "/usr/local/lib/python2.7/dist-packages/urllib3/contrib/socks.py:37: DependencyWarning: SOCKS support in urllib3 requires the installation of optional dependencies: specifically, PySocks. For more information, see https://urllib3.readthedocs.io/en/latest/contrib.html#socks-proxies",
-        "  DependencyWarning",
-        "pip 1.5.6 from /usr/lib/python2.7/dist-packages (python 2.7)"
-      ]
-      allow(described_class).to receive(:execpipe).with([pip, '--version']).and_yield(process)
-
-      expect(described_class.pip_version(pip)).to eq('1.5.6')
-    end
-
-    it "raises if there isn't a version string" do
-      allow(described_class).to receive(:pip_cmd).and_return(pip)
-      allow(described_class).to receive(:execpipe).with([pip, '--version']).and_yield([""])
-      expect {
-        described_class.pip_version(pip)
-      }.to raise_error(Puppet::Error, 'Cannot resolve pip version')
-    end
-
-    it "quotes commands with spaces" do
-      pip = 'C:\Program Files\Python27\Scripts\pip.exe'
-      allow(described_class).to receive(:pip_cmd).and_return(pip)
-      process = ["pip 18.1 from c:\program files\python27\lib\site-packages\pip (python 2.7)\r\n"]
-      allow(described_class).to receive(:execpipe).with(["\"#{pip}\"", '--version']).and_yield(process)
-
-      expect(described_class.pip_version(pip)).to eq('18.1')
+      allow(described_class).to receive(:pip_cmd).and_return('/fake/bin/pip')
+      p = double("process")
+      expect(p).to receive(:collect).and_yield('pip 8.0.2 from /usr/local/lib/python2.7/dist-packages (python 2.7)')
+      expect(described_class).to receive(:execpipe).with(['/fake/bin/pip', '--version']).and_yield(p)
+      expect(described_class.pip_version('/fake/bin/pip')).to eq('8.0.2')
     end
   end
 

data/spec/unit/provider/package/portage_spec.rb

@@ -44,13 +44,13 @@ describe Puppet::Type.type(:package).provider(:portage) do
     allow(@unslotted_provider).to receive(:qatom).and_return({:category=>"dev-lang", :pn=>"ruby", :pv=>nil, :pr=>nil, :slot=>nil, :pfx=>nil, :sfx=>nil})
     allow(@unslotted_provider.class).to receive(:emerge).with('--list-sets').and_return(package_sets)
     @slotted_provider = described_class.new(@slotted_resource)
-    allow(@slotted_provider).to receive(:qatom).and_return({:category=>"dev-lang", :pn=>"ruby", :pv=>nil, :pr=>nil, :slot=>"2.1", :pfx=>nil, :sfx=>nil})
+    allow(@slotted_provider).to receive(:qatom).and_return({:category=>"dev-lang", :pn=>"ruby", :pv=>nil, :pr=>nil, :slot=>":2.1", :pfx=>nil, :sfx=>nil})
     allow(@slotted_provider.class).to receive(:emerge).with('--list-sets').and_return(package_sets)
     @versioned_provider = described_class.new(@versioned_resource)
     allow(@versioned_provider).to receive(:qatom).and_return({:category=>"dev-lang", :pn=>"ruby", :pv=>"1.9.3", :pr=>nil, :slot=>nil, :pfx=>"=", :sfx=>nil})
     allow(@versioned_provider.class).to receive(:emerge).with('--list-sets').and_return(package_sets)
     @versioned_slotted_provider = described_class.new(@versioned_slotted_resource)
-    allow(@versioned_slotted_provider).to receive(:qatom).and_return({:category=>"dev-lang", :pn=>"ruby", :pv=>"1.9.3", :pr=>nil, :slot=>"1.9", :pfx=>"=", :sfx=>nil})
+    allow(@versioned_slotted_provider).to receive(:qatom).and_return({:category=>"dev-lang", :pn=>"ruby", :pv=>"1.9.3", :pr=>nil, :slot=>":1.9", :pfx=>"=", :sfx=>nil})
     allow(@versioned_slotted_provider.class).to receive(:emerge).with('--list-sets').and_return(package_sets)
     @set_provider = described_class.new(@set_resource)
     allow(@set_provider).to receive(:qatom).and_return({:category=>nil, :pn=>"@system", :pv=>nil, :pr=>nil, :slot=>nil, :pfx=>nil, :sfx=>nil})
@@ -180,7 +180,7 @@ describe Puppet::Type.type(:package).provider(:portage) do
   end
 
   it "can extract the slot from the package name" do
-    expect(@slotted_provider.qatom[:slot]).to eq('2.1')
+    expect(@slotted_provider.qatom[:slot]).to eq(':2.1')
   end
 
   it "returns nil for as the slot when no slot is specified" do
@@ -196,7 +196,7 @@ describe Puppet::Type.type(:package).provider(:portage) do
     expect(@versioned_slotted_provider.qatom[:category]).to eq('dev-lang')
     expect(@versioned_slotted_provider.qatom[:pn]).to eq('ruby')
     expect(@versioned_slotted_provider.qatom[:pv]).to eq('1.9.3')
-    expect(@versioned_slotted_provider.qatom[:slot]).to eq('1.9')
+    expect(@versioned_slotted_provider.qatom[:slot]).to eq(':1.9')
   end
 
   it "can handle search output with slots for unslotted packages" do

data/spec/unit/provider/package/rpm_spec.rb

@@ -9,8 +9,6 @@ describe Puppet::Type.type(:package).provider(:rpm) do
     myresource 0 1.2.3.4 5.el4 noarch
     mysummaryless 0 1.2.3.4 5.el4 noarch
     tomcat 1 1.2.3.4 5.el4 x86_64
-    kernel 1 1.2.3.4 5.el4 x86_64
-    kernel 1 1.2.3.6 5.el4 x86_64
     '
     RPM_OUTPUT
   end
@@ -59,7 +57,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
     describe "with a modern version of RPM" do
       it "includes all the modern flags" do
         expect(Puppet::Util::Execution).to receive(:execpipe)
-          .with("/bin/rpm -qa --nosignature --nodigest --qf '#{nevra_format}' | sort")
+          .with("/bin/rpm -qa --nosignature --nodigest --qf '#{nevra_format}'")
           .and_yield(packages)
 
         described_class.instances
@@ -71,7 +69,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       it "excludes the --nosignature flag" do
         expect(Puppet::Util::Execution).to receive(:execpipe)
-          .with("/bin/rpm -qa  --nodigest --qf '#{nevra_format}' | sort")
+          .with("/bin/rpm -qa  --nodigest --qf '#{nevra_format}'")
           .and_yield(packages)
 
         described_class.instances
@@ -83,7 +81,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       it "excludes the --nodigest flag" do
         expect(Puppet::Util::Execution).to receive(:execpipe)
-        .with("/bin/rpm -qa   --qf '#{nevra_format}' | sort")
+        .with("/bin/rpm -qa   --qf '#{nevra_format}'")
         .and_yield(packages)
 
         described_class.instances
@@ -92,7 +90,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
     it "returns an array of packages" do
       expect(Puppet::Util::Execution).to receive(:execpipe)
-        .with("/bin/rpm -qa --nosignature --nodigest --qf '#{nevra_format}' | sort")
+        .with("/bin/rpm -qa --nosignature --nodigest --qf '#{nevra_format}'")
         .and_yield(packages)
 
       installed_packages = described_class.instances
@@ -163,17 +161,6 @@ describe Puppet::Type.type(:package).provider(:rpm) do
           :ensure      => "1:1.2.3.4-5.el4",
         }
       )
-      expect(installed_packages[6].properties).to eq(
-        {
-          :provider    => :rpm,
-          :name        => "kernel",
-          :epoch       => "1",
-          :version     => "1.2.3.4",
-          :release     => "5.el4",
-          :arch        => "x86_64",
-          :ensure      => "1:1.2.3.4-5.el4; 1:1.2.3.6-5.el4",
-        }
-      )
     end
   end
 
@@ -231,7 +218,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
     it "retrieves version string after querying rpm for version from source file" do
       expect(resource).to receive(:[]).with(:source).and_return('source-string')
       expect(Puppet::Util::Execution).to receive(:execute)
-        .with(["/bin/rpm", "-q", "--qf", "#{nevra_format}", "-p", "source-string"])
+        .with(["/bin/rpm", "-q", "--qf", "'#{nevra_format}'", "-p", "source-string"])
         .and_return(Puppet::Util::Execution::ProcessOutput.new("myresource 0 1.2.3.4 5.el4 noarch\n", 0))
       expect(provider.latest).to eq("1.2.3.4-5.el4")
     end
@@ -239,7 +226,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
     it "raises an error if the rpm command fails" do
       expect(resource).to receive(:[]).with(:source).and_return('source-string')
       expect(Puppet::Util::Execution).to receive(:execute)
-        .with(["/bin/rpm", "-q", "--qf", "#{nevra_format}", "-p", "source-string"])
+        .with(["/bin/rpm", "-q", "--qf", "'#{nevra_format}'", "-p", "source-string"])
         .and_raise(Puppet::ExecutionFailure, 'rpm command failed')
 
       expect {
@@ -261,7 +248,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       before(:each) do
         expect(Puppet::Util::Execution).to receive(:execute)
-          .with(["/bin/rpm", "-q", resource_name, '', '', '--qf', "#{nevra_format}"], execute_options)
+          .with(["/bin/rpm", "-q", resource_name, '', '', '--qf', "'#{nevra_format}'"], execute_options)
           .and_return(Puppet::Util::Execution::ProcessOutput.new("#{resource_name} 0 1.2.3.4 5.el4 noarch\n", 0))
       end
 
@@ -279,7 +266,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       before(:each) do
         expect(Puppet::Util::Execution).to receive(:execute)
-          .with(["/bin/rpm", "-q", resource_name, '--nosignature', '--nodigest', "--qf", "#{nevra_format}"], execute_options)
+          .with(["/bin/rpm", "-q", resource_name, '--nosignature', '--nodigest', "--qf", "'#{nevra_format}'"], execute_options)
           .and_return(Puppet::Util::Execution::ProcessOutput.new("#{resource_name} 0 1.2.3.4 5.el4 noarch\n", 0))
       end
 
@@ -303,7 +290,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       before(:each) do
         expect(Puppet::Util::Execution).to receive(:execute)
-          .with(["/bin/rpm", "-q", "#{resource_name}.noarch", '--nosignature', '--nodigest', "--qf", "#{nevra_format}"], execute_options)
+          .with(["/bin/rpm", "-q", "#{resource_name}.noarch", '--nosignature', '--nodigest', "--qf", "'#{nevra_format}'"], execute_options)
           .and_return(Puppet::Util::Execution::ProcessOutput.new("#{resource_name} 0 1.2.3.4 5.el4 noarch\n", 0))
       end
 
@@ -325,7 +312,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       before(:each) do
         expect(Puppet::Util::Execution).to receive(:execute)
-          .with(["/bin/rpm", "-q", "#{resource_name}-1.2.3.4-5.el4", '--nosignature', '--nodigest', "--qf", "#{nevra_format}"], execute_options)
+          .with(["/bin/rpm", "-q", "#{resource_name}-1.2.3.4-5.el4", '--nosignature', '--nodigest', "--qf", "'#{nevra_format}'"], execute_options)
           .and_return(Puppet::Util::Execution::ProcessOutput.new("#{resource_name} 0 1.2.3.4 5.el4 noarch\n", 0))
       end
 
@@ -347,7 +334,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       before(:each) do
         expect(Puppet::Util::Execution).to receive(:execute)
-          .with(["/bin/rpm", "-q", "#{resource_name}-1.2.3.4", '--nosignature', '--nodigest', "--qf", "#{nevra_format}"], execute_options)
+          .with(["/bin/rpm", "-q", "#{resource_name}-1.2.3.4", '--nosignature', '--nodigest', "--qf", "'#{nevra_format}'"], execute_options)
           .and_return(Puppet::Util::Execution::ProcessOutput.new("#{resource_name} 0 1.2.3.4 5.el4 noarch\n", 0))
       end
 
@@ -371,7 +358,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       before(:each) do
         expect(Puppet::Util::Execution).to receive(:execute)
-          .with(["/bin/rpm", "-q", resource_name, '--nosignature', '--nodigest', "--qf", "#{nevra_format}"], execute_options)
+          .with(["/bin/rpm", "-q", resource_name, '--nosignature', '--nodigest', "--qf", "'#{nevra_format}'"], execute_options)
           .and_return(Puppet::Util::Execution::ProcessOutput.new("#{resource_name} 0 1.2.3.4 5.el4 noarch\n", 0))
       end
 
@@ -387,7 +374,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
     def parser_test(rpm_output_string, gold_hash, number_of_debug_logs = 0)
       expect(Puppet).to receive(:debug).exactly(number_of_debug_logs).times()
       expect(Puppet::Util::Execution).to receive(:execute)
-        .with(["/bin/rpm", "-q", resource_name, "--nosignature", "--nodigest", "--qf", "#{nevra_format}"], execute_options)
+        .with(["/bin/rpm", "-q", resource_name, "--nosignature", "--nodigest", "--qf", "'#{nevra_format}'"], execute_options)
         .and_return(Puppet::Util::Execution::ProcessOutput.new(rpm_output_string, 0))
       expect(provider.query).to eq(gold_hash)
     end
@@ -426,7 +413,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
     describe "when the package is not found" do
       before do
         expect(Puppet).not_to receive(:debug)
-        expected_args = ["/bin/rpm", "-q", resource_name, "--nosignature", "--nodigest", "--qf", "#{nevra_format}"]
+        expected_args = ["/bin/rpm", "-q", resource_name, "--nosignature", "--nodigest", "--qf", "'#{nevra_format}'"]
         expect(Puppet::Util::Execution).to receive(:execute)
           .with(expected_args, execute_options)
           .and_raise(Puppet::ExecutionFailure.new("package #{resource_name} is not installed"))
@@ -439,7 +426,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
       it "does not log or fail if allow_virtual is true" do
         resource[:allow_virtual] = true
-        expected_args = ['/bin/rpm', '-q', resource_name, '--nosignature', '--nodigest', '--qf', "#{nevra_format}", '--whatprovides']
+        expected_args = ['/bin/rpm', '-q', resource_name, '--nosignature', '--nodigest', '--qf', "'#{nevra_format}'", '--whatprovides']
         expect(Puppet::Util::Execution).to receive(:execute)
           .with(expected_args, execute_options)
           .and_raise(Puppet::ExecutionFailure.new("package #{resource_name} is not provided"))
@@ -449,7 +436,7 @@ describe Puppet::Type.type(:package).provider(:rpm) do
 
     it "parses virtual package" do
       provider.resource[:allow_virtual] = true
-      expected_args = ["/bin/rpm", "-q", resource_name, "--nosignature", "--nodigest", "--qf", "#{nevra_format}"]
+      expected_args = ["/bin/rpm", "-q", resource_name, "--nosignature", "--nodigest", "--qf", "'#{nevra_format}'"]
       expect(Puppet::Util::Execution).to receive(:execute)
         .with(expected_args, execute_options)
         .and_raise(Puppet::ExecutionFailure.new("package #{resource_name} is not installed"))
@@ -762,125 +749,4 @@ describe Puppet::Type.type(:package).provider(:rpm) do
       provider.compare_values('s1', 's2')
     end
   end
-
-
-  describe 'insync?' do
-    context 'for multiple versions' do
-      let(:is) { '1:1.2.3.4-5.el4; 1:5.6.7.8-5.el4' }
-      it 'returns true if there is match and feature is enabled' do
-        resource[:install_only] = true
-        resource[:ensure] = '1:1.2.3.4-5.el4'
-        expect(provider).to be_insync(is)
-      end
-      it 'returns false if there is match and feature is not enabled' do
-        resource[:ensure] = '1:1.2.3.4-5.el4'
-        expect(provider).to_not be_insync(is)
-      end
-      it 'returns false if no match and feature is enabled' do
-        resource[:install_only] = true
-        resource[:ensure] = '1:1.2.3.6-5.el4'
-        expect(provider).to_not be_insync(is)
-      end
-      it 'returns false if no match and feature is not enabled' do
-        resource[:ensure] = '1:1.2.3.6-5.el4'
-        expect(provider).to_not be_insync(is)
-      end
-    end
-    context 'for simple versions' do
-      let(:is) { '1:1.2.3.4-5.el4' }
-      it 'returns true if there is match and feature is enabled' do
-        resource[:install_only] = true
-        resource[:ensure] = '1:1.2.3.4-5.el4'
-        expect(provider).to be_insync(is)
-      end
-      it 'returns true if there is match and feature is not enabled' do
-        resource[:ensure] = '1:1.2.3.4-5.el4'
-        expect(provider).to be_insync(is)
-      end
-      it 'returns false if no match and feature is enabled' do
-        resource[:install_only] = true
-        resource[:ensure] = '1:1.2.3.6-5.el4'
-        expect(provider).to_not be_insync(is)
-      end
-      it 'returns false if no match and feature is not enabled' do
-        resource[:ensure] = '1:1.2.3.6-5.el4'
-        expect(provider).to_not be_insync(is)
-      end
-    end
-  end
-
-  describe 'rpm multiversion to hash' do
-    it 'should return empty hash for empty imput' do
-      package_hash = described_class.nevra_to_multiversion_hash('')
-      expect(package_hash).to eq({})
-    end
-
-    it 'should return package hash for one package input' do
-      package_list = <<-RPM_OUTPUT
-kernel-devel 1 1.2.3.4 5.el4 x86_64
-RPM_OUTPUT
-      package_hash = described_class.nevra_to_multiversion_hash(package_list)
-      expect(package_hash).to eq(
-        {
-          :arch => "x86_64",
-          :ensure => "1:1.2.3.4-5.el4",
-          :epoch => "1",
-          :name => "kernel-devel",
-          :provider => :rpm,
-          :release => "5.el4",
-          :version => "1.2.3.4",
-        }
-      )
-    end
-
-    it 'should return package hash with versions concatenated in ensure for two package input' do
-      package_list = <<-RPM_OUTPUT
-kernel-devel 1 1.2.3.4 5.el4 x86_64
-kernel-devel 1 5.6.7.8 5.el4 x86_64
-RPM_OUTPUT
-      package_hash = described_class.nevra_to_multiversion_hash(package_list)
-      expect(package_hash).to eq(
-        {
-          :arch => "x86_64",
-          :ensure => "1:1.2.3.4-5.el4; 1:5.6.7.8-5.el4",
-          :epoch => "1",
-          :name => "kernel-devel",
-          :provider => :rpm,
-          :release => "5.el4",
-          :version => "1.2.3.4",
-        }
-      )
-    end
-
-    it 'should return list of packages for one multiversion and one package input' do
-      package_list = <<-RPM_OUTPUT
-kernel-devel 1 1.2.3.4 5.el4 x86_64
-kernel-devel 1 5.6.7.8 5.el4 x86_64
-basesystem 0 8.0 5.1.1.el5.centos noarch
-RPM_OUTPUT
-      package_hash = described_class.nevra_to_multiversion_hash(package_list)
-      expect(package_hash).to eq(
-        [
-          {
-            :arch => "x86_64",
-            :ensure => "1:1.2.3.4-5.el4; 1:5.6.7.8-5.el4",
-            :epoch => "1",
-            :name => "kernel-devel",
-            :provider => :rpm,
-            :release => "5.el4",
-            :version => "1.2.3.4",
-          },
-          {
-            :provider => :rpm,
-            :name => "basesystem",
-            :epoch => "0",
-            :version => "8.0",
-            :release => "5.1.1.el5.centos",
-            :arch => "noarch",
-            :ensure => "8.0-5.1.1.el5.centos",
-          }
-        ]
-      )
-    end
-  end
 end

data/spec/unit/provider/package/yum_spec.rb

@@ -64,13 +64,6 @@ describe Puppet::Type.type(:package).provider(:yum) do
 
     before { allow(described_class).to receive(:command).with(:cmd).and_return("/usr/bin/yum") }
 
-  describe 'provider features' do
-    it { is_expected.to be_versionable }
-    it { is_expected.to be_install_options }
-    it { is_expected.to be_virtual_packages }
-    it { is_expected.to be_install_only }
-  end
-
     context "when installing" do
       it "should use the supplied source as the explicit path to a package to install" do
         resource[:ensure] = :present