chef 18.8.11 → 18.8.54

data/spec/unit/provider/package_spec.rb

@@ -1,791 +0,0 @@
-#
-# Author:: Adam Jacob (<adam@chef.io>)
-# Copyright:: Copyright (c) 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"
-
-describe Chef::Provider::Package do
-  let(:node) do
-    node = Chef::Node.new
-    node.automatic_attrs[:platform] = :just_testing
-    node.automatic_attrs[:platform_version] = :just_testing
-    node
-  end
-  let(:events) { Chef::EventDispatch::Dispatcher.new }
-  let(:logger) { double("Mixlib::Log::Child").as_null_object }
-  let(:run_context) { Chef::RunContext.new(node, {}, events) }
-  let(:new_resource) { Chef::Resource::Package.new("install emacs") }
-  let(:current_resource) { Chef::Resource::Package.new("install emacs") }
-  let(:candidate_version) { "1.0" }
-  let(:provider) do
-    new_resource.package_name = "emacs"
-    current_resource.package_name = "emacs"
-    provider = Chef::Provider::Package.new(new_resource, run_context)
-    provider.current_resource = current_resource
-    provider.candidate_version = candidate_version
-    provider
-  end
-
-  before do
-    allow(run_context).to receive(:logger).and_return(logger)
-  end
-
-  describe "when installing a package" do
-    before(:each) do
-      provider.current_resource = current_resource
-      allow(provider).to receive(:install_package).and_return(true)
-    end
-
-    it "raises a Chef::Exceptions::InvalidResourceSpecification if both multipackage and source are provided" do
-      new_resource.package_name(%w{a b})
-      new_resource.source("foo")
-      expect { provider.run_action(:install) }.to raise_error(Chef::Exceptions::InvalidResourceSpecification)
-    end
-
-    it "should raise a Chef::Exceptions::Package if no version is specified, and no candidate is available" do
-      provider.candidate_version = nil
-      expect { provider.run_action(:install) }.to raise_error(Chef::Exceptions::Package)
-    end
-
-    it "should install the package at the candidate_version if it is not already installed" do
-      expect(provider).to receive(:install_package).with(
-        new_resource.package_name,
-        provider.candidate_version
-      ).and_return(true)
-      provider.run_action(:install)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should install the package at the version specified if it is not already installed" do
-      new_resource.version("1.0")
-      expect(provider).to receive(:install_package).with(
-        new_resource.package_name,
-        new_resource.version
-      ).and_return(true)
-      provider.run_action(:install)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should install the package at the version specified if a different version is installed" do
-      new_resource.version("1.0")
-      allow(current_resource).to receive(:version).and_return("0.99")
-      expect(provider).to receive(:install_package).with(
-        new_resource.package_name,
-        new_resource.version
-      ).and_return(true)
-      provider.run_action(:install)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not install the package if it is already installed and no version is specified" do
-      current_resource.version("1.0")
-      expect(provider).not_to receive(:install_package)
-      provider.run_action(:install)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should not install the package if it is already installed at the version specified" do
-      current_resource.version("1.0")
-      new_resource.version("1.0")
-      expect(provider).not_to receive(:install_package)
-      provider.run_action(:install)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should call the candidate_version accessor only once if the package is already installed and no version is specified" do
-      current_resource.version("1.0")
-      allow(provider).to receive(:candidate_version).and_return("1.0")
-      provider.run_action(:install)
-    end
-
-    it "should call the candidate_version accessor only once if the package is already installed at the version specified" do
-      current_resource.version("1.0")
-      new_resource.version("1.0")
-      provider.run_action(:install)
-    end
-
-    it "should set the resource to updated if it installs the package" do
-      provider.run_action(:install)
-      expect(new_resource).to be_updated
-    end
-
-  end
-
-  describe "when upgrading the package" do
-    before(:each) do
-      allow(provider).to receive(:upgrade_package).and_return(true)
-    end
-
-    it "should upgrade the package if the current version is not the candidate version" do
-      expect(provider).to receive(:upgrade_package).with(
-        new_resource.package_name,
-        provider.candidate_version
-      ).and_return(true)
-      provider.run_action(:upgrade)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should set the resource to updated if it installs the package" do
-      provider.run_action(:upgrade)
-      expect(new_resource).to be_updated
-    end
-
-    it "should not install the package if the current version is the candidate version" do
-      current_resource.version "1.0"
-      expect(provider).not_to receive(:upgrade_package)
-      provider.run_action(:upgrade)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should print the word 'uninstalled' if there was no original version" do
-      allow(current_resource).to receive(:version).and_return(nil)
-      expect(logger).to receive(:info).with("package[install emacs] upgraded(allow_downgrade) emacs to 1.0")
-      provider.run_action(:upgrade)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should raise a Chef::Exceptions::Package if current version and candidate are nil" do
-      allow(current_resource).to receive(:version).and_return(nil)
-      provider.candidate_version = nil
-      expect { provider.run_action(:upgrade) }.to raise_error(Chef::Exceptions::Package)
-    end
-
-    it "should not install the package if candidate version is nil" do
-      current_resource.version "1.0"
-      provider.candidate_version = nil
-      expect(provider).not_to receive(:upgrade_package)
-      provider.run_action(:upgrade)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-  end
-
-  describe "When removing the package" do
-    before(:each) do
-      allow(provider).to receive(:remove_package).and_return(true)
-      current_resource.version "1.4.2"
-    end
-
-    it "should remove the package if it is installed" do
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:remove_package).with("emacs", nil)
-      provider.run_action(:remove)
-      expect(new_resource).to be_updated
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should remove the package at a specific version if it is installed at that version" do
-      new_resource.version "1.4.2"
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:remove_package).with("emacs", "1.4.2")
-      provider.run_action(:remove)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not remove the package at a specific version if it is not installed at that version" do
-      new_resource.version "1.0"
-      expect(provider).not_to be_removing_package
-      expect(provider).not_to receive(:remove_package)
-      provider.run_action(:remove)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should not remove the package if it is not installed" do
-      expect(provider).not_to receive(:remove_package)
-      allow(current_resource).to receive(:version).and_return(nil)
-      provider.run_action(:remove)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should set the resource to updated if it removes the package" do
-      provider.run_action(:remove)
-      expect(new_resource).to be_updated
-    end
-
-  end
-
-  describe "When purging the package" do
-    before(:each) do
-      allow(provider).to receive(:purge_package).and_return(true)
-      current_resource.version "1.4.2"
-    end
-
-    it "should purge the package if it is installed" do
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:purge_package).with("emacs", nil)
-      provider.run_action(:purge)
-      expect(new_resource).to be_updated
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should purge the package at a specific version if it is installed at that version" do
-      new_resource.version "1.4.2"
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:purge_package).with("emacs", "1.4.2")
-      provider.run_action(:purge)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not purge the package at a specific version if it is not installed at that version" do
-      new_resource.version "1.0"
-      expect(provider).not_to be_removing_package
-      expect(provider).not_to receive(:purge_package)
-      provider.run_action(:purge)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should not purge the package if it is not installed" do
-      current_resource.instance_variable_set(:@version, nil)
-      expect(provider).not_to be_removing_package
-
-      expect(provider).not_to receive(:purge_package)
-      provider.run_action(:purge)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should set the resource to updated if it purges the package" do
-      provider.run_action(:purge)
-      expect(new_resource).to be_updated
-    end
-
-  end
-
-  describe "When locking the package" do
-    before(:each) do
-      allow(provider).to receive(:lock_package).with(
-        new_resource.package_name,
-        nil
-      ).and_return(true)
-    end
-
-    it "should lock the package if it is unlocked" do
-      allow(provider).to receive(:package_locked).with(
-        new_resource.package_name,
-        nil
-      ).and_return(false)
-      provider.run_action(:lock)
-    end
-
-    it "should not lock the package if it is already locked" do
-      allow(provider).to receive(:package_locked).with(
-        new_resource.package_name,
-        nil
-      ).and_return(true)
-      expect(provider).not_to receive(:lock_package)
-      provider.run_action(:lock)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should set the resource to updated if it locks the package" do
-      allow(provider).to receive(:package_locked).with(
-        new_resource.package_name,
-        nil
-      ).and_return(false)
-      provider.run_action(:lock)
-      expect(new_resource).to be_updated
-    end
-  end
-
-  describe "When unlocking the package" do
-    before(:each) do
-      allow(provider).to receive(:unlock_package).and_return(true)
-    end
-
-    it "should unlock the package if it is locked" do
-      allow(provider).to receive(:package_locked).with(
-        new_resource.package_name,
-        nil
-      ).and_return(true)
-      expect(provider).to receive(:unlock_package)
-      provider.run_action(:unlock)
-    end
-
-    it "should not unlock the package if it is already unlocked" do
-      allow(provider).to receive(:package_locked).with(
-        new_resource.package_name,
-        nil
-      ).and_return(false)
-      expect(provider).not_to receive(:unlock_package)
-      provider.run_action(:unlock)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should set the resource to updated if it unlocks the package" do
-      allow(provider).to receive(:package_locked).with(
-        new_resource.package_name,
-        nil
-      ).and_return(true)
-      provider.run_action(:unlock)
-      expect(new_resource).to be_updated
-    end
-  end
-
-  describe "when running commands to be implemented by subclasses" do
-    it "should raises UnsupportedAction for install" do
-      expect { provider.install_package("emacs", "1.4.2") }.to raise_error(Chef::Exceptions::UnsupportedAction)
-    end
-
-    it "should raises UnsupportedAction for upgrade" do
-      expect { provider.upgrade_package("emacs", "1.4.2") }.to raise_error(Chef::Exceptions::UnsupportedAction)
-    end
-
-    it "should raises UnsupportedAction for remove" do
-      expect { provider.remove_package("emacs", "1.4.2") }.to raise_error(Chef::Exceptions::UnsupportedAction)
-    end
-
-    it "should raises UnsupportedAction for purge" do
-      expect { provider.purge_package("emacs", "1.4.2") }.to raise_error(Chef::Exceptions::UnsupportedAction)
-    end
-
-    it "should raise UnsupportedAction for preseed_package" do
-      preseed_file = "/tmp/sun-jdk-package-preseed-file.seed"
-      expect { provider.preseed_package(preseed_file) }.to raise_error(Chef::Exceptions::UnsupportedAction)
-    end
-
-    it "should raise UnsupportedAction for reconfig" do
-      expect { provider.reconfig_package("emacs") }.to raise_error(Chef::Exceptions::UnsupportedAction)
-    end
-
-    it "should raise UnsupportedAction for lock" do
-      expect { provider.lock_package("emacs", nil) }.to raise_error(Chef::Exceptions::UnsupportedAction)
-    end
-
-    it "should raise UnsupportedAction for unlock" do
-      expect { provider.unlock_package("emacs", nil) }.to raise_error(Chef::Exceptions::UnsupportedAction)
-    end
-  end
-end
-
-describe "Subclass with use_multipackage_api" do
-  class MyPackageResource < Chef::Resource::Package
-  end
-
-  class MyPackageProvider < Chef::Provider::Package
-    use_multipackage_api
-  end
-
-  let(:node) { Chef::Node.new }
-  let(:events) { Chef::EventDispatch::Dispatcher.new }
-  let(:run_context) { Chef::RunContext.new(node, {}, events) }
-  let(:new_resource) { MyPackageResource.new("installs the packages") }
-  let(:current_resource) { MyPackageResource.new("installs the packages") }
-  let(:provider) do
-    provider = MyPackageProvider.new(new_resource, run_context)
-    provider.current_resource = current_resource
-    provider
-  end
-
-  it "has use_multipackage_api? methods on the class and instance" do
-    expect(MyPackageProvider.use_multipackage_api?).to be true
-    expect(provider.use_multipackage_api?).to be true
-  end
-
-  it "when user passes string to package_name, passes arrays to install_package" do
-    new_resource.package_name "vim"
-    new_resource.version nil
-    provider.candidate_version = [ "1.0" ]
-    expect(provider).to receive(:install_package).with(
-      [ "vim" ],
-      [ "1.0" ]
-    ).and_return(true)
-    provider.run_action(:install)
-    expect(new_resource).to be_updated_by_last_action
-    expect(new_resource.version).to eql(nil)
-  end
-
-  it "when user pases string to package_name and version, passes arrays to install_package" do
-    new_resource.package_name "vim"
-    new_resource.version "1.0"
-    provider.candidate_version = [ "1.0" ]
-    expect(provider).to receive(:install_package).with(
-      [ "vim" ],
-      [ "1.0" ]
-    ).and_return(true)
-    provider.run_action(:install)
-    expect(new_resource).to be_updated_by_last_action
-    expect(new_resource.version).to eql("1.0")
-  end
-
-  it "when user passes string to package_name, passes arrays to upgrade_package" do
-    new_resource.package_name "vim"
-    new_resource.version nil
-    provider.candidate_version = [ "1.0" ]
-    expect(provider).to receive(:upgrade_package).with(
-      [ "vim" ],
-      [ "1.0" ]
-    ).and_return(true)
-    provider.run_action(:upgrade)
-    expect(new_resource).to be_updated_by_last_action
-    expect(new_resource.version).to eql(nil)
-  end
-
-  it "when user pases string to package_name and version, passes arrays to upgrade_package" do
-    new_resource.package_name "vim"
-    new_resource.version "1.0"
-    provider.candidate_version = [ "1.0" ]
-    expect(provider).to receive(:upgrade_package).with(
-      [ "vim" ],
-      [ "1.0" ]
-    ).and_return(true)
-    provider.run_action(:upgrade)
-    expect(new_resource).to be_updated_by_last_action
-    expect(new_resource.version).to eql("1.0")
-  end
-
-  it "when user passes string to package_name, passes arrays to remove_package" do
-    new_resource.package_name "vim"
-    current_resource.package_name "vim"
-    current_resource.version [ "1.0" ]
-    provider.candidate_version = [ "1.0" ]
-    expect(provider).to receive(:remove_package).with(
-      [ "vim" ],
-      [ nil ]
-    ).and_return(true)
-    provider.run_action(:remove)
-    expect(new_resource).to be_updated_by_last_action
-    expect(new_resource.version).to eql(nil)
-  end
-
-  it "when user passes string to package_name, passes arrays to purge_package" do
-    new_resource.package_name "vim"
-    current_resource.package_name "vim"
-    current_resource.version [ "1.0" ]
-    provider.candidate_version = [ "1.0" ]
-    expect(provider).to receive(:purge_package).with(
-      [ "vim" ],
-      [ nil ]
-    ).and_return(true)
-    provider.run_action(:purge)
-    expect(new_resource).to be_updated_by_last_action
-    expect(new_resource.version).to eql(nil)
-  end
-end
-
-describe "Chef::Provider::Package - Multi" do
-  let(:node) { Chef::Node.new }
-  let(:events) { Chef::EventDispatch::Dispatcher.new }
-  let(:run_context) { Chef::RunContext.new(node, {}, events) }
-  let(:new_resource) { Chef::Resource::Package.new(%w{emacs vi}) }
-  let(:current_resource) { Chef::Resource::Package.new(%w{emacs vi}) }
-  let(:candidate_version) { [ "1.0", "6.2" ] }
-  class MyPackageProvider < Chef::Provider::Package
-    use_multipackage_api
-  end
-  let(:provider) do
-    provider = MyPackageProvider.new(new_resource, run_context)
-    provider.current_resource = current_resource
-    provider.candidate_version = candidate_version
-    provider
-  end
-
-  describe "when installing multiple packages" do
-    before(:each) do
-      provider.current_resource = current_resource
-      allow(provider).to receive(:install_package).and_return(true)
-    end
-
-    it "installs the candidate versions when none are installed" do
-      expect(provider).to receive(:install_package).with(
-        %w{emacs vi},
-        ["1.0", "6.2"]
-      ).and_return(true)
-      provider.run_action(:install)
-      expect(new_resource).to be_updated
-    end
-
-    it "installs the candidate versions when some are installed" do
-      expect(provider).to receive(:install_package).with(
-        [ "vi" ],
-        [ "6.2" ]
-      ).and_return(true)
-      current_resource.version(["1.0", nil])
-      provider.run_action(:install)
-      expect(new_resource).to be_updated
-    end
-
-    it "installs the specified version when some are out of date" do
-      current_resource.version(["1.0", "6.1"])
-      new_resource.version(["1.0", "6.2"])
-      provider.run_action(:install)
-      expect(new_resource).to be_updated
-    end
-
-    it "does not install any version if all are installed at the right version" do
-      current_resource.version(["1.0", "6.2"])
-      new_resource.version(["1.0", "6.2"])
-      provider.run_action(:install)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "does install an older version by default" do
-      current_resource.version(["1.1", "6.2"])
-      new_resource.version(["1.0", "6.1"])
-      provider.run_action(:install)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "does not install an older version if the resource subclass has allow_downgrade set to false" do
-      allow(new_resource).to receive(:allow_downgrade).and_return(false)
-      current_resource.version(["1.1", "6.2"])
-      new_resource.version(["1.0", "6.1"])
-      provider.run_action(:install)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "does not install any version if all are installed, and no version was specified" do
-      current_resource.version(["1.0", "6.2"])
-      provider.run_action(:install)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "raises an exception if both are not installed and no candidates are available" do
-      current_resource.version([nil, nil])
-      provider.candidate_version = [nil, nil]
-      expect { provider.run_action(:install) }.to raise_error(Chef::Exceptions::Package)
-    end
-
-    it "raises an exception if one is not installed and no candidates are available" do
-      current_resource.version(["1.0", nil])
-      provider.candidate_version = ["1.0", nil]
-      expect { provider.run_action(:install) }.to raise_error(Chef::Exceptions::Package)
-    end
-
-    it "does not raise an exception if the packages are installed or have a candidate" do
-      current_resource.version(["1.0", nil])
-      provider.candidate_version = [nil, "6.2"]
-      expect { provider.run_action(:install) }.not_to raise_error
-    end
-
-    it "raises an exception if an explicit version is asked for, an old version is installed, but no candidate" do
-      new_resource.version ["1.0", "6.2"]
-      current_resource.version(["1.0", "6.1"])
-      provider.candidate_version = ["1.0", nil]
-      expect { provider.run_action(:install) }.to raise_error(Chef::Exceptions::Package)
-    end
-
-    it "does not raise an exception if an explicit version is asked for, and is installed, but no candidate" do
-      new_resource.version ["1.0", "6.2"]
-      current_resource.version(["1.0", "6.2"])
-      provider.candidate_version = ["1.0", nil]
-      expect { provider.run_action(:install) }.not_to raise_error
-    end
-
-    it "raise an exception if an explicit version is asked for, and is not installed, and no candidate" do
-      new_resource.version ["1.0", "6.2"]
-      current_resource.version(["1.0", nil])
-      provider.candidate_version = ["1.0", nil]
-      expect { provider.run_action(:install) }.to raise_error(Chef::Exceptions::Package)
-    end
-
-    it "does not raise an exception if an explicit version is asked for, and is not installed, and there is a candidate" do
-      new_resource.version ["1.0", "6.2"]
-      current_resource.version(["1.0", nil])
-      provider.candidate_version = ["1.0", "6.2"]
-      expect { provider.run_action(:install) }.not_to raise_error
-    end
-  end
-
-  describe "when upgrading multiple packages" do
-    before(:each) do
-      provider.current_resource = current_resource
-      allow(provider).to receive(:upgrade_package).and_return(true)
-    end
-
-    it "should upgrade the package if the current versions are not the candidate version" do
-      current_resource.version ["0.9", "6.1"]
-      expect(provider).to receive(:upgrade_package).with(
-        new_resource.package_name,
-        provider.candidate_version
-      ).and_return(true)
-      provider.run_action(:upgrade)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should upgrade the package if some of current versions are not the candidate versions" do
-      current_resource.version ["1.0", "6.1"]
-      expect(provider).to receive(:upgrade_package).with(
-        ["vi"],
-        ["6.2"]
-      ).and_return(true)
-      provider.run_action(:upgrade)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not install the package if the current versions are the candidate version" do
-      current_resource.version ["1.0", "6.2"]
-      expect(provider).not_to receive(:upgrade_package)
-      provider.run_action(:upgrade)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should raise an exception if both are not installed and no candidates are available" do
-      current_resource.version([nil, nil])
-      provider.candidate_version = [nil, nil]
-      expect { provider.run_action(:upgrade) }.to raise_error(Chef::Exceptions::Package)
-    end
-
-    it "should raise an exception if one is not installed and no candidates are available" do
-      current_resource.version(["1.0", nil])
-      provider.candidate_version = ["1.0", nil]
-      expect { provider.run_action(:upgrade) }.to raise_error(Chef::Exceptions::Package)
-    end
-
-    it "should not raise an exception if the packages are installed or have a candidate" do
-      current_resource.version(["1.0", nil])
-      provider.candidate_version = [nil, "6.2"]
-      expect { provider.run_action(:upgrade) }.not_to raise_error
-    end
-
-    it "should not raise an exception if the packages are installed or have a candidate" do
-      current_resource.version(["1.0", nil])
-      provider.candidate_version = [nil, "6.2"]
-      expect { provider.run_action(:upgrade) }.not_to raise_error
-    end
-  end
-
-  describe "When removing multiple packages " do
-    before(:each) do
-      allow(provider).to receive(:remove_package).and_return(true)
-      current_resource.version ["1.0", "6.2"]
-    end
-
-    it "should remove the packages if all are installed" do
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:remove_package).with(%w{emacs vi}, [nil])
-      provider.run_action(:remove)
-      expect(new_resource).to be_updated
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should remove the packages if some are installed" do
-      current_resource.version ["1.0", nil]
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:remove_package).with(%w{emacs vi}, [nil])
-      provider.run_action(:remove)
-      expect(new_resource).to be_updated
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should remove the packages at a specific version if they are installed at that version" do
-      new_resource.version ["1.0", "6.2"]
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:remove_package).with(%w{emacs vi}, ["1.0", "6.2"])
-      provider.run_action(:remove)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should remove the packages at a specific version any are is installed at that version" do
-      new_resource.version ["0.5", "6.2"]
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:remove_package).with(%w{emacs vi}, ["0.5", "6.2"])
-      provider.run_action(:remove)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not remove the packages at a specific version if they are not installed at that version" do
-      new_resource.version ["0.5", "6.0"]
-      expect(provider).not_to be_removing_package
-      expect(provider).not_to receive(:remove_package)
-      provider.run_action(:remove)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should not remove the packages if they are not installed" do
-      expect(provider).not_to receive(:remove_package)
-      allow(current_resource).to receive(:version).and_return(nil)
-      provider.run_action(:remove)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-  end
-
-  describe "When purging multiple packages " do
-    before(:each) do
-      allow(provider).to receive(:purge_package).and_return(true)
-      current_resource.version ["1.0", "6.2"]
-    end
-
-    it "should purge the packages if all are installed" do
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:purge_package).with(%w{emacs vi}, [nil])
-      provider.run_action(:purge)
-      expect(new_resource).to be_updated
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should purge the packages if some are installed" do
-      current_resource.version ["1.0", nil]
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:purge_package).with(%w{emacs vi}, [nil])
-      provider.run_action(:purge)
-      expect(new_resource).to be_updated
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should purge the packages at a specific version if they are installed at that version" do
-      new_resource.version ["1.0", "6.2"]
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:purge_package).with(%w{emacs vi}, ["1.0", "6.2"])
-      provider.run_action(:purge)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should purge the packages at a specific version any are is installed at that version" do
-      new_resource.version ["0.5", "6.2"]
-      expect(provider).to be_removing_package
-      expect(provider).to receive(:purge_package).with(%w{emacs vi}, ["0.5", "6.2"])
-      provider.run_action(:purge)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not purge the packages at a specific version if they are not installed at that version" do
-      new_resource.version ["0.5", "6.0"]
-      expect(provider).not_to be_removing_package
-      expect(provider).not_to receive(:purge_package)
-      provider.run_action(:purge)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should not purge the packages if they are not installed" do
-      expect(provider).not_to receive(:purge_package)
-      allow(current_resource).to receive(:version).and_return(nil)
-      provider.run_action(:purge)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-  end
-
-  describe "version_compare" do
-    it "tests equality" do
-      expect(provider.version_compare("1.3", "1.3")).to eql(0)
-    end
-
-    it "tests less than" do
-      expect(provider.version_compare("1.2", "1.3")).to eql(-1)
-    end
-
-    it "tests greater than" do
-      expect(provider.version_compare("1.5", "1.3")).to eql(1)
-    end
-
-    it "x.10 is greater than x.2 (so does not do floating point comparisons)" do
-      expect(provider.version_compare("1.10", "1.2")).to eql(1)
-    end
-
-    it "sanitizes inputs" do
-      expect(provider.version_compare("1.3_3", "1.3")).to eql(0)
-    end
-  end
-end