puppet-autostager 0.0.9 → 0.0.10

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NzU0ZTkxMjMwYzUyNDRiNjg3ZTdiNmY1YTc3MjUyYjkwZGYwMjczMQ==
+    ZTc4YTFlMGVhODg4YjcxODdhNjlmNjc5ZGZjNDFmMWRlMTU5MmNkNQ==
   data.tar.gz: !binary |-
-    ZGU1ZjkzYTAzZTU4NDY5YWYyY2VmMThkMTYxMjQ5YjhhODQxOTY0ZA==
+    YTU4ZmI3YzI2MmFhMjFjNjczNGZmY2E3NzEzNWE2NThjNjUxMzNhOA==
 SHA512:
   metadata.gz: !binary |-
-    YWI5MWIxOWZmMDhhYjU5OWUyNTkwYzM4ZGRlY2Y0ZjM0YmQ4N2RmMDU3M2Uy
-    ZDUxODNiNmNiYzQ2ZTg1Nzk1Yzk3NmFhMzkxNWFhYjFkYzE3OGFlNjBmMzcy
-    NjEyMjFmMjk2ZWZjYmRiNzcxZTIzOWI3NjcwMGI1MWUzMzE2YjE=
+    YWFlZWE2YmRlYjBkMDk4OTQ0MDVlYWY5OWI0MTAwNjJjMGYzMDAzN2ZlOTMy
+    NmM5MTA4YzYyMzVmNmI0MmY3NTZmNjAxOTA3YWVjZjZjYWYyZWQxYzBlNjI3
+    NzFjNWQ1OTZlMjNiYjhhYmIxZjAzOTcyNTg4M2ZiZjI0YWE5Y2M=
   data.tar.gz: !binary |-
-    YjEyOTg3ZjdjM2ViNzUzMWE0NzI2ODE2ZjliZjhkNTgxZWIyZDY0MzA2ZjE2
-    NzE3YjNlYzMxYzgwZTkzYjIxNDJhMWJjZDFhYWQyNzUxODA5NjE2NmY1YjVl
-    YWUzNmFjZDhlZTk2M2FhZWRiNzY0YTI1NWI4ZTkzZmM2ZmJmY2Y=
+    MmMwMmZkZTRhNTYyNmRhMTk3MjU5MWZiZDZmMmNlN2NjZDc4YjBjYzJmMWE5
+    N2JhOTI4YmE2MzQ0NDUyODFhYjJlZjg3ODA5MjQ2NTA3YTQwYmFlYzkxMGQw
+    NzMzYjEzNDg4NDkyMzRhNWIxZGM2YTVhMGJiZjgyYTM2ZjJmNGY=

data/circle.yml

@@ -1,7 +1,7 @@
 machine:
   environment:
     # If you change this, you need to change lib/autostager/version.rb
-    VERSION: 0.0.9
+    VERSION: 0.0.10
 
 test:
   post:

data/lib/autostager.rb

@@ -127,9 +127,12 @@ module Autostager
   end
 
   def timeout_seconds
-    ENV['timeout'].to_i || 120
-  rescue
-    120
+    result = 120
+    if ENV.key?('timeout')
+      result = ENV['timeout'].to_i
+      fail 'timeout must be greater than zero seconds' if result <= 0
+    end
+    result
   end
 
   # A list of directories we never discard.

data/lib/autostager/cli.rb

@@ -8,6 +8,7 @@ module Autostager
 
     # Entry point for the app.
     # Stage pull requests on a 30-second loop.
+    # rubocop:disable MethodLength
     def run
       trap_interrupt
       loop do
@@ -23,6 +24,7 @@ module Autostager
       log 'Exit on interrupt'
       exit!(0)
     end
+    # rubocop:enable MethodLength
 
     def sleep_interval
       ENV['sleep_interval'].to_i || 30

data/lib/autostager/version.rb

@@ -3,5 +3,5 @@
 #
 # If you change this, you need to also change circle.yaml
 module Autostager
-  VERSION = '0.0.9'
+  VERSION = '0.0.10'
 end

data/spec/spec_helper.rb

@@ -4,6 +4,7 @@ require 'mocha/setup'
 require 'English'
 require 'tempfile'
 require 'pp'
+require 'autostager'
 
 RSpec.configure do |c|
   c.mock_with :mocha

data/spec/unit/autostager_spec.rb

@@ -0,0 +1,34 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe Autostager do
+  describe 'timeout_seconds' do
+    it 'defaults to 120 seconds' do
+      ENV['timeout'] = nil
+      Autostager.timeout_seconds.should eql(120)
+    end
+
+    it 'should use ENV["timeout"] if exists' do
+      ENV['timeout'] = '1'
+      Autostager.timeout_seconds.should eql(1)
+    end
+
+    it 'fails if timeout is zero' do
+      ENV['timeout'] = '0'
+      expect { Autostager.timeout_seconds }.to raise_error(RuntimeError)
+    end
+
+    it 'fails if timeout is negative' do
+      ENV['timeout'] = '-5'
+      expect { Autostager.timeout_seconds }.to raise_error(RuntimeError)
+    end
+
+    it 'dynamically adjusts based on env var' do
+      ENV['timeout'] = '1'
+      Autostager.timeout_seconds.should eql(1)
+
+      ENV['timeout'] = '2'
+      Autostager.timeout_seconds.should eql(2)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puppet-autostager
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.10
 platform: ruby
 authors:
 - Paul Morgan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-03 00:00:00.000000000 Z
+date: 2015-09-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -231,6 +231,7 @@ files:
 - script/test
 - spec/functional/placeholder_spec.rb
 - spec/spec_helper.rb
+- spec/unit/autostager_spec.rb
 - spec/unit/friction_spec.rb
 - templates/credentials
 homepage: https://github.com/jumanjiman/autostager
@@ -260,4 +261,5 @@ summary: Create a puppet enviroment for each pull request
 test_files:
 - spec/functional/placeholder_spec.rb
 - spec/spec_helper.rb
+- spec/unit/autostager_spec.rb
 - spec/unit/friction_spec.rb