Merge pull request #299 from ghoneycutt/strict_vars

Strict vars

.fixtures.yml

@@ -5,6 +5,6 @@ fixtures:
       ref: '4.6.0'
     'epel':
       repo: 'git://github.com/stahnma/puppet-module-epel.git'
-      ref: '1.0.2'
+      ref: '1.2.2'
   symlinks:
     python: "#{source_dir}"

.travis.yml

@@ -27,7 +27,7 @@ env:
     - PUPPET_GEM_VERSION="~> 4.3.0"
     - PUPPET_GEM_VERSION="~> 4.4.0"
     - PUPPET_GEM_VERSION="~> 4.5.0"
-    - PUPPET_GEM_VERSION="~> 4"
+    - PUPPET_GEM_VERSION="~> 4" STRICT_VARIABLES="yes"
 
 sudo: false
 
@@ -59,7 +59,7 @@ matrix:
     - rvm: 1.8.7
       env: PUPPET_GEM_VERSION="~> 4.5.0"
     - rvm: 1.8.7
-      env: PUPPET_GEM_VERSION="~> 4"
+      env: PUPPET_GEM_VERSION="~> 4" STRICT_VARIABLES="yes"
     - rvm: 2.3.1
       env: PUPPET_GEM_VERSION="~> 3.1.0"
     - rvm: 2.3.1

Gemfile

@@ -3,7 +3,7 @@ source ENV['GEM_SOURCE'] || "https://rubygems.org"
 group :development, :test do
   gem 'metadata-json-lint',      :require => false
   gem 'rspec-puppet',            :require => false
-  gem 'puppetlabs_spec_helper', '>= 1.1.1'
+  gem 'puppetlabs_spec_helper', '1.1.1'
   gem 'puppet-lint',             :require => false
   gem 'pry',                     :require => false
   gem 'simplecov',               :require => false

manifests/install.pp

@@ -190,9 +190,10 @@
 
         $virtualenv_package = "${python}-virtualenv"
       } else {
-        $virtualenv_package = $::lsbdistcodename ? {
-          'jessie' => 'virtualenv',
-          default  => 'python-virtualenv',
+        if $::lsbdistcodename == 'jessie' {
+          $virtualenv_package = 'virtualenv'
+        } else {
+          $virtualenv_package = 'python-virtualenv'
         }
       }
 

manifests/pyvenv.pp

@@ -59,6 +59,8 @@
   $environment      = [],
 ) {
 
+  include ::python
+
   if $ensure == 'present' {
 
     $virtualenv_cmd = $version ? {

manifests/requirements.pp

@@ -80,6 +80,8 @@
   $timeout                = 1800,
 ) {
 
+  include ::python
+
   if $virtualenv == 'system' and ($owner != 'root' or $group != 'root') {
     fail('python::pip: root user must be used when virtualenv is system')
   }
@@ -98,8 +100,8 @@
   }
 
   $pip_env = $virtualenv ? {
-    'system' => "${python::exec_prefix} pip",
-    default  => "${python::exec_prefix} ${virtualenv}/bin/pip",
+    'system' => "${::python::exec_prefix} pip",
+    default  => "${::python::exec_prefix} ${virtualenv}/bin/pip",
   }
 
   $proxy_flag = $proxy ? {
@@ -136,5 +138,4 @@
     subscribe   => File[$requirements],
     environment => $environment,
   }
-
 }

metadata.json

@@ -54,6 +54,6 @@
   ],
   "dependencies": [
     {"name":"puppetlabs/stdlib","version_requirement":">= 4.6.0 < 6.0.0"},
-    {"name":"stahnma/epel","version_requirement":">= 1.0.1 < 2.0.0"}
+    {"name":"stahnma/epel","version_requirement":">= 1.2.2 < 2.0.0"}
   ]
 }

spec/classes/python_spec.rb

@@ -187,6 +187,7 @@
       {
         :id => 'root',
         :kernel => 'Linux',
+        :lsbdistcodename => nil,
         :osfamily => 'Suse',
         :operatingsystem => 'SLES',
         :operatingsystemrelease => '11.3',

spec/defines/pyvenv_spec.rb

@@ -2,6 +2,12 @@
 
 describe 'python::pyvenv', :type => :define do
   let (:title) { '/opt/env' }
+  let (:facts) do
+    {
+      :lsbdistcodename => 'jessie',
+      :osfamily => 'Debian',
+    }
+  end
 
   it {
     is_expected.to contain_file( '/opt/env')