greenjaguar 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e4fc61d4fffe390ea3af3fabff9edd69042b4586
-  data.tar.gz: 64b48478ccdfa7af0887c669d969aa11865b135c
+  metadata.gz: 84523e4db04a5527e3dab130b50a0b4a68bf6ab1
+  data.tar.gz: 515888c081b94b8733ad865b21cfafd62b32789e
 SHA512:
-  metadata.gz: e5fc9b39e21fbe19bc83d2bf767005614298df222cd990c8853bbcc987b37fa3b7bcd13ae1ca50c8fc23767a1e328c3700bfd85730709b69f2235fde25caaf92
-  data.tar.gz: 1d30022a6c2d4506cdf8c200aec18af14b3dfd598c84886e85804a6d776db767b00b8d38750a5b265cece72269af86d40d58ee005f09bc1f684c59eb128ca112
+  metadata.gz: 6c60c04211a67629397bbbe3666b834ce481296b48e13d44541dc1263898052ec57c0f8e5f19bd372212e1a8a78b410548f3da4ba30693dfb8dd4ded179b8c35
+  data.tar.gz: 40f915a3bcff528b9233c485eeb14d7a61ef5730aa68bd1fc9b4b14c9455badf2b3216bec423282a00dee0c0d71d8224fdb860ca915ef7f5924b721e8018aadb

data/lib/greenjaguar/policy_builder.rb

@@ -1,6 +1,6 @@
 module Greenjaguar
   class PolicyBuilder
-    attr_accessor :count, :wait_strategy, :timeout, :logger, :strategy, :exceptions
+    attr_accessor :count, :wait_strategy, :timeout, :logger, :exceptions
 
     def initialize(&block)
       @count = 1
@@ -41,7 +41,7 @@ module Greenjaguar
     end
 
     def measure_time_in(time_unit)
-      @strategy.time_unit = time_unit
+      strategy.time_unit = time_unit
       self
     end
 
@@ -67,12 +67,12 @@ module Greenjaguar
       strategy.wait
     end
 
-    private
-
     def strategy
       @strategy ||= init_wait_strategy(:default)
     end
 
+    private
+
     def init_wait_strategy(wait_strategy, *args)
       case wait_strategy
         when :fibonacci

data/lib/greenjaguar/version.rb

@@ -1,3 +1,3 @@
 module Greenjaguar
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

data/spec/greenjaguar_spec.rb

@@ -15,7 +15,7 @@ module Specs
       WebMock.reset!
     end
 
-    it '#run should call the passed code block 4 times' do
+    it 'should call the passed code block 4 times' do
       policy = @object_under_test.build_policy do
         times 3
       end
@@ -28,7 +28,7 @@ module Specs
       assert_requested :get, "http://www.example.com", :times => 4
     end
 
-    it '#run should call the passed code block only 1 time if successful response is received' do
+    it 'should call the passed code block only 1 time if successful response is received' do
       @stub = stub_request(:get, "http://www.example.com")
 
       policy = @object_under_test.build_policy do
@@ -41,7 +41,7 @@ module Specs
       assert_requested :get, "http://www.example.com", :times => 1
     end
 
-    it '#run should raise the error once retrying is completed' do
+    it 'should raise the error once retrying is completed' do
       policy = @object_under_test.build_policy do
         times 3
       end
@@ -53,7 +53,7 @@ module Specs
       end.to raise_error
     end
 
-    it '#run should call the passed code block 4 times according to fibonacci sequence' do
+    it 'should call the passed code block 4 times according to fibonacci sequence' do
       policy = @object_under_test.build_policy do
         times 3
         with_strategy :fibonacci
@@ -68,7 +68,7 @@ module Specs
       assert_requested :get, "http://www.example.com", :times => 4
     end
 
-    it '#run should call the passed code block 4 times according to fixed interval strategy' do
+    it 'should call the passed code block 4 times according to fixed interval strategy' do
       policy = @object_under_test.build_policy do
         times 3
         with_strategy :fixed_interval, 2
@@ -83,7 +83,7 @@ module Specs
       assert_requested :get, "http://www.example.com", :times => 4
     end
 
-    it '#run should call the passed code block 4 times according to exponential backoff sequence' do
+    it 'should call the passed code block 4 times according to exponential backoff sequence' do
       policy = @object_under_test.build_policy do
         times 5
         with_strategy :exponential_backoff
@@ -98,7 +98,7 @@ module Specs
       assert_requested :get, "http://www.example.com", :times => 6
     end
 
-    it '#run does not call the passed code block if exception is not part of allowed exception(s)' do
+    it 'does not call the passed code block if exception is not part of allowed exception(s)' do
       @stub = stub_request(:get, "www.example.com").to_raise(RegexpError)
       policy = @object_under_test.build_policy do
         times 5
@@ -114,7 +114,7 @@ module Specs
       assert_requested :get, "http://www.example.com", :times => 1
     end
 
-    it '#run should call the passed code block if exception is part of allowed exception(s)' do
+    it 'should call the passed code block if exception is part of allowed exception(s)' do
       @stub = stub_request(:get, "http://www.example.com").to_raise(ZeroDivisionError)
 
       policy = @object_under_test.build_policy do
@@ -132,7 +132,7 @@ module Specs
       assert_requested :get, "http://www.example.com", :times => 11
     end
 
-    it '#run should not raise the error if set to fail silently' do
+    it 'should not raise the error if set to fail silently' do
       policy = @object_under_test.build_policy do
         times 3
         fail_silently
@@ -143,5 +143,15 @@ module Specs
       end
     end
 
+    it 'should use default strategy when none is specified' do
+      policy = @object_under_test.build_policy do
+        times 3
+        measure_time_in :ms
+        fail_silently
+      end
+
+      expect(policy.strategy.class).to eql(Greenjaguar::Strategies::DefaultWaitStrategy)
+    end
+
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: greenjaguar
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Aniruddha Deshpande