Merge pull request #607 from brabiega/patch-1

Use $real_pkgname for pip uninstall command

manifests/pip.pp

@@ -220,7 +220,7 @@
 
       default: {
         # Anti-action, uninstall.
-        $command        = "echo y | ${pip_env} uninstall ${uninstall_args} ${proxy_flag} ${name}"
+        $command        = "echo y | ${pip_env} uninstall ${uninstall_args} ${proxy_flag} ${real_pkgname}"
         $unless_command = "! ${pip_env} list | grep -i -e '${grep_regex}'"
       }
     }

spec/acceptance/pip_spec.rb

@@ -0,0 +1,75 @@
+require 'spec_helper_acceptance'
+
+describe 'python::pip defined resource' do
+  context 'install package with custom name' do
+    it 'works with no errors' do
+      pp = <<-PUPPET
+      class { 'python':
+        version => '3',
+        dev     => 'present',
+      }
+
+      python::pyvenv { '/opt/test-venv':
+        ensure      => 'present',
+        systempkgs  => false,
+        mode        => '0755',
+        pip_version => '<= 20.3.4',
+      }
+
+      python::pip { 'agent package':
+        virtualenv => '/opt/test-venv',
+        pkgname    => 'agent',
+        ensure     => '0.1.2',
+      }
+      PUPPET
+
+      apply_manifest(pp, catch_failures: true)
+      apply_manifest(pp, catch_changes: true)
+    end
+  end
+
+  describe command('/opt/test-venv/bin/pip list') do
+    its(:exit_status) { is_expected.to eq 0 }
+    its(:stdout) { is_expected.to match %r{agent.* 0\.1\.2} }
+  end
+
+  context 'uninstall package with custom name' do
+    it 'works with no errors' do
+      pp = <<-PUPPET
+      class { 'python':
+        version => '3',
+        dev     => 'present',
+      }
+
+      python::pyvenv { '/opt/test-venv':
+        ensure      => 'present',
+        systempkgs  => false,
+        mode        => '0755',
+        pip_version => '<= 20.3.4',
+      }
+
+      python::pip { 'agent package install':
+        ensure     => '0.1.2',
+        pkgname    => 'agent',
+        virtualenv => '/opt/test-venv',
+      }
+
+      python::pip { 'agent package uninstall custom pkgname':
+        ensure  => 'absent',
+        pkgname => 'agent',
+        virtualenv => '/opt/test-venv',
+        require => Python::Pip['agent package install'],
+      }
+
+      PUPPET
+
+      apply_manifest(pp, catch_failures: true)
+    end
+  end
+
+  describe command('/opt/test-venv/bin/pip list') do
+    its(:exit_status) { is_expected.to eq 0 }
+    its(:stdout) { is_expected.not_to match %r{agent.* 0\.1\.2} }
+  end
+end
+

spec/defines/pip_spec.rb

@@ -116,6 +116,14 @@
 
         it { is_expected.to contain_exec('pip_uninstall_rpyc').with_command(%r{uninstall.*rpyc$}) }
       end
+
+      context 'passes correct package name' do
+        let(:params) { { ensure: 'absent', 'pkgname': 'r-pyc' } }
+
+        it { is_expected.not_to contain_exec('pip_install_rpyc') }
+
+        it { is_expected.to contain_exec('pip_uninstall_rpyc').with_command(%r{uninstall.*r-pyc$}) }
+      end
     end
   end
 end