build_status_server 0.11 → 0.12

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    build_status_server (0.11)
+    build_status_server (0.12)
       json
 
 GEM

data/lib/build_status_server/server.rb

@@ -147,13 +147,20 @@ The address configured is not available (#{address})
         STDERR.puts "Error: #{ex} while trying to send #{light}"
         retry unless attempts > tcp_client["attempts"]
       rescue Errno::ECONNREFUSED, Errno::EHOSTUNREACH => ex
+        wait = wait_for(attempts)
         STDERR.puts "Error: #{ex} while trying to send #{light}"
-        STDERR.puts "Will wait for 2 seconds and try again..."
-        sleep 2
+        STDERR.puts "Will wait for #{wait} seconds and try again..."
+        # sleep 2 seconds the first attempt, 4 the next, 8 the following...
+        sleep wait
         retry unless attempts > tcp_client["attempts"]
       ensure
         client.close if client
       end
     end
+
+    def wait_for(attempt = 1)
+      return 60 if attempt > 5 # Cap at one minute wait
+      2**attempt
+    end
   end
 end

data/lib/build_status_server/version.rb

@@ -1,3 +1,3 @@
 module BuildStatusServer
-  VERSION = "0.11"
+  VERSION = "0.12"
 end

data/spec/lib/build_status_server/server_spec.rb

@@ -352,13 +352,15 @@ describe BuildStatusServer::Server do
         it "should not connect and retry 2 times" do
           STDERR.should_receive(:puts).with("Error: Connection refused while trying to send fail")
           STDERR.should_receive(:puts).with("Error: No route to host while trying to send fail")
-          STDERR.should_receive(:puts).exactly(2).with("Will wait for 2 seconds and try again...")
+          STDERR.should_receive(:puts).with("Will wait for 2 seconds and try again...")
+          STDERR.should_receive(:puts).with("Will wait for 4 seconds and try again...")
 
           TCPSocket.should_receive(:new).with("host", "port").and_raise(Errno::ECONNREFUSED)
           TCPSocket.should_receive(:new).with("host", "port").and_raise(Errno::EHOSTUNREACH)
           TCPSocket.should_receive(:new).with("host", "port").and_return(client)
 
-          server.should_receive(:sleep).twice.with(2)
+          server.should_receive(:sleep).with(2)
+          server.should_receive(:sleep).with(4)
 
           client.should_receive(:print).with("GET fail HTTP/1.0\n\n")
 
@@ -424,6 +426,18 @@ describe BuildStatusServer::Server do
       end
     end
 
+    describe "#wait_for" do
+      it "should return 2**n for each attempt n up to 5" do
+        server.send(:wait_for, 1).should == 2
+        server.send(:wait_for, 5).should == 32
+      end
+
+      it "should return 60 for anything over 5" do
+        server.send(:wait_for, 6).should == 60
+        server.send(:wait_for, 1000).should == 60
+      end
+    end
+
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: build_status_server
 version: !ruby/object:Gem::Version
-  version: '0.11'
+  version: '0.12'
   prerelease: 
 platform: ruby
 authors: