run_loop 1.1.1.pre6 → 1.1.1.pre7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d9b31f9972e25c59de2b72ab0afbc80b9b466e38
-  data.tar.gz: e6f49c6b0685ccc5278bbd132a86cd224810ae3d
+  metadata.gz: 1420a614a1d2f74b9cbb6e7bf376fd1521eebd01
+  data.tar.gz: 26965697486e6081811a4f876e333d0d77f4750e
 SHA512:
-  metadata.gz: 61340a659acd2465aac81330d51fb4b8ccd09e9123d02204f5356152743ef4c742a505e10864aed55c4c57776725ca8d85f5448a084fc8d398afdf4f8540ff47
-  data.tar.gz: 7d07874bd804d87ceab4b1db43d129c71d2077cc8c42b8c2ebf7e6d48ca6ec8256fcd17d8226a7c54d172b95b6d1f9fcfa0c652144d0ac9c8125447f22d9e02d
+  metadata.gz: 805cfd4180e6566a36e4b518c102a10dd5ce93f2e405d14db8d13904e9ef6425aeaf2b64d316b963b718492250cf1582b8dc4f82bc487fe6124dc40689151d7f
+  data.tar.gz: a43ef171976bb68f1ed6b19a582a29b1fec64dcfe6cbdaf3a710c229a941bf6c18ff091948aabdc74a2af21fcee7f984a01fe7ba180a9a9bbf39aa814b019921

data/lib/run_loop/core.rb

@@ -721,21 +721,64 @@ module RunLoop
     Core.run_with_options(options)
   end
 
-  def self.send_command(run_loop, cmd, timeout=60)
+  def self.send_command(run_loop, cmd, options={timeout: 60}, num_retries=0, last_error=nil)
+    if num_retries > 3
+      if last_error
+        raise last_error
+      else
+        raise "Max retries exceeded #{num_retries} > 3. No error recorded."
+      end
+    end
+
+    if options.is_a?(Numeric)
+      options = {timeout: options}
+    end
 
     if not cmd.is_a?(String)
       raise "Illegal command #{cmd} (must be a string)"
     end
 
+    if not options.is_a?(Hash)
+      raise "Illegal options #{options} (must be a Hash (or number for compatibility))"
+    end
+
+    timeout = options[:timeout] || 60
+    logger = options[:logger]
+    interrupt_retry_timeout = options[:interrupt_retry_timeout] || 25
 
-    expected_index = Core.write_request(run_loop, cmd)
+    expected_index = run_loop[:index]
     result = nil
+    begin
+      expected_index = Core.write_request(run_loop, cmd)
+    rescue Errno::EINTR => intr_error
+      # Attempt recover from interrupt by attempting to read result (assuming write went OK)
+      # or retry if attempted read result fails
+      run_loop[:index] = expected_index # restore expected index in case it changed
+      log_info(logger, "Core.write_request was interrupted: #{intr_error}. Attempting recovery...")
+      sleep(1) # Arbitrary wait in hope that the system condition causing the interrupt passes
+      if intr_error && intr_error.backtrace
+        log_info(logger, "backtrace: #{intr_error.backtrace.join("\n")}")
+      end
+      log_info(logger, "Attempting read in case the request was received... Please wait (#{interrupt_retry_timeout})...")
+      begin
+        Timeout::timeout(interrupt_retry_timeout, TimeoutError) do
+          result = Core.read_response(run_loop, expected_index)
+        end
+        # Update run_loop expected index since we succeeded in reading the index
+        run_loop[:index] = expected_index + 1
+        log_info(logger, "Did read response for interrupted request of index #{expected_index}... Proceeding.")
+        return result
+      rescue TimeoutError => _
+        log_info(logger, "Read did not result in a response for index #{expected_index}... Retrying send_command...")
+        return send_command(run_loop, cmd, options, num_retries+1, intr_error)
+      end
+    end
+
 
     begin
       Timeout::timeout(timeout, TimeoutError) do
         result = Core.read_response(run_loop, expected_index)
       end
-
     rescue TimeoutError => _
       raise TimeoutError, "Time out waiting for UIAutomation run-loop for command #{cmd}. Waiting for index:#{expected_index}"
     end
@@ -776,4 +819,12 @@ module RunLoop
     script
   end
 
+  def self.log_info(device_logger, message)
+    msg = "#{Time.now}: #{message}"
+    if device_logger && device_logger.respond_to?(:info)
+      logger.info(msg)
+    else
+      puts msg if ENV['DEBUG'] == '1'
+    end
+  end
 end

data/lib/run_loop/version.rb

@@ -1,5 +1,5 @@
 module RunLoop
-  VERSION = '1.1.1.pre6'
+  VERSION = '1.1.1.pre7'
 
   # A model of a software release version that can be used to compare two versions.
   #

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: run_loop
 version: !ruby/object:Gem::Version
-  version: 1.1.1.pre6
+  version: 1.1.1.pre7
 platform: ruby
 authors:
 - Karl Krukow