(maint) Use match? methods

RuboCop's Performance/RegexpMatch cop flagged several files for
using `=~` to regex match when the `match?` method (introduced
in Ruby 2.4) is often more performant.

This commit updates instances of `=~` to use `match?` instead.

Author: mhashizume

install.rb

@@ -153,7 +153,7 @@ def prepare_installation
     # /System/Library/Frameworks/Ruby.framework/Versions/1.8/usr/bin
     # which is not generally where people expect executables to be installed
     # These settings are appropriate defaults for all OS X versions.
-    RbConfig::CONFIG['bindir'] = '/usr/bin' if RUBY_PLATFORM =~ /^universal-darwin[\d\.]+$/
+    RbConfig::CONFIG['bindir'] = '/usr/bin' if RUBY_PLATFORM.match?(/^universal-darwin[\d\.]+$/)
 
     # if InstallOptions.configdir
     #   configdir = InstallOptions.configdir
@@ -176,7 +176,7 @@ def prepare_installation
         sitelibdir = $LOAD_PATH.find { |x| x =~ /site_ruby/ }
         if sitelibdir.nil?
           sitelibdir = File.join(libdir, 'site_ruby')
-        elsif sitelibdir !~ Regexp.quote(version)
+        elsif !sitelibdir&.match?(Regexp.quote(version))
           sitelibdir = File.join(sitelibdir, version)
         end
       end
@@ -232,7 +232,7 @@ def install_binfile(from, op_file, target)
       File.open(tmp_file.path, 'w') do |op|
         op.puts "#!#{ruby}"
         contents = ip.readlines
-        contents.shift if contents[0] =~ /^#!/
+        contents.shift if /^#!/.match?(contents[0])
         op.write contents.join
       end
     end

lib/facter/custom_facts/core/execution/base.rb

@@ -140,7 +140,7 @@ def log_stderr(msg, command, logger)
 
         def builtin_command?(command)
           output, _status = Open3.capture2("type #{command}")
-          output.chomp =~ /builtin/ ? true : false
+          /builtin/.match?(output.chomp) ? true : false
         end
       end
     end

lib/facter/custom_facts/core/execution/posix.rb

@@ -44,7 +44,7 @@ def expand_command(command)
 
           return unless exe && (expanded = which(exe))
 
-          expanded = "'#{expanded}'" if expanded =~ /\s/
+          expanded = "'#{expanded}'" if /\s/.match?(expanded)
           expanded << " #{args}" if args
 
           expanded

lib/facter/custom_facts/core/execution/windows.rb

@@ -11,7 +11,7 @@ def search_paths
         def which(bin)
           # `echo` is allowed for facter 3.x compatibility, otherwise
           # all commands much be found on the PATH or absolute.
-          return bin if /^echo$/i =~ bin
+          return bin if /^echo$/i.match?(bin)
 
           if absolute_path?(bin)
             return bin if File.executable?(bin)
@@ -56,7 +56,7 @@ def expand_command(command)
 
           return unless exe && (expanded = which(exe))
 
-          expanded = "\"#{expanded}\"" if expanded =~ /\s+/
+          expanded = "\"#{expanded}\"" if /\s+/.match?(expanded)
           expanded << " #{args}" if args
 
           expanded

lib/facter/custom_facts/util/normalization.rb

@@ -51,7 +51,7 @@ def normalize(value)
       # @param value [String]
       # @return [void]
 
-      if RUBY_VERSION =~ /^1\.8/
+      if RUBY_VERSION.match?(/^1\.8/)
         require 'iconv'
 
         def normalize_string(value)

lib/facter/facts/debian/architecture.rb

@@ -10,7 +10,7 @@ class Architecture
         def call_the_resolver
           fact_value = Facter::Resolvers::Uname.resolve(:machine)
           fact_value = 'amd64' if fact_value == 'x86_64'
-          fact_value = 'i386' if fact_value =~ /i[3456]86|pentium/
+          fact_value = 'i386' if /i[3456]86|pentium/.match?(fact_value)
 
           [Facter::ResolvedFact.new(FACT_NAME, fact_value), Facter::ResolvedFact.new(ALIASES, fact_value, :legacy)]
         end

lib/facter/facts/linux/hypervisors/kvm.rb

@@ -52,9 +52,9 @@ def discover_provider
 
           return { google: true } if manufacturer == 'Google'
 
-          return { openstack: true } if manufacturer =~ /^OpenStack/
+          return { openstack: true } if /^OpenStack/.match?(manufacturer)
 
-          return { amazon: true } if manufacturer =~ /^Amazon/
+          return { amazon: true } if /^Amazon/.match?(manufacturer)
 
           {}
         end

lib/facter/facts/linux/os/architecture.rb

@@ -10,7 +10,7 @@ class Architecture
         def call_the_resolver
           fact_value = Facter::Resolvers::Uname.resolve(:machine)
 
-          fact_value = 'i386' if fact_value =~ /i[3456]86|pentium/
+          fact_value = 'i386' if /i[3456]86|pentium/.match?(fact_value)
 
           [Facter::ResolvedFact.new(FACT_NAME, fact_value), Facter::ResolvedFact.new(ALIASES, fact_value, :legacy)]
         end

lib/facter/facts/windows/hypervisors/kvm.rb

@@ -27,9 +27,9 @@ def discover_provider
 
           return { google: true } if manufacturer == 'Google'
 
-          return { openstack: true } if Facter::Resolvers::DMIComputerSystem.resolve(:name) =~ /^OpenStack/
+          return { openstack: true } if /^OpenStack/.match?(Facter::Resolvers::DMIComputerSystem.resolve(:name))
 
-          return { amazon: true } if manufacturer =~ /^Amazon/
+          return { amazon: true } if /^Amazon/.match?(manufacturer)
         end
       end
     end

lib/facter/framework/config/fact_groups.rb

@@ -64,7 +64,7 @@ def get_group_ttls(group_name)
     def get_fact(fact_name)
       return @facts_ttls[fact_name] if @facts_ttls[fact_name]
 
-      result = @facts_ttls.select { |name, fact| break fact if fact_name =~ /^#{name}\..*/ }
+      result = @facts_ttls.select { |name, fact| break fact if /^#{name}\..*/.match?(fact_name) }
       return nil if result == {}
 
       result

lib/facter/framework/core/fact_loaders/fact_loader.rb

@@ -103,7 +103,7 @@ def construct_reject_lists(blocked_facts, facts)
 
       blocked_facts.each do |blocked|
         facts.each do |fact|
-          next unless fact.name =~ /^#{blocked}\..*|^#{blocked}$/
+          next unless /^#{blocked}\..*|^#{blocked}$/.match?(fact.name)
 
           if fact.type == :core
             reject_list_core << fact

lib/facter/framework/formatters/legacy_fact_formatter.rb

@@ -95,7 +95,7 @@ def remove_comma_and_quotation(output)
       # quotation marks that come after \ are not removed
       @log.debug('Remove unnecessary comma and quotation marks on root facts')
       output.split("\n")
-            .map! { |line| line =~ /^[\s]+/ ? line : line.gsub(/,$|(?<!\\)\"/, '').gsub('\\"', '"') }.join("\n")
+            .map! { |line| /^[\s]+/.match?(line) ? line : line.gsub(/,$|(?<!\\)\"/, '').gsub('\\"', '"') }.join("\n")
     end
   end
 end

lib/facter/framework/formatters/yaml_fact_formatter.rb

@@ -52,9 +52,9 @@ def unquote_keys(fact_hash)
     end
 
     def needs_quote?(value)
-      return false if value =~ /true|false/
+      return false if /true|false/.match?(value)
       return false if value[/^[0-9]+$/]
-      return true if value =~ /y|Y|yes|Yes|YES|n|N|no|No|NO|True|TRUE|False|FALSE|on|On|ON|off|Off|OFF|:/
+      return true if /y|Y|yes|Yes|YES|n|N|no|No|NO|True|TRUE|False|FALSE|on|On|ON|off|Off|OFF|:/.match?(value)
       return false if value[/[a-zA-Z]/]
       return false if value[/[0-9]+\.[0-9]+/]
 

lib/facter/patches/sysfilesystem/sys/statvfs.rb

@@ -10,7 +10,7 @@ class Statvfs < FFI::Struct
         # it the second time will make FFI log a warning message.
         remove_instance_variable(:@layout) if @layout
 
-        if RbConfig::CONFIG['host_os'] =~ /darwin|osx|mach/i
+        if /darwin|osx|mach/i.match?(RbConfig::CONFIG['host_os'])
           layout(
             :f_bsize, :ulong,
             :f_frsize, :ulong,
@@ -24,7 +24,7 @@ class Statvfs < FFI::Struct
             :f_flag, :ulong,
             :f_namemax, :ulong
           )
-        elsif RbConfig::CONFIG['host'] =~ /bsd/i
+        elsif /bsd/i.match?(RbConfig::CONFIG['host'])
           layout(
             :f_bavail, :uint64,
             :f_bfree, :uint64,
@@ -38,7 +38,7 @@ class Statvfs < FFI::Struct
             :f_fsid, :ulong,
             :f_namemax, :ulong
           )
-        elsif RbConfig::CONFIG['host'] =~ /sunos|solaris/i
+        elsif /sunos|solaris/i.match?(RbConfig::CONFIG['host'])
           layout(
             :f_bsize, :ulong,
             :f_frsize, :ulong,
@@ -55,7 +55,7 @@ class Statvfs < FFI::Struct
             :f_fstr, [:char, 32],
             :f_filler, [:ulong, 16]
           )
-        elsif RbConfig::CONFIG['host'] =~ /i686/i
+        elsif /i686/i.match?(RbConfig::CONFIG['host'])
           layout(
             :f_bsize, :ulong,
             :f_frsize, :ulong,

lib/facter/resolvers/aix/filesystem.rb

@@ -18,7 +18,7 @@ def read_vtf_file(fact_name)
             return if file_content.empty?
 
             file_content = file_content.map do |line|
-              next if line =~ /#|%/ # skip lines that are comments or defaultvfs line
+              next if /#|%/.match?(line) # skip lines that are comments or defaultvfs line
 
               line.split(' ').first
             end

lib/facter/resolvers/aix/memory.rb

@@ -25,7 +25,7 @@ def execute_svmon(fact_name)
 
             result.each_line do |line|
               @fact_list[:system] = populate_system(line, pagesize) if line.include?('memory')
-              @fact_list[:swap] = populate_swap(line, pagesize) if line =~ /pg\sspace/
+              @fact_list[:swap] = populate_swap(line, pagesize) if /pg\sspace/.match?(line)
             end
 
             @fact_list[fact_name]

lib/facter/resolvers/aix/mountpoints.rb

@@ -19,7 +19,7 @@ def read_mount(fact_name)
             @fact_list[:mountpoints] = {}
             output = Facter::Core::Execution.execute('mount', logger: log)
             output.split("\n").drop(2).map do |line|
-              next if line =~ /procfs|ahafs/
+              next if /procfs|ahafs/.match?(line)
 
               add_mount_points_fact(line)
             end
@@ -40,7 +40,7 @@ def add_mount_points_fact(line)
           def retrieve_sizes_for_mounts
             output = Facter::Core::Execution.execute('df -P', logger: log)
             output.split("\n").drop(1).map do |line|
-              next if line =~ /-\s+-\s+-/
+              next if /-\s+-\s+-/.match?(line)
 
               mount_info = line.split("\s")
               mount_info[3] = translate_to_bytes(mount_info[3])

lib/facter/resolvers/aix/networking.rb

@@ -43,7 +43,7 @@ def build_bindings(name, ip, mask_length, is_ipv4)
           def populate_with_mtu_and_mac!(interfaces)
             output = Facter::Core::Execution.execute('netstat -in', logger: log)
             output.each_line do |line|
-              next if line =~ /Name\s/
+              next if /Name\s/.match?(line)
 
               info = line.split("\s")
               interface_name = info[0]

lib/facter/resolvers/aix/partitions.rb

@@ -50,8 +50,8 @@ def populate_from_lslv(name)
             }
             mount = info_hash['MOUNT POINT']
             label = info_hash['LABEL']
-            part_info[:mount] = mount unless %r{N/A} =~ mount
-            part_info[:label] = label.strip unless /None/ =~ label
+            part_info[:mount] = mount unless %r{N/A}.match?(mount)
+            part_info[:label] = label.strip unless /None/.match?(label)
             part_info
           end
 

lib/facter/resolvers/lspci.rb

@@ -26,7 +26,7 @@ def lspci_command(fact_name)
         end
 
         def retrieve_vm(output)
-          output.each_line { |line| REGEX_VALUES.each { |key, value| return value if line =~ /#{key}/ } }
+          output.each_line { |line| REGEX_VALUES.each { |key, value| return value if /#{key}/.match?(line) } }
 
           nil
         end

lib/facter/resolvers/networking.rb

@@ -71,7 +71,7 @@ def extract_mac(raw_data, parsed_interface_data)
         end
 
         def extract_dhcp(interface_name, raw_data, parsed_interface_data)
-          return unless raw_data =~ /status:\s+active/
+          return unless /status:\s+active/.match?(raw_data)
 
           result = Facter::Core::Execution.execute("ipconfig getoption #{interface_name} " \
                                                      'server_identifier', logger: log)

lib/facter/resolvers/open_vz.rb

@@ -29,7 +29,7 @@ def read_proc_status
             parts = line.split("\s")
             next unless parts.size.equal?(2)
 
-            next unless /^envID:/ =~ parts[0]
+            next unless /^envID:/.match?(parts[0])
 
             @fact_list[:id] = parts[1]
 

lib/facter/resolvers/os_release.rb

@@ -63,13 +63,13 @@ def fill_fact_list(pairs)
         def process_version_id
           return unless @fact_list[:version_id]
 
-          @fact_list[:version_id] = "#{@fact_list[:version_id]}.0" unless @fact_list[:version_id] =~ /\./
+          @fact_list[:version_id] = "#{@fact_list[:version_id]}.0" unless /\./.match?(@fact_list[:version_id])
         end
 
         def process_id
           return unless @fact_list[:id]
 
-          @fact_list[:id] = 'opensuse' if @fact_list[:id] =~ /opensuse/i
+          @fact_list[:id] = 'opensuse' if /opensuse/i.match?(@fact_list[:id])
         end
 
         def process_name

lib/facter/resolvers/selinux.rb

@@ -26,7 +26,7 @@ def selinux_mountpoint
           mountpoint = ''
 
           output.each_line do |line|
-            next unless line =~ /selinuxfs/
+            next unless /selinuxfs/.match?(line)
 
             mountpoint = line.split("\s")[1]
             break
@@ -54,8 +54,8 @@ def read_selinux_config
           file_lines = Facter::Util::FileHelper.safe_readlines('/etc/selinux/config')
 
           file_lines.map do |line|
-            @fact_list[:config_mode] = line.split('=').last.strip if line =~ /^SELINUX=/
-            @fact_list[:config_policy] = line.split('=').last.strip if line =~ /^SELINUXTYPE=/
+            @fact_list[:config_mode] = line.split('=').last.strip if /^SELINUX=/.match?(line)
+            @fact_list[:config_policy] = line.split('=').last.strip if /^SELINUXTYPE=/.match?(line)
           end
 
           !file_lines.empty? ? true : false

lib/facter/resolvers/solaris/filesystems.rb

@@ -18,7 +18,7 @@ def read_sysdef_file(fact_name)
 
             file_content = Facter::Core::Execution.execute('/usr/sbin/sysdef', logger: log)
             files = file_content.split("\n").map do |line|
-              line.split('/').last if line =~ /^fs\.*/
+              line.split('/').last if /^fs\.*/.match?(line)
             end
 
             @fact_list[:file_systems] = files.compact.sort.join(',')

lib/facter/resolvers/virt_what.rb

@@ -28,18 +28,18 @@ def determine_xen(output)
           return unless xen_info
 
           xen_info = xen_info.to_s
-          return 'xenu' if xen_info =~ /xen-domu/
-          return 'xenhvm' if xen_info =~ /xen-hvm/
-          return 'xen0' if xen_info =~ /xen-dom0/
+          return 'xenu' if /xen-domu/.match?(xen_info)
+          return 'xenhvm' if /xen-hvm/.match?(xen_info)
+          return 'xen0' if /xen-dom0/.match?(xen_info)
         end
 
         def determine_other(output)
           values = output.split("\n")
           other_vm = values.first
           return unless other_vm
 
-          return 'zlinux' if other_vm =~ /ibm_systemz/
-          return retrieve_vserver if other_vm =~ /linux_vserver/
+          return 'zlinux' if /ibm_systemz/.match?(other_vm)
+          return retrieve_vserver if /linux_vserver/.match?(other_vm)
           return (values - ['redhat']).first if values.include?('redhat')
 
           other_vm
@@ -53,7 +53,7 @@ def retrieve_vserver
             parts = line.split("\s")
             next unless parts.size.equal?(2)
 
-            next unless parts[0] =~ /^s_context:|^VxID:/
+            next unless /^s_context:|^VxID:/.match?(parts[0])
             return @fact_list[:vserver] = 'vserver_host' if parts[1] == '0'
 
             return @fact_list[:vserver] = 'vserver'

lib/facter/resolvers/windows/virtualization.rb

@@ -42,9 +42,9 @@ def determine_hypervisor_by_manufacturer(comp)
             manufacturer = comp.Manufacturer
             if comp.Model =~ /^Virtual Machine/ && manufacturer =~ /^Microsoft/
               'hyperv'
-            elsif manufacturer =~ /^Xen/
+            elsif /^Xen/.match?(manufacturer)
               'xen'
-            elsif manufacturer =~ /^Amazon EC2/
+            elsif /^Amazon EC2/.match?(manufacturer)
               'kvm'
             else
               'physical'

lib/facter/resolvers/xen.rb

@@ -45,7 +45,7 @@ def detect_domains
           return if output.empty?
 
           output.each_line do |line|
-            next if line =~ /Domain-0|Name/
+            next if /Domain-0|Name/.match?(line)
 
             domain = line.match(/^([^\s]*)\s/)
             domain = domain&.captures&.first

lib/facter/util/facts/windows_release_finder.rb

@@ -13,7 +13,7 @@ def find_release(input)
             description = input[:description]
             kernel_version = input[:kernel_version]
 
-            if version =~ /10.0/
+            if /10.0/.match?(version)
               check_version_10_11(consumerrel, kernel_version)
             else
               check_version_6(version, consumerrel) || check_version_5(version, consumerrel, description) || version
@@ -47,7 +47,7 @@ def check_version_6(version, consumerrel)
           end
 
           def check_version_5(version, consumerrel, description)
-            return unless version =~ /5.2/
+            return unless /5.2/.match?(version)
             return 'XP' if consumerrel
 
             description == 'R2' ? '2003 R2' : '2003'

lib/facter/util/linux/dhcp.rb

@@ -45,7 +45,7 @@ def search_dhclient_leases(interface_name)
 
               lease_files.select do |file|
                 content = Facter::Util::FileHelper.safe_read("#{dir}#{file}", nil)
-                next unless content =~ /interface.*#{interface_name}/
+                next unless /interface.*#{interface_name}/.match?(content)
 
                 dhcp = content.match(/dhcp-server-identifier ([0-9]+\.[0-9]+\.[0-9]+\.[0-9]+)/)
                 return dhcp[1] if dhcp