wifi-wand 2.19.1 → 2.20.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b9da3f2d69b3deb5046f7ea3394a43235733d9784c41bf372febdcca09c5d15d
-  data.tar.gz: 63f83f663cb0ad11f8fe5087aeda730e8dcd4a5108dad4b84c653d37cd2cd21e
+  metadata.gz: f6064b0904166cac0ccc9768c6bd385f422616d42418e6c42db8f38235fde354
+  data.tar.gz: 07a0a469d659dd82a5fbb80562dfbef16629aa2344ca22134e1f92c585717cac
 SHA512:
-  metadata.gz: 0be501742d8ad2e8ac317bf622b6f448af53293a8b90e563768aba12cf49a79488b3e8897928300be1150c6d021aa531c2fb809db0e9ff1346813268b347693c
-  data.tar.gz: a645d6e65afb64c19bbca0dd3443fc1b92067084897384df8da330df876093bfd5906b754dc18826b59810aeab65050dc9a64918692a4f291361ada7f87a9b2c
+  metadata.gz: 4366219cfc3d754a69f21055ee489e2e458d7d969f6103353b203acbf8bc2d3419389288e90dc039c350f77bb8dbe90d8821e61e884fa8098483036810a7f9eb
+  data.tar.gz: cd4fe96013c357f5cea0e09d740ef723a460c046b0f8f48d2726b957675e87f305455a6f46f91d38d50b49c27ba87c8a7b7cdb39c55333a02e60cd689b0dc1b7

data/RELEASE_NOTES.md

@@ -1,3 +1,10 @@
+## v2.20.0
+
+* Change detect_wifi_interface and available_network_names to use system_profiler JSON output.
+* Previously, detect_wifi_interface parsed human readable text; parsing JSON is more reliable.
+* Previously, available_network_names used Swift and CoreLAN and required XCode installation.
+
+
 ## v2.19.1
 
 * Fix connected_network_name when wifi is on but no network is connected.

data/lib/wifi-wand/command_line_interface.rb

@@ -219,9 +219,17 @@ When in interactive shell mode:
       if post_processor
         puts post_processor.(info)
       else
-        puts model.wifi_on? \
-            ? "Available networks are:\n\n#{fancy_string(info)}" \
-            : "Wifi is off, cannot see available networks."
+        message = if model.wifi_on?
+          <<~MESSAGE
+            Available networks, in descending signal strength order, 
+            and not including any currently connected network, are:
+  
+            #{fancy_string(info)}"
+          MESSAGE
+        else
+          "Wifi is off, cannot see available networks."
+        end
+        puts message
       end
     end
   end

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

@@ -14,17 +14,16 @@ class BaseModel
   def initialize(options)
     @verbose_mode = options.verbose
 
-    if options.wifi_interface && (! is_wifi_interface?(options.wifi_interface))
-      raise Error.new("#{options.wifi_interface} is not a Wi-Fi interface.")
+    if options.wifi_interface
+      if is_wifi_interface?(options.wifi_interface)
+        @wifi_interface = options.wifi_interface
+      else
+        raise Error.new("#{options.wifi_interface} is not a Wi-Fi interface.")
+      end
     end
-    @wifi_interface = options.wifi_interface
   end
 
-
-
-
   def run_os_command(command, raise_on_error = true)
-
     if verbose_mode
       puts CommandOutputFormatter.command_attempt_as_string(command)
     end
@@ -46,16 +45,15 @@ class BaseModel
 
 
   # This method returns whether or not there is a working Internet connection,
-  # which is defined as name resolution and HTTP get being successful.
-  # Domains attempted are google.com and baidu.com. Success is either being successful.
+  # which is defined as success for both name resolution and an HTTP get.
+  # Domains attempted are google.com and baidu.com.
+  # Success is defined as either being successful.
   # Commands for the multiple sites are run in parallel, in threads, to save time.
   def connected_to_internet?
-    return nil unless wifi_on? # no need to try
+    return false unless wifi_on? # no need to try
 
     # We cannot use run_os_command for the running of external processes here,
     # because they are multithreaded, and the output will get mixed up.
-
-    # First pass to fail quickly if name resolving does not work.
     test_using_dig = -> do
       domains = %w(google.com  baidu.com)
       puts "Calling dig on domains #{domains}..." if verbose_mode
@@ -185,8 +183,8 @@ class BaseModel
   # @param wait_interval_in_secs sleeps this interval between retries; if nil or absent,
   #        a default will be provided
   #
+  # Connected is defined as being able to connect to an external web site.
   def till(target_status, wait_interval_in_secs = nil)
-
     # One might ask, why not just put the 0.5 up there as the default argument.
     # We could do that, but we'd still need the line below in case nil
     # was explicitly specified. The default argument of nil above emphasizes that
@@ -217,7 +215,7 @@ class BaseModel
   # Tries an OS command until the stop condition is true.
   # @command the command to run in the OS
   # @stop_condition a lambda taking the command's stdout as its sole parameter
-  # @return the stdout produced by the command
+  # @return the stdout produced by the command, or nil if max_tries was reached
   def try_os_command_until(command, stop_condition, max_tries = 100)
 
     report_attempt_count = ->(attempt_count) do
@@ -225,10 +223,10 @@ class BaseModel
     end
 
     max_tries.times do |n|
-      stdout = run_os_command(command)
-      if stop_condition.(stdout)
+      stdout_text = run_os_command(command)
+      if stop_condition.(stdout_text)
         report_attempt_count.(n + 1)
-        return stdout
+        return stdout_text
       end
     end
 

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

@@ -16,7 +16,7 @@ class MacOsModel < BaseModel
 
   # Identifies the (first) wireless network hardware interface 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_interface
+  def detect_wifi_interface_using_networksetup
 
     lines = run_os_command("networksetup -listallhardwareports").split("\n")
     # Produces something like this:
@@ -39,10 +39,34 @@ class MacOsModel < BaseModel
     end
   end
 
+  # Identifies the (first) wireless network hardware interface 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_interface
+    json_text = run_os_command('system_profiler -json SPNetworkDataType')
+    net_data = JSON.parse(json_text)
+    nets = net_data['SPNetworkDataType']
+    wifi = nets.detect { |net| net['_name'] == 'Wi-Fi'}
+    wifi['interface']
+  end
+
+  # Returns the network names sorted in descending order of signal strength.
   def available_network_names
     return nil unless wifi_on? # no need to try
 
-    run_swift_command('AvailableWifiNetworkLister').split("\n")
+    # run_swift_command('AvailableWifiNetworkLister').split("\n").uniq
+
+    json_text = run_os_command('system_profiler -json SPAirPortDataType')
+    data = JSON.parse(json_text)
+
+    inner_key = connected_network_name ? 'spairport_airport_other_local_wireless_networks' : 'spairport_airport_local_wireless_networks'
+
+    nets = data['SPAirPortDataType'] \
+       .detect { |h| h.key?('spairport_airport_interfaces') } \
+        ['spairport_airport_interfaces'] \
+       .detect { |h| h['_name'] == wifi_interface } \
+        [inner_key] \
+        .sort_by { |net| -net['spairport_signal_noise'].split('/').first.to_i }
+    nets.map { |h| h['_name']}.uniq
   end
 
 

data/lib/wifi-wand/version.rb

@@ -1,3 +1,3 @@
 module WifiWand
-  VERSION = '2.19.1' unless defined?(VERSION)
+  VERSION = '2.20.0' unless defined?(VERSION)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wifi-wand
 version: !ruby/object:Gem::Version
-  version: 2.19.1
+  version: 2.20.0
 platform: ruby
 authors:
 - Keith Bennett