From 2d2fe182c0b8860f8976a693c66eb5a676fe7445 Mon Sep 17 00:00:00 2001
From: Jonathan Hartman <j@p4nt5.com>
Date: Mon, 8 Feb 2016 23:40:11 -0800
Subject: [PATCH] Resolve new RuboCop offenses

---
 libraries/helpers.rb                                 | 2 +-
 libraries/resource_vlc_app.rb                        | 6 ++++--
 libraries/resource_vlc_app_mac_os_x.rb               | 2 +-
 metadata.rb                                          | 4 +---
 spec/support/cookbooks/vlc_app_test/metadata.rb      | 4 +---
 test/fixtures/cookbooks/vlc_test/metadata.rb         | 4 +---
 test/integration/default/serverspec/spec_helper.rb   | 2 +-
 test/integration/uninstall/serverspec/spec_helper.rb | 2 +-
 8 files changed, 11 insertions(+), 15 deletions(-)

diff --git a/libraries/helpers.rb b/libraries/helpers.rb
index 8c51427..fafd5f9 100644
--- a/libraries/helpers.rb
+++ b/libraries/helpers.rb
@@ -73,7 +73,7 @@ def vlc_site_body
     # @return [TrueClass, FalseClass] whether the string is valid
     #
     def self.valid_version?(arg)
-      arg.match(/^[0-9]+\.[0-9]+\.[0-9]+$/) ? true : false
+      arg =~ /^[0-9]+\.[0-9]+\.[0-9]+$/ ? true : false
     end
   end
 end
diff --git a/libraries/resource_vlc_app.rb b/libraries/resource_vlc_app.rb
index 0110b38..aeea204 100644
--- a/libraries/resource_vlc_app.rb
+++ b/libraries/resource_vlc_app.rb
@@ -38,8 +38,10 @@ class VlcApp < Resource
 
       %i(install remove).each do |a|
         action a do
-          fail(NotImplementedError,
-               "Action `#{a}` must be implemented for `#{self.class}` resource")
+          raise(
+            NotImplementedError,
+            "Action `#{a}` must be implemented for `#{self.class}` resource"
+          )
         end
       end
     end
diff --git a/libraries/resource_vlc_app_mac_os_x.rb b/libraries/resource_vlc_app_mac_os_x.rb
index 77f02fa..8128a8f 100644
--- a/libraries/resource_vlc_app_mac_os_x.rb
+++ b/libraries/resource_vlc_app_mac_os_x.rb
@@ -29,7 +29,7 @@ class Resource
     class VlcAppMacOsX < VlcApp
       include Vlc::Helpers
 
-      PATH ||= '/Applications/VLC.app'
+      PATH ||= '/Applications/VLC.app'.freeze
 
       provides :vlc_app, platform_family: 'mac_os_x'
 
diff --git a/metadata.rb b/metadata.rb
index 3aa9a93..ec48bc9 100644
--- a/metadata.rb
+++ b/metadata.rb
@@ -1,6 +1,5 @@
 # Encoding: UTF-8
-#
-# rubocop:disable SingleSpaceBeforeFirstArg
+
 name             'vlc'
 maintainer       'Jonathan Hartman'
 maintainer_email 'j@p4nt5.com'
@@ -24,4 +23,3 @@
 supports         'centos'
 supports         'scientific'
 supports         'freebsd'
-# rubocop:enable SingleSpaceBeforeFirstArg
diff --git a/spec/support/cookbooks/vlc_app_test/metadata.rb b/spec/support/cookbooks/vlc_app_test/metadata.rb
index 4efe684..a3876c6 100644
--- a/spec/support/cookbooks/vlc_app_test/metadata.rb
+++ b/spec/support/cookbooks/vlc_app_test/metadata.rb
@@ -1,6 +1,5 @@
 # Encoding: UTF-8
-#
-# rubocop:disable SingleSpaceBeforeFirstArg
+
 name             'vlc_app_test'
 maintainer       'Jonathan Hartman'
 maintainer_email 'j@p4nt5.com'
@@ -19,4 +18,3 @@
 supports         'centos'
 supports         'scientific'
 supports         'freebsd'
-# rubocop:enable SingleSpaceBeforeFirstArg
diff --git a/test/fixtures/cookbooks/vlc_test/metadata.rb b/test/fixtures/cookbooks/vlc_test/metadata.rb
index a6ba50a..ea2cc41 100644
--- a/test/fixtures/cookbooks/vlc_test/metadata.rb
+++ b/test/fixtures/cookbooks/vlc_test/metadata.rb
@@ -1,6 +1,5 @@
 # Encoding: UTF-8
-#
-# rubocop:disable SingleSpaceBeforeFirstArg
+
 name             'vlc_test'
 maintainer       'Jonathan Hartman'
 maintainer_email 'j@p4nt5.com'
@@ -19,4 +18,3 @@
 supports         'centos'
 supports         'scientific'
 supports         'freebsd'
-# rubocop:enable SingleSpaceBeforeFirstArg
diff --git a/test/integration/default/serverspec/spec_helper.rb b/test/integration/default/serverspec/spec_helper.rb
index eef2022..04da6e6 100644
--- a/test/integration/default/serverspec/spec_helper.rb
+++ b/test/integration/default/serverspec/spec_helper.rb
@@ -2,7 +2,7 @@
 
 require 'serverspec'
 
-if RUBY_PLATFORM.match(/mswin|mingw32|windows/)
+if RUBY_PLATFORM =~ /mswin|mingw32|windows/
   set :os, family: 'windows'
   set :backend, :cmd
 else
diff --git a/test/integration/uninstall/serverspec/spec_helper.rb b/test/integration/uninstall/serverspec/spec_helper.rb
index eef2022..04da6e6 100644
--- a/test/integration/uninstall/serverspec/spec_helper.rb
+++ b/test/integration/uninstall/serverspec/spec_helper.rb
@@ -2,7 +2,7 @@
 
 require 'serverspec'
 
-if RUBY_PLATFORM.match(/mswin|mingw32|windows/)
+if RUBY_PLATFORM =~ /mswin|mingw32|windows/
   set :os, family: 'windows'
   set :backend, :cmd
 else