Merge pull request #518 from syseleven/fix/venv_duplicate_declaration

Fix duplicate declaration for python-venv package

Author: bastelfreak

manifests/pyvenv.pp

@@ -32,13 +32,12 @@
   $path             = [ '/bin', '/usr/bin', '/usr/sbin', '/usr/local/bin' ],
   $environment      = [],
 ) {
-
   include python
 
   if $ensure == 'present' {
     $python_version = $version ? {
-        'system' => $facts['python3_version'],
-        default  => $version,
+      'system' => $facts['python3_version'],
+      default  => $version,
     }
 
     $python_version_parts = split($python_version, '[.]')
@@ -50,11 +49,10 @@
       case $facts['lsbdistcodename'] {
         'xenial','bionic','cosmic','disco',
         'jessie','stretch','buster': {
-          ensure_packages ($python3_venv_package, {
-            before => File[$venv_dir],
-          })
+          ensure_packages ($python3_venv_package)
+          Package[$python3_venv_package] -> File[$venv_dir]
         }
-          default: {}
+        default: {}
       }
     }
 

spec/classes/python_spec.rb

@@ -72,6 +72,28 @@
             end
           end
 
+          describe 'with python::python_pyvenvs' do
+            context 'with two pyenvs' do
+              let(:params) do
+                {
+                  python_pyvenvs: {
+                    '/opt/env1' => {
+                      version: '3.8'
+                    },
+                    '/opt/env2' => {
+                      version: '3.8'
+                    }
+                  }
+                }
+              end
+
+              it { is_expected.to compile }
+
+              it { is_expected.to contain_python__pyvenv('/opt/env1').with_ensure('present') }
+              it { is_expected.to contain_python__pyvenv('/opt/env2').with_ensure('present') }
+            end
+          end
+
           describe 'with manage_gunicorn' do
             context 'true' do
               let(:params) { { manage_gunicorn: true } }

spec/defines/pyvenv_spec.rb

@@ -13,10 +13,14 @@
         '/opt/env'
       end
 
-      it {
-        is_expected.to contain_file('/opt/env')
-        is_expected.to contain_exec('python_virtualenv_/opt/env').with_command('pyvenv-3.5 --clear  /opt/env')
-      }
+      context 'with default parameters' do
+        it { is_expected.to contain_file('/opt/env') }
+        it { is_expected.to contain_exec('python_virtualenv_/opt/env').with_command('pyvenv-3.5 --clear  /opt/env') }
+
+        if %w[xenial bionic cosmic disco jessie stretch buster].include?(facts[:lsbdistcodename])
+          it { is_expected.to contain_package('python3.5-venv').that_comes_before('File[/opt/env]') }
+        end
+      end
 
       describe 'when ensure' do
         context 'is absent' do