calabash-cucumber 0.9.145.pre2 → 0.9.145.pre3

data/features-skeleton/support/01_launch.rb

@@ -42,5 +42,7 @@ end
 
 at_exit do
   launcher = Calabash::Cucumber::Launcher.new
-  Calabash::Cucumber::SimulatorHelper.stop unless launcher.calabash_no_stop?
+  if launcher.simulator_target?
+    Calabash::Cucumber::SimulatorHelper.stop unless launcher.calabash_no_stop?
+  end
 end

data/lib/calabash-cucumber/core.rb

@@ -7,7 +7,10 @@ module Calabash
 
       DATA_PATH = File.expand_path(File.dirname(__FILE__))
       CAL_HTTP_RETRY_COUNT=3
-      RETRYABLE_ERRORS = [Errno::ECONNREFUSED, Errno::ECONNRESET, Errno::ECONNABORTED, Errno::ETIMEDOUT]
+      RETRYABLE_ERRORS = [HTTPClient::TimeoutError,
+                          HTTPClient::KeepAliveDisconnected,
+                          Errno::ECONNREFUSED, Errno::ECONNRESET, Errno::ECONNABORTED,
+                          Errno::ETIMEDOUT]
 
       def macro(txt)
         if self.respond_to? :step
@@ -466,10 +469,11 @@ EOF
 
       def calabash_exit
         # Exiting the app shuts down the HTTP connection and generates ECONNREFUSED,
+        # or HTTPClient::KeepAliveDisconnected
         # which needs to be suppressed.
         begin
-          http({:method =>:post, :path => 'exit', :retryable_errors => RETRYABLE_ERRORS - [Errno::ECONNREFUSED]})
-        rescue Errno::ECONNREFUSED
+          http({:method =>:post, :path => 'exit', :retryable_errors => RETRYABLE_ERRORS - [Errno::ECONNREFUSED,HTTPClient::KeepAliveDisconnected]})
+        rescue Errno::ECONNREFUSED, HTTPClient::KeepAliveDisconnected
           []
         end
       end
@@ -548,6 +552,7 @@ EOF
       end
 
       def make_http_request(options)
+
         body = nil
         retryable_errors = options[:retryable_errors] || RETRYABLE_ERRORS
         CAL_HTTP_RETRY_COUNT.times do |count|
@@ -561,29 +566,20 @@ EOF
               body = @http.get(options[:uri], options[:body]).body
             end
             break
-          rescue HTTPClient::TimeoutError, HTTPClient::KeepAliveDisconnected => e
-            if count < CAL_HTTP_RETRY_COUNT-1
-              @http.reset_all
-              @http=nil
-              STDOUT.write "Waiting 5 secs before retry...\n"
-              sleep(5)
-              STDOUT.write "Retrying.. #{e.class}: (#{e})\n"
-              STDOUT.flush
+          rescue Exception => e
 
-            else
-              puts "Failing... #{e.class}"
-              raise e
-            end
+            if retryable_errors.include?(e) || retryable_errors.any?{|c| e.is_a?(c)}
 
-          rescue Exception => e
-            if retryable_errors.include?(e)
               if count < CAL_HTTP_RETRY_COUNT-1
-                sleep(0.5)
+                if e.is_a?(HTTPClient::TimeoutError)
+                  sleep(3)
+                else
+                  sleep(0.5)
+                end
                 @http.reset_all
                 @http=nil
                 STDOUT.write "Retrying.. #{e.class}: (#{e})\n"
                 STDOUT.flush
-
               else
                 puts "Failing... #{e.class}"
                 raise e

data/lib/calabash-cucumber/launcher.rb

@@ -25,7 +25,7 @@ class Calabash::Cucumber::Launcher
   end
 
   def device_target?
-    ENV['DEVICE_TARGET'] == 'device'
+    (ENV['DEVICE_TARGET'] != nil) && (not simulator_target?)
   end
 
   def simulator_target?
@@ -68,6 +68,10 @@ class Calabash::Cucumber::Launcher
 
     if device_target?
       default_args = {:app => ENV['BUNDLE_ID']}
+      target = ENV['DEVICE_TARGET']
+      if target != 'DEVICE'
+        default_args[:udid] = target
+      end
       self.run_loop = RunLoop.run(default_args.merge(args))
     else
 

data/lib/calabash-cucumber/version.rb

@@ -1,6 +1,6 @@
 module Calabash
   module Cucumber
-    VERSION = '0.9.145.pre2'
-    FRAMEWORK_VERSION = '0.9.145.pre2'
+    VERSION = '0.9.145.pre3'
+    FRAMEWORK_VERSION = '0.9.145.pre3'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: calabash-cucumber
 version: !ruby/object:Gem::Version
-  version: 0.9.145.pre2
+  version: 0.9.145.pre3
   prerelease: 8
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-07 00:00:00.000000000 Z
+date: 2013-05-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber