circuit_breakage 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 44c83ed5565612433627a4466aa4486ebdf25543
-  data.tar.gz: a0f65eb73e9dd27f832adf19d860e979112efb9e
+  metadata.gz: cba4ea6a897024a9a64f381298b155806e47ade7
+  data.tar.gz: bda065d0f7132b76891004752add0cc948bcfc6e
 SHA512:
-  metadata.gz: 1eaece305ff4a9fb83f1cd4f1a138930b28bde2724bd70c8d0d6fae16a67afa0a94e8514034fe9272133530fb7cd5ad349a264316490ddbf026c2e02e28ea879
-  data.tar.gz: 79ca29de3a040028ff01023c07889ce12fcf88b249fbb694082ce87e48737e3083bbfbd59ba1b875027d3e0965891c68215a0888d6e8ccb0e1fd324b58f6bef1
+  metadata.gz: 04e698f1eff6b98ad18a4ad6359c96db40d7756a3bec6c04c1e44f78aecf5f05781cd1f506631139535c1ee9f8c47585a915750e8d44e3dcafa79eb0cf4196c7
+  data.tar.gz: 9d5cfd0d6bf4e8b3ba7ef1f349e28f27ab92b9951c3dde89a94bc30e7ff0b5bb43bc3eabcdd79600a3f60a445c1a6112d5748a9098cf30e036a210f9f626fadd

data/README.md

@@ -18,10 +18,16 @@ proc = ->(*args) do
 end
 
 breaker = CircuitBreakage::Breaker.new(proc)
+
+# These options are required.
 breaker.failure_threshold =   3 # only 3 failures before tripping circuit
 breaker.duration          =  10 # 10 seconds before retry
 breaker.timeout           = 0.5 # 500 milliseconds allowed before auto-fail
 
+# These options are, uh, optional.
+breaker.only_trip_on  = [ExpensiveFailureException]
+breaker.never_trip_on = [CheapUnimportantFailureException]
+
 begin
   breaker.call(*some_args)    # args are passed through to the proc
 rescue CircuitBreakage::CircuitOpen

data/lib/circuit_breakage/breaker.rb

@@ -10,16 +10,21 @@ module CircuitBreakage
   class Breaker
     attr_accessor :failure_count, :last_failed, :state, :block
     attr_accessor :failure_threshold, :duration, :timeout, :last_exception
+    attr_accessor :only_trip_on, :never_trip_on
 
-    DEFAULT_FAILURE_THRESHOLD = 5     # Number of failures required to trip circuit
-    DEFAULT_DURATION          = 300   # Number of seconds the circuit stays tripped
-    DEFAULT_TIMEOUT           = 10    # Number of seconds before the call times out
+    DEFAULT_FAILURE_THRESHOLD = 5           # Number of failures required to trip circuit
+    DEFAULT_DURATION          = 300         # Number of seconds the circuit stays tripped
+    DEFAULT_TIMEOUT           = 10          # Number of seconds before the call times out
+    DEFAULT_ONLY_TRIP_ON      = [Exception] # Exceptions that trigger the breaker
+    DEFAULT_NEVER_TRIP_ON     = []          # Exceptions that won't trigger the breaker
 
     def initialize(block=nil)
       self.block              = block
       self.failure_threshold  = DEFAULT_FAILURE_THRESHOLD
       self.duration           = DEFAULT_DURATION
       self.timeout            = DEFAULT_TIMEOUT
+      self.only_trip_on       = DEFAULT_ONLY_TRIP_ON
+      self.never_trip_on      = DEFAULT_NEVER_TRIP_ON
       self.failure_count      ||= 0
       self.last_failed        ||= 0
       self.state              ||= 'closed'
@@ -55,7 +60,8 @@ module CircuitBreakage
       handle_success
 
       return ret_value
-    rescue Exception => e
+    rescue *self.only_trip_on => e
+      raise if never_trip_on.any? { |t| e.instance_of?(t) }
       handle_failure(e)
     end
 

data/lib/circuit_breakage/version.rb

@@ -1,3 +1,3 @@
 module CircuitBreakage
-  VERSION = "1.0.0"
+  VERSION = "1.1.0"
 end

data/spec/breaker_spec.rb

@@ -1,4 +1,7 @@
 module CircuitBreakage
+  class VerySpecificException < StandardError
+  end
+
   describe Breaker do
     let(:breaker) { Breaker.new(block) }
     let(:block) { ->(x) { return x } }
@@ -78,6 +81,85 @@ module CircuitBreakage
             expect { breaker.call(arg) }.to raise_exception(CircuitBreakage::CircuitTimeout)
           end
         end
+
+        context 'with specific exceptions defined' do
+          before do
+            breaker.only_trip_on = [VerySpecificException]
+          end
+
+          context 'and the call fails with one of the specific exceptions' do
+            let(:block) { ->(_) { raise VerySpecificException } }
+
+            it { is_expected.to change { breaker.failure_count }.by(1) }
+            it { is_expected.to change { breaker.last_failed } }
+            it { is_expected.to change { breaker.last_exception }.from(nil) }
+
+            it 'raises the exception that caused the failure' do
+              expect { breaker.call(arg) }.to raise_exception(VerySpecificException)
+            end
+          end
+
+          context 'and the call fails with a different exception (including timeouts)' do
+            let(:block) { ->(_) { raise Timeout::Error } }
+
+            it { is_expected.not_to change { breaker.failure_count } }
+            it { is_expected.not_to change { breaker.last_failed } }
+            it { is_expected.not_to change { breaker.last_exception } }
+
+            it 'raises the exception that caused the failure' do
+              expect { breaker.call(arg) }.to raise_exception(Timeout::Error)
+            end
+          end
+        end
+
+        context 'with specific exceptions excluded' do
+          before do
+            breaker.never_trip_on = [VerySpecificException]
+          end
+
+          context 'and the call fails with one of the specific exceptions' do
+            let(:block) { ->(_) { raise VerySpecificException } }
+
+            it { is_expected.not_to change { breaker.failure_count } }
+            it { is_expected.not_to change { breaker.last_failed } }
+            it { is_expected.not_to change { breaker.last_exception } }
+
+            it 'raises the exception that caused the failure' do
+              expect { breaker.call(arg) }.to raise_exception(VerySpecificException)
+            end
+          end
+
+          context 'and the call fails with a different exception' do
+            let(:block) { ->(_) { raise RuntimeError } }
+
+            it { is_expected.to change { breaker.failure_count }.by(1) }
+            it { is_expected.to change { breaker.last_failed } }
+            it { is_expected.to change { breaker.last_exception }.from(nil) }
+
+            it 'raises the exception that caused the failure' do
+              expect { breaker.call(arg) }.to raise_exception(RuntimeError)
+            end
+          end
+        end
+
+        context 'with overlapping whitelisted and blacklisted exceptions' do
+          before do
+            breaker.only_trip_on  = [StandardError]
+            breaker.never_trip_on = [VerySpecificException] # inherits from StandardError
+          end
+
+          context 'and the call fails with an overlapped exception' do
+            let(:block) { ->(_) { raise VerySpecificException } }
+
+            it { is_expected.not_to change { breaker.failure_count } }
+            it { is_expected.not_to change { breaker.last_failed } }
+            it { is_expected.not_to change { breaker.last_exception } }
+
+            it 'raises the exception that caused the failure' do
+              expect { breaker.call(arg) }.to raise_exception(VerySpecificException)
+            end
+          end
+        end
       end
 
       context 'when the circuit is open' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: circuit_breakage
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - John Hyland
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-19 00:00:00.000000000 Z
+date: 2016-03-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -108,7 +108,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Provides a simple circuit breaker pattern.
@@ -116,4 +116,3 @@ test_files:
 - spec/breaker_spec.rb
 - spec/redis_backed_breaker_spec.rb
 - spec/spec_helper.rb
-has_rdoc: