Add Gemfile and puppet 4 checks

In anticipation of puppet 4, start trying to deal with puppet 4 things
that can be helpfully predicted by puppet lint plugins. This patch also
corrects lint errors caught by the puppet-lint-absolute_classname-check
and puppet-lint-empty_string-check gems as well as arrow alignment
which wasn't being caught under the system version of puppet-lint.

Change-Id: I91cb2fdeb458edbd41e777be74d292a564bf2d58

.gitignore

@@ -0,0 +1,2 @@
+Gemfile.lock
+.bundled_gems/

Gemfile

@@ -0,0 +1,30 @@
+source 'https://rubygems.org'
+
+group :development, :test do
+  gem 'puppetlabs_spec_helper', :require => false
+
+  gem 'metadata-json-lint'
+  # This is nice and all, but let's not worry about it until we've actually
+  # got puppet 4.x sorted
+  # gem 'puppet-lint-param-docs'
+  gem 'puppet-lint-absolute_classname-check'
+  gem 'puppet-lint-absolute_template_path'
+  gem 'puppet-lint-trailing_newline-check'
+
+  # Puppet 4.x related lint checks
+  gem 'puppet-lint-unquoted_string-check'
+  gem 'puppet-lint-empty_string-check'
+  gem 'puppet-lint-leading_zero-check'
+  gem 'puppet-lint-variable_contains_upcase'
+  gem 'puppet-lint-spaceship_operator_without_tag-check'
+  gem 'puppet-lint-undef_in_function-check'
+
+  if puppetversion = ENV['PUPPET_GEM_VERSION']
+    gem 'puppet', puppetversion, :require => false
+  else
+    gem 'puppet', '~> 3.0', :require => false
+  end
+
+end
+
+# vim:ft=ruby

manifests/init.pp

@@ -79,9 +79,9 @@ class drupal (
   $site_ssl_cert_file_contents = undef,
   $site_ssl_key_file_contents = undef,
   $site_ssl_chain_file_contents = undef,
-  $site_ssl_cert_file = '',
+  $site_ssl_cert_file = undef,
-  $site_ssl_key_file = '',
+  $site_ssl_key_file = undef,
-  $site_ssl_chain_file = '',
+  $site_ssl_chain_file = undef,
   $package_repository = undef,
   $package_branch = undef,
   $conf = undef,
@@ -90,7 +90,7 @@ class drupal (
   $conf_openid_provider = undef,
 ) {
   include ::httpd
-  include pear
+  include ::pear
 
   # ssl certificates
   if $site_ssl_enabled == true {
@@ -207,10 +207,10 @@ class drupal (
   # drush site-alias definition
 
   file { '/etc/drush':
-    ensure  => directory,
+    ensure => directory,
-    owner   => 'root',
+    owner  => 'root',
-    group   => 'root',
+    group  => 'root',
-    mode    => '0755',
+    mode   => '0755',
   }
 
   file { '/etc/drush/aliases.drushrc.php':
@@ -226,41 +226,41 @@ class drupal (
   # site custom configuration
 
   file { "${site_root}/etc":
-    ensure    => directory,
+    ensure  => directory,
-    owner     => 'root',
+    owner   => 'root',
-    group     => 'root',
+    group   => 'root',
-    mode      => '0755',
+    mode    => '0755',
-    require   => File[$site_root],
+    require => File[$site_root],
   }
 
   file { "${site_root}/etc/settings.php":
-    ensure    => file,
+    ensure  => file,
-    owner     => 'root',
+    owner   => 'root',
-    group     => 'root',
+    group   => 'root',
-    mode      => '0400',
+    mode    => '0400',
-    content   => template('drupal/settings.php.erb'),
+    content => template('drupal/settings.php.erb'),
-    replace   => true,
+    replace => true,
-    require   => File["${site_root}/etc"],
+    require => File["${site_root}/etc"],
   }
 
   # deploy a site from scratch when site status is 'NOT INSTALLED'
   exec { "sitedeploy-${site_name}":
-    command     => "/usr/bin/drush dsd-init @${site_alias}",
+    command   => "/usr/bin/drush dsd-init @${site_alias}",
-    logoutput   => true,
+    logoutput => true,
-    timeout     => 600,
+    timeout   => 600,
-    onlyif      => "/usr/bin/drush dsd-status @${site_alias} | /bin/grep -c 'NOT INSTALLED'",
+    onlyif    => "/usr/bin/drush dsd-status @${site_alias} | /bin/grep -c 'NOT INSTALLED'",
-    require     => [
+    require   => [
       File['/etc/drush/aliases.drushrc.php'],
       ]
   }
 
   # update the site into a new slot when a remote update available
   exec { "siteupdate-${site_name}":
-    command     => "/usr/bin/drush dsd-update @${site_alias}",
+    command   => "/usr/bin/drush dsd-update @${site_alias}",
-    logoutput   => true,
+    logoutput => true,
-    timeout     => 600,
+    timeout   => 600,
-    onlyif      => "/usr/bin/drush dsd-status @${site_alias} | /bin/grep -c 'UPDATE'",
+    onlyif    => "/usr/bin/drush dsd-status @${site_alias} | /bin/grep -c 'UPDATE'",
-    require     => [
+    require   => [
       File['/etc/drush/aliases.drushrc.php'],
       Exec["sitedeploy-${site_name}"],
       ]