wifi-wand 2.10.1 → 2.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 301b4e83a32c377d251c8cdef985299c26220e4c7322ba72fe43488aa050b94e
-  data.tar.gz: 1880892a4a794728a932ca5e42830ca546c46ce55403d0cf3b2059d92381c707
+  metadata.gz: 5f21e7e9af16e0eea5c7d254ef75ce061fbae1ada6c707201a189ec78f9238be
+  data.tar.gz: ec4b2ef369ca6e3186fe02b59e57c059f89a17b03eba2cdf760035b93fdaa9b9
 SHA512:
-  metadata.gz: a8aede3fee3958093e2c5cca0dbf8c01118d79cb2cca7cac815d83aa69f1c45a51f6fb94846cf0ec9bbbf6e5a88e29dd3644af1f5a51d3ad7d191a3c07238134
-  data.tar.gz: 6a7ed65e40957546032570d54e36db58157885de5bc4e11723e4564e7b4d0040ec433876c8a5b5c67943130be5293ae6ad2c159cabf82147464e7abb9338fcb3
+  metadata.gz: a86d2fd563801ca4a3259d200de197d8811c6f46360a805b85779fc488107045c8df20f6013c69866b61bdddcc40f11ff77a338bb2e0984eef8690d86a2016fd
+  data.tar.gz: cd16ad95aed459d769e51539a911797f004be30e7ad9dfab8c6352d2aaa226cbadcbe46d5d0c4075c58b6f478bb1ccb3f6a66408aa932b1111cf0205d5ea2e3c

data/RELEASE_NOTES.md

@@ -1,3 +1,9 @@
+## v2.11.0
+
+* Various fixes and clarifications.
+* Change implementation of available_network_names to use REXML; first implemented w/position number, then XPath.
+* Add attempt count to try_os_command_until in verbose mode.
+
 ## v2.10.1
 
 * Fix egregious bug; the 'a' command did not work if `airport` was not in the path; I should have been using the AIRPORT_CMD constant but hard coded `airport` instead.

data/lib/wifi-wand/command_line_interface.rb

@@ -1,5 +1,6 @@
 require_relative 'operating_systems'
 require 'ostruct'
+require_relative 'error'
 require_relative 'version'
 
 module WifiWand
@@ -79,11 +80,11 @@ ro[pen]                   - open resource (#{OPEN_RESOURCES.help_string})
 t[ill]                    - returns when the desired Internet connection state is true. Options:
                             1) 'on'/:on, 'off'/:off, 'conn'/:conn, or 'disc'/:disc
                             2) wait interval, in seconds (optional, defaults to 0.5 seconds)
-w[ifion]                  - is the wifi on?
+w[ifi_on]                 - is the wifi on?
 x[it]                     - exits this program (interactive shell mode only) (see also 'q')
 
 When in interactive shell mode:
-  * use quotes for string parameters such as method names.
+  * remember to quote string literals.
   * for pry commands, use prefix `%`.
 
 "
@@ -91,7 +92,7 @@ When in interactive shell mode:
   def initialize(options)
     @options = options
     current_os = OperatingSystems.new.current_os
-    raise "Could not determine operating system" if current_os.nil?
+    raise Error.new("Could not determine operating system") if current_os.nil?
     model_options = OpenStruct.new({
         verbose:   options.verbose,
         wifi_port: options.wifi_port
@@ -107,8 +108,6 @@ When in interactive shell mode:
   end
 
 
-  # Until command line option parsing is added, the only way to specify
-  # verbose mode is in the environment variable MAC_WIFI_OPTS.
   def verbose_mode
     options.verbose
   end
@@ -177,9 +176,9 @@ When in interactive shell mode:
     begin
       require 'pry'
     rescue LoadError
-      puts "The 'pry' gem and/or one of its prerequisites, required for running the shell, was not found." +
-               " Please `gem install pry` or, if necessary, `sudo gem install pry`."
-      exit(-1)
+      message = "The 'pry' gem and/or one of its prerequisites, required for running the shell, was not found." +
+          " Please `gem install pry` or, if necessary, `sudo gem install pry`."
+      raise Error.new(message)
     end
 
     print_help
@@ -197,13 +196,25 @@ When in interactive shell mode:
   end
 
 
-  # For use by the shell; when typing a command and options, it is passed to process_command_line
-  def method_missing(method_name, *method_args)
-    method_name = method_name.to_s
-    if find_command_action(method_name)
-      process_command_line(method_name, method_args)
+  # Look up the command name and, if found, run it. If not, execute the passed block.
+  def attempt_command_action(command, *args, &error_handler_block)
+    action = find_command_action(command)
+
+    if action
+      action.(*args)
     else
-      puts(%Q{"#{method_name}" is not a valid command or option. If you intend for this to be a string literal, use quotes or %q/Q{}.})
+      error_handler_block.call
+      nil
+    end
+  end
+
+
+  # For use by the shell when the user types the DSL commands
+  def method_missing(method_name, *method_args)
+    attempt_command_action(method_name.to_s, *method_args) do
+      puts(%Q{"#{method_name}" is not a valid command or option. } \
+          << 'If you intend for this to be a string literal, ' \
+          << 'use quotes or %q{}/%Q{}.')
     end
   end
 
@@ -214,18 +225,16 @@ When in interactive shell mode:
   # be in a form that the Ruby interpreter will recognize as a string,
   # i.e. single or double quotes, %q, %Q, etc.
   # Otherwise it will assume it's a method name and pass it to method_missing!
-  def process_command_line(command, args)
-    action = find_command_action(command)
-    if action
-      action.(*args)
-    else
+  def process_command_line
+    attempt_command_action(ARGV[0], *ARGV[1..-1]) do
       print_help
       raise BadCommandError.new(
-          %Q{Unrecognized command. Command was "#{command}" and options were #{args.inspect}.})
+          %Q{! Unrecognized command. Command was "#{ARGV.first.inspect}" and options were #{ARGV[1..-1].inspect}.})
     end
   end
 
 
+
   def quit
     if interactive_mode
       exit(0)
@@ -313,7 +322,6 @@ When in interactive shell mode:
   # Performs nameserver functionality.
   # @param subcommand 'get' or no arg to get, 'clear' to clear, and an array of IP addresses to set
   def cmd_na(*args)
-
     subcommand = if [[], ['get']].include?(args)
       :get
     elsif args == ['clear']
@@ -473,7 +481,6 @@ When in interactive shell mode:
       && \
       command_string.length >= cmd.min_string.length  # e.g. 'c' by itself should not work
     end
-
     result ? result.action : nil
   end
 
@@ -494,7 +501,7 @@ When in interactive shell mode:
     begin
       # By this time, the Main class has removed the command line options, and all that is left
       # in ARGV is the commands and their options.
-      process_command_line(ARGV[0], ARGV[1..-1])
+      process_command_line
     rescue BadCommandError => error
       separator_line = "! #{'-' * 75} !\n"
       puts '' << separator_line << error.to_s << "\n" << separator_line

data/lib/wifi-wand/error.rb

@@ -0,0 +1,4 @@
+module WifiWand
+  class Error < RuntimeError
+  end
+end

data/lib/wifi-wand/main.rb

@@ -37,8 +37,10 @@ class Main
         choice = v[0].downcase
 
         unless formatters.keys.include?(choice)
-          raise %Q{Output format "#{choice}" not in list of available formats} +
-                    " (#{formatters.keys.inspect})."
+          message = %Q{Output format "#{choice}" not in list of available formats} <<
+              " (#{formatters.keys})."
+          puts; puts message; puts
+          raise Error.new(message)
         end
 
         options.post_processor = formatters[choice]

data/lib/wifi-wand/models/base_model.rb

@@ -2,6 +2,8 @@ require 'json'
 require 'net/http'
 require 'tempfile'
 require 'uri'
+require_relative 'helpers/command_output_formatter'
+require_relative '../error'
 require_relative '../../wifi-wand'
 
 module WifiWand
@@ -33,30 +35,18 @@ class BaseModel
     @verbose_mode = options.verbose
 
     if options.wifi_port && (! is_wifi_port?(options.wifi_port))
-      raise "#{options.wifi_port} is not a Wi-Fi interface."
+      raise Error.new("#{options.wifi_port} is not a Wi-Fi interface.")
     end
     @wifi_port = options.wifi_port
   end
 
 
-  def banner_line
-    @banner_line ||= '-' * 79
-  end
-
-  def command_attempt_as_string(command)
-    "\n\n#{banner_line}\nCommand: #{command}\n"
-  end
-
-  def command_result_as_string(output)
-    "#{output}#{banner_line}\n\n"
-  end
-
 
 
   def run_os_command(command, raise_on_error = true)
 
     if @verbose_mode
-      puts command_attempt_as_string(command)
+      puts CommandOutputFormatter.command_attempt_as_string(command)
     end
 
     start_time = Time.now
@@ -64,7 +54,7 @@ class BaseModel
 
     if @verbose_mode
       puts "Duration: #{'%.4f' % [Time.now - start_time]} seconds"
-      puts command_result_as_string(output)
+      puts CommandOutputFormatter.command_result_as_string(output)
     end
 
     if $?.exitstatus != 0 && raise_on_error
@@ -94,7 +84,7 @@ class BaseModel
       success = true
 
       if @verbose_mode
-        puts command_attempt_as_string("[Calling Net:HTTP.start(#{url.host})]")
+        puts CommandOutputFormatter.command_attempt_as_string("[Calling Net:HTTP.start(#{url.host})]")
       end
 
       start_time = Time.now
@@ -110,7 +100,7 @@ class BaseModel
 
       if @verbose_mode
         puts "Duration: #{'%.4f' % [Time.now - start_time]} seconds"
-        puts command_result_as_string("#{success}\n")
+        puts CommandOutputFormatter.command_result_as_string("#{success}\n")
       end
 
       success
@@ -146,7 +136,7 @@ class BaseModel
     password     = password.to_s     if password
 
     if network_name.nil? || network_name.empty?
-      raise "A network name is required but was not provided."
+      raise Error.new("A network name is required but was not provided.")
     end
     wifi_on
     os_level_connect(network_name, password)
@@ -161,7 +151,7 @@ class BaseModel
         message << %Q{connected to "#{connected_network_name}" instead.}
       end
       message << ' Did you ' << (password ? "provide the correct password?" : "need to provide a password?")
-      raise message
+      raise Error.new(message)
     end
     nil
   end
@@ -181,7 +171,7 @@ class BaseModel
     if preferred_networks.include?(preferred_network_name)
       os_level_preferred_network_password(preferred_network_name)
     else
-      raise "Network #{preferred_network_name} not in preferred networks list."
+      raise Error.new("Network #{preferred_network_name} not in preferred networks list.")
     end
   end
 
@@ -225,12 +215,20 @@ class BaseModel
   # @stop_condition a lambda taking the command's stdout as its sole parameter
   # @return the stdout produced by the command
   def try_os_command_until(command, stop_condition, max_tries = 100)
-    max_tries.times do
+
+    report_attempt_count = ->(attempt_count) do
+      puts "Command was executed #{attempt_count} time(s)." if @verbose_mode
+    end
+
+    max_tries.times do |n|
       stdout = run_os_command(command)
       if stop_condition.(stdout)
+        report_attempt_count.(n + 1)
         return stdout
       end
     end
+
+    report_attempt_count.(max_tries)
     nil
   end
 

data/lib/wifi-wand/models/helpers/command_output_formatter.rb

@@ -0,0 +1,18 @@
+module WifiWand
+  module CommandOutputFormatter
+
+    module_function
+
+    def banner_line
+      @banner_line ||= '-' * 79
+    end
+
+    def command_attempt_as_string(command)
+      "\n\n#{banner_line}\nCommand: #{command}\n"
+    end
+
+    def command_result_as_string(output)
+      "#{output}#{banner_line}\n\n"
+    end
+  end
+end

data/lib/wifi-wand/models/mac_os_model.rb

@@ -1,8 +1,10 @@
 require 'ipaddr'
 require 'ostruct'
+require 'rexml/document'
 require 'shellwords'
 
 require_relative 'base_model'
+require_relative '../error'
 
 module WifiWand
 
@@ -16,6 +18,21 @@ class MacOsModel < BaseModel
   end
 
 
+  # Although at this time the airport command utility is predictable,
+  # allow putting it elsewhere in the path for overriding and easier fix
+  # if that location should change.
+  def airport_command
+    airport_in_path = `which airport`.chomp
+    if ! airport_in_path.empty?
+      airport_in_path
+    elsif File.exist?(AIRPORT_CMD)
+      AIRPORT_CMD
+    else
+      raise Error.new("Airport command not found.")
+    end
+  end
+
+
   # Identifies the (first) wireless network hardware port in the system, e.g. en0 or en1
   # This may not detect wifi ports with nonstandard names, such as USB wifi devices.
   def detect_wifi_port
@@ -35,7 +52,7 @@ class MacOsModel < BaseModel
     end
 
     if wifi_port_line_num.nil?
-      raise %Q{Wifi port (e.g. "en0") not found in output of: networksetup -listallhardwareports}
+      raise Error.new(%Q{Wifi port (e.g. "en0") not found in output of: networksetup -listallhardwareports})
     else
       lines[wifi_port_line_num + 1].split(': ').last
     end
@@ -56,7 +73,7 @@ class MacOsModel < BaseModel
   #     "DIRECT-sq-BRAVIA                 02:71:cc:87:4a:8c -76  6       Y  -- WPA2(PSK/AES/AES) ",  #
   def available_network_info
     return nil unless wifi_on? # no need to try
-    command = "#{AIRPORT_CMD} -s"
+    command = "#{airport_command} -s"
     max_attempts = 50
 
 
@@ -87,27 +104,59 @@ class MacOsModel < BaseModel
     if output
       process_tabular_data.(output)
     else
-      raise "Unable to get available network information after #{max_attempts} attempts."
+      raise Error.new("Unable to get available network information after #{max_attempts} attempts.")
     end
   end
 
 
-  # @return an array of unique available network names only, sorted alphabetically
-  # The reason we don't use an XML parser to get the exactly correct result is that we don't want
-  # users to need to install any external dependencies in order to run this script.
+  # The Mac OS airport utility (at
+  # /System/Library/PrivateFrameworks/Apple80211.framework/Versions/Current/Resources/airport)
+  # outputs the network names right padded with spaces so there is no way to differentiate a
+  # network name *with* leading space(s) from one without:
+  #
+  #                   SSID BSSID             RSSI CHANNEL HT CC SECURITY (auth/unicast/group)
+  #    ngHub_319442NL0293C 04:a1:51:58:5b:05 -65  11      Y  US WPA2(PSK/AES/AES)
+  #        NETGEAR89_2GEXT 9c:3d:cf:11:69:b4 -67  8       Y  US NONE
+  #
+  # To remedy this, they offer a "-x" option that outputs the information in (pseudo) XML.
+  # This XML has 'dict' elements that contain many elements. The SSID can be found in the
+  # XML element <string> which immediately follows an XML element whose text is "SSID_STR".
+  # Unfortunately, since there is no way to connect the two other than their physical location,
+  # the key is rather useless for XML parsing.
+  #
+  # I tried extracting the arrays of keys and strings, and finding the string element
+  # at the same position in the string array as the 'SSID_STR' was in the keys array.
+  # However, not all keys had string elements, so the index in the key array was the wrong index.
+  # Here is an excerpt from the XML output:
+  #
+  # 		<key>RSSI</key>
+  # 		<integer>-91</integer>
+  # 		<key>SSID</key>
+  # 		<data>
+  # 		TkVUR0VBUjY1
+  # 		</data>
+  # 		<key>SSID_STR</key>
+  # 		<string>NETGEAR65</string>
+  #
+  # The kludge I came up with was that the ssid was always the 2nd value in the <string> element
+  # array, so that's what is used here.
+  #
+  # But now even that approach has been superseded by the XPath approach now used.
+  #
+  # REXML is used here to avoid the need for the user to install Nokogiri.
   def available_network_names
+    # For some reason, the airport command very often returns nothing, so we need to try until
+    # we get data in the response:
 
-    # awk command below kindly provided by @nohillside at https://apple.stackexchange.com/questions/320323/how-to-programmatically-get-available-wifi-networks-without-airport-utility.
-    command = %Q{#{AIRPORT_CMD} -s -x | awk '{ if (catch == 1) { print; catch=0 } } /SSID_STR/ { catch=1 }'}
+    command = "#{airport_command} -s -x"
     stop_condition = ->(response) { ! [nil, ''].include?(response) }
-
     output = try_os_command_until(command, stop_condition)
-    output = output.split("\n")
-    output.map!(&:strip)
-    output.map! { |s| s.gsub(/<\/?string>/, '') }
-    output.sort! { |s1, s2| s1.casecmp(s2) }    # sort alphabetically, case insensitively
-    output.uniq!
-    output
+    doc = REXML::Document.new(output)
+    xpath = '//key[text() = "SSID_STR"][1]/following-sibling::*[1]' # provided by @ScreenStaring on Twitter
+    REXML::XPath.match(doc, xpath) \
+        .map(&:text) \
+        .sort { |x,y| x.casecmp(y) } \
+        .uniq
   end
 
 
@@ -136,7 +185,7 @@ class MacOsModel < BaseModel
 
   # Returns true if wifi is on, else false.
   def wifi_on?
-    lines = run_os_command("#{AIRPORT_CMD} -I").split("\n")
+    lines = run_os_command("#{airport_command} -I").split("\n")
     lines.grep("AirPort: Off").none?
   end
 
@@ -145,7 +194,7 @@ class MacOsModel < BaseModel
   def wifi_on
     return if wifi_on?
     run_os_command("networksetup -setairportpower #{wifi_port} on")
-    wifi_on? ? nil : raise("Wifi could not be enabled.")
+    wifi_on? ? nil : Error.new(raise("Wifi could not be enabled."))
   end
 
 
@@ -153,7 +202,7 @@ class MacOsModel < BaseModel
   def wifi_off
     return unless wifi_on?
     run_os_command("networksetup -setairportpower #{wifi_port} off")
-    wifi_on? ? raise("Wifi could not be disabled.") : nil
+    wifi_on? ? Error.new(raise("Wifi could not be disabled.")) : nil
   end
 
 
@@ -210,7 +259,7 @@ class MacOsModel < BaseModel
 
   # Returns the network currently connected to, or nil if none.
   def connected_network_name
-    lines = run_os_command("#{AIRPORT_CMD} -I").split("\n")
+    lines = run_os_command("#{airport_command} -I").split("\n")
     ssid_lines = lines.grep(/ SSID:/)
     ssid_lines.empty? ? nil : ssid_lines.first.split('SSID: ').last.lstrip
   end
@@ -218,7 +267,7 @@ class MacOsModel < BaseModel
 
   # Disconnects from the currently connected network. Does not turn off wifi.
   def disconnect
-    run_os_command("sudo #{AIRPORT_CMD} -z")
+    run_os_command("sudo #{airport_command} -z")
     nil
   end
 
@@ -253,7 +302,7 @@ class MacOsModel < BaseModel
         'nameservers' => nameservers_using_scutil,
         'timestamp'   => Time.now,
     }
-    more_output = run_os_command(AIRPORT_CMD + " -I")
+    more_output = run_os_command(airport_command + " -I")
     more_info   = colon_output_to_hash(more_output)
     info.merge!(more_info)
     info.delete('AirPort') # will be here if off, but info is already in wifi_on key
@@ -285,11 +334,12 @@ class MacOsModel < BaseModel
       end
 
       unless bad_addresses.empty?
-        raise "Bad IP addresses provided: #{bad_addresses.join(', ')}"
+        raise Error.new("Bad IP addresses provided: #{bad_addresses.join(', ')}")
       end
       nameservers.join(' ')
     end
     run_os_command("networksetup -setdnsservers Wi-Fi #{arg}")
+    nameservers
   end
 
 
@@ -313,6 +363,7 @@ class MacOsModel < BaseModel
   end
   private :colon_output_to_hash
 
+
   # @return array of nameserver IP addresses from /etc/resolv.conf, or nil if not found
   # Though this is strictly *not* OS-agnostic, it will be used by most OS's,
   # and can be overridden by subclasses (e.g. Windows).

data/lib/wifi-wand/models/model_validator.rb

@@ -0,0 +1,60 @@
+module WifiWand
+
+  class ModelValidator
+
+    BASE_MODEL_ESSENTIAL_METHODS = [
+        :connect,
+        :connected_to?,
+        :connected_to_internet?,
+        :cycle_network,
+        :preferred_network_password,
+        :public_ip_address_info,
+        :random_mac_address,
+        :remove_preferred_networks,
+        :run_os_command,
+        :till,
+        :try_os_command_until,
+        :verbose_mode,
+        :verbose_mode=,
+        :wifi_port,
+        :wifi_port=
+    ]
+
+
+    BASE_MODEL_NONESSENTIAL_METHODS = [
+    ]
+
+
+    MAC_OS_MODEL_ESSENTIAL_METHODS = [
+        :airport_command,
+        :available_network_info,
+        :available_network_names,
+        :connected_network_name,
+        :detect_wifi_port,
+        :disconnect,
+        :ip_address,
+        :is_wifi_port?,
+        :mac_address,
+        :nameservers_using_networksetup,
+        :nameservers_using_resolv_conf,
+        :nameservers_using_scutil,
+        :open_resource,
+        :os_level_connect,
+        :os_level_preferred_network_password,
+        :preferred_networks,
+        :remove_preferred_network,
+        :set_nameservers,
+        :wifi_info,
+        :wifi_off,
+        :wifi_on,
+        :wifi_on?
+    ]
+
+    MAC_OS_MODEL_ESSENTIAL_METHODS = [
+        ]
+
+    ALL_MODEL_METHODS = BASE_MODEL_ESSENTIAL_METHODS + MAC_OS_MODEL_ESSENTIAL_METHODS
+
+
+  end
+end

data/lib/wifi-wand/operating_systems.rb

@@ -1,3 +1,4 @@
+require_relative 'error'
 require_relative 'os/base_os'
 require_relative 'os/imaginary_os'
 require_relative 'os/mac_os'
@@ -31,7 +32,7 @@ class OperatingSystems
       matches = supported_operating_systems.select { |os| os.current_os_is_this_os? }
       if matches.size > 1
         matching_names = matches.map(&:display_name)
-        raise "There should only be 1 matching OS, but there were multiple: #{matching_names.inspect}"
+        raise Error.new("There should only be 1 matching OS, but there were multiple: #{matching_names.inspect}")
       end
       @current_os = matches.first
     end
@@ -39,8 +40,8 @@ class OperatingSystems
   end
 
 
-  def current_id;            current_os&.id;            end
-  def current_display_name;  current_os&.display_name;  end
+  def current_id;            current_os ? current_os.id : nil;            end
+  def current_display_name;  current_os ? current_os.display_name : nil;  end
 
 end
 end

data/lib/wifi-wand/version.rb

@@ -1,5 +1,5 @@
 module WifiWand
 
-  VERSION = '2.10.1'
+  VERSION = '2.11.0'
 
 end

data/wifi-wand.gemspec

@@ -7,8 +7,8 @@ Gem::Specification.new do |spec|
   spec.version       = WifiWand::VERSION
   spec.authors       = ["Keith Bennett"]
   spec.email         = ["keithrbennett@gmail.com"]
-  spec.description   = %q{A command line interface for managing wifi on a Mac.}
-  spec.summary       = %q{Mac wifi utility}
+  spec.description   = %q{A command line interface for managing WiFi on a Mac.}
+  spec.summary       = %q{Mac WiFi utility}
   spec.homepage      = "https://github.com/keithrbennett/wifi-wand"
   spec.license       = "MIT"
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: wifi-wand
 version: !ruby/object:Gem::Version
-  version: 2.10.1
+  version: 2.11.0
 platform: ruby
 authors:
 - Keith Bennett
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-04-28 00:00:00.000000000 Z
+date: 2018-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -52,7 +52,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
-description: A command line interface for managing wifi on a Mac.
+description: A command line interface for managing WiFi on a Mac.
 email:
 - keithrbennett@gmail.com
 executables:
@@ -68,9 +68,12 @@ files:
 - exe/wifi-wand
 - lib/wifi-wand.rb
 - lib/wifi-wand/command_line_interface.rb
+- lib/wifi-wand/error.rb
 - lib/wifi-wand/main.rb
 - lib/wifi-wand/models/base_model.rb
+- lib/wifi-wand/models/helpers/command_output_formatter.rb
 - lib/wifi-wand/models/mac_os_model.rb
+- lib/wifi-wand/models/model_validator.rb
 - lib/wifi-wand/operating_systems.rb
 - lib/wifi-wand/os/base_os.rb
 - lib/wifi-wand/os/imaginary_os.rb
@@ -105,5 +108,5 @@ rubyforge_project:
 rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
-summary: Mac wifi utility
+summary: Mac WiFi utility
 test_files: []