Yum package provider rewrite

squash and rebase of all the work

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>

2 files changed, 8 insertions, 2300 deletions

diff --git a/spec/unit/provider/package/yum/yum_cache_spec.rb b/spec/unit/provider/package/yum/python_helper_spec.rb
index e9d615d734..ba91e2f24a 100644
--- a/spec/unit/provider/package/yum/yum_cache_spec.rb
+++ b/spec/unit/provider/package/yum/python_helper_spec.rb
@@ -1,6 +1,5 @@
 #
-# Author:: Adam Jacob (<adam@chef.io>)
-# Copyright:: Copyright 2008-2016, Chef Software, Inc.
+# Copyright:: Copyright 2017-2017, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -18,10 +17,13 @@
 
 require "spec_helper"
 
-describe Chef::Provider::Package::Yum::YumCache do
+# NOTE: most of the tests of this functionality are baked into the func tests for the yum package provider
 
-  it "can find yum-dump.py" do
-    expect(File.exist?(Chef::Provider::Package::Yum::YumCache.instance.yum_dump_path)).to be true
-  end
+describe Chef::Provider::Package::Yum::PythonHelper do
+  let(:helper) { Chef::Provider::Package::Yum::PythonHelper.instance }
 
+  it "propagates stacktraces on stderr from the forked subprocess", :rhel do
+    allow(helper).to receive(:yum_command).and_return("ruby -e 'raise \"your hands in the air\"'")
+    expect { helper.package_query(:whatprovides, "tcpdump") }.to raise_error(/your hands in the air/)
+  end
 end
diff --git a/spec/unit/provider/package/yum_spec.rb b/spec/unit/provider/package/yum_spec.rb
deleted file mode 100644
index c8fb04cc21..0000000000
--- a/spec/unit/provider/package/yum_spec.rb
+++ /dev/null
@@ -1,2294 +0,0 @@
-#
-# Author:: Adam Jacob (<adam@chef.io>)
-# Copyright:: Copyright 2008-2017, Chef Software Inc.
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require "spec_helper"
-require "securerandom"
-
-describe Chef::Provider::Package::Yum do
-  before(:each) do
-    @node = Chef::Node.new
-    @events = Chef::EventDispatch::Dispatcher.new
-    @run_context = Chef::RunContext.new(@node, {}, @events)
-    @new_resource = Chef::Resource::YumPackage.new("cups")
-    @status = double("Status", :exitstatus => 0)
-    @yum_cache = double(
-      "Chef::Provider::Yum::YumCache",
-      :reload_installed => true,
-      :reset => true,
-      :installed_version => "1.2.4-11.18.el5",
-      :candidate_version => "1.2.4-11.18.el5_2.3",
-      :package_available? => true,
-      :version_available? => true,
-      :allow_multi_install => [ "kernel" ],
-      :package_repository => "base",
-      :disable_extra_repo_control => true
-    )
-    allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-    allow(@yum_cache).to receive(:yum_binary=).with("yum")
-    allow(::File).to receive(:exist?).with("/usr/bin/yum-deprecated").and_return(false)
-    @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-    @pid = double("PID")
-  end
-
-  describe "when loading the current system state" do
-    it "should create a current resource with the name of the new_resource" do
-      @provider.load_current_resource
-      expect(@provider.current_resource.name).to eq("cups")
-    end
-
-    it "should set the current resources package name to the new resources package name" do
-      @provider.load_current_resource
-      expect(@provider.current_resource.package_name).to eq("cups")
-    end
-
-    it "should set the installed version to nil on the current resource if no installed package" do
-      allow(@yum_cache).to receive(:installed_version).and_return(nil)
-      @provider.load_current_resource
-      expect(@provider.current_resource.version).to be_nil
-    end
-
-    it "should set the installed version if yum has one" do
-      @provider.load_current_resource
-      expect(@provider.current_resource.version).to eq("1.2.4-11.18.el5")
-    end
-
-    it "should set the candidate version if yum info has one" do
-      @provider.load_current_resource
-      expect(@provider.candidate_version).to eql("1.2.4-11.18.el5_2.3")
-    end
-
-    it "should return the current resouce" do
-      expect(@provider.load_current_resource).to eql(@provider.current_resource)
-    end
-
-    describe "when source is provided" do
-      it "should set the candidate version" do
-        @new_resource = Chef::Resource::YumPackage.new("testing.source")
-        @new_resource.source "chef-server-core-12.0.5-1.rpm"
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        allow(File).to receive(:exist?).with(@new_resource.source).and_return(true)
-        allow(@yum_cache).to receive(:installed_version).and_return(nil)
-        shellout_double = double(:stdout => "chef-server-core 12.0.5-1 i386")
-        allow(@provider).to receive(:shell_out!).and_return(shellout_double)
-        @provider.load_current_resource
-        expect(@provider.candidate_version).to eql("12.0.5-1")
-        expect(@provider.new_resource.arch).to eql("i386")
-      end
-    end
-
-    describe "yum_binary accessor" do
-      it "when yum-deprecated exists" do
-        expect(File).to receive(:exist?).with("/usr/bin/yum-deprecated").and_return(true)
-        expect(@yum_cache).to receive(:yum_binary=).with("yum-deprecated")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        expect(@provider.send(:yum_binary)).to eql("yum-deprecated")
-      end
-
-      it "when yum-deprecated does not exist" do
-        expect(File).to receive(:exist?).with("/usr/bin/yum-deprecated").and_return(false)
-        expect(@yum_cache).to receive(:yum_binary=).with("yum")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        expect(@provider.send(:yum_binary)).to eql("yum")
-      end
-
-      it "when the yum_binary is set on the resource" do
-        @new_resource.yum_binary "/usr/bin/yum-something"
-        expect(File).not_to receive(:exist?)
-        expect(@yum_cache).to receive(:yum_binary=).with("/usr/bin/yum-something")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        expect(@provider.send(:yum_binary)).to eql("/usr/bin/yum-something")
-      end
-
-      it "when the new_resource is a vanilla package class and yum-deprecated exists" do
-        @new_resource = Chef::Resource::YumPackage.new("cups")
-        expect(File).to receive(:exist?).with("/usr/bin/yum-deprecated").and_return(true)
-        expect(@yum_cache).to receive(:yum_binary=).with("yum-deprecated")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        expect(@provider.send(:yum_binary)).to eql("yum-deprecated")
-      end
-
-      it "when the new_resource is a vanilla package class and yum-deprecated does not exist" do
-        @new_resource = Chef::Resource::YumPackage.new("cups")
-        expect(File).to receive(:exist?).with("/usr/bin/yum-deprecated").and_return(false)
-        expect(@yum_cache).to receive(:yum_binary=).with("yum")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        expect(@provider.send(:yum_binary)).to eql("yum")
-      end
-    end
-
-    describe "when arch in package_name" do
-      it "should set the arch if no existing package_name is found and new_package_name+new_arch is available" do
-        @new_resource = Chef::Resource::YumPackage.new("testing.noarch")
-        @yum_cache = double(
-          "Chef::Provider::Yum::YumCache"
-        )
-        allow(@yum_cache).to receive(:installed_version) do |package_name, arch|
-          # nothing installed for package_name/new_package_name
-          nil
-        end
-        allow(@yum_cache).to receive(:candidate_version) do |package_name, arch|
-          if package_name == "testing.noarch" || package_name == "testing.more.noarch"
-            nil
-          # candidate for new_package_name
-          elsif package_name == "testing" || package_name == "testing.more"
-            "1.1"
-          end
-        end
-        allow(@yum_cache).to receive(:package_available?).and_return(true)
-        allow(@yum_cache).to receive(:disable_extra_repo_control).and_return(true)
-        allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-        allow(@yum_cache).to receive(:yum_binary=).with("yum")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@provider.new_resource.package_name).to eq("testing")
-        expect(@provider.new_resource.arch).to eq("noarch")
-
-        @new_resource = Chef::Resource::YumPackage.new("testing.more.noarch")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@provider.new_resource.package_name).to eq("testing.more")
-        expect(@provider.new_resource.arch).to eq("noarch")
-      end
-
-      describe "when version constraint in package_name" do
-        it "should set package_version if no existing package_name is found and new_package_name is available" do
-          @new_resource = Chef::Resource::YumPackage.new("cups = 1.2.4-11.18.el5_2.3")
-          @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-          allow(@yum_cache).to receive(:package_available?) { |pkg| pkg == "cups" ? true : false }
-          allow(@yum_cache).to receive(:packages_from_require) do |pkg|
-            [Chef::Provider::Package::Yum::RPMDbPackage.new("cups", "1.2.4-11.18.el5_2.3", "noarch", [], false, true, "base"),
-            Chef::Provider::Package::Yum::RPMDbPackage.new("cups", "1.2.4-11.18.el5_2.2", "noarch", [], false, true, "base")]
-          end
-          expect(Chef::Log).to receive(:debug).exactly(1).times.with(%r{checking yum info})
-          expect(Chef::Log).to receive(:debug).exactly(1).times.with(%r{installed version})
-          expect(Chef::Log).to receive(:debug).exactly(1).times.with(%r{matched 2 packages,})
-          @provider.load_current_resource
-          expect(@provider.new_resource.package_name).to eq("cups")
-          expect(@provider.new_resource.version).to eq("1.2.4-11.18.el5_2.3")
-          expect(@provider.send(:new_version_array)).to eq(["1.2.4-11.18.el5_2.3"])
-          expect(@provider.send(:package_name_array)).to eq(["cups"])
-        end
-      end
-
-      it "should not set the arch when an existing package_name is found" do
-        @new_resource = Chef::Resource::YumPackage.new("testing.beta3")
-        @yum_cache = double(
-          "Chef::Provider::Yum::YumCache"
-        )
-        allow(@yum_cache).to receive(:installed_version) do |package_name, arch|
-          # installed for package_name
-          if package_name == "testing.beta3" || package_name == "testing.beta3.more"
-            "1.1"
-          elsif package_name == "testing" || package_name == "testing.beta3"
-            nil
-          end
-        end
-        allow(@yum_cache).to receive(:candidate_version) do |package_name, arch|
-          # no candidate for package_name/new_package_name
-          nil
-        end
-        allow(@yum_cache).to receive(:package_available?).and_return(true)
-        allow(@yum_cache).to receive(:disable_extra_repo_control).and_return(true)
-        allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-        allow(@yum_cache).to receive(:yum_binary=).with("yum")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        # annoying side effect of the fun stub'ing above
-        @provider.load_current_resource
-        expect(@provider.new_resource.package_name).to eq("testing.beta3")
-        expect(@provider.new_resource.arch).to eq(nil)
-
-        @new_resource = Chef::Resource::YumPackage.new("testing.beta3.more")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@provider.new_resource.package_name).to eq("testing.beta3.more")
-        expect(@provider.new_resource.arch).to eq(nil)
-      end
-
-      it "should not set the arch when no existing package_name or new_package_name+new_arch is found" do
-        @new_resource = Chef::Resource::YumPackage.new("testing.beta3")
-        @yum_cache = double(
-          "Chef::Provider::Yum::YumCache"
-        )
-        allow(@yum_cache).to receive(:installed_version) do |package_name, arch|
-          # nothing installed for package_name/new_package_name
-          nil
-        end
-        allow(@yum_cache).to receive(:candidate_version) do |package_name, arch|
-          # no candidate for package_name/new_package_name
-          nil
-        end
-        allow(@yum_cache).to receive(:package_available?).and_return(true)
-        allow(@yum_cache).to receive(:disable_extra_repo_control).and_return(true)
-        allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-        allow(@yum_cache).to receive(:yum_binary=).with("yum")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@provider.new_resource.package_name).to eq("testing.beta3")
-        expect(@provider.new_resource.arch).to eq(nil)
-
-        @new_resource = Chef::Resource::YumPackage.new("testing.beta3.more")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@provider.new_resource.package_name).to eq("testing.beta3.more")
-        expect(@provider.new_resource.arch).to eq(nil)
-      end
-
-      it "should ensure it doesn't clobber an existing arch if passed" do
-        @new_resource = Chef::Resource::YumPackage.new("testing.i386")
-        @new_resource.arch("x86_64")
-        @yum_cache = double(
-          "Chef::Provider::Yum::YumCache"
-        )
-        allow(@yum_cache).to receive(:installed_version) do |package_name, arch|
-          # nothing installed for package_name/new_package_name
-          nil
-        end
-        allow(@yum_cache).to receive(:candidate_version) do |package_name, arch|
-          if package_name == "testing.noarch"
-            nil
-          # candidate for new_package_name
-          elsif package_name == "testing"
-            "1.1"
-          end
-        end.and_return("something")
-        allow(@yum_cache).to receive(:package_available?).and_return(true)
-        allow(@yum_cache).to receive(:disable_extra_repo_control).and_return(true)
-        allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-        allow(@yum_cache).to receive(:yum_binary=).with("yum")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@provider.new_resource.package_name).to eq("testing.i386")
-        expect(@provider.new_resource.arch).to eq("x86_64")
-      end
-    end
-
-    it "should flush the cache if :before is true" do
-      @new_resource.flush_cache({ :after => false, :before => true })
-      expect(@yum_cache).to receive(:reload).once
-      @provider.load_current_resource
-    end
-
-    it "should flush the cache if :before is false" do
-      @new_resource.flush_cache({ :after => false, :before => false })
-      expect(@yum_cache).not_to receive(:reload)
-      @provider.load_current_resource
-    end
-
-    it "should detect --enablerepo or --disablerepo when passed among options, collect them preserving order and notify the yum cache" do
-      @new_resource.options("--stuff --enablerepo=foo --otherthings --disablerepo=a,b,c  --enablerepo=bar")
-      expect(@yum_cache).to receive(:enable_extra_repo_control).with("--enablerepo=foo --disablerepo=a,b,c --enablerepo=bar")
-      @provider.load_current_resource
-    end
-
-    it "should let the yum cache know extra repos are disabled if --enablerepo or --disablerepo aren't among options" do
-      @new_resource.options("--stuff --otherthings")
-      expect(@yum_cache).to receive(:disable_extra_repo_control)
-      @provider.load_current_resource
-    end
-
-    it "should let the yum cache know extra repos are disabled if options aren't set" do
-      @new_resource.options(nil)
-      expect(@yum_cache).to receive(:disable_extra_repo_control)
-      @provider.load_current_resource
-    end
-
-    context "when the package name isn't found" do
-      let(:yum_cache) do
-        double(
-          "Chef::Provider::Yum::YumCache",
-          :reload_installed => true,
-          :reset => true,
-          :installed_version => "1.0.1.el5",
-          :candidate_version => "2.0.1.el5",
-          :package_available? => false,
-          :version_available? => true,
-          :disable_extra_repo_control => true
-      )
-      end
-
-      before do
-        allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(yum_cache)
-        allow(yum_cache).to receive(:yum_binary=).with("yum")
-        @pkg = Chef::Provider::Package::Yum::RPMPackage.new("test-package", "2.0.1.el5", "x86_64", [])
-        expect(yum_cache).to receive(:packages_from_require).and_return([@pkg])
-      end
-
-      it "should search provides then set package_name to match" do
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@new_resource.package_name).to eq("test-package")
-        expect(@new_resource.version).to eq(nil)
-      end
-
-      it "should search provides then set version to match if a requirement was passed in the package name" do
-        @new_resource = Chef::Resource::YumPackage.new("test-package = 2.0.1.el5")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@new_resource.package_name).to eq("test-package")
-        expect(@new_resource.version).to eq("2.0.1.el5")
-      end
-
-      it "should search provides then set version to match if a requirement was passed in the version" do
-        @new_resource = Chef::Resource::YumPackage.new("test-package")
-        @new_resource.version("= 2.0.1.el5")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@new_resource.package_name).to eq("test-package")
-        expect(@new_resource.version).to eq("2.0.1.el5")
-      end
-
-      it "should search provides and not set the version to match if a specific version was requested" do
-        @new_resource = Chef::Resource::YumPackage.new("test-package")
-        @new_resource.version("3.0.1.el5")
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@new_resource.package_name).to eq("test-package")
-        expect(@new_resource.version).to eq("3.0.1.el5")
-      end
-
-      it "should search provides then set versions to match if requirements were passed in the package name as an array" do
-        @new_resource = Chef::Resource::YumPackage.new(["test-package = 2.0.1.el5"])
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@new_resource.package_name).to eq(["test-package"])
-        expect(@new_resource.version).to eq(["2.0.1.el5"])
-      end
-
-      it "should search provides and not set the versions to match if specific versions were requested in an array" do
-        @new_resource = Chef::Resource::YumPackage.new(["test-package"])
-        @new_resource.version(["3.0.1.el5"])
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        @provider.load_current_resource
-        expect(@new_resource.package_name).to eq(["test-package"])
-        expect(@new_resource.version).to eq(["3.0.1.el5"])
-      end
-
-    end
-
-    it "should not return an error if no version number is specified in the resource" do
-      @new_resource = Chef::Resource::YumPackage.new("test-package")
-      @yum_cache = double(
-          "Chef::Provider::Yum::YumCache",
-          :reload_installed => true,
-          :reset => true,
-          :installed_version => "1.0.1.el5",
-          :candidate_version => "2.0.1.el5",
-          :package_available? => false,
-          :version_available? => true,
-          :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      pkg = Chef::Provider::Package::Yum::RPMPackage.new("test-package", "2.0.1.el5", "x86_64", [])
-      expect(@yum_cache).to receive(:packages_from_require).and_return([pkg])
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect(@new_resource.package_name).to eq("test-package")
-      expect(@new_resource.version).to eq(nil)
-    end
-
-    it "should give precedence to the version attribute when both a requirement in the resource name and a version attribute are specified" do
-      @new_resource = Chef::Resource::YumPackage.new("test-package")
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_installed => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.18.el5",
-        :package_available? => false,
-        :version_available? => true,
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      pkg = Chef::Provider::Package::Yum::RPMPackage.new("test-package", "2.0.1.el5", "x86_64", [])
-      expect(@yum_cache).to receive(:packages_from_require).and_return([pkg])
-      @new_resource = Chef::Resource::YumPackage.new("test-package = 2.0.1.el5")
-      @new_resource.version("3.0.1.el5")
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect(@new_resource.package_name).to eq("test-package")
-      expect(@new_resource.version).to eq("3.0.1.el5")
-    end
-
-    it "should correctly detect the installed states of an array of package names and version numbers" do
-      @yum_cache = double(
-          "Chef::Provider::Yum::YumCache",
-          :reload_installed => true,
-          :reset => true,
-          :installed_version => "1.0.1.el5",
-          :candidate_version => "2.0.1.el5",
-          :package_available? => false,
-          :version_available? => true,
-          :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-
-      expect(@yum_cache).to receive(:packages_from_require).exactly(4).times.and_return([])
-      expect(@yum_cache).to receive(:reload_provides).twice
-
-      @new_resource = Chef::Resource::YumPackage.new(["test-package", "test-package2"])
-      @new_resource.version(["2.0.1.el5", "3.0.1.el5"])
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect(@new_resource.package_name).to eq(["test-package", "test-package2"])
-      expect(@new_resource.version).to eq(["2.0.1.el5", "3.0.1.el5"])
-    end
-
-    it "should search provides if no package is available - if no match in installed provides then load the complete set" do
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_installed => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.18.el5",
-        :package_available? => false,
-        :version_available? => true,
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      expect(@yum_cache).to receive(:packages_from_require).twice.and_return([])
-      expect(@yum_cache).to receive(:reload_provides)
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect(@new_resource.version).to eq(nil)
-    end
-
-    it "should search provides if no package is available and not load the complete set if action is :remove or :purge" do
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_installed => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.18.el5",
-        :package_available? => false,
-        :version_available? => true,
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      expect(@yum_cache).to receive(:packages_from_require).once.and_return([])
-      expect(@yum_cache).not_to receive(:reload_provides)
-      @new_resource.action(:remove)
-      @provider.load_current_resource
-      expect(@yum_cache).to receive(:packages_from_require).once.and_return([])
-      expect(@yum_cache).not_to receive(:reload_provides)
-      @new_resource.action(:purge)
-      @provider.load_current_resource
-    end
-
-    it "should search provides if no package is available - if no match in provides leave the name intact" do
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_provides => true,
-        :reload_installed => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.18.el5",
-        :package_available? => false,
-        :version_available? => true,
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      expect(@yum_cache).to receive(:packages_from_require).twice.and_return([])
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect(@new_resource.package_name).to eq("cups")
-    end
-  end
-
-  describe "when installing a package" do
-    it "should run yum install with the package name and version" do
-      @provider.load_current_resource
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.19.el5"
-      )
-      @provider.install_package("cups", "1.2.4-11.19.el5")
-    end
-
-    it "should run yum localinstall if given a path to an rpm" do
-      @new_resource.source("/tmp/emacs-21.4-20.el5.i386.rpm")
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y localinstall /tmp/emacs-21.4-20.el5.i386.rpm"
-      )
-      @provider.install_package("emacs", "21.4-20.el5")
-    end
-
-    it "should run yum localinstall if given a path to an rpm as the package" do
-      @new_resource = Chef::Resource::YumPackage.new("/tmp/emacs-21.4-20.el5.i386.rpm")
-      allow(::File).to receive(:exist?).with("/tmp/emacs-21.4-20.el5.i386.rpm").and_return(true)
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      expect(@new_resource.source).to eq("/tmp/emacs-21.4-20.el5.i386.rpm")
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y localinstall /tmp/emacs-21.4-20.el5.i386.rpm"
-      )
-      @provider.install_package("/tmp/emacs-21.4-20.el5.i386.rpm", "21.4-20.el5")
-    end
-
-    it "should run yum install with the package name, version and arch" do
-      @new_resource.arch("i386")
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      @provider.load_current_resource
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.19.el5.i386"
-      )
-      @provider.install_package("cups", "1.2.4-11.19.el5")
-    end
-
-    it "installs the package with the options given in the resource" do
-      @provider.load_current_resource
-      allow(@provider).to receive(:candidate_version).and_return("11")
-      @new_resource.options("--disablerepo epmd")
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y --disablerepo epmd install cups-11"
-      )
-      @provider.install_package(@new_resource.name, @provider.candidate_version)
-    end
-
-    it "should raise an exception if the package is not available" do
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_from_cache => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.18.el5_2.3",
-        :package_available? => true,
-        :version_available? => nil,
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      expect { @provider.install_package("lolcats", "0.99") }.to raise_error(Chef::Exceptions::Package, %r{Version .* not found})
-    end
-
-    it "should raise an exception if candidate version is older than the installed version and allow_downgrade is false" do
-      @new_resource.allow_downgrade(false)
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_installed => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.15.el5",
-        :package_available? => true,
-        :version_available? => true,
-        :allow_multi_install => [ "kernel" ],
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect { @provider.install_package("cups", "1.2.4-11.15.el5") }.to raise_error(Chef::Exceptions::Package, %r{is newer than candidate package})
-    end
-
-    it "should not raise an exception if candidate version is older than the installed version and the package is list in yum's installonlypkg option" do
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_installed => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.15.el5",
-        :package_available? => true,
-        :version_available? => true,
-        :allow_multi_install => [ "cups" ],
-        :package_repository => "base",
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.15.el5"
-      )
-      @provider.install_package("cups", "1.2.4-11.15.el5")
-    end
-
-    it "should run yum downgrade if candidate version is older than the installed version and allow_downgrade is true" do
-      @new_resource.allow_downgrade(true)
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_installed => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.15.el5",
-        :package_available? => true,
-        :version_available? => true,
-        :allow_multi_install => [],
-        :package_repository => "base",
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y downgrade cups-1.2.4-11.15.el5"
-      )
-      @provider.install_package("cups", "1.2.4-11.15.el5")
-    end
-
-    it "should run yum install then flush the cache if :after is true" do
-      @new_resource.flush_cache({ :after => true, :before => false })
-      @provider.load_current_resource
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.15.el5"
-      )
-      expect(@yum_cache).to receive(:reload).once
-      @provider.install_package("cups", "1.2.4-11.15.el5")
-    end
-
-    it "should run yum install then not flush the cache if :after is false" do
-      @new_resource.flush_cache({ :after => false, :before => false })
-      @provider.load_current_resource
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.15.el5"
-      )
-      expect(@yum_cache).not_to receive(:reload)
-      @provider.install_package("cups", "1.2.4-11.15.el5")
-    end
-  end
-
-  describe "when upgrading a package" do
-    it "should run yum install if the package is installed and a version is given" do
-      @provider.load_current_resource
-      allow(@provider).to receive(:candidate_version).and_return("11")
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-11"
-      )
-      @provider.upgrade_package(@new_resource.name, @provider.candidate_version)
-    end
-
-    it "should run yum install if the package is not installed" do
-      @provider.load_current_resource
-      @current_resource = Chef::Resource::YumPackage.new("cups")
-      allow(@provider).to receive(:candidate_version).and_return("11")
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-11"
-      )
-      @provider.upgrade_package(@new_resource.name, @provider.candidate_version)
-    end
-
-    it "should raise an exception if candidate version is older than the installed version" do
-      @yum_cache = double(
-        "Chef::Provider::Yum::YumCache",
-        :reload_installed => true,
-        :reset => true,
-        :installed_version => "1.2.4-11.18.el5",
-        :candidate_version => "1.2.4-11.15.el5",
-        :package_available? => true,
-        :version_available? => true,
-        :allow_multi_install => [ "kernel" ],
-        :disable_extra_repo_control => true
-      )
-      allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-      allow(@yum_cache).to receive(:yum_binary=).with("yum")
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @provider.load_current_resource
-      expect { @provider.upgrade_package("cups", "1.2.4-11.15.el5") }.to raise_error(Chef::Exceptions::Package, %r{is newer than candidate package})
-    end
-
-    # Test our little workaround, some crossover into Chef::Provider::Package territory
-    it "should call action_upgrade in the parent if the current resource version is nil" do
-      allow(@yum_cache).to receive(:installed_version).and_return(nil)
-      @current_resource = Chef::Resource::YumPackage.new("cups")
-      allow(@provider).to receive(:candidate_version).and_return("11")
-      expect(@provider).to receive(:upgrade_package).with(["cups"], ["11"])
-      @provider.run_action(:upgrade)
-    end
-
-    it "should call action_upgrade in the parent if the candidate version is nil" do
-      @provider.load_current_resource
-      @current_resource = Chef::Resource::YumPackage.new("cups")
-      allow(@provider).to receive(:candidate_version).and_return(nil)
-      expect(@provider).not_to receive(:upgrade_package)
-      @provider.run_action(:upgrade)
-    end
-
-    it "should call action_upgrade in the parent if the candidate is newer" do
-      @provider.load_current_resource
-      @current_resource = Chef::Resource::YumPackage.new("cups")
-      allow(@provider).to receive(:candidate_version).and_return("11")
-      expect(@provider).to receive(:upgrade_package).with(["cups"], ["11"])
-      @provider.run_action(:upgrade)
-    end
-
-    it "should not call action_upgrade in the parent if the candidate is older" do
-      allow(@yum_cache).to receive(:installed_version).and_return("12")
-      @provider.load_current_resource
-      @current_resource = Chef::Resource::YumPackage.new("cups")
-      allow(@provider).to receive(:candidate_version).and_return("11")
-      expect(@provider).not_to receive(:upgrade_package)
-      @provider.run_action(:upgrade)
-    end
-  end
-
-  describe "when removing a package" do
-    it "should run yum remove with the package name" do
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y remove emacs-1.0"
-      )
-      @provider.remove_package("emacs", "1.0")
-    end
-
-    it "should run yum remove with the package name and arch" do
-      @new_resource.arch("x86_64")
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y remove emacs-1.0.x86_64"
-      )
-      @provider.remove_package("emacs", "1.0")
-    end
-  end
-
-  describe "when purging a package" do
-    it "should run yum remove with the package name" do
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y remove emacs-1.0"
-      )
-      @provider.purge_package("emacs", "1.0")
-    end
-  end
-
-  describe "when locking a package" do
-    it "should run yum versionlock add with the package name" do
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y versionlock add emacs"
-      )
-      @provider.lock_package("emacs", nil)
-    end
-  end
-
-  describe "when unlocking a package" do
-    it "should run yum versionlock delete with the package name" do
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y versionlock delete emacs"
-      )
-      @provider.unlock_package("emacs", nil)
-    end
-  end
-
-  describe "when running yum" do
-    it "should run yum once if it exits with a return code of 0" do
-      @status = double("Status", :exitstatus => 0, :stdout => "", :stderr => "")
-      allow(@provider).to receive(:shell_out).and_return(@status)
-      expect(@provider).to receive(:shell_out).once.with(
-        "yum -d0 -e0 -y install emacs-1.0",
-        { :timeout => Chef::Config[:yum_timeout] }
-      )
-      @provider.send(:yum_command, "-d0 -e0 -y install emacs-1.0")
-    end
-
-    it "should run yum once if it exits with a return code > 0 and no scriptlet failures" do
-      @status = double("Status", :exitstatus => 2, :stdout => "failure failure", :stderr => "problem problem")
-      allow(@provider).to receive(:shell_out).and_return(@status)
-      expect(@provider).to receive(:shell_out).once.with(
-        "yum -d0 -e0 -y install emacs-1.0",
-        { :timeout => Chef::Config[:yum_timeout] }
-      )
-      expect { @provider.send(:yum_command, "-d0 -e0 -y install emacs-1.0") }.to raise_error(Chef::Exceptions::Exec)
-    end
-
-    it "should run yum once if it exits with a return code of 1 and %pre scriptlet failures" do
-      @status = double("Status", :exitstatus => 1, :stdout => "error: %pre(demo-1-1.el5.centos.x86_64) scriptlet failed, exit status 2",
-                                 :stderr => "")
-      allow(@provider).to receive(:shell_out).and_return(@status)
-      expect(@provider).to receive(:shell_out).once.with(
-        "yum -d0 -e0 -y install emacs-1.0",
-        { :timeout => Chef::Config[:yum_timeout] }
-      )
-      # will still raise an exception, can't stub out the subsequent call
-      expect { @provider.send(:yum_command, "-d0 -e0 -y install emacs-1.0") }.to raise_error(Chef::Exceptions::Exec)
-    end
-
-    it "should run yum twice if it exits with a return code of 1 and %post scriptlet failures" do
-      @status = double("Status", :exitstatus => 1, :stdout => "error: %post(demo-1-1.el5.centos.x86_64) scriptlet failed, exit status 2",
-                                 :stderr => "")
-      allow(@provider).to receive(:shell_out).and_return(@status)
-      expect(@provider).to receive(:shell_out).twice.with(
-        "yum -d0 -e0 -y install emacs-1.0",
-        { :timeout => Chef::Config[:yum_timeout] }
-      )
-      # will still raise an exception, can't stub out the subsequent call
-      expect { @provider.send(:yum_command, "-d0 -e0 -y install emacs-1.0") }.to raise_error(Chef::Exceptions::Exec)
-    end
-
-    it "should pass the yum_binary to the command if its specified" do
-      @new_resource.yum_binary "yum-deprecated"
-      expect(@yum_cache).to receive(:yum_binary=).with("yum-deprecated")
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      @status = double("Status", :exitstatus => 0, :stdout => "", :stderr => "")
-      allow(@provider).to receive(:shell_out).and_return(@status)
-      expect(@provider).to receive(:shell_out).once.with(
-        "yum-deprecated -d0 -e0 -y install emacs-1.0",
-        { :timeout => Chef::Config[:yum_timeout] }
-      )
-      @provider.send(:yum_command, "-d0 -e0 -y install emacs-1.0")
-    end
-  end
-end
-
-describe Chef::Provider::Package::Yum::RPMUtils do
-  describe "version_parse" do
-    before do
-      @rpmutils = Chef::Provider::Package::Yum::RPMUtils
-    end
-
-    it "parses known good epoch strings" do
-      [
-        [ "0:3.3", [ 0, "3.3", nil ] ],
-        [ "9:1.7.3", [ 9, "1.7.3", nil ] ],
-        [ "15:20020927", [ 15, "20020927", nil ] ],
-      ].each do |x, y|
-        expect(@rpmutils.version_parse(x)).to eq(y)
-      end
-    end
-
-    it "parses strange epoch strings" do
-      [
-        [ ":3.3", [ 0, "3.3", nil ] ],
-        [ "-1:1.7.3", [ nil, nil, "1:1.7.3" ] ],
-        [ "-:20020927", [ nil, nil, ":20020927" ] ],
-      ].each do |x, y|
-        expect(@rpmutils.version_parse(x)).to eq(y)
-      end
-    end
-
-    it "parses known good version strings" do
-      [
-        [ "3.3", [ nil, "3.3", nil ] ],
-        [ "1.7.3", [ nil, "1.7.3", nil ] ],
-        [ "20020927", [ nil, "20020927", nil ] ],
-      ].each do |x, y|
-        expect(@rpmutils.version_parse(x)).to eq(y)
-      end
-    end
-
-    it "parses strange version strings" do
-      [
-        [ "3..3", [ nil, "3..3", nil ] ],
-        [ "0001.7.3", [ nil, "0001.7.3", nil ] ],
-        [ "20020927,3", [ nil, "20020927,3", nil ] ],
-      ].each do |x, y|
-        expect(@rpmutils.version_parse(x)).to eq(y)
-      end
-    end
-
-    it "parses known good version release strings" do
-      [
-        [ "3.3-0.pre3.1.60.el5_5.1", [ nil, "3.3", "0.pre3.1.60.el5_5.1" ] ],
-        [ "1.7.3-1jpp.2.el5", [ nil, "1.7.3", "1jpp.2.el5" ] ],
-        [ "20020927-46.el5", [ nil, "20020927", "46.el5" ] ],
-      ].each do |x, y|
-        expect(@rpmutils.version_parse(x)).to eq(y)
-      end
-    end
-
-    it "parses strange version release strings" do
-      [
-        [ "3.3-", [ nil, "3.3", nil ] ],
-        [ "-1jpp.2.el5", [ nil, nil, "1jpp.2.el5" ] ],
-        [ "-0020020927-46.el5", [ nil, "-0020020927", "46.el5" ] ],
-      ].each do |x, y|
-        expect(@rpmutils.version_parse(x)).to eq(y)
-      end
-    end
-  end
-
-  describe "rpmvercmp" do
-    before do
-      @rpmutils = Chef::Provider::Package::Yum::RPMUtils
-    end
-
-    it "should validate version compare logic for standard examples" do
-      [
-        # numeric
-        [ "0.0.2", "0.0.1", 1 ],
-        [ "0.2.0", "0.1.0", 1 ],
-        [ "2.0.0", "1.0.0", 1 ],
-        [ "0.0.1", "0.0.1", 0 ],
-        [ "0.0.1", "0.0.2", -1 ],
-        [ "0.1.0", "0.2.0", -1 ],
-        [ "1.0.0", "2.0.0", -1 ],
-        # alpha
-        [ "bb", "aa", 1 ],
-        [ "ab", "aa", 1 ],
-        [ "aa", "aa", 0 ],
-        [ "aa", "bb", -1 ],
-        [ "aa", "ab", -1 ],
-        [ "BB", "AA", 1 ],
-        [ "AA", "AA", 0 ],
-        [ "AA", "BB", -1 ],
-        [ "aa", "AA", 1 ],
-        [ "AA", "aa", -1 ],
-        # alphanumeric
-        [ "0.0.1b", "0.0.1a", 1 ],
-        [ "0.1b.0", "0.1a.0", 1 ],
-        [ "1b.0.0", "1a.0.0", 1 ],
-        [ "0.0.1a", "0.0.1a", 0 ],
-        [ "0.0.1a", "0.0.1b", -1 ],
-        [ "0.1a.0", "0.1b.0", -1 ],
-        [ "1a.0.0", "1b.0.0", -1 ],
-        # alphanumeric against alphanumeric
-        [ "0.0.1", "0.0.a", 1 ],
-        [ "0.1.0", "0.a.0", 1 ],
-        [ "1.0.0", "a.0.0", 1 ],
-        [ "0.0.a", "0.0.a", 0 ],
-        [ "0.0.a", "0.0.1", -1 ],
-        [ "0.a.0", "0.1.0", -1 ],
-        [ "a.0.0", "1.0.0", -1 ],
-        # alphanumeric against numeric
-        [ "0.0.2", "0.0.1a", 1 ],
-        [ "0.0.2a", "0.0.1", 1 ],
-        [ "0.0.1", "0.0.2a", -1 ],
-        [ "0.0.1a", "0.0.2", -1 ],
-        # length
-        [ "0.0.1aa", "0.0.1a", 1 ],
-        [ "0.0.1aa", "0.0.1aa", 0 ],
-        [ "0.0.1a", "0.0.1aa", -1 ],
-     ].each do |x, y, result|
-        expect(@rpmutils.rpmvercmp(x, y)).to eq(result)
-      end
-    end
-
-    it "should validate version compare logic for strange examples" do
-      [
-        [ "2,0,0", "1.0.0", 1 ],
-        [ "0.0.1", "0,0.1", 0 ],
-        [ "1.0.0", "2,0,0", -1 ],
-        [ "002.0.0", "001.0.0", 1 ],
-        [ "001..0.1", "001..0.0", 1 ],
-        [ "-001..1", "-001..0", 1 ],
-        [ "1.0.1", nil, 1 ],
-        [ nil, nil, 0 ],
-        [ nil, "1.0.1", -1 ],
-        [ "1.0.1", "", 1 ],
-        [ "", "", 0 ],
-        [ "", "1.0.1", -1 ],
-     ].each do |x, y, result|
-        expect(@rpmutils.rpmvercmp(x, y)).to eq(result)
-      end
-    end
-
-    it "tests isalnum good input" do
-      %w{a z A Z 0 9}.each do |t|
-        expect(@rpmutils.isalnum(t)).to eq(true)
-      end
-    end
-
-    it "tests isalnum bad input" do
-      [ "-", ".", "!", "^", ":", "_" ].each do |t|
-        expect(@rpmutils.isalnum(t)).to eq(false)
-      end
-    end
-
-    it "tests isalpha good input" do
-      %w{a z A Z}.each do |t|
-        expect(@rpmutils.isalpha(t)).to eq(true)
-      end
-    end
-
-    it "tests isalpha bad input" do
-      [ "0", "9", "-", ".", "!", "^", ":", "_" ].each do |t|
-        expect(@rpmutils.isalpha(t)).to eq(false)
-      end
-    end
-
-    it "tests isdigit good input" do
-      %w{0 9}.each do |t|
-        expect(@rpmutils.isdigit(t)).to eq(true)
-      end
-    end
-
-    it "tests isdigit bad input" do
-      [ "A", "z", "-", ".", "!", "^", ":", "_" ].each do |t|
-        expect(@rpmutils.isdigit(t)).to eq(false)
-      end
-    end
-  end
-
-end
-
-describe Chef::Provider::Package::Yum::RPMVersion do
-  describe "new - with parsing" do
-    before do
-      @rpmv = Chef::Provider::Package::Yum::RPMVersion.new("1:1.6.5-9.36.el5")
-    end
-
-    it "should expose evr (name-version-release) available" do
-      expect(@rpmv.e).to eq(1)
-      expect(@rpmv.v).to eq("1.6.5")
-      expect(@rpmv.r).to eq("9.36.el5")
-
-      expect(@rpmv.evr).to eq("1:1.6.5-9.36.el5")
-    end
-
-    it "should output a version-release string" do
-      expect(@rpmv.to_s).to eq("1.6.5-9.36.el5")
-    end
-  end
-
-  describe "new - no parsing" do
-    before do
-      @rpmv = Chef::Provider::Package::Yum::RPMVersion.new("1", "1.6.5", "9.36.el5")
-    end
-
-    it "should expose evr (name-version-release) available" do
-      expect(@rpmv.e).to eq(1)
-      expect(@rpmv.v).to eq("1.6.5")
-      expect(@rpmv.r).to eq("9.36.el5")
-
-      expect(@rpmv.evr).to eq("1:1.6.5-9.36.el5")
-    end
-
-    it "should output a version-release string" do
-      expect(@rpmv.to_s).to eq("1.6.5-9.36.el5")
-    end
-  end
-
-  it "should raise an error unless passed 1 or 3 args" do
-    expect do
-      Chef::Provider::Package::Yum::RPMVersion.new()
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMVersion.new("1:1.6.5-9.36.el5")
-    end.not_to raise_error
-    expect do
-      Chef::Provider::Package::Yum::RPMVersion.new("1:1.6.5-9.36.el5", "extra")
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMVersion.new("1", "1.6.5", "9.36.el5")
-    end.not_to raise_error
-    expect do
-      Chef::Provider::Package::Yum::RPMVersion.new("1", "1.6.5", "9.36.el5", "extra")
-    end.to raise_error(ArgumentError)
-  end
-
-  # thanks version_class_spec.rb!
-  describe "compare" do
-    it "should sort based on complete epoch-version-release data" do
-      [
-        # smaller, larger
-        [ "0:1.6.5-9.36.el5",
-          "1:1.6.5-9.36.el5" ],
-        [ "0:2.3-15.el5",
-          "0:3.3-15.el5" ],
-        [ "0:alpha9.8-27.2",
-          "0:beta9.8-27.2" ],
-        [ "0:0.09-14jpp.3",
-          "0:0.09-15jpp.3" ],
-        [ "0:0.9.0-0.6.20110211.el5",
-          "0:0.9.0-0.6.20120211.el5" ],
-        [ "0:1.9.1-4.el5",
-          "0:1.9.1-5.el5" ],
-        [ "0:1.4.10-7.20090624svn.el5",
-          "0:1.4.10-7.20090625svn.el5" ],
-        [ "0:2.3.4-2.el5",
-          "0:2.3.4-2.el6" ],
-      ].each do |smaller, larger|
-        sm = Chef::Provider::Package::Yum::RPMVersion.new(smaller)
-        lg = Chef::Provider::Package::Yum::RPMVersion.new(larger)
-        expect(sm).to be < lg
-        expect(lg).to be > sm
-        expect(sm).not_to eq(lg)
-      end
-    end
-
-    it "should sort based on partial epoch-version-release data" do
-      [
-        # smaller, larger
-        [ ":1.6.5-9.36.el5",
-          "1:1.6.5-9.36.el5" ],
-        [ "2.3-15.el5",
-          "3.3-15.el5" ],
-        [ "alpha9.8",
-          "beta9.8" ],
-        %w{14jpp
-15jpp},
-        [ "0.9.0-0.6",
-          "0.9.0-0.7" ],
-        [ "0:1.9",
-          "3:1.9" ],
-        [ "2.3-2.el5",
-          "2.3-2.el6" ],
-      ].each do |smaller, larger|
-        sm = Chef::Provider::Package::Yum::RPMVersion.new(smaller)
-        lg = Chef::Provider::Package::Yum::RPMVersion.new(larger)
-        expect(sm).to be < lg
-        expect(lg).to be > sm
-        expect(sm).not_to eq(lg)
-      end
-    end
-
-    it "should verify equality of complete epoch-version-release data" do
-      [
-        [ "0:1.6.5-9.36.el5",
-          "0:1.6.5-9.36.el5" ],
-        [ "0:2.3-15.el5",
-          "0:2.3-15.el5" ],
-        [ "0:alpha9.8-27.2",
-          "0:alpha9.8-27.2" ],
-      ].each do |smaller, larger|
-        sm = Chef::Provider::Package::Yum::RPMVersion.new(smaller)
-        lg = Chef::Provider::Package::Yum::RPMVersion.new(larger)
-        expect(sm).to eq(lg)
-      end
-    end
-
-    it "should verify equality of partial epoch-version-release data" do
-      [
-        [ ":1.6.5-9.36.el5",
-          "0:1.6.5-9.36.el5" ],
-        [ "2.3-15.el5",
-          "2.3-15.el5" ],
-        [ "alpha9.8-3",
-          "alpha9.8-3" ],
-      ].each do |smaller, larger|
-        sm = Chef::Provider::Package::Yum::RPMVersion.new(smaller)
-        lg = Chef::Provider::Package::Yum::RPMVersion.new(larger)
-        expect(sm).to eq(lg)
-      end
-    end
-  end
-
-  describe "partial compare" do
-    it "should compare based on partial epoch-version-release data" do
-      [
-        # smaller, larger
-        [ "0:1.1.1-1",
-          "1:" ],
-        [ "0:1.1.1-1",
-          "0:1.1.2" ],
-        [ "0:1.1.1-1",
-          "0:1.1.2-1" ],
-        [ "0:",
-          "1:1.1.1-1" ],
-        [ "0:1.1.1",
-          "0:1.1.2-1" ],
-        [ "0:1.1.1-1",
-          "0:1.1.2-1" ],
-      ].each do |smaller, larger|
-        sm = Chef::Provider::Package::Yum::RPMVersion.new(smaller)
-        lg = Chef::Provider::Package::Yum::RPMVersion.new(larger)
-        expect(sm.partial_compare(lg)).to eq(-1)
-        expect(lg.partial_compare(sm)).to eq(1)
-        expect(sm.partial_compare(lg)).not_to eq(0)
-      end
-    end
-
-    it "should verify equality based on partial epoch-version-release data" do
-      [
-        [ "0:",
-          "0:1.1.1-1" ],
-        [ "0:1.1.1",
-          "0:1.1.1-1" ],
-        [ "0:1.1.1-1",
-          "0:1.1.1-1" ],
-      ].each do |smaller, larger|
-        sm = Chef::Provider::Package::Yum::RPMVersion.new(smaller)
-        lg = Chef::Provider::Package::Yum::RPMVersion.new(larger)
-        expect(sm.partial_compare(lg)).to eq(0)
-      end
-    end
-  end
-
-end
-
-describe Chef::Provider::Package::Yum::RPMPackage do
-  describe "new - with parsing" do
-    before do
-      @rpm = Chef::Provider::Package::Yum::RPMPackage.new("testing", "1:1.6.5-9.36.el5", "x86_64", [])
-    end
-
-    it "should expose nevra (name-epoch-version-release-arch) available" do
-      expect(@rpm.name).to eq("testing")
-      expect(@rpm.version.e).to eq(1)
-      expect(@rpm.version.v).to eq("1.6.5")
-      expect(@rpm.version.r).to eq("9.36.el5")
-      expect(@rpm.arch).to eq("x86_64")
-
-      expect(@rpm.nevra).to eq("testing-1:1.6.5-9.36.el5.x86_64")
-      expect(@rpm.to_s).to eq(@rpm.nevra)
-    end
-
-    it "should always have at least one provide, itself" do
-      expect(@rpm.provides.size).to eq(1)
-      expect(@rpm.provides[0].name).to eql("testing")
-      expect(@rpm.provides[0].version.evr).to eql("1:1.6.5-9.36.el5")
-      expect(@rpm.provides[0].flag).to eql(:==)
-    end
-  end
-
-  describe "new - no parsing" do
-    before do
-      @rpm = Chef::Provider::Package::Yum::RPMPackage.new("testing", "1", "1.6.5", "9.36.el5", "x86_64", [])
-    end
-
-    it "should expose nevra (name-epoch-version-release-arch) available" do
-      expect(@rpm.name).to eq("testing")
-      expect(@rpm.version.e).to eq(1)
-      expect(@rpm.version.v).to eq("1.6.5")
-      expect(@rpm.version.r).to eq("9.36.el5")
-      expect(@rpm.arch).to eq("x86_64")
-
-      expect(@rpm.nevra).to eq("testing-1:1.6.5-9.36.el5.x86_64")
-      expect(@rpm.to_s).to eq(@rpm.nevra)
-    end
-
-    it "should always have at least one provide, itself" do
-      expect(@rpm.provides.size).to eq(1)
-      expect(@rpm.provides[0].name).to eql("testing")
-      expect(@rpm.provides[0].version.evr).to eql("1:1.6.5-9.36.el5")
-      expect(@rpm.provides[0].flag).to eql(:==)
-    end
-  end
-
-  it "should raise an error unless passed 4 or 6 args" do
-    expect do
-      Chef::Provider::Package::Yum::RPMPackage.new()
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMPackage.new("testing")
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMPackage.new("testing", "1:1.6.5-9.36.el5")
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMPackage.new("testing", "1:1.6.5-9.36.el5", "x86_64")
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMPackage.new("testing", "1:1.6.5-9.36.el5", "x86_64", [])
-    end.not_to raise_error
-    expect do
-      Chef::Provider::Package::Yum::RPMPackage.new("testing", "1", "1.6.5", "9.36.el5", "x86_64")
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMPackage.new("testing", "1", "1.6.5", "9.36.el5", "x86_64", [])
-    end.not_to raise_error
-    expect do
-      Chef::Provider::Package::Yum::RPMPackage.new("testing", "1", "1.6.5", "9.36.el5", "x86_64", [], "extra")
-    end.to raise_error(ArgumentError)
-  end
-
-  describe "<=>" do
-    it "should sort alphabetically based on package name" do
-      [
-        [ "a-test",
-          "b-test" ],
-        [ "B-test",
-          "a-test" ],
-        [ "A-test",
-          "B-test" ],
-        [ "Aa-test",
-          "aA-test" ],
-        %w{1test
-2test},
-      ].each do |smaller, larger|
-        sm = Chef::Provider::Package::Yum::RPMPackage.new(smaller, "0:0.0.1-1", "x86_64", [])
-        lg = Chef::Provider::Package::Yum::RPMPackage.new(larger, "0:0.0.1-1", "x86_64", [])
-        expect(sm).to be < lg
-        expect(lg).to be > sm
-        expect(sm).not_to eq(lg)
-      end
-    end
-
-    it "should sort alphabetically based on package arch" do
-      [
-        %w{i386
-x86_64},
-        %w{i386
-noarch},
-        %w{noarch
-x86_64},
-      ].each do |smaller, larger|
-        sm = Chef::Provider::Package::Yum::RPMPackage.new("test-package", "0:0.0.1-1", smaller, [])
-        lg = Chef::Provider::Package::Yum::RPMPackage.new("test-package", "0:0.0.1-1", larger, [])
-        expect(sm).to be < lg
-        expect(lg).to be > sm
-        expect(sm).not_to eq(lg)
-      end
-    end
-  end
-
-end
-
-describe Chef::Provider::Package::Yum::RPMDbPackage do
-  before(:each) do
-    # name, version, arch, installed, available, repoid
-    @rpm_x = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-b", "0:1.6.5-9.36.el5", "noarch", [], false, true, "base")
-    @rpm_y = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-b", "0:1.6.5-9.36.el5", "noarch", [], true, true, "extras")
-    @rpm_z = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-b", "0:1.6.5-9.36.el5", "noarch", [], true, false, "other")
-  end
-
-  describe "initialize" do
-    it "should return a Chef::Provider::Package::Yum::RPMDbPackage object" do
-      expect(@rpm_x).to be_kind_of(Chef::Provider::Package::Yum::RPMDbPackage)
-    end
-  end
-
-  describe "available" do
-    it "should return true" do
-      expect(@rpm_x.available).to eq(true)
-      expect(@rpm_y.available).to eq(true)
-      expect(@rpm_z.available).to eq(false)
-    end
-  end
-
-  describe "installed" do
-    it "should return true" do
-      expect(@rpm_x.installed).to eq(false)
-      expect(@rpm_y.installed).to eq(true)
-      expect(@rpm_z.installed).to eq(true)
-    end
-  end
-
-  describe "repoid" do
-    it "should return the source repository repoid" do
-      expect(@rpm_x.repoid).to eq("base")
-      expect(@rpm_y.repoid).to eq("extras")
-      expect(@rpm_z.repoid).to eq("other")
-    end
-  end
-end
-
-describe Chef::Provider::Package::Yum::RPMDependency do
-  describe "new - with parsing" do
-    before do
-      @rpmdep = Chef::Provider::Package::Yum::RPMDependency.new("testing", "1:1.6.5-9.36.el5", :==)
-    end
-
-    it "should expose name, version, flag available" do
-      expect(@rpmdep.name).to eq("testing")
-      expect(@rpmdep.version.e).to eq(1)
-      expect(@rpmdep.version.v).to eq("1.6.5")
-      expect(@rpmdep.version.r).to eq("9.36.el5")
-      expect(@rpmdep.flag).to eq(:==)
-    end
-  end
-
-  describe "new - no parsing" do
-    before do
-      @rpmdep = Chef::Provider::Package::Yum::RPMDependency.new("testing", "1", "1.6.5", "9.36.el5", :==)
-    end
-
-    it "should expose name, version, flag available" do
-      expect(@rpmdep.name).to eq("testing")
-      expect(@rpmdep.version.e).to eq(1)
-      expect(@rpmdep.version.v).to eq("1.6.5")
-      expect(@rpmdep.version.r).to eq("9.36.el5")
-      expect(@rpmdep.flag).to eq(:==)
-    end
-  end
-
-  it "should raise an error unless passed 3 or 5 args" do
-    expect do
-      Chef::Provider::Package::Yum::RPMDependency.new()
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMDependency.new("testing")
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMDependency.new("testing", "1:1.6.5-9.36.el5")
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMDependency.new("testing", "1:1.6.5-9.36.el5", :==)
-    end.not_to raise_error
-    expect do
-      Chef::Provider::Package::Yum::RPMDependency.new("testing", "1:1.6.5-9.36.el5", :==, "extra")
-    end.to raise_error(ArgumentError)
-    expect do
-      Chef::Provider::Package::Yum::RPMDependency.new("testing", "1", "1.6.5", "9.36.el5", :==)
-    end.not_to raise_error
-    expect do
-      Chef::Provider::Package::Yum::RPMDependency.new("testing", "1", "1.6.5", "9.36.el5", :==, "extra")
-    end.to raise_error(ArgumentError)
-  end
-
-  describe "parse" do
-    it "should parse a name, flag, version string into a valid RPMDependency object" do
-      @rpmdep = Chef::Provider::Package::Yum::RPMDependency.parse("testing >= 1:1.6.5-9.36.el5")
-
-      expect(@rpmdep.name).to eq("testing")
-      expect(@rpmdep.version.e).to eq(1)
-      expect(@rpmdep.version.v).to eq("1.6.5")
-      expect(@rpmdep.version.r).to eq("9.36.el5")
-      expect(@rpmdep.flag).to eq(:>=)
-    end
-
-    it "should parse a name into a valid RPMDependency object" do
-      @rpmdep = Chef::Provider::Package::Yum::RPMDependency.parse("testing")
-
-      expect(@rpmdep.name).to eq("testing")
-      expect(@rpmdep.version.e).to eq(nil)
-      expect(@rpmdep.version.v).to eq(nil)
-      expect(@rpmdep.version.r).to eq(nil)
-      expect(@rpmdep.flag).to eq(:==)
-    end
-
-    it "should parse an invalid string into the name of a RPMDependency object" do
-      @rpmdep = Chef::Provider::Package::Yum::RPMDependency.parse("testing blah >")
-
-      expect(@rpmdep.name).to eq("testing blah >")
-      expect(@rpmdep.version.e).to eq(nil)
-      expect(@rpmdep.version.v).to eq(nil)
-      expect(@rpmdep.version.r).to eq(nil)
-      expect(@rpmdep.flag).to eq(:==)
-    end
-
-    it "should parse various valid flags" do
-      [
-        [ ">", :> ],
-        [ ">=", :>= ],
-        [ "=", :== ],
-        [ "==", :== ],
-        [ "<=", :<= ],
-        [ "<", :< ],
-      ].each do |before, after|
-        @rpmdep = Chef::Provider::Package::Yum::RPMDependency.parse("testing #{before} 1:1.1-1")
-        expect(@rpmdep.flag).to eq(after)
-      end
-    end
-
-    it "should parse various invalid flags and treat them as names" do
-      [
-        [ "<>", :== ],
-        [ "!=", :== ],
-        [ ">>", :== ],
-        [ "<<", :== ],
-        [ "!", :== ],
-        [ "~", :== ],
-      ].each do |before, after|
-        @rpmdep = Chef::Provider::Package::Yum::RPMDependency.parse("testing #{before} 1:1.1-1")
-        expect(@rpmdep.name).to eq("testing #{before} 1:1.1-1")
-        expect(@rpmdep.flag).to eq(after)
-      end
-    end
-  end
-
-  describe "satisfy?" do
-    it "should raise an error unless a RPMDependency is passed" do
-      @rpmprovide = Chef::Provider::Package::Yum::RPMDependency.new("testing", "1:1.6.5-9.36.el5", :==)
-      @rpmrequire = Chef::Provider::Package::Yum::RPMDependency.new("testing", "1:1.6.5-9.36.el5", :>=)
-      expect do
-        @rpmprovide.satisfy?("hi")
-      end.to raise_error(ArgumentError)
-      expect do
-        @rpmprovide.satisfy?(@rpmrequire)
-      end.not_to raise_error
-    end
-
-    it "should validate dependency satisfaction logic for standard examples" do
-      [
-        # names
-        [ "test", "test", true ],
-        [ "test", "foo", false ],
-        # full: epoch:version-relese
-        [ "testing = 1:1.1-1", "testing > 1:1.1-0", true ],
-        [ "testing = 1:1.1-1", "testing >= 1:1.1-0", true ],
-        [ "testing = 1:1.1-1", "testing >= 1:1.1-1", true ],
-        [ "testing = 1:1.1-1", "testing = 1:1.1-1", true ],
-        [ "testing = 1:1.1-1", "testing == 1:1.1-1", true ],
-        [ "testing = 1:1.1-1", "testing <= 1:1.1-1", true ],
-        [ "testing = 1:1.1-1", "testing <= 1:1.1-0", false ],
-        [ "testing = 1:1.1-1", "testing < 1:1.1-0", false ],
-        # partial: epoch:version
-        [ "testing = 1:1.1", "testing > 1:1.0", true ],
-        [ "testing = 1:1.1", "testing >= 1:1.0", true ],
-        [ "testing = 1:1.1", "testing >= 1:1.1", true ],
-        [ "testing = 1:1.1", "testing = 1:1.1", true ],
-        [ "testing = 1:1.1", "testing == 1:1.1", true ],
-        [ "testing = 1:1.1", "testing <= 1:1.1", true ],
-        [ "testing = 1:1.1", "testing <= 1:1.0", false ],
-        [ "testing = 1:1.1", "testing < 1:1.0", false ],
-        # partial: epoch
-        [ "testing = 1:", "testing > 0:", true ],
-        [ "testing = 1:", "testing >= 0:", true ],
-        [ "testing = 1:", "testing >= 1:", true ],
-        [ "testing = 1:", "testing = 1:", true ],
-        [ "testing = 1:", "testing == 1:", true ],
-        [ "testing = 1:", "testing <= 1:", true ],
-        [ "testing = 1:", "testing <= 0:", false ],
-        [ "testing = 1:", "testing < 0:", false ],
-        # mix and match!
-        [ "testing = 1:1.1-1", "testing == 1:1.1", true ],
-        [ "testing = 1:1.1-1", "testing == 1:", true ],
-     ].each do |prov, req, result|
-        @rpmprovide = Chef::Provider::Package::Yum::RPMDependency.parse(prov)
-        @rpmrequire = Chef::Provider::Package::Yum::RPMDependency.parse(req)
-
-        expect(@rpmprovide.satisfy?(@rpmrequire)).to eq(result)
-        expect(@rpmrequire.satisfy?(@rpmprovide)).to eq(result)
-      end
-    end
-  end
-
-end
-
-# thanks resource_collection_spec.rb!
-describe Chef::Provider::Package::Yum::RPMDb do
-  before(:each) do
-    @rpmdb = Chef::Provider::Package::Yum::RPMDb.new
-    # name, version, arch, installed, available
-    deps_v = [
-      Chef::Provider::Package::Yum::RPMDependency.parse("libz.so.1()(64bit)"),
-      Chef::Provider::Package::Yum::RPMDependency.parse("test-package-a = 0:1.6.5-9.36.el5"),
-    ]
-    deps_z = [
-      Chef::Provider::Package::Yum::RPMDependency.parse("libz.so.1()(64bit)"),
-      Chef::Provider::Package::Yum::RPMDependency.parse("config(test) = 0:1.6.5-9.36.el5"),
-      Chef::Provider::Package::Yum::RPMDependency.parse("test-package-c = 0:1.6.5-9.36.el5"),
-    ]
-    @rpm_v = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-a", "0:1.6.5-9.36.el5", "i386", deps_v, true, false, "base")
-    @rpm_w = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-b", "0:1.6.5-9.36.el5", "i386", [], true, true, "extras")
-    @rpm_x = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-b", "0:1.6.5-9.36.el5", "x86_64", [], false, true, "extras")
-    @rpm_y = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-b", "1:1.6.5-9.36.el5", "x86_64", [], true, true, "extras")
-    @rpm_z = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-c", "0:1.6.5-9.36.el5", "noarch", deps_z, true, true, "base")
-    @rpm_z_mirror = Chef::Provider::Package::Yum::RPMDbPackage.new("test-package-c", "0:1.6.5-9.36.el5", "noarch", deps_z, true, true, "base")
-  end
-
-  describe "initialize" do
-    it "should return a Chef::Provider::Package::Yum::RPMDb object" do
-      expect(@rpmdb).to be_kind_of(Chef::Provider::Package::Yum::RPMDb)
-    end
-  end
-
-  describe "push" do
-    it "should accept an RPMDbPackage object through pushing" do
-      expect { @rpmdb.push(@rpm_w) }.not_to raise_error
-    end
-
-    it "should accept multiple RPMDbPackage object through pushing" do
-      expect { @rpmdb.push(@rpm_w, @rpm_x, @rpm_y, @rpm_z) }.not_to raise_error
-    end
-
-    it "should only accept an RPMDbPackage object" do
-      expect { @rpmdb.push("string") }.to raise_error(ArgumentError)
-    end
-
-    it "should add the package to the package db" do
-      @rpmdb.push(@rpm_w)
-      expect(@rpmdb["test-package-b"]).not_to eq(nil)
-    end
-
-    it "should add conditionally add the package to the available list" do
-      expect(@rpmdb.available_size).to eq(0)
-      @rpmdb.push(@rpm_v, @rpm_w)
-      expect(@rpmdb.available_size).to eq(1)
-    end
-
-    it "should add conditionally add the package to the installed list" do
-      expect(@rpmdb.installed_size).to eq(0)
-      @rpmdb.push(@rpm_w, @rpm_x)
-      expect(@rpmdb.installed_size).to eq(1)
-    end
-
-    it "should have a total of 2 packages in the RPMDb" do
-      expect(@rpmdb.size).to eq(0)
-      @rpmdb.push(@rpm_w, @rpm_x, @rpm_y, @rpm_z)
-      expect(@rpmdb.size).to eq(2)
-    end
-
-    it "should keep the Array unique when a duplicate is pushed" do
-      @rpmdb.push(@rpm_z, @rpm_z_mirror)
-      expect(@rpmdb["test-package-c"].size).to eq(1)
-    end
-
-    it "should register the package provides in the provides index" do
-      @rpmdb.push(@rpm_v, @rpm_w, @rpm_z)
-      expect(@rpmdb.lookup_provides("test-package-a")[0]).to eq(@rpm_v)
-      expect(@rpmdb.lookup_provides("config(test)")[0]).to eq(@rpm_z)
-      expect(@rpmdb.lookup_provides("libz.so.1()(64bit)")[0]).to eq(@rpm_v)
-      expect(@rpmdb.lookup_provides("libz.so.1()(64bit)")[1]).to eq(@rpm_z)
-    end
-  end
-
-  describe "<<" do
-    it "should accept an RPMPackage object through the << operator" do
-      expect { @rpmdb << @rpm_w }.not_to raise_error
-    end
-  end
-
-  describe "lookup" do
-    it "should return an Array of RPMPackage objects by index" do
-      @rpmdb << @rpm_w
-      expect(@rpmdb.lookup("test-package-b")).to be_kind_of(Array)
-    end
-  end
-
-  describe "[]" do
-    it "should return an Array of RPMPackage objects though the [index] operator" do
-      @rpmdb << @rpm_w
-      expect(@rpmdb["test-package-b"]).to be_kind_of(Array)
-    end
-
-    it "should return an Array of 3 RPMPackage objects" do
-      @rpmdb.push(@rpm_w, @rpm_x, @rpm_y, @rpm_z)
-      expect(@rpmdb["test-package-b"].size).to eq(3)
-    end
-
-    it "should return an Array of RPMPackage objects sorted from newest to oldest" do
-      @rpmdb.push(@rpm_w, @rpm_x, @rpm_y, @rpm_z)
-      expect(@rpmdb["test-package-b"][0]).to eq(@rpm_y)
-      expect(@rpmdb["test-package-b"][1]).to eq(@rpm_x)
-      expect(@rpmdb["test-package-b"][2]).to eq(@rpm_w)
-    end
-  end
-
-  describe "lookup_provides" do
-    it "should return an Array of RPMPackage objects by index" do
-      @rpmdb << @rpm_z
-      x = @rpmdb.lookup_provides("config(test)")
-      expect(x).to be_kind_of(Array)
-      expect(x[0]).to eq(@rpm_z)
-    end
-  end
-
-  describe "clear" do
-    it "should clear the RPMDb" do
-      expect(@rpmdb).to receive(:clear_available).once
-      expect(@rpmdb).to receive(:clear_installed).once
-      @rpmdb.push(@rpm_w, @rpm_x, @rpm_y, @rpm_z)
-      expect(@rpmdb.size).not_to eq(0)
-      expect(@rpmdb.lookup_provides("config(test)")).to be_kind_of(Array)
-      @rpmdb.clear
-      expect(@rpmdb.lookup_provides("config(test)")).to eq(nil)
-      expect(@rpmdb.size).to eq(0)
-    end
-  end
-
-  describe "clear_available" do
-    it "should clear the available list" do
-      @rpmdb.push(@rpm_w, @rpm_x, @rpm_y, @rpm_z)
-      expect(@rpmdb.available_size).not_to eq(0)
-      @rpmdb.clear_available
-      expect(@rpmdb.available_size).to eq(0)
-    end
-  end
-
-  describe "available?" do
-    it "should return true if a package is available" do
-      expect(@rpmdb.available?(@rpm_w)).to eq(false)
-      @rpmdb.push(@rpm_v, @rpm_w)
-      expect(@rpmdb.available?(@rpm_v)).to eq(false)
-      expect(@rpmdb.available?(@rpm_w)).to eq(true)
-    end
-  end
-
-  describe "clear_installed" do
-    it "should clear the installed list" do
-      @rpmdb.push(@rpm_w, @rpm_x, @rpm_y, @rpm_z)
-      expect(@rpmdb.installed_size).not_to eq(0)
-      @rpmdb.clear_installed
-      expect(@rpmdb.installed_size).to eq(0)
-    end
-  end
-
-  describe "installed?" do
-    it "should return true if a package is installed" do
-      expect(@rpmdb.installed?(@rpm_w)).to eq(false)
-      @rpmdb.push(@rpm_w, @rpm_x)
-      expect(@rpmdb.installed?(@rpm_w)).to eq(true)
-      expect(@rpmdb.installed?(@rpm_x)).to eq(false)
-    end
-  end
-
-  describe "whatprovides" do
-    it "should raise an error unless a RPMDependency is passed" do
-      @rpmprovide = Chef::Provider::Package::Yum::RPMDependency.new("testing", "1:1.6.5-9.36.el5", :==)
-      @rpmrequire = Chef::Provider::Package::Yum::RPMDependency.new("testing", "1:1.6.5-9.36.el5", :>=)
-      expect do
-        @rpmdb.whatprovides("hi")
-      end.to raise_error(ArgumentError)
-      expect do
-        @rpmdb.whatprovides(@rpmrequire)
-      end.not_to raise_error
-    end
-
-    it "should return an Array of packages statisfying a RPMDependency" do
-      @rpmdb.push(@rpm_v, @rpm_w, @rpm_z)
-
-      @rpmrequire = Chef::Provider::Package::Yum::RPMDependency.parse("test-package-a >= 1.6.5")
-      x = @rpmdb.whatprovides(@rpmrequire)
-      expect(x).to be_kind_of(Array)
-      expect(x[0]).to eq(@rpm_v)
-
-      @rpmrequire = Chef::Provider::Package::Yum::RPMDependency.parse("libz.so.1()(64bit)")
-      x = @rpmdb.whatprovides(@rpmrequire)
-      expect(x).to be_kind_of(Array)
-      expect(x[0]).to eq(@rpm_v)
-      expect(x[1]).to eq(@rpm_z)
-    end
-  end
-
-end
-
-describe Chef::Provider::Package::Yum::YumCache do
-  # allow for the reset of a Singleton
-  # thanks to Ian White (http://blog.ardes.com/2006/12/11/testing-singletons-with-ruby)
-  class << Chef::Provider::Package::Yum::YumCache
-    def reset_instance
-      Singleton.send :__init__, self
-      self
-    end
-  end
-
-  let(:yum_exe) do
-    StringIO.new("#!/usr/bin/python\n\naldsjfa\ldsajflkdsjf\lajsdfj")
-  end
-
-  let(:bin_exe) do
-    StringIO.new(SecureRandom.random_bytes)
-  end
-
-  before(:each) do
-    @stdin = double("STDIN", :nil_object => true)
-    @stdout = double("STDOUT", :nil_object => true)
-
-    @stdout_good = <<EOF
-[option installonlypkgs] kernel kernel-bigmem kernel-enterprise
-erlang-mochiweb 0 1.4.1 5.el5 x86_64 ['erlang-mochiweb = 1.4.1-5.el5', 'mochiweb = 1.4.1-5.el5'] i installed
-zip 0 2.31 2.el5 x86_64 ['zip = 2.31-2.el5'] r base
-zisofs-tools 0 1.0.6 3.2.2 x86_64 [] a extras
-zlib 0 1.2.3 3 x86_64 ['zlib = 1.2.3-3', 'libz.so.1()(64bit)'] r base
-zlib 0 1.2.3 3 i386 ['zlib = 1.2.3-3', 'libz.so.1'] r base
-zlib-devel 0 1.2.3 3 i386 [] a extras
-zlib-devel 0 1.2.3 3 x86_64 ['zlib-devel = 1.2.3-3'] r base
-znc 0 0.098 1.el5 x86_64 [] a base
-znc-devel 0 0.098 1.el5 i386 [] a extras
-znc-devel 0 0.098 1.el5 x86_64 [] a base
-znc-extra 0 0.098 1.el5 x86_64 [] a base
-znc-modtcl 0 0.098 1.el5 x86_64 [] a base
-znc-test.beta1 0 0.098 1.el5 x86_64 [] a extras
-znc-test.test.beta1 0 0.098 1.el5 x86_64 [] a base
-EOF
-    @stdout_bad_type = <<EOF
-zip 0 2.31 2.el5 x86_64 ['zip = 2.31-2.el5'] r base
-zlib 0 1.2.3 3 x86_64 ['zlib = 1.2.3-3', 'libz.so.1()(64bit)'] c base
-zlib-devel 0 1.2.3 3 i386 [] a extras
-zlib-devel 0 1.2.3 3 x86_64 ['zlib-devel = 1.2.3-3'] bad installed
-znc-modtcl 0 0.098 1.el5 x86_64 [] a base
-EOF
-
-    @stdout_bad_separators = <<EOF
-zip 0 2.31 2.el5 x86_64 ['zip = 2.31-2.el5'] r base
-zlib 0 1.2.3 3 x86_64 ['zlib = 1.2.3-3', 'libz.so.1()(64bit)'] i base bad
-zlib-devel 0 1.2.3 3 i386 [] a extras
-bad zlib-devel 0 1.2.3 3 x86_64 ['zlib-devel = 1.2.3-3'] i installed
-znc-modtcl 0 0.098 1.el5 x86_64 [] a base bad
-EOF
-
-    @stdout_no_output = ""
-
-    @stderr = <<EOF
-yum-dump Config Error: File contains no section headers.
-file: file://///etc/yum.repos.d/CentOS-Base.repo, line: 12
-'qeqwewe\n'
-EOF
-    @status = double("Status", :exitstatus => 0, :stdin => @stdin, :stdout => @stdout_good, :stderr => @stderr)
-    # new singleton each time
-    Chef::Provider::Package::Yum::YumCache.reset_instance
-    @yc = Chef::Provider::Package::Yum::YumCache.instance
-    # load valid data
-    @yc.yum_binary = "yum"
-    allow(@yc).to receive(:shell_out!).and_return(@status)
-    allow_any_instance_of(described_class).to receive(:which).with("yum").and_return("/usr/bin/yum")
-    allow(::File).to receive(:open).with("/usr/bin/yum", "r") do |&block|
-      res = block.call(yum_exe)
-      # a bit of a hack. rewind this since it seem that no matter what
-      # I do, we get the same StringIO objects on multiple calls to
-      # ::File.open
-      yum_exe.rewind; res
-    end
-  end
-
-  describe "initialize" do
-    it "should return a Chef::Provider::Package::Yum::YumCache object" do
-      expect(@yc).to be_kind_of(Chef::Provider::Package::Yum::YumCache)
-    end
-
-    it "should register reload for start of Chef::Client runs" do
-      Chef::Provider::Package::Yum::YumCache.reset_instance
-      expect(Chef::Client).to receive(:when_run_starts) do |&b|
-        expect(b).not_to be_nil
-      end
-      @yc = Chef::Provider::Package::Yum::YumCache.instance
-    end
-  end
-
-  describe "python_bin" do
-    it "should return the default python if an error occurs" do
-      allow(::File).to receive(:open).with("/usr/bin/yum", "r").and_raise(StandardError)
-      expect(@yc.python_bin).to eq("/usr/bin/python")
-    end
-
-    it "should return the default python if the yum-executable doesn't start with #!" do
-      allow(::File).to receive(:open).with("/usr/bin/yum", "r") { |&b| r = b.call(bin_exe); bin_exe.rewind; r }
-      expect(@yc.python_bin).to eq("/usr/bin/python")
-    end
-
-    it "should return /usr/bin/python if the interpreter is /bin/bash" do
-      other = StringIO.new("#!/bin/bash\n# The yum executable redirecting to dnf from dnf-yum compatible package.")
-      allow(::File).to receive(:open).with("/usr/bin/yum", "r") { |&b| r = b.call(other); other.rewind; r }
-      expect(@yc.python_bin).to eq("/usr/bin/python")
-    end
-
-    it "should return the interpreter for yum" do
-      other = StringIO.new("#!/usr/bin/super_python\n\nlasjdfdsaljf\nlasdjfs")
-      allow(::File).to receive(:open).with("/usr/bin/yum", "r") { |&b| r = b.call(other); other.rewind; r }
-      expect(@yc.python_bin).to eq("/usr/bin/super_python")
-    end
-  end
-
-  describe "refresh" do
-    it "should implicitly call yum-dump.py only once by default after being instantiated" do
-      expect(@yc).to receive(:shell_out!).once
-      @yc.installed_version("zlib")
-      @yc.reset
-      @yc.installed_version("zlib")
-    end
-
-    it "should run yum-dump.py using the system python when next_refresh is for :all" do
-      @yc.reload
-      expect(@yc).to receive(:shell_out!).with(%r{^/usr/bin/python .*/yum-dump.py --options --installed-provides --yum-lock-timeout 30$}, :timeout => Chef::Config[:yum_timeout])
-      @yc.refresh
-    end
-
-    it "should run yum-dump.py with the installed flag when next_refresh is for :installed" do
-      @yc.reload_installed
-      expect(@yc).to receive(:shell_out!).with(%r{^/usr/bin/python .*/yum-dump.py --installed --yum-lock-timeout 30$}, :timeout => Chef::Config[:yum_timeout])
-      @yc.refresh
-    end
-
-    it "should run yum-dump.py with the all-provides flag when next_refresh is for :provides" do
-      @yc.reload_provides
-      expect(@yc).to receive(:shell_out!).with(%r{^/usr/bin/python .*/yum-dump.py --options --all-provides --yum-lock-timeout 30$}, :timeout => Chef::Config[:yum_timeout])
-      @yc.refresh
-    end
-
-    it "should pass extra_repo_control args to yum-dump.py" do
-      @yc.enable_extra_repo_control("--enablerepo=foo --disablerepo=bar")
-      expect(@yc).to receive(:shell_out!).with(%r{^/usr/bin/python .*/yum-dump.py --options --installed-provides --enablerepo=foo --disablerepo=bar --yum-lock-timeout 30$}, :timeout => Chef::Config[:yum_timeout])
-      @yc.refresh
-    end
-
-    it "should pass extra_repo_control args and configured yum lock timeout to yum-dump.py" do
-      Chef::Config[:yum_lock_timeout] = 999
-      @yc.enable_extra_repo_control("--enablerepo=foo --disablerepo=bar")
-      expect(@yc).to receive(:shell_out!).with(%r{^/usr/bin/python .*/yum-dump.py --options --installed-provides --enablerepo=foo --disablerepo=bar --yum-lock-timeout 999$}, :timeout => Chef::Config[:yum_timeout])
-      @yc.refresh
-    end
-
-    it "should warn about invalid data with too many separators" do
-      @status = double("Status", :exitstatus => 0, :stdin => @stdin, :stdout => @stdout_bad_separators, :stderr => @stderr)
-      allow(@yc).to receive(:shell_out!).and_return(@status)
-      expect(Chef::Log).to receive(:warn).exactly(3).times.with(%r{Problem parsing})
-      @yc.refresh
-    end
-
-    it "should warn about invalid data with an incorrect type" do
-      @status = double("Status", :exitstatus => 0, :stdin => @stdin, :stdout => @stdout_bad_type, :stderr => @stderr)
-      allow(@yc).to receive(:shell_out!).and_return(@status)
-      expect(Chef::Log).to receive(:warn).exactly(2).times.with(%r{Problem parsing})
-      @yc.refresh
-    end
-
-    it "should warn about no output from yum-dump.py" do
-      @status = double("Status", :exitstatus => 0, :stdin => @stdin, :stdout => @stdout_no_output, :stderr => @stderr)
-      allow(@yc).to receive(:shell_out!).and_return(@status)
-      expect(Chef::Log).to receive(:warn).exactly(1).times.with(%r{no output from yum-dump.py})
-      @yc.refresh
-    end
-
-    it "should raise exception yum-dump.py exits with a non zero status" do
-      @status = double("Status", :exitstatus => 1, :stdin => @stdin, :stdout => @stdout_no_output, :stderr => @stderr)
-      allow(@yc).to receive(:shell_out!).and_return(@status)
-      expect { @yc.refresh }.to raise_error(Chef::Exceptions::Package, %r{CentOS-Base.repo, line: 12})
-    end
-
-    it "should parse type 'i' into an installed state for a package" do
-      expect(@yc.available_version("erlang-mochiweb")).to eq(nil)
-      expect(@yc.installed_version("erlang-mochiweb")).not_to eq(nil)
-    end
-
-    it "should parse type 'a' into an available state for a package" do
-      expect(@yc.available_version("znc")).not_to eq(nil)
-      expect(@yc.installed_version("znc")).to eq(nil)
-    end
-
-    it "should parse type 'r' into an installed and available states for a package" do
-      expect(@yc.available_version("zip")).not_to eq(nil)
-      expect(@yc.installed_version("zip")).not_to eq(nil)
-    end
-
-    it "should parse installonlypkgs from yum-dump.py options output" do
-      expect(@yc.allow_multi_install).to eq(%w{kernel kernel-bigmem kernel-enterprise})
-    end
-  end
-
-  describe "installed_version" do
-    it "should take one or two arguments" do
-      expect { @yc.installed_version("zip") }.not_to raise_error
-      expect { @yc.installed_version("zip", "i386") }.not_to raise_error
-      expect { @yc.installed_version("zip", "i386", "extra") }.to raise_error(ArgumentError)
-    end
-
-    it "should return version-release for matching package regardless of arch" do
-      expect(@yc.installed_version("zip", "x86_64")).to eq("2.31-2.el5")
-      expect(@yc.installed_version("zip", nil)).to eq("2.31-2.el5")
-    end
-
-    it "should return version-release for matching package and arch" do
-      expect(@yc.installed_version("zip", "x86_64")).to eq("2.31-2.el5")
-      expect(@yc.installed_version("zisofs-tools", "i386")).to eq(nil)
-    end
-
-    it "should return nil for an unmatched package" do
-      expect(@yc.installed_version(nil, nil)).to eq(nil)
-      expect(@yc.installed_version("test1", nil)).to eq(nil)
-      expect(@yc.installed_version("test2", "x86_64")).to eq(nil)
-    end
-  end
-
-  describe "available_version" do
-    it "should take one or two arguments" do
-      expect { @yc.available_version("zisofs-tools") }.not_to raise_error
-      expect { @yc.available_version("zisofs-tools", "i386") }.not_to raise_error
-      expect { @yc.available_version("zisofs-tools", "i386", "extra") }.to raise_error(ArgumentError)
-    end
-
-    it "should return version-release for matching package regardless of arch" do
-      expect(@yc.available_version("zip", "x86_64")).to eq("2.31-2.el5")
-      expect(@yc.available_version("zip", nil)).to eq("2.31-2.el5")
-    end
-
-    it "should return version-release for matching package and arch" do
-      expect(@yc.available_version("zip", "x86_64")).to eq("2.31-2.el5")
-      expect(@yc.available_version("zisofs-tools", "i386")).to eq(nil)
-    end
-
-    it "should return nil for an unmatched package" do
-      expect(@yc.available_version(nil, nil)).to eq(nil)
-      expect(@yc.available_version("test1", nil)).to eq(nil)
-      expect(@yc.available_version("test2", "x86_64")).to eq(nil)
-    end
-  end
-
-  describe "version_available?" do
-    it "should take two or three arguments" do
-      expect { @yc.version_available?("zisofs-tools") }.to raise_error(ArgumentError)
-      expect { @yc.version_available?("zisofs-tools", "1.0.6-3.2.2") }.not_to raise_error
-      expect { @yc.version_available?("zisofs-tools", "1.0.6-3.2.2", "x86_64") }.not_to raise_error
-    end
-
-    it "should return true if our package-version-arch is available" do
-      expect(@yc.version_available?("zisofs-tools", "1.0.6-3.2.2", "x86_64")).to eq(true)
-    end
-
-    it "should return true if our package-version, no arch, is available" do
-      expect(@yc.version_available?("zisofs-tools", "1.0.6-3.2.2", nil)).to eq(true)
-      expect(@yc.version_available?("zisofs-tools", "1.0.6-3.2.2")).to eq(true)
-    end
-
-    it "should return false if our package-version-arch isn't available" do
-      expect(@yc.version_available?("zisofs-tools", "1.0.6-3.2.2", "pretend")).to eq(false)
-      expect(@yc.version_available?("zisofs-tools", "pretend", "x86_64")).to eq(false)
-      expect(@yc.version_available?("pretend", "1.0.6-3.2.2", "x86_64")).to eq(false)
-    end
-
-    it "should return false if our package-version, no arch, isn't available" do
-      expect(@yc.version_available?("zisofs-tools", "pretend", nil)).to eq(false)
-      expect(@yc.version_available?("zisofs-tools", "pretend")).to eq(false)
-      expect(@yc.version_available?("pretend", "1.0.6-3.2.2")).to eq(false)
-    end
-  end
-
-  describe "package_repository" do
-    it "should take two or three arguments" do
-      expect { @yc.package_repository("zisofs-tools") }.to raise_error(ArgumentError)
-      expect { @yc.package_repository("zisofs-tools", "1.0.6-3.2.2") }.not_to raise_error
-      expect { @yc.package_repository("zisofs-tools", "1.0.6-3.2.2", "x86_64") }.not_to raise_error
-    end
-
-    it "should return repoid for package-version-arch" do
-      expect(@yc.package_repository("zlib-devel", "1.2.3-3", "i386")).to eq("extras")
-      expect(@yc.package_repository("zlib-devel", "1.2.3-3", "x86_64")).to eq("base")
-    end
-
-    it "should return repoid for package-version, no arch" do
-      expect(@yc.package_repository("zisofs-tools", "1.0.6-3.2.2", nil)).to eq("extras")
-      expect(@yc.package_repository("zisofs-tools", "1.0.6-3.2.2")).to eq("extras")
-    end
-
-    it "should return nil when no match for package-version-arch" do
-      expect(@yc.package_repository("zisofs-tools", "1.0.6-3.2.2", "pretend")).to eq(nil)
-      expect(@yc.package_repository("zisofs-tools", "pretend", "x86_64")).to eq(nil)
-      expect(@yc.package_repository("pretend", "1.0.6-3.2.2", "x86_64")).to eq(nil)
-    end
-
-    it "should return nil when no match for package-version, no arch" do
-      expect(@yc.package_repository("zisofs-tools", "pretend", nil)).to eq(nil)
-      expect(@yc.package_repository("zisofs-tools", "pretend")).to eq(nil)
-      expect(@yc.package_repository("pretend", "1.0.6-3.2.2")).to eq(nil)
-    end
-  end
-
-  describe "reset" do
-    it "should empty the installed and available packages RPMDb" do
-      expect(@yc.available_version("zip", "x86_64")).to eq("2.31-2.el5")
-      expect(@yc.installed_version("zip", "x86_64")).to eq("2.31-2.el5")
-      @yc.reset
-      expect(@yc.available_version("zip", "x86_64")).to eq(nil)
-      expect(@yc.installed_version("zip", "x86_64")).to eq(nil)
-    end
-  end
-
-  describe "package_available?" do
-    it "should return true a package name is available" do
-      expect(@yc.package_available?("zisofs-tools")).to eq(true)
-      expect(@yc.package_available?("moo")).to eq(false)
-      expect(@yc.package_available?(nil)).to eq(false)
-    end
-
-    it "should return true a package name + arch is available" do
-      expect(@yc.package_available?("zlib-devel.i386")).to eq(true)
-      expect(@yc.package_available?("zisofs-tools.x86_64")).to eq(true)
-      expect(@yc.package_available?("znc-test.beta1.x86_64")).to eq(true)
-      expect(@yc.package_available?("znc-test.beta1")).to eq(true)
-      expect(@yc.package_available?("znc-test.test.beta1")).to eq(true)
-      expect(@yc.package_available?("moo.i386")).to eq(false)
-      expect(@yc.package_available?("zisofs-tools.beta")).to eq(false)
-      expect(@yc.package_available?("znc-test.test")).to eq(false)
-    end
-  end
-
-  describe "enable_extra_repo_control" do
-    it "should set @extra_repo_control to arg" do
-      @yc.enable_extra_repo_control("--enablerepo=test")
-      expect(@yc.extra_repo_control).to eq("--enablerepo=test")
-    end
-
-    it "should call reload once when set to flag cache for update" do
-      expect(@yc).to receive(:reload).once
-      @yc.enable_extra_repo_control("--enablerepo=test")
-      @yc.enable_extra_repo_control("--enablerepo=test")
-    end
-  end
-
-  describe "disable_extra_repo_control" do
-    it "should set @extra_repo_control to nil" do
-      @yc.enable_extra_repo_control("--enablerepo=test")
-      @yc.disable_extra_repo_control
-      expect(@yc.extra_repo_control).to eq(nil)
-    end
-
-    it "should call reload once when cleared to flag cache for update" do
-      expect(@yc).to receive(:reload).once
-      @yc.enable_extra_repo_control("--enablerepo=test")
-      expect(@yc).to receive(:reload).once
-      @yc.disable_extra_repo_control
-      @yc.disable_extra_repo_control
-    end
-  end
-
-end
-
-describe "Chef::Provider::Package::Yum - Multi" do
-  before(:each) do
-    @node = Chef::Node.new
-    @events = Chef::EventDispatch::Dispatcher.new
-    @run_context = Chef::RunContext.new(@node, {}, @events)
-    @new_resource = Chef::Resource::YumPackage.new(%w{cups vim})
-    @status = double("Status", :exitstatus => 0)
-    @yum_cache = double(
-      "Chef::Provider::Yum::YumCache",
-      :reload_installed => true,
-      :reset => true,
-      :installed_version => "XXXX",
-      :candidate_version => "YYYY",
-      :package_available? => true,
-      :version_available? => true,
-      :allow_multi_install => [ "kernel" ],
-      :package_repository => "base",
-      :disable_extra_repo_control => true
-    )
-    allow(Chef::Provider::Package::Yum::YumCache).to receive(:instance).and_return(@yum_cache)
-    allow(@yum_cache).to receive(:yum_binary=).with("yum")
-    allow(@yum_cache).to receive(:yum_binary=).with("yum")
-    @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-    @pid = double("PID")
-  end
-
-  describe "when evaluating the correctness of the resource" do
-    it "raises an error if the array lengths of package name, arch, and version do not match up" do
-      @new_resource.version(["1.1"])
-      @new_resource.arch(%w{x86_64 i386 i686})
-      expect { @provider.check_resource_semantics! }.to raise_error(Chef::Exceptions::InvalidResourceSpecification)
-    end
-  end
-
-  describe "when loading the current system state" do
-    it "should create a current resource with the name of the new_resource" do
-      @provider.load_current_resource
-      expect(@provider.current_resource.name).to eq("cups, vim")
-    end
-
-    it "should set the current resources package name to the new resources package name" do
-      @provider.load_current_resource
-      expect(@provider.current_resource.package_name).to eq(%w{cups vim})
-    end
-
-    it "should set the installed version to nil on the current resource if no installed package" do
-      allow(@yum_cache).to receive(:installed_version).and_return(nil)
-      @provider.load_current_resource
-      expect(@provider.current_resource.version).to eq([nil, nil])
-    end
-
-    it "should set the installed version if yum has one" do
-      allow(@yum_cache).to receive(:installed_version).with("cups", nil).and_return("1.2.4-11.18.el5")
-      allow(@yum_cache).to receive(:installed_version).with("vim", nil).and_return("1.0")
-      allow(@yum_cache).to receive(:candidate_version).with("cups", nil).and_return("1.2.4-11.18.el5_2.3")
-      allow(@yum_cache).to receive(:candidate_version).with("vim", nil).and_return("1.5")
-      @provider.load_current_resource
-      expect(@provider.current_resource.version).to eq(["1.2.4-11.18.el5", "1.0"])
-    end
-
-    it "should set the candidate version if yum info has one" do
-      allow(@yum_cache).to receive(:installed_version).with("cups", nil).and_return("1.2.4-11.18.el5")
-      allow(@yum_cache).to receive(:installed_version).with("vim", nil).and_return("1.0")
-      allow(@yum_cache).to receive(:candidate_version).with("cups", nil).and_return("1.2.4-11.18.el5_2.3")
-      allow(@yum_cache).to receive(:candidate_version).with("vim", nil).and_return("1.5")
-      @provider.load_current_resource
-      expect(@provider.candidate_version).to eql(["1.2.4-11.18.el5_2.3", "1.5"])
-    end
-
-    it "should return the current resouce" do
-      expect(@provider.load_current_resource).to eql(@provider.current_resource)
-    end
-
-    describe "when version constraint in package_name" do
-      it "should set package_version if no existing package_name is found and new_package_name is available" do
-        @new_resource = Chef::Resource::YumPackage.new(["cups = 1.2.4-11.18.el5_2.3", "emacs = 24.4"])
-        @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-        allow(@yum_cache).to receive(:package_available?) { |pkg| %w{cups emacs}.include?(pkg) ? true : false }
-        allow(@yum_cache).to receive(:candidate_version) do |pkg|
-          if pkg == "cups"
-            "1.2.4-11.18.el5_2.3"
-          elsif pkg == "emacs"
-            "24.4"
-          end
-        end
-        allow(@yum_cache).to receive(:packages_from_require) do |pkg|
-          if pkg.name == "cups"
-            [Chef::Provider::Package::Yum::RPMDbPackage.new("cups", "1.2.4-11.18.el5_2.3", "noarch", [], false, true, "base")]
-          elsif pkg.name == "emacs"
-            [Chef::Provider::Package::Yum::RPMDbPackage.new("emacs", "24.4", "noarch", [], false, true, "base")]
-          end
-        end
-        expect(Chef::Log).to receive(:debug).exactly(2).times.with(%r{matched 1 package,})
-        expect(Chef::Log).to receive(:debug).exactly(1).times.with(%r{candidate version: 1.2.4-11.18.el5_2.3})
-        expect(Chef::Log).to receive(:debug).exactly(1).times.with(%r{candidate version: 24.4})
-        expect(Chef::Log).to receive(:debug).at_least(2).times.with(%r{checking yum info})
-        @provider.load_current_resource
-        expect(@provider.new_resource.package_name).to eq(%w{cups emacs})
-        expect(@provider.new_resource.version).to eq(["1.2.4-11.18.el5_2.3", "24.4"])
-        expect(@provider.send(:package_name_array)).to eq(%w{cups emacs})
-        expect(@provider.send(:new_version_array)).to eq(["1.2.4-11.18.el5_2.3", "24.4"])
-      end
-    end
-  end
-
-  describe "when installing a package" do
-    it "should run yum install with the package name and version" do
-      @provider.load_current_resource
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      allow(@yum_cache).to receive(:installed_version).with("cups", nil).and_return("1.2.4-11.18.el5")
-      allow(@yum_cache).to receive(:installed_version).with("vim", nil).and_return("0.9")
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.19.el5 vim-1.0"
-      )
-      @provider.install_package(%w{cups vim}, ["1.2.4-11.19.el5", "1.0"])
-    end
-
-    it "should not run yum install with nil package name" do
-      @provider.load_current_resource
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.19.el5"
-      )
-      @provider.install_package(["cups", nil], ["1.2.4-11.19.el5", nil])
-    end
-
-    it "should run yum install with the package name, version and arch" do
-      @provider.load_current_resource
-      @new_resource.arch(%w{i386 i386})
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.19.el5.i386 vim-1.0.i386"
-      )
-      @provider.install_package(%w{cups vim}, ["1.2.4-11.19.el5", "1.0"])
-    end
-
-    it "installs the package with the options given in the resource" do
-      @provider.load_current_resource
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-      allow(@yum_cache).to receive(:installed_version).with("cups", nil).and_return("1.2.4-11.18.el5")
-      allow(@yum_cache).to receive(:installed_version).with("vim", nil).and_return("0.9")
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y --disablerepo epmd install cups-1.2.4-11.19.el5 vim-1.0"
-      )
-      @new_resource.options("--disablerepo epmd")
-      @provider.install_package(%w{cups vim}, ["1.2.4-11.19.el5", "1.0"])
-    end
-
-    it "should run yum install with the package name and version when name has arch" do
-      @new_resource = Chef::Resource::YumPackage.new(["cups.x86_64", "vim"])
-      @provider = Chef::Provider::Package::Yum.new(@new_resource, @run_context)
-      allow(Chef::Provider::Package::Yum::RPMUtils).to receive(:rpmvercmp).and_return(-1)
-
-      # Inside of load_current_resource() we'll call parse_arch for cups,
-      # and we need to craft the right response. The default mock setup above
-      # will just return valid versions all the time which won't work for this
-      # test.
-      allow(@yum_cache).to receive(:installed_version).with("cups", "x86_64").and_return("XXXX")
-      allow(@yum_cache).to receive(:candidate_version).with("cups", "x86_64").and_return("1.2.4-11.18.el5")
-      allow(@yum_cache).to receive(:installed_version).with("cups.x86_64").and_return(nil)
-      allow(@yum_cache).to receive(:candidate_version).with("cups.x86_64").and_return(nil)
-
-      # Normal mock's for the idempotency check
-      allow(@yum_cache).to receive(:installed_version).with("cups", nil).and_return("1.2.4-11.18.el5")
-      allow(@yum_cache).to receive(:installed_version).with("vim", nil).and_return("0.9")
-
-      @provider.load_current_resource
-      expect(@provider).to receive(:yum_command).with(
-        "-d0 -e0 -y install cups-1.2.4-11.19.el5.x86_64 vim-1.0"
-      )
-      @provider.install_package(%w{cups vim}, ["1.2.4-11.19.el5", "1.0"])
-    end
-
-  end
-end