Don't delegate to bulkhead, it might be nil

lib/semian/circuit_breaker.rb

@@ -2,6 +2,8 @@ module Semian
   class CircuitBreaker #:nodoc:
     extend Forwardable
 
+    attr_reader :name
+
     def_delegators :@state, :closed?, :open?, :half_open?
 
     def initialize(name, exceptions:, success_threshold:, error_threshold:, error_timeout:, implementation:)

lib/semian/protected_resource.rb

@@ -2,15 +2,15 @@ module Semian
   class ProtectedResource
     extend Forwardable
 
-    def_delegators :@bulkhead, :destroy, :count, :semid, :tickets, :registered_workers, :name
     def_delegators :@circuit_breaker, :reset, :mark_failed, :mark_success, :request_allowed?,
                    :open?, :closed?, :half_open?
 
-    attr_reader :bulkhead, :circuit_breaker
+    attr_reader :bulkhead, :circuit_breaker, :name
 
     def initialize(bulkhead, circuit_breaker)
       @bulkhead = bulkhead
       @circuit_breaker = circuit_breaker
+      assign_name
     end
 
     def destroy
@@ -26,6 +26,22 @@ def acquire(timeout: nil, scope: nil, adapter: nil)
       end
     end
 
+    def tickets
+      @bulkhead ? @bulkhead.tickets : 0
+    end
+
+    def count
+      @bulkhead ? @bulkhead.count : 0
+    end
+
+    def registered_workers
+      @bulkhead ? @bulkhead.registered_workers : 0
+    end
+
+    def semid
+      @bulkhead ? @bulkhead.semid : 0
+    end
+
     private
 
     def acquire_circuit_breaker(scope, adapter)
@@ -54,5 +70,10 @@ def acquire_bulkhead(timeout, scope, adapter)
       Semian.notify(:busy, self, scope, adapter)
       raise
     end
+
+    def assign_name
+      @name = @bulkhead.name if @bulkhead
+      @name ||= @circuit_breaker.name if @circuit_breaker
+    end
   end
 end

test/protected_resource_test.rb

@@ -16,6 +16,85 @@ def teardown
     super
   end
 
+  def test_get_name_without_bulkhead
+    Semian.register(
+      :testing,
+      exceptions: [SomeError],
+      error_threshold: 2,
+      error_timeout: 5,
+      success_threshold: 1,
+      bulkhead: false,
+    )
+
+    refute_nil Semian.resources[:testing].name
+  end
+
+  def test_get_name_without_bulkhead_or_circuit_breaker
+    Semian.register(
+      :testing,
+      exceptions: [SomeError],
+      error_threshold: 2,
+      error_timeout: 5,
+      success_threshold: 1,
+      bulkhead: false,
+      circuit_breaker: false,
+    )
+
+    assert_nil Semian.resources[:testing].name
+  end
+
+  def test_get_semid_without_bulkhead
+    Semian.register(
+      :testing,
+      exceptions: [SomeError],
+      error_threshold: 2,
+      error_timeout: 5,
+      success_threshold: 1,
+      bulkhead: false,
+    )
+
+    assert_equal(0, Semian.resources[:testing].semid)
+  end
+
+  def test_get_tickets_without_bulkhead
+    Semian.register(
+      :testing,
+      exceptions: [SomeError],
+      error_threshold: 2,
+      error_timeout: 5,
+      success_threshold: 1,
+      bulkhead: false,
+    )
+
+    assert_equal(0, Semian.resources[:testing].tickets)
+  end
+
+  def test_get_count_without_bulkhead
+    Semian.register(
+      :testing,
+      exceptions: [SomeError],
+      error_threshold: 2,
+      error_timeout: 5,
+      success_threshold: 1,
+      bulkhead: false,
+    )
+
+    assert_equal(0, Semian.resources[:testing].count)
+  end
+
+  def test_get_registered_workers_without_bulkhead
+    Semian.register(
+      :testing,
+      exceptions: [SomeError],
+      error_threshold: 2,
+      error_timeout: 5,
+      success_threshold: 1,
+      bulkhead: false,
+    )
+
+    assert_equal(0, Semian.resources[:testing].registered_workers)
+  end
+
   def test_acquire_without_bulkhead
     Semian.register(
       :testing,