puppet 6.14.0 → 6.15.0

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

@@ -123,6 +123,36 @@ Version table:
       provider.install
     end
 
+    it "should select latest available version if range is specified" do
+      resource[:ensure] = '>60.0'
+      expect(provider).to receive(:aptget) do |*command|
+        expect(command[-1]).to eq("#{name}=72.0.1+build1-0ubuntu0.19.04.1")
+      end
+      expect(provider).to receive(:aptcache).with(:madison, name).and_return(<<-HERE)
+   #{name} | 72.0.1+build1-0ubuntu0.19.04.1 | http://ro.archive.ubuntu.com/ubuntu disco-updates/main amd64 Packages
+   #{name} | 72.0.1+build1-0ubuntu0.19.04.1 | http://security.ubuntu.com/ubuntu disco-security/main amd64 Packages
+   #{name} | 66.0.3+build1-0ubuntu1 | http://ro.archive.ubuntu.com/ubuntu disco/main amd64 Packages
+    HERE
+      expect(provider).to receive(:properties).and_return({:mark => :none})
+
+      provider.install
+    end
+
+    it "should pass through ensure is no version can be selected" do
+      resource[:ensure] = '>74.0'
+      expect(provider).to receive(:aptget) do |*command|
+        expect(command[-1]).to eq("#{name}=>74.0")
+      end
+      expect(provider).to receive(:aptcache).with(:madison, name).and_return(<<-HERE)
+   #{name} | 72.0.1+build1-0ubuntu0.19.04.1 | http://ro.archive.ubuntu.com/ubuntu disco-updates/main amd64 Packages
+   #{name} | 72.0.1+build1-0ubuntu0.19.04.1 | http://security.ubuntu.com/ubuntu disco-security/main amd64 Packages
+   #{name} | 66.0.3+build1-0ubuntu1 | http://ro.archive.ubuntu.com/ubuntu disco/main amd64 Packages
+    HERE
+      expect(provider).to receive(:properties).and_return({:mark => :none})
+
+      provider.install
+    end
+
     it "should use --force-yes if a package version is specified" do
       resource[:ensure] = '1.0'
       expect(provider).to receive(:aptget) do |*command|

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

@@ -18,7 +18,7 @@ describe Puppet::Type.type(:package).provider(:dnfmodule) do
     {:failonfail => true, :combine => true, :custom_environment => {}}
   end
 
-  let(:packages) { File.read(my_fixture("dnf-module-list-installed.txt")) }
+  let(:packages) { File.read(my_fixture("dnf-module-list-enabled.txt")) }
   let(:dnf_path) { '/usr/bin/dnf' }
 
   before(:each) { allow(Puppet::Util).to receive(:which).with('/usr/bin/dnf').and_return(dnf_path) }
@@ -123,6 +123,23 @@ describe Puppet::Type.type(:package).provider(:dnfmodule) do
         provider.install
       end
 
+      it "should just enable the module if it has no default profile" do
+        dnf_exception = Puppet::ExecutionFailure.new("Error: Problems in request:\nmissing groups or modules: #{resource[:name]}")
+        allow(provider).to receive(:execute).with(array_including('install')).and_raise(dnf_exception)
+        resource[:ensure] = :present
+        expect(provider).to receive(:execute).with(array_including('install')).ordered
+        expect(provider).to receive(:execute).with(array_including('enable')).ordered
+        provider.install
+      end
+
+      it "should just enable the module if enable_only = true" do
+        resource[:ensure] = :present
+        resource[:enable_only] = true
+        expect(provider).to receive(:execute).with(array_including('enable'))
+        expect(provider).not_to receive(:execute).with(array_including('install'))
+        provider.install
+      end
+
       it "should install the default stream and flavor" do
         resource[:ensure] = :present
         expect(provider).to receive(:execute).with(array_including('baz'))
@@ -185,24 +202,26 @@ describe Puppet::Type.type(:package).provider(:dnfmodule) do
     end
   end
 
-  context "parsing the output of module list --installed" do
+  context "parsing the output of module list --enabled" do
     before { allow(described_class).to receive(:command).with(:dnf).and_return(dnf_path) }
 
-    it "returns an array of installed modules" do
+    it "returns an array of enabled modules" do
       allow(Puppet::Util::Execution).to receive(:execute)
-        .with("/usr/bin/dnf module list --installed -d 0 -e 1")
+        .with("/usr/bin/dnf module list --enabled -d 0 -e 1")
         .and_return(packages)
 
-      installed_packages = described_class.instances.map { |package| package.properties }
-      expected_packages = [{name: "gimp", ensure: "2.8", flavor: "devel", :provider => :dnfmodule},
-                           {name: "mariadb", ensure: "10.3", flavor: "client", :provider => :dnfmodule},
-                           {name: "nodejs", ensure: "10", flavor: "minimal", :provider => :dnfmodule},
-                           {name: "perl", ensure: "5.26", flavor: "minimal", :provider => :dnfmodule},
-                           {name: "postgresql", ensure: "10", flavor: "server", :provider => :dnfmodule},
-                           {name: "rust-toolset", ensure: "rhel8", flavor: "common", :provider => :dnfmodule},
-                           {name: "subversion", ensure: "1.10", flavor: "server", :provider => :dnfmodule}]
-
-      expect(installed_packages).to eql(expected_packages)
+      enabled_packages = described_class.instances.map { |package| package.properties }
+      expected_packages = [{name: "389-ds", ensure: "1.4", flavor: :absent, provider: :dnfmodule},
+                           {name: "gimp", ensure: "2.8", flavor: "devel", provider: :dnfmodule},
+                           {name: "mariadb", ensure: "10.3", flavor: "client", provider: :dnfmodule},
+                           {name: "nodejs", ensure: "10", flavor: "minimal", provider: :dnfmodule},
+                           {name: "perl", ensure: "5.26", flavor: "minimal", provider: :dnfmodule},
+                           {name: "postgresql", ensure: "10", flavor: "server", provider: :dnfmodule},
+                           {name: "ruby", ensure: "2.5", flavor: :absent, provider: :dnfmodule},
+                           {name: "rust-toolset", ensure: "rhel8", flavor: "common", provider: :dnfmodule},
+                           {name: "subversion", ensure: "1.10", flavor: "server", provider: :dnfmodule}]
+
+      expect(enabled_packages).to eql(expected_packages)
     end
   end
 end

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

@@ -257,6 +257,26 @@ context Puppet::Type.type(:package).provider(:gem) do
           resource[:ensure] = '3.6.2'
           expect(provider).to_not be_insync(is)
         end
+
+        it 'returns true for >2, <4' do
+          resource[:ensure] = '>2, <4'
+          expect(provider).to be_insync(is)
+        end
+
+        it 'returns false for >=4, <5' do
+          resource[:ensure] = '>=4, <5'
+          expect(provider).to_not be_insync(is)
+        end
+
+        it 'returns true for >2 <4' do
+          resource[:ensure] = '>2 <4'
+          expect(provider).to be_insync(is)
+        end
+
+        it 'returns false for >=4 <5' do
+          resource[:ensure] = '>=4 <5'
+          expect(provider).to_not be_insync(is)
+        end
       end
 
       context 'for string version' do
@@ -286,6 +306,26 @@ context Puppet::Type.type(:package).provider(:gem) do
           resource[:ensure] = '3.6.1'
           expect(provider).to_not be_insync(is)
         end
+
+        it 'returns true for >=1.3, <2' do
+          resource[:ensure] = '>=1.3, <2'
+          expect(provider).to be_insync(is)
+        end
+
+        it 'returns false for >1, <=1.3' do
+          resource[:ensure] = '>1, <=1.3'
+          expect(provider).to_not be_insync(is)
+        end
+
+        it 'returns true for >=1.3 <2' do
+          resource[:ensure] = '>=1.3 <2'
+          expect(provider).to be_insync(is)
+        end
+
+        it 'returns false for >1 <=1.3' do
+          resource[:ensure] = '>1 <=1.3'
+          expect(provider).to_not be_insync(is)
+        end
       end
 
       it 'should return false for bad version specifiers' do

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

@@ -26,14 +26,14 @@ describe Puppet::Type.type(:package).provider(:pacman) do
     end
 
     it "should call pacman to install the right package quietly when yaourt is not installed" do
-      args = ['--noconfirm', '--needed', '--noprogressbar', '-Sy', resource[:name]]
+      args = ['--noconfirm', '--needed', '--noprogressbar', '-S', resource[:name]]
       expect(provider).to receive(:pacman).at_least(:once).with(*args).and_return('')
       provider.install
     end
 
     it "should call yaourt to install the right package quietly when yaourt is installed" do
       allow(described_class).to receive(:yaourt?).and_return(true)
-      args = ['--noconfirm', '--needed', '--noprogressbar', '-Sy', resource[:name]]
+      args = ['--noconfirm', '--needed', '--noprogressbar', '-S', resource[:name]]
       expect(provider).to receive(:yaourt).at_least(:once).with(*args).and_return('')
       provider.install
     end
@@ -68,14 +68,14 @@ describe Puppet::Type.type(:package).provider(:pacman) do
       end
 
       it "should call pacman to install the right package quietly when yaourt is not installed" do
-        args = ['--noconfirm', '--needed', '--noprogressbar', '-x', '--arg=value', '-Sy', resource[:name]]
+        args = ['--noconfirm', '--needed', '--noprogressbar', '-x', '--arg=value', '-S', resource[:name]]
         expect(provider).to receive(:pacman).at_least(:once).with(*args).and_return('')
         provider.install
       end
 
       it "should call yaourt to install the right package quietly when yaourt is installed" do
         expect(described_class).to receive(:yaourt?).and_return(true)
-        args = ['--noconfirm', '--needed', '--noprogressbar', '-x', '--arg=value', '-Sy', resource[:name]]
+        args = ['--noconfirm', '--needed', '--noprogressbar', '-x', '--arg=value', '-S', resource[:name]]
         expect(provider).to receive(:yaourt).at_least(:once).with(*args).and_return('')
         provider.install
       end
@@ -94,7 +94,7 @@ describe Puppet::Type.type(:package).provider(:pacman) do
             resource[:source] = source
 
             expect(executor).to receive(:execute).
-              with(include("-Sy") & include("--noprogressbar"), no_extra_options).
+              with(include("-S") & include("--noprogressbar"), no_extra_options).
               ordered.
               and_return("")
 
@@ -117,7 +117,7 @@ describe Puppet::Type.type(:package).provider(:pacman) do
 
         it "should install from the path segment of the URL" do
           expect(executor).to receive(:execute).
-            with(include("-Sy") & include("--noprogressbar") & include("--noconfirm"),
+            with(include("-S") & include("--noprogressbar") & include("--noconfirm"),
                  no_extra_options).
             ordered.
             and_return("")
@@ -348,21 +348,7 @@ EOF
   end
 
   describe "when determining the latest version" do
-    it "should refresh package list" do
-      expect(executor).to receive(:execute).
-        ordered.
-        with(['/usr/bin/pacman', '-Sy'], no_extra_options)
-
-      expect(executor).to receive(:execute).
-        ordered.
-        and_return("")
-
-      provider.latest
-    end
-
     it "should get query pacman for the latest version" do
-      expect(executor).to receive(:execute).ordered
-
       expect(executor).to receive(:execute).
         ordered.
         with(['/usr/bin/pacman', '-Sp', '--print-format', '%v', resource[:name]], no_extra_options).
@@ -379,7 +365,6 @@ EOF
 
     it "should return a virtual group version when resource is a package group" do
       allow(described_class).to receive(:group?).and_return(true)
-      expect(executor).to receive(:execute).with(['/usr/bin/pacman', '-Sy'], no_extra_options).ordered
       expect(executor).to receive(:execute).with(['/usr/bin/pacman', '-Sp', '--print-format', '%n %v', resource[:name]], no_extra_options).ordered.
         and_return(<<EOF)
 package2 1.0.1

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

@@ -10,6 +10,7 @@ describe Puppet::Type.type(:package).provider(:pip) do
   it { is_expected.to be_versionable }
   it { is_expected.to be_install_options }
   it { is_expected.to be_targetable }
+  it { is_expected.to be_version_ranges }
 
   before do
     @resource = Puppet::Resource.new(:package, "fake_package")
@@ -150,6 +151,14 @@ describe Puppet::Type.type(:package).provider(:pip) do
     end
   end
 
+  context "when comparing versions" do
+    it "an abnormal version should still be compared (using default implementation) but a debug message should also be printed regarding it" do
+      expect(Puppet).to receive(:debug).with("Cannot compare 1.0 and abnormal-version.0.1. abnormal-version.0.1 is not a valid python package version. Please refer to https://www.python.org/dev/peps/pep-0440/. Falling through default comparison mechanism.")
+      expect(Puppet::Util::Package).to receive(:versioncmp).with('1.0', 'abnormal-version.0.1')
+      expect{ described_class.compare_pip_versions('1.0', 'abnormal-version.0.1') }.not_to raise_error
+    end
+  end
+
   context "latest" do
     context "with pip version < 1.5.4" do
       before :each do
@@ -206,6 +215,21 @@ describe Puppet::Type.type(:package).provider(:pip) do
         expect(@provider.latest).to eq(nil)
       end
 
+      it "should handle out-of-order version numbers for real_package" do
+        p = StringIO.new(
+          <<-EOS
+          Downloading/unpacking fake-package
+            Using version 2!2.3.4.alpha5.rev6.dev7+abc89 (newest of versions: 1.11, 13.0.3, 1.6, 1!1.2.rev33+123456, 1.9, 1.3.2, 14.0.1, 12.0.7, 13.0.3, 1.7.2, 1.8.4, 1.2+123abc456, 1.6.1, 0.9.2, 1.3, 1.8.3, 12.1.1, 1.1, 1.11.6, 1.2+123456, 1.4.8, 1.6.3, 1!1.0b2.post345.dev456, 1.10.1, 14.0.2, 1.11.3, 14.0.3, 1.4rc1, 1.0b2.post345.dev456, 0.8.4, 1.0, 1!1.0.post456, 12.0.5, 14.0.6, 1.11.5, 1.0rc2, 1.7.1.1, 1.11.4, 13.0.1, 13.1.2, 1.3.3, 0.8.2, 14.0.0, 12.0, 1.8, 1.3.4, 12.0, 1.2, 12.0.6, 0.9.1, 13.1.1, 2!2.3.4.alpha5.rev6.dev7+abc89, 14.0.5, 15.0.2, 15.0.0, 1.4.5, 1.4.3, 13.1.1, 1.11.2, 13.1.2, 1.2+abc123def, 1.3.1, 13.1.0, 12.0.2, 1.11.1, 12.0.1, 12.1.0, 0.9, 1.4.4, 1.2+abc123, 13.0.0, 1.4.9, 1.1.dev1, 12.1.0, 1.7.1, 1.4.2, 14.0.5, 0.8.1, 1.4.6, 0.8.3, 1.11.3, 1.5.1, 1.4.7, 13.0.2, 12.0.7, 1!13.0, 0!13.0, 1.9.1, 1.0.post456.dev34, 1.8.2, 14.0.1, 14.0.0, 1.2.rev33+123456, 14.0.4, 1.6.2, 15.0.1, 13.1.0, 0.8, 1.2+1234.abc, 1.7, 15.0.2, 12.0.5, 13.0.1, 1.8.1, 1.11.6, 15.0.1, 12.0.4, 1.2+123abc, 12.1.1, 13.0.2, 1.11.4, 1.10, 1.2.r32+123456, 14.0.4, 14.0.6, 1.4.1, 1.4, 1.5.2, 12.0.2, 12.0.1, 14.0.3, 14.0.2, 1.11.1, 1.7.1.2, 15.0.0, 12.0.4, 1.6.4, 1.11.2, 1.5, 0.1, 0.10, 0.10.1, 0.10.1.0.1, 1.0.dev456, 1.0a1, 1.0a2.dev456, 1.0a12.dev456, 1.0a12, 1.0b1.dev456, 1.0b2, 1.0b2.post345, 1.0b2-346, 1.0c1.dev456, 1.0c1, 1.0c3, 1.0, 1.0.post456, 1.2+abc, 1!1.0, 1!1.0.post456.dev34)
+            Downloading real-package-2!2.3.4.alpha5.rev6.dev7+abc89.tar.gz (544Kb): 544Kb downloaded
+          Saved ./foo/real-package-2!2.3.4.alpha5.rev6.dev7+abc89.tar.gz
+          Successfully downloaded real-package
+          EOS
+        )
+        expect(Puppet::Util::Execution).to receive(:execpipe).and_yield(p).once
+        @resource[:name] = "real_package"
+        expect(@provider.latest).to eq('2!2.3.4.alpha5.rev6.dev7+abc89')
+      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"
@@ -257,14 +281,13 @@ describe Puppet::Type.type(:package).provider(:pip) do
         p = StringIO.new(
           <<-EOS
           Collecting real-package==versionplease
-            Could not find a version that satisfies the requirement real-package==versionplease (from versions: 1.11, 13.0.3, 1.6, 1.9, 1.3.2, 14.0.1, 12.0.7, 13.0.3, 1.7.2, 1.8.4, 1.6.1, 0.9.2, 1.3, 1.8.3, 12.1.1, 1.1, 1.11.6, 1.4.8, 1.6.3, 1.10.1, 14.0.2, 1.11.3, 14.0.3, 1.4rc1, 0.8.4, 1.0, 12.0.5, 14.0.6, 1.11.5, 1.7.1.1, 1.11.4, 13.0.1, 13.1.2, 1.3.3, 0.8.2, 14.0.0, 12.0, 1.8, 1.3.4, 12.0, 1.2, 12.0.6, 0.9.1, 13.1.1, 14.0.5, 15.0.2, 15.0.0, 1.4.5, 1.4.3, 13.1.1, 1.11.2, 13.1.2, 1.3.1, 13.1.0, 12.0.2, 1.11.1, 12.0.1, 12.1.0, 0.9, 1.4.4, 13.0.0, 1.4.9, 12.1.0, 1.7.1, 1.4.2, 14.0.5, 0.8.1, 1.4.6, 0.8.3, 1.11.3, 1.5.1, 1.4.7, 13.0.2, 12.0.7, 13.0.0, 1.9.1, 1.8.2, 14.0.1, 14.0.0, 14.0.4, 1.6.2, 15.0.1, 13.1.0, 0.8, 1.7, 15.0.2, 12.0.5, 13.0.1, 1.8.1, 1.11.6, 15.0.1, 12.0.4, 12.1.1, 13.0.2, 1.11.4, 1.10, 14.0.4, 14.0.6, 1.4.1, 1.4, 1.5.2, 12.0.2, 12.0.1, 14.0.3, 14.0.2, 1.11.1, 1.7.1.2, 15.0.0, 12.0.4, 1.6.4, 1.11.2, 1.5)
+            Could not find a version that satisfies the requirement real-package==versionplease (from versions: 1.11, 13.0.3, 1.6, 1!1.2.rev33+123456, 1.9, 1.3.2, 14.0.1, 12.0.7, 13.0.3, 1.7.2, 1.8.4, 1.2+123abc456, 1.6.1, 0.9.2, 1.3, 1.8.3, 12.1.1, 1.1, 1.11.6, 1.2+123456, 1.4.8, 1.6.3, 1!1.0b2.post345.dev456, 1.10.1, 14.0.2, 1.11.3, 14.0.3, 1.4rc1, 1.0b2.post345.dev456, 0.8.4, 1.0, 1!1.0.post456, 12.0.5, 14.0.6, 1.11.5, 1.0rc2, 1.7.1.1, 1.11.4, 13.0.1, 13.1.2, 1.3.3, 0.8.2, 14.0.0, 12.0, 1.8, 1.3.4, 12.0, 1.2, 12.0.6, 0.9.1, 13.1.1, 2!2.3.4.alpha5.rev6.dev7+abc89, 14.0.5, 15.0.2, 15.0.0, 1.4.5, 1.4.3, 13.1.1, 1.11.2, 13.1.2, 1.2+abc123def, 1.3.1, 13.1.0, 12.0.2, 1.11.1, 12.0.1, 12.1.0, 0.9, 1.4.4, 1.2+abc123, 13.0.0, 1.4.9, 1.1.dev1, 12.1.0, 1.7.1, 1.4.2, 14.0.5, 0.8.1, 1.4.6, 0.8.3, 1.11.3, 1.5.1, 1.4.7, 13.0.2, 12.0.7, 1!13.0, 0!13.0, 1.9.1, 1.0.post456.dev34, 1.8.2, 14.0.1, 14.0.0, 1.2.rev33+123456, 14.0.4, 1.6.2, 15.0.1, 13.1.0, 0.8, 1.2+1234.abc, 1.7, 15.0.2, 12.0.5, 13.0.1, 1.8.1, 1.11.6, 15.0.1, 12.0.4, 1.2+123abc, 12.1.1, 13.0.2, 1.11.4, 1.10, 1.2.r32+123456, 14.0.4, 14.0.6, 1.4.1, 1.4, 1.5.2, 12.0.2, 12.0.1, 14.0.3, 14.0.2, 1.11.1, 1.7.1.2, 15.0.0, 12.0.4, 1.6.4, 1.11.2, 1.5, 0.1, 0.10, 0.10.1, 0.10.1.0.1, 1.0.dev456, 1.0a1, 1.0a2.dev456, 1.0a12.dev456, 1.0a12, 1.0b1.dev456, 1.0b2, 1.0b2.post345, 1.0b2-346, 1.0c1.dev456, 1.0c1, 1.0c3, 1.0, 1.0.post456, 1.2+abc, 1!1.0, 1!1.0.post456.dev34)
           No distributions matching the version for real-package==versionplease
           EOS
         )
         expect(Puppet::Util::Execution).to receive(:execpipe).with(["/fake/bin/pip", "install", "real_package==versionplease"]).and_yield(p).once
         @resource[:name] = "real_package"
-        latest = @provider.latest
-        expect(latest).to eq('15.0.2')
+        expect(@provider.latest).to eq('2!2.3.4.alpha5.rev6.dev7+abc89')
       end
 
       it "should use 'install_options' when specified" do

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

@@ -45,7 +45,7 @@ describe Puppet::Type.type(:package).provider(:pkgdmg) do
     it "should call hdiutil to mount and eject the disk image" do
       allow(Dir).to receive(:entries).and_return([])
       expect(provider.class).to receive(:hdiutil).with("eject", fake_mountpoint).and_return(0)
-      expect(provider.class).to receive(:hdiutil).with("mount", "-plist", "-nobrowse", "-readonly", "-noidme", "-mountrandom", "/tmp", '/tmp/foo').and_return('a plist')
+      expect(provider.class).to receive(:hdiutil).with("mount", "-plist", "-nobrowse", "-readonly", "-mountrandom", "/tmp", '/tmp/foo').and_return('a plist')
       expect(Puppet::Util::Plist).to receive(:parse_plist).with('a plist').and_return(fake_hdiutil_plist)
       provider.install
     end

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

@@ -110,6 +110,30 @@ describe Puppet::Type.type(:package).provider(:pkgng) do
       end
       resource.provider.install
     end
+
+    it "should call pkg with the specified install options string" do
+      resource = Puppet::Type.type(:package).new(
+        :name            => 'curl',
+        :provider        => :pkgng,
+        :install_options => ['--foo', '--bar']
+      )
+      expect(resource.provider).to receive(:pkg) do |arg|
+        expect(arg).to include('--foo', '--bar')
+      end
+      resource.provider.install
+    end
+
+    it "should call pkg with the specified install options hash" do
+      resource = Puppet::Type.type(:package).new(
+        :name            => 'curl',
+        :provider        => :pkgng,
+        :install_options => ['--foo', { '--bar' => 'baz', '--baz' => 'foo' }]
+      )
+      expect(resource.provider).to receive(:pkg) do |arg|
+        expect(arg).to include('--foo', '--bar=baz', '--baz=foo')
+      end
+      resource.provider.install
+    end
   end
 
   context "#prefetch" do
@@ -177,6 +201,20 @@ describe Puppet::Type.type(:package).provider(:pkgng) do
 
       expect(bash_comp_latest_version).to eq('2.1_3')
     end
+
+    it "should return nil when the package is orphaned" do
+      version_list = File.read(my_fixture('pkg.version'))
+      allow(described_class).to receive(:get_version_list).and_return(version_list)
+      orphan_latest_version = described_class.get_latest_version('sysutils/orphan')
+      expect(orphan_latest_version).to be_nil
+    end
+
+    it "should return nil when the package is broken" do
+      version_list = File.read(my_fixture('pkg.version'))
+      allow(described_class).to receive(:get_version_list).and_return(version_list)
+      broken_latest_version = described_class.get_latest_version('sysutils/broken')
+      expect(broken_latest_version).to be_nil
+    end
   end
 
   describe "confine" do

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

@@ -67,6 +67,14 @@ describe Puppet::Type.type(:package).provider(:puppet_gem) do
       expect(described_class).to receive(:execute_gem_command).with(provider_gem_cmd, %w{uninstall --executables --all myresource --force --bindir=/usr/bin}).and_return('')
       provider.uninstall
     end
+
+    it 'should invalidate the rubygems cache' do
+      gem_source = double('gem_source')
+      allow(Puppet::Util::Autoload).to receive(:gem_source).and_return(gem_source)
+      expect(described_class).to receive(:execute_gem_command).with(provider_gem_cmd, %w{uninstall --executables --all myresource}).and_return('')
+      expect(gem_source).to receive(:clear_paths)
+      provider.uninstall
+    end
   end
 
   context 'calculated specificity' do

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

@@ -552,218 +552,6 @@ describe Puppet::Type.type(:package).provider(:rpm) do
     end
   end
 
-  describe 'version comparison' do
-    # test cases munged directly from rpm's own
-    # tests/rpmvercmp.at
-    it { expect(provider.rpmvercmp("1.0", "1.0")).to eq(0) }
-    it { expect(provider.rpmvercmp("1.0", "2.0")).to eq(-1) }
-    it { expect(provider.rpmvercmp("2.0", "1.0")).to eq(1) }
-    it { expect(provider.rpmvercmp("2.0.1", "2.0.1")).to eq(0) }
-    it { expect(provider.rpmvercmp("2.0", "2.0.1")).to eq(-1) }
-    it { expect(provider.rpmvercmp("2.0.1", "2.0")).to eq(1) }
-    it { expect(provider.rpmvercmp("2.0.1a", "2.0.1a")).to eq(0) }
-    it { expect(provider.rpmvercmp("2.0.1a", "2.0.1")).to eq(1) }
-    it { expect(provider.rpmvercmp("2.0.1", "2.0.1a")).to eq(-1) }
-    it { expect(provider.rpmvercmp("5.5p1", "5.5p1")).to eq(0) }
-    it { expect(provider.rpmvercmp("5.5p1", "5.5p2")).to eq(-1) }
-    it { expect(provider.rpmvercmp("5.5p2", "5.5p1")).to eq(1) }
-    it { expect(provider.rpmvercmp("5.5p10", "5.5p10")).to eq(0) }
-    it { expect(provider.rpmvercmp("5.5p1", "5.5p10")).to eq(-1) }
-    it { expect(provider.rpmvercmp("5.5p10", "5.5p1")).to eq(1) }
-    it { expect(provider.rpmvercmp("10xyz", "10.1xyz")).to eq(-1) }
-    it { expect(provider.rpmvercmp("10.1xyz", "10xyz")).to eq(1) }
-    it { expect(provider.rpmvercmp("xyz10", "xyz10")).to eq(0) }
-    it { expect(provider.rpmvercmp("xyz10", "xyz10.1")).to eq(-1) }
-    it { expect(provider.rpmvercmp("xyz10.1", "xyz10")).to eq(1) }
-    it { expect(provider.rpmvercmp("xyz.4", "xyz.4")).to eq(0) }
-    it { expect(provider.rpmvercmp("xyz.4", "8")).to eq(-1) }
-    it { expect(provider.rpmvercmp("8", "xyz.4")).to eq(1) }
-    it { expect(provider.rpmvercmp("xyz.4", "2")).to eq(-1) }
-    it { expect(provider.rpmvercmp("2", "xyz.4")).to eq(1) }
-    it { expect(provider.rpmvercmp("5.5p2", "5.6p1")).to eq(-1) }
-    it { expect(provider.rpmvercmp("5.6p1", "5.5p2")).to eq(1) }
-    it { expect(provider.rpmvercmp("5.6p1", "6.5p1")).to eq(-1) }
-    it { expect(provider.rpmvercmp("6.5p1", "5.6p1")).to eq(1) }
-    it { expect(provider.rpmvercmp("6.0.rc1", "6.0")).to eq(1) }
-    it { expect(provider.rpmvercmp("6.0", "6.0.rc1")).to eq(-1) }
-    it { expect(provider.rpmvercmp("10b2", "10a1")).to eq(1) }
-    it { expect(provider.rpmvercmp("10a2", "10b2")).to eq(-1) }
-    it { expect(provider.rpmvercmp("1.0aa", "1.0aa")).to eq(0) }
-    it { expect(provider.rpmvercmp("1.0a", "1.0aa")).to eq(-1) }
-    it { expect(provider.rpmvercmp("1.0aa", "1.0a")).to eq(1) }
-    it { expect(provider.rpmvercmp("10.0001", "10.0001")).to eq(0) }
-    it { expect(provider.rpmvercmp("10.0001", "10.1")).to eq(0) }
-    it { expect(provider.rpmvercmp("10.1", "10.0001")).to eq(0) }
-    it { expect(provider.rpmvercmp("10.0001", "10.0039")).to eq(-1) }
-    it { expect(provider.rpmvercmp("10.0039", "10.0001")).to eq(1) }
-    it { expect(provider.rpmvercmp("4.999.9", "5.0")).to eq(-1) }
-    it { expect(provider.rpmvercmp("5.0", "4.999.9")).to eq(1) }
-    it { expect(provider.rpmvercmp("20101121", "20101121")).to eq(0) }
-    it { expect(provider.rpmvercmp("20101121", "20101122")).to eq(-1) }
-    it { expect(provider.rpmvercmp("20101122", "20101121")).to eq(1) }
-    it { expect(provider.rpmvercmp("2_0", "2_0")).to eq(0) }
-    it { expect(provider.rpmvercmp("2.0", "2_0")).to eq(0) }
-    it { expect(provider.rpmvercmp("2_0", "2.0")).to eq(0) }
-    it { expect(provider.rpmvercmp("a", "a")).to eq(0) }
-    it { expect(provider.rpmvercmp("a+", "a+")).to eq(0) }
-    it { expect(provider.rpmvercmp("a+", "a_")).to eq(0) }
-    it { expect(provider.rpmvercmp("a_", "a+")).to eq(0) }
-    it { expect(provider.rpmvercmp("+a", "+a")).to eq(0) }
-    it { expect(provider.rpmvercmp("+a", "_a")).to eq(0) }
-    it { expect(provider.rpmvercmp("_a", "+a")).to eq(0) }
-    it { expect(provider.rpmvercmp("+_", "+_")).to eq(0) }
-    it { expect(provider.rpmvercmp("_+", "+_")).to eq(0) }
-    it { expect(provider.rpmvercmp("_+", "_+")).to eq(0) }
-    it { expect(provider.rpmvercmp("+", "_")).to eq(0) }
-    it { expect(provider.rpmvercmp("_", "+")).to eq(0) }
-    it { expect(provider.rpmvercmp("1.0~rc1", "1.0~rc1")).to eq(0) }
-    it { expect(provider.rpmvercmp("1.0~rc1", "1.0")).to eq(-1) }
-    it { expect(provider.rpmvercmp("1.0", "1.0~rc1")).to eq(1) }
-    it { expect(provider.rpmvercmp("1.0~rc1", "1.0~rc2")).to eq(-1) }
-    it { expect(provider.rpmvercmp("1.0~rc2", "1.0~rc1")).to eq(1) }
-    it { expect(provider.rpmvercmp("1.0~rc1~git123", "1.0~rc1~git123")).to eq(0) }
-    it { expect(provider.rpmvercmp("1.0~rc1~git123", "1.0~rc1")).to eq(-1) }
-    it { expect(provider.rpmvercmp("1.0~rc1", "1.0~rc1~git123")).to eq(1) }
-    it { expect(provider.rpmvercmp("1.0~rc1", "1.0arc1")).to eq(-1) }
-    it { expect(provider.rpmvercmp("", "~")).to eq(1) }
-    it { expect(provider.rpmvercmp("~", "~~")).to eq(1) }
-    it { expect(provider.rpmvercmp("~", "~+~")).to eq(1) }
-    it { expect(provider.rpmvercmp("~", "~a")).to eq(-1) }
-
-    # non-upstream test cases
-    it { expect(provider.rpmvercmp("405", "406")).to eq(-1) }
-    it { expect(provider.rpmvercmp("1", "0")).to eq(1) }
-  end
-
-  describe 'package evr parsing' do
-    it 'should parse full simple evr' do
-      v = provider.rpm_parse_evr('0:1.2.3-4.el5')
-      expect(v[:epoch]).to eq('0')
-      expect(v[:version]).to eq('1.2.3')
-      expect(v[:release]).to eq('4.el5')
-    end
-
-    it 'should parse version only' do
-      v = provider.rpm_parse_evr('1.2.3')
-      expect(v[:epoch]).to eq(nil)
-      expect(v[:version]).to eq('1.2.3')
-      expect(v[:release]).to eq(nil)
-    end
-
-    it 'should parse version-release' do
-      v = provider.rpm_parse_evr('1.2.3-4.5.el6')
-      expect(v[:epoch]).to eq(nil)
-      expect(v[:version]).to eq('1.2.3')
-      expect(v[:release]).to eq('4.5.el6')
-    end
-
-    it 'should parse release with git hash' do
-      v = provider.rpm_parse_evr('1.2.3-4.1234aefd')
-      expect(v[:epoch]).to eq(nil)
-      expect(v[:version]).to eq('1.2.3')
-      expect(v[:release]).to eq('4.1234aefd')
-    end
-
-    it 'should parse single integer versions' do
-      v = provider.rpm_parse_evr('12345')
-      expect(v[:epoch]).to eq(nil)
-      expect(v[:version]).to eq('12345')
-      expect(v[:release]).to eq(nil)
-    end
-
-    it 'should parse text in the epoch to 0' do
-      v = provider.rpm_parse_evr('foo0:1.2.3-4')
-      expect(v[:epoch]).to eq(nil)
-      expect(v[:version]).to eq('1.2.3')
-      expect(v[:release]).to eq('4')
-    end
-
-    it 'should parse revisions with text' do
-      v = provider.rpm_parse_evr('1.2.3-SNAPSHOT20140107')
-      expect(v[:epoch]).to eq(nil)
-      expect(v[:version]).to eq('1.2.3')
-      expect(v[:release]).to eq('SNAPSHOT20140107')
-    end
-
-    # test cases for PUP-682
-    it 'should parse revisions with text and numbers' do
-      v = provider.rpm_parse_evr('2.2-SNAPSHOT20121119105647')
-      expect(v[:epoch]).to eq(nil)
-      expect(v[:version]).to eq('2.2')
-      expect(v[:release]).to eq('SNAPSHOT20121119105647')
-    end
-  end
-
-  describe 'rpm evr comparison' do
-    # currently passing tests
-    it 'should evaluate identical version-release as equal' do
-      v = provider.rpm_compareEVR({:epoch => '0', :version => '1.2.3', :release => '1.el5'},
-                                  {:epoch => '0', :version => '1.2.3', :release => '1.el5'})
-      expect(v).to eq(0)
-    end
-
-    it 'should evaluate identical version as equal' do
-      v = provider.rpm_compareEVR({:epoch => '0', :version => '1.2.3', :release => nil},
-                                  {:epoch => '0', :version => '1.2.3', :release => nil})
-      expect(v).to eq(0)
-    end
-
-    it 'should evaluate identical version but older release as less' do
-      v = provider.rpm_compareEVR({:epoch => '0', :version => '1.2.3', :release => '1.el5'},
-                                  {:epoch => '0', :version => '1.2.3', :release => '2.el5'})
-      expect(v).to eq(-1)
-    end
-
-    it 'should evaluate identical version but newer release as greater' do
-      v = provider.rpm_compareEVR({:epoch => '0', :version => '1.2.3', :release => '3.el5'},
-                                  {:epoch => '0', :version => '1.2.3', :release => '2.el5'})
-      expect(v).to eq(1)
-    end
-
-    it 'should evaluate a newer epoch as greater' do
-      v = provider.rpm_compareEVR({:epoch => '1', :version => '1.2.3', :release => '4.5'},
-                                  {:epoch => '0', :version => '1.2.3', :release => '4.5'})
-      expect(v).to eq(1)
-    end
-
-    # these tests describe PUP-1244 logic yet to be implemented
-    it 'should evaluate any version as equal to the same version followed by release' do
-      v = provider.rpm_compareEVR({:epoch => '0', :version => '1.2.3', :release => nil},
-                                  {:epoch => '0', :version => '1.2.3', :release => '2.el5'})
-      expect(v).to eq(0)
-    end
-
-    # test cases for PUP-682
-    it 'should evaluate same-length numeric revisions numerically' do
-      expect(provider.rpm_compareEVR({:epoch => '0', :version => '2.2', :release => '405'},
-                               {:epoch => '0', :version => '2.2', :release => '406'})).to eq(-1)
-    end
-  end
-
-  describe 'version segment comparison' do
-    it 'should treat two nil values as equal' do
-      v = provider.compare_values(nil, nil)
-      expect(v).to eq(0)
-    end
-
-    it 'should treat a nil value as less than a non-nil value' do
-      v = provider.compare_values(nil, '0')
-      expect(v).to eq(-1)
-    end
-
-    it 'should treat a non-nil value as greater than a nil value' do
-      v = provider.compare_values('0', nil)
-      expect(v).to eq(1)
-    end
-
-    it 'should pass two non-nil values on to rpmvercmp' do
-      allow(provider).to receive(:rpmvercmp).and_return(0)
-      expect(provider).to receive(:rpmvercmp).with('s1', 's2')
-      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' }