From 59654d741243469a37eac1b58ed01f835c742730 Mon Sep 17 00:00:00 2001
From: Tim Smith <tsmith84@gmail.com>
Date: Mon, 14 Mar 2016 10:47:12 -0700
Subject: [PATCH] Update rubocop config to 0.38 and resolve warnings

---
 .rubocop.yml                                           | 10 ++++++----
 libraries/helpers.rb                                   |  2 +-
 libraries/provider_priority_linux.rb                   |  2 +-
 tasks/maintainers.rb                                   |  4 ++--
 .../client51/serverspec/assert_functioning_spec.rb     |  2 +-
 .../client55/serverspec/assert_functioning_spec.rb     |  2 +-
 test/integration/config51/serverspec/linux_spec.rb     |  2 +-
 test/integration/config55/serverspec/linux_spec.rb     |  2 +-
 8 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/.rubocop.yml b/.rubocop.yml
index 12b61476b..42bc867b5 100644
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -15,17 +15,19 @@ Encoding:
   Enabled: false
 Style/FileName:
   Enabled: false
+Style/Alias:
+  Enabled: false
+Style/SpaceBeforeFirstArg:
+  Enabled: false
 LineLength:
   Enabled: false
 MethodLength:
   Enabled: false
 Metrics/AbcSize:
   Enabled: false
-PerceivedComplexity:
+Metrics/ModuleLength:
   Enabled: false
-SingleSpaceBeforeFirstArg:
+PerceivedComplexity:
   Enabled: false
 Style/ClassAndModuleChildren:
   Enabled: false
-Style/FileName:
-  Enabled: false
diff --git a/libraries/helpers.rb b/libraries/helpers.rb
index d171da3fa..8b47559a1 100644
--- a/libraries/helpers.rb
+++ b/libraries/helpers.rb
@@ -392,7 +392,7 @@ def package_name_for(platform, platform_family, platform_version, version, type)
         Chef::Log.error("Unsupported Version: You requested to install a Mysql #{type_label} version that is not supported by your platform")
         Chef::Log.error("Platform: #{platform_family} #{platform_version} - Request Mysql #{type_label} version: #{version}")
         Chef::Log.error("Availabe versions for your platform are: #{info.map { |k, _v| k }.join(' - ')}")
-        fail "Unsupported Mysql #{type_label} Version"
+        raise "Unsupported Mysql #{type_label} Version"
       end
       info[version][type]
     end
diff --git a/libraries/provider_priority_linux.rb b/libraries/provider_priority_linux.rb
index e2840256e..cea5b11a4 100644
--- a/libraries/provider_priority_linux.rb
+++ b/libraries/provider_priority_linux.rb
@@ -1,7 +1,7 @@
 
 begin
   require 'chef/platform/provider_priority_map'
-rescue LoadError
+rescue LoadError # rubocop: disable Lint/HandleExceptions
 end
 
 require_relative 'provider_mysql_service_smf'
diff --git a/tasks/maintainers.rb b/tasks/maintainers.rb
index 80a19416f..cc5d072e0 100644
--- a/tasks/maintainers.rb
+++ b/tasks/maintainers.rb
@@ -34,9 +34,9 @@
       out << project_lieutenant
       out << all_maintainers
 
-      File.open(TARGET, 'w') { |fn|
+      File.open(TARGET, 'w') do |fn|
         fn.write out
-      }
+      end
     end
   end
 
diff --git a/test/integration/client51/serverspec/assert_functioning_spec.rb b/test/integration/client51/serverspec/assert_functioning_spec.rb
index ae1e6a658..c81a302bb 100644
--- a/test/integration/client51/serverspec/assert_functioning_spec.rb
+++ b/test/integration/client51/serverspec/assert_functioning_spec.rb
@@ -5,7 +5,7 @@
 puts "os: #{os}"
 
 def mysql_cmd
-  return "scl enable mysql51 \"mysql --version\"" if os[:family] == 'redhat' && os[:release] =~ /^5\./
+  return 'scl enable mysql51 "mysql --version"' if os[:family] == 'redhat' && os[:release] =~ /^5\./
   '/usr/bin/mysql --version'
 end
 
diff --git a/test/integration/client55/serverspec/assert_functioning_spec.rb b/test/integration/client55/serverspec/assert_functioning_spec.rb
index 1e6f9cc6f..76c0feab6 100644
--- a/test/integration/client55/serverspec/assert_functioning_spec.rb
+++ b/test/integration/client55/serverspec/assert_functioning_spec.rb
@@ -5,7 +5,7 @@
 puts "os: #{os}"
 
 def mysql_cmd
-  return "scl enable mysql55 \"mysql --version\"" if os[:family] == 'redhat' && os[:release] =~ /^5\./
+  return 'scl enable mysql55 "mysql --version"' if os[:family] == 'redhat' && os[:release] =~ /^5\./
   return '/opt/local/bin/mysql --version' if os[:family] == 'smartos'
   return '/opt/omni/bin/mysql --version' if os[:family] == 'solaris'
   '/usr/bin/mysql --version'
diff --git a/test/integration/config51/serverspec/linux_spec.rb b/test/integration/config51/serverspec/linux_spec.rb
index 624cc6783..18b964ede 100644
--- a/test/integration/config51/serverspec/linux_spec.rb
+++ b/test/integration/config51/serverspec/linux_spec.rb
@@ -4,7 +4,7 @@
 
 puts "os: #{os}"
 
-os[:family] == 'redhat' ? prefix_dir = '/opt/rh/mysql51/root' : prefix_dir = nil
+prefix_dir = os[:family] == 'redhat' ? '/opt/rh/mysql51/root' : nil
 
 if %w(debian ubuntu redhat suse fedora).include? os[:family]
   describe file("#{prefix_dir}/etc/mysql-default") do
diff --git a/test/integration/config55/serverspec/linux_spec.rb b/test/integration/config55/serverspec/linux_spec.rb
index 6c086f10c..51bdab8b4 100644
--- a/test/integration/config55/serverspec/linux_spec.rb
+++ b/test/integration/config55/serverspec/linux_spec.rb
@@ -4,7 +4,7 @@
 
 puts "os: #{os}"
 
-os[:family] == 'redhat' && os[:release] =~ /^5\./ ? prefix_dir = '/opt/rh/mysql55/root' : prefix_dir = nil
+prefix_dir = os[:family] == 'redhat' && os[:release] =~ /^5\./ ? '/opt/rh/mysql55/root' : nil
 
 if %w(debian ubuntu redhat suse fedora).include? os[:family]
   describe file("#{prefix_dir}/etc/mysql-default") do