puppet 4.5.3-x86-mingw32 → 4.6.1-x86-mingw32

data/spec/integration/util/windows/security_spec.rb

@@ -12,11 +12,16 @@ describe "Puppet::Util::Windows::Security", :if => Puppet.features.microsoft_win
   include PuppetSpec::Files
 
   before :all do
+
+    # necessary for localized name of guests
+    guests_name = Puppet::Util::Windows::SID.sid_to_name('S-1-5-32-546')
+    guests = Puppet::Util::Windows::ADSI::Group.new(guests_name)
+
     @sids = {
       :current_user => Puppet::Util::Windows::SID.name_to_sid(Puppet::Util::Windows::ADSI::User.current_user_name),
       :system => Puppet::Util::Windows::SID::LocalSystem,
       :administrators => Puppet::Util::Windows::SID::BuiltinAdministrators,
-      :guest => Puppet::Util::Windows::SID.name_to_sid("Guest"),
+      :guest => Puppet::Util::Windows::SID.name_to_sid(guests.members[0]),
       :users => Puppet::Util::Windows::SID::BuiltinUsers,
       :power_users => Puppet::Util::Windows::SID::PowerUsers,
       :none => Puppet::Util::Windows::SID::Nobody,
@@ -132,7 +137,10 @@ describe "Puppet::Util::Windows::Security", :if => Puppet.features.microsoft_win
           end
 
           it "should raise an exception when setting to a different user" do
-            expect { winsec.set_owner(sids[:guest], path) }.to raise_error(Puppet::Error, /This security ID may not be assigned as the owner of this object./)
+            expect { winsec.set_owner(sids[:guest], path) }.to raise_error do |error|
+              expect(error).to be_a(Puppet::Util::Windows::Error)
+              expect(error.code).to eq(1307) # ERROR_INVALID_OWNER
+            end
           end
         end
 
@@ -142,7 +150,10 @@ describe "Puppet::Util::Windows::Security", :if => Puppet.features.microsoft_win
           end
 
           it "should raise an exception if an invalid path is provided" do
-            expect { winsec.get_owner("c:\\doesnotexist.txt") }.to raise_error(Puppet::Error, /The system cannot find the file specified./)
+            expect { winsec.get_owner("c:\\doesnotexist.txt") }.to raise_error do |error|
+              expect(error).to be_a(Puppet::Util::Windows::Error)
+              expect(error.code).to eq(2) # ERROR_FILE_NOT_FOUND
+            end
           end
         end
 
@@ -164,7 +175,10 @@ describe "Puppet::Util::Windows::Security", :if => Puppet.features.microsoft_win
           end
 
           it "should raise an exception if an invalid path is provided" do
-            expect { winsec.get_group("c:\\doesnotexist.txt") }.to raise_error(Puppet::Error, /The system cannot find the file specified./)
+            expect { winsec.get_group("c:\\doesnotexist.txt") }.to raise_error do |error|
+              expect(error).to be_a(Puppet::Util::Windows::Error)
+              expect(error.code).to eq(2) # ERROR_FILE_NOT_FOUND
+            end
           end
         end
 
@@ -300,7 +314,10 @@ describe "Puppet::Util::Windows::Security", :if => Puppet.features.microsoft_win
           end
 
           it "should raise an exception if an invalid path is provided" do
-            expect { winsec.set_mode(sids[:guest], "c:\\doesnotexist.txt") }.to raise_error(Puppet::Error, /The system cannot find the file specified./)
+            expect { winsec.set_mode(sids[:guest], "c:\\doesnotexist.txt") }.to raise_error do |error|
+              expect(error).to be_a(Puppet::Util::Windows::Error)
+              expect(error.code).to eq(2) # ERROR_FILE_NOT_FOUND
+            end
           end
         end
 
@@ -344,7 +361,10 @@ describe "Puppet::Util::Windows::Security", :if => Puppet.features.microsoft_win
           end
 
           it "should raise an exception if an invalid path is provided" do
-            expect { winsec.get_mode("c:\\doesnotexist.txt") }.to raise_error(Puppet::Error, /The system cannot find the file specified./)
+            expect { winsec.get_mode("c:\\doesnotexist.txt") }.to raise_error do |error|
+              expect(error).to be_a(Puppet::Util::Windows::Error)
+              expect(error.code).to eq(2) # ERROR_FILE_NOT_FOUND
+            end
           end
         end
 
@@ -424,7 +444,10 @@ describe "Puppet::Util::Windows::Security", :if => Puppet.features.microsoft_win
           end
 
           it "should raise an exception if an invalid path is provided" do
-            expect { winsec.set_owner(sids[:guest], "c:\\doesnotexist.txt") }.to raise_error(Puppet::Error, /The system cannot find the file specified./)
+            expect { winsec.set_owner(sids[:guest], "c:\\doesnotexist.txt") }.to raise_error do |error|
+              expect(error).to be_a(Puppet::Util::Windows::Error)
+              expect(error.code).to eq(2) # ERROR_FILE_NOT_FOUND
+            end
           end
         end
 
@@ -463,7 +486,10 @@ describe "Puppet::Util::Windows::Security", :if => Puppet.features.microsoft_win
           end
 
           it "should raise an exception if an invalid path is provided" do
-            expect { winsec.set_group(sids[:guest], "c:\\doesnotexist.txt") }.to raise_error(Puppet::Error, /The system cannot find the file specified./)
+            expect { winsec.set_group(sids[:guest], "c:\\doesnotexist.txt") }.to raise_error do |error|
+              expect(error).to be_a(Puppet::Util::Windows::Error)
+              expect(error.code).to eq(2) # ERROR_FILE_NOT_FOUND
+            end
           end
         end
 

data/spec/integration/util/windows/user_spec.rb

@@ -106,13 +106,35 @@ describe "Puppet::Util::Windows::User", :if => Puppet.features.microsoft_windows
         expect(Puppet::Util::Windows::User.password_is?(username, bad_password)).to be_falsey
       end
 
-      it "should return false given an incorrect username and nil password" do
-        expect(Puppet::Util::Windows::User.password_is?(username, nil)).to be_falsey
-      end
-
       it "should return false given a nil username and an incorrect password" do
         expect(Puppet::Util::Windows::User.password_is?(nil, bad_password)).to be_falsey
       end
+
+      context "with a correct password" do
+        it "should return true even if account restrictions are in place " do
+          error = Puppet::Util::Windows::Error.new('', Puppet::Util::Windows::User::ERROR_ACCOUNT_RESTRICTION)
+          Puppet::Util::Windows::User.stubs(:logon_user).raises(error)
+          expect(Puppet::Util::Windows::User.password_is?(username, 'p@ssword')).to be(true)
+        end
+
+        it "should return true even for an account outside of logon hours" do
+          error = Puppet::Util::Windows::Error.new('', Puppet::Util::Windows::User::ERROR_INVALID_LOGON_HOURS)
+          Puppet::Util::Windows::User.stubs(:logon_user).raises(error)
+          expect(Puppet::Util::Windows::User.password_is?(username, 'p@ssword')).to be(true)
+        end
+
+        it "should return true even for an account not allowed to log into this workstation" do
+          error = Puppet::Util::Windows::Error.new('', Puppet::Util::Windows::User::ERROR_INVALID_WORKSTATION)
+          Puppet::Util::Windows::User.stubs(:logon_user).raises(error)
+          expect(Puppet::Util::Windows::User.password_is?(username, 'p@ssword')).to be(true)
+        end
+
+        it "should return true even for a disabled account" do
+          error = Puppet::Util::Windows::Error.new('', Puppet::Util::Windows::User::ERROR_ACCOUNT_DISABLED)
+          Puppet::Util::Windows::User.stubs(:logon_user).raises(error)
+          expect(Puppet::Util::Windows::User.password_is?(username, 'p@ssword')).to be(true)
+        end
+      end
     end
 
     describe "check_token_membership" do

data/spec/lib/puppet_spec/files.rb

@@ -28,7 +28,7 @@ module PuppetSpec::Files
   def self.tmpfile(name, dir = nil)
     # Generate a temporary file, just for the name...
     source = dir ? Tempfile.new(name, dir) : Tempfile.new(name)
-    path = source.path.encode(Encoding::UTF_8)
+    path = Puppet::FileSystem.expand_path(source.path.encode(Encoding::UTF_8))
     source.close!
 
     record_tmp(File.expand_path(path))
@@ -59,7 +59,7 @@ module PuppetSpec::Files
 
   def tmpdir(name) PuppetSpec::Files.tmpdir(name) end
   def self.tmpdir(name)
-    dir = Dir.mktmpdir(name).encode!(Encoding::UTF_8)
+    dir = Puppet::FileSystem.expand_path(Dir.mktmpdir(name).encode!(Encoding::UTF_8))
 
     record_tmp(dir)
 

data/spec/lib/puppet_spec/network.rb

@@ -5,20 +5,24 @@ require 'puppet/network/http/api/indirected_routes'
 require 'puppet/indirector_testing'
 
 module PuppetSpec::Network
-  def not_found_code
-    Puppet::Network::HTTP::Error::HTTPNotFoundError::CODE
+  def not_found_error
+    Puppet::Network::HTTP::Error::HTTPNotFoundError
   end
 
-  def not_acceptable_code
-    Puppet::Network::HTTP::Error::HTTPNotAcceptableError::CODE
+  def not_acceptable_error
+    Puppet::Network::HTTP::Error::HTTPNotAcceptableError
   end
 
-  def bad_request_code
-    Puppet::Network::HTTP::Error::HTTPBadRequestError::CODE
+  def bad_request_error
+    Puppet::Network::HTTP::Error::HTTPBadRequestError
   end
 
-  def not_authorized_code
-    Puppet::Network::HTTP::Error::HTTPNotAuthorizedError::CODE
+  def not_authorized_error
+    Puppet::Network::HTTP::Error::HTTPNotAuthorizedError
+  end
+
+  def method_not_allowed_error
+    Puppet::Network::HTTP::Error::HTTPMethodNotAllowedError
   end
 
   def params

data/spec/shared_examples/rhel_package_provider.rb

@@ -0,0 +1,341 @@
+shared_examples "RHEL package provider" do |provider_class, provider_name|
+  describe provider_name do
+
+    let(:name) { 'mypackage' }
+    let(:resource) do
+      Puppet::Type.type(:package).new(
+        :name     => name,
+        :ensure   => :installed,
+        :provider => provider_name
+      )
+    end
+    let(:provider) do
+      provider = provider_class.new
+      provider.resource = resource
+      provider
+    end
+    let(:arch) { 'x86_64' }
+    let(:arch_resource) do
+      Puppet::Type.type(:package).new(
+        :name     => "#{name}.#{arch}",
+        :ensure   => :installed,
+        :provider => provider_name
+      )
+    end
+    let(:arch_provider) do
+      provider = provider_class.new
+      provider.resource = arch_resource
+      provider
+    end
+
+    case provider_name
+    when 'yum'
+      let(:error_level) { '0' }
+    when 'dnf'
+      let(:error_level) { '1' }
+    when 'tdnf'
+      let(:error_level) { '1' }
+    end
+
+    case provider_name
+    when 'yum'
+      let(:upgrade_command) { 'update' }
+    when 'dnf'
+      let(:upgrade_command) { 'upgrade' }
+    when 'tdnf'
+      let(:upgrade_command) { 'upgrade' }
+    end
+
+    before do
+      provider_class.stubs(:command).with(:cmd).returns("/usr/bin/#{provider_name}")
+      provider.stubs(:rpm).returns 'rpm'
+      provider.stubs(:get).with(:version).returns '1'
+      provider.stubs(:get).with(:release).returns '1'
+      provider.stubs(:get).with(:arch).returns 'i386'
+    end
+    describe 'provider features' do
+      it { is_expected.to be_versionable }
+      it { is_expected.to be_install_options }
+      it { is_expected.to be_virtual_packages }
+    end
+    # provider should repond to the following methods
+     [:install, :latest, :update, :purge, :install_options].each do |method|
+       it "should have a(n) #{method}" do
+         expect(provider).to respond_to(method)
+      end
+    end
+    describe 'when installing' do
+      before(:each) do
+        Puppet::Util.stubs(:which).with("rpm").returns("/bin/rpm")
+        provider.stubs(:which).with("rpm").returns("/bin/rpm")
+        Puppet::Util::Execution.expects(:execute).with(["/bin/rpm", "--version"], {:combine => true, :custom_environment => {}, :failonfail => true}).returns("4.10.1\n").at_most_once
+        Facter.stubs(:value).with(:operatingsystemmajrelease).returns('6')
+      end
+      it "should call #{provider_name} install for :installed" do
+        resource.stubs(:should).with(:ensure).returns :installed
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', :install, 'mypackage'])
+        provider.install
+      end
+
+      if provider_name == 'yum'
+        context 'on el-5' do
+          before(:each) do
+            Facter.stubs(:value).with(:operatingsystemmajrelease).returns('5')
+          end
+          it "should catch #{provider_name} install failures when status code is wrong" do
+            resource.stubs(:should).with(:ensure).returns :installed
+            Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-e', error_level, '-y', :install, name]).returns("No package #{name} available.")
+            expect {
+              provider.install
+            }.to raise_error(Puppet::Error, "Could not find package #{name}")
+          end
+        end
+      end
+      it 'should use :install to update' do
+        provider.expects(:install)
+        provider.update
+      end
+      it 'should be able to set version' do
+        version = '1.2'
+        resource[:ensure] = version
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', :install, "#{name}-#{version}"])
+        provider.stubs(:query).returns :ensure => version
+        provider.install
+      end
+      it 'should handle partial versions specified' do
+        version = '1.3.4'
+        resource[:ensure] = version
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', :install, 'mypackage-1.3.4'])
+        provider.stubs(:query).returns :ensure => '1.3.4-1.el6'
+        provider.install
+      end
+      it 'should be able to downgrade' do
+        current_version = '1.2'
+        version = '1.0'
+        resource[:ensure] = '1.0'
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', :downgrade, "#{name}-#{version}"])
+        provider.stubs(:query).returns(:ensure => current_version).then.returns(:ensure => version)
+        provider.install
+      end
+      it 'should be able to upgrade' do
+        current_version = '1.0'
+        version = '1.2'
+        resource[:ensure] = '1.2'
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', upgrade_command, "#{name}-#{version}"])
+        provider.stubs(:query).returns(:ensure => current_version).then.returns(:ensure => version)
+        provider.install
+      end
+      it 'should accept install options' do
+        resource[:ensure] = :installed
+        resource[:install_options] = ['-t', {'-x' => 'expackage'}]
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', ['-t', '-x=expackage'], :install, name])
+        provider.install
+      end
+      it 'allow virtual packages' do
+        resource[:ensure] = :installed
+        resource[:allow_virtual] = true
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', :list, name]).never
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', :install, name])
+        provider.install
+      end
+      it 'moves architecture to end of version' do
+        version = '1.2.3'
+        arch_resource[:ensure] = version
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-d', '0', '-e', error_level, '-y', :install, "#{name}-#{version}.#{arch}"])
+        arch_provider.stubs(:query).returns :ensure => version
+        arch_provider.install
+      end
+    end
+    describe 'when uninstalling' do
+      it 'should use erase to purge' do
+        Puppet::Util::Execution.expects(:execute).with(["/usr/bin/#{provider_name}", '-y', :erase, name])
+        provider.purge
+      end
+    end
+    it 'should be versionable' do
+      expect(provider).to be_versionable
+    end
+    describe 'determining the latest version available for a package' do
+      it "passes the value of enablerepo install_options when querying" do
+        resource[:install_options] = [
+          {'--enablerepo' => 'contrib'},
+          {'--enablerepo' => 'centosplus'},
+        ]
+        provider.stubs(:properties).returns({:ensure => '3.4.5'})
+        described_class.expects(:latest_package_version).with(name, ['contrib', 'centosplus'], [], [])
+        provider.latest
+      end
+      it "passes the value of disablerepo install_options when querying" do
+        resource[:install_options] = [
+          {'--disablerepo' => 'updates'},
+          {'--disablerepo' => 'centosplus'},
+        ]
+        provider.stubs(:properties).returns({:ensure => '3.4.5'})
+        described_class.expects(:latest_package_version).with(name, [], ['updates', 'centosplus'], [])
+        provider.latest
+      end
+      it "passes the value of disableexcludes install_options when querying" do
+        resource[:install_options] = [
+          {'--disableexcludes' => 'main'},
+          {'--disableexcludes' => 'centosplus'},
+        ]
+        provider.stubs(:properties).returns({:ensure => '3.4.5'})
+        described_class.expects(:latest_package_version).with(name, [], [], ['main', 'centosplus'])
+        provider.latest
+      end
+      describe 'and a newer version is not available' do
+        before :each do
+          described_class.stubs(:latest_package_version).with(name, [], [], []).returns nil
+        end
+        it 'raises an error the package is not installed' do
+          provider.stubs(:properties).returns({:ensure => :absent})
+          expect {
+            provider.latest
+          }.to raise_error(Puppet::DevError, 'Tried to get latest on a missing package')
+        end
+        it 'returns version of the currently installed package' do
+          provider.stubs(:properties).returns({:ensure => '3.4.5'})
+          expect(provider.latest).to eq('3.4.5')
+        end
+      end
+      describe 'and a newer version is available' do
+        let(:latest_version) do
+          {
+            :name     => name,
+            :epoch    => '1',
+            :version  => '2.3.4',
+            :release  => '5',
+            :arch     => 'i686',
+          }
+        end
+        it 'includes the epoch in the version string' do
+          described_class.stubs(:latest_package_version).with(name, [], [], []).returns(latest_version)
+          expect(provider.latest).to eq('1:2.3.4-5')
+        end
+      end
+    end
+    describe "lazy loading of latest package versions" do
+      before { described_class.clear }
+      after { described_class.clear }
+      let(:mypackage_version) do
+        {
+          :name     => name,
+          :epoch    => '1',
+          :version  => '2.3.4',
+          :release  => '5',
+          :arch     => 'i686',
+        }
+      end
+      let(:mypackage_newerversion) do
+        {
+          :name     => name,
+          :epoch    => '1',
+          :version  => '4.5.6',
+          :release  => '7',
+          :arch     => 'i686',
+        }
+      end
+      let(:latest_versions) { {name => [mypackage_version]} }
+      let(:enabled_versions) { {name => [mypackage_newerversion]} }
+      it "returns the version hash if the package was found" do
+        described_class.expects(:check_updates).with([], [], []).once.returns(latest_versions)
+        version = described_class.latest_package_version(name, [], [], [])
+        expect(version).to eq(mypackage_version)
+      end
+      it "is nil if the package was not found in the query" do
+        described_class.expects(:check_updates).with([], [], []).once.returns(latest_versions)
+        version = described_class.latest_package_version('nopackage', [], [], [])
+        expect(version).to be_nil
+      end
+      it "caches the package list and reuses that for subsequent queries" do
+        described_class.expects(:check_updates).with([], [], []).once.returns(latest_versions)
+        2.times {
+          version = described_class.latest_package_version(name, [], [], [])
+          expect(version).to eq mypackage_version
+        }
+      end
+      it "caches separate lists for each combination of 'enablerepo' and 'disablerepo' and 'disableexcludes'" do
+        described_class.expects(:check_updates).with([], [], []).once.returns(latest_versions)
+        described_class.expects(:check_updates).with(['enabled'], ['disabled'], ['disableexcludes']).once.returns(enabled_versions)
+        2.times {
+          version = described_class.latest_package_version(name, [], [], [])
+          expect(version).to eq mypackage_version
+        }
+        2.times {
+          version = described_class.latest_package_version(name, ['enabled'], ['disabled'], ['disableexcludes'])
+          expect(version).to eq(mypackage_newerversion)
+        }
+      end
+    end
+    describe "executing #{provider_name} check-update" do
+      it "passes repos to enable to '#{provider_name} check-update'" do
+        Puppet::Util::Execution.expects(:execute).with do |args, *rest|
+          expect(args).to eq %W[/usr/bin/#{provider_name} check-update --enablerepo=updates --enablerepo=centosplus]
+        end.returns(stub(:exitstatus => 0))
+        described_class.check_updates(%W[updates centosplus], [], [])
+      end
+      it "passes repos to disable to '#{provider_name} check-update'" do
+        Puppet::Util::Execution.expects(:execute).with do |args, *rest|
+          expect(args).to eq %W[/usr/bin/#{provider_name} check-update --disablerepo=updates --disablerepo=centosplus]
+        end.returns(stub(:exitstatus => 0))
+        described_class.check_updates([],%W[updates centosplus], [])
+      end
+      it "passes a combination of repos to enable and disable to '#{provider_name} check-update'" do
+        Puppet::Util::Execution.expects(:execute).with do |args, *rest|
+          expect(args).to eq %W[/usr/bin/#{provider_name} check-update --enablerepo=os --enablerepo=contrib --disablerepo=updates --disablerepo=centosplus]
+        end.returns(stub(:exitstatus => 0))
+        described_class.check_updates(%W[os contrib], %W[updates centosplus], [])
+      end
+      it "passes disableexcludes to '#{provider_name} check-update'" do
+        Puppet::Util::Execution.expects(:execute).with do |args, *rest|
+          expect(args).to eq %W[/usr/bin/#{provider_name} check-update --disableexcludes=main --disableexcludes=centosplus]
+        end.returns(stub(:exitstatus => 0))
+        described_class.check_updates([], [], %W[main centosplus])
+      end
+      it "passes all options to '#{provider_name} check-update'" do
+        Puppet::Util::Execution.expects(:execute).with do |args, *rest|
+          expect(args).to eq %W[/usr/bin/#{provider_name} check-update --enablerepo=a --enablerepo=b --disablerepo=c
+                                --disablerepo=d --disableexcludes=e --disableexcludes=f]
+        end.returns(stub(:exitstatus => 0))
+        described_class.check_updates(%W[a b], %W[c d], %W[e f])
+      end
+      it "returns an empty hash if '#{provider_name} check-update' returned 0" do
+        Puppet::Util::Execution.expects(:execute).returns(stub :exitstatus => 0)
+        expect(described_class.check_updates([], [], [])).to be_empty
+      end
+      it "returns a populated hash if '#{provider_name} check-update returned 100'" do
+        output = stub(:exitstatus => 100)
+        Puppet::Util::Execution.expects(:execute).returns(output)
+        described_class.expects(:parse_updates).with(output).returns({:has => :updates})
+        expect(described_class.check_updates([], [], [])).to eq({:has => :updates})
+      end
+      it "returns an empty hash if '#{provider_name} check-update' returned an exit code that was not 0 or 100" do
+        Puppet::Util::Execution.expects(:execute).returns(stub(:exitstatus => 1))
+        described_class.expects(:warning).with("Could not check for updates, \'/usr/bin/#{provider_name} check-update\' exited with 1")
+        expect(described_class.check_updates([], [], [])).to eq({})
+      end
+    end
+    describe "parsing a line from #{provider_name} check-update" do
+      it "splits up the package name and architecture fields" do
+        checkupdate = %W[curl.i686 7.32.0-10.fc20]
+        parsed = described_class.update_to_hash(*checkupdate)
+        expect(parsed[:name]).to eq 'curl'
+        expect(parsed[:arch]).to eq 'i686'
+      end
+      it "splits up the epoch, version, and release fields" do
+        checkupdate = %W[dhclient.i686 12:4.1.1-38.P1.el6.centos]
+        parsed = described_class.update_to_hash(*checkupdate)
+        expect(parsed[:epoch]).to eq '12'
+        expect(parsed[:version]).to eq '4.1.1'
+        expect(parsed[:release]).to eq '38.P1.el6.centos'
+      end
+      it "sets the epoch to 0 when an epoch is not specified" do
+        checkupdate = %W[curl.i686 7.32.0-10.fc20]
+        parsed = described_class.update_to_hash(*checkupdate)
+        expect(parsed[:epoch]).to eq '0'
+        expect(parsed[:version]).to eq '7.32.0'
+        expect(parsed[:release]).to eq '10.fc20'
+      end
+    end
+  end
+end