Simplify to_s and inspect

Author: pitr-ch

lib/concurrent/edge/promises.rb

@@ -580,17 +580,12 @@ def chain_on(executor, *args, &task)
         ChainPromise.new_blocked1(self, @DefaultExecutor, executor, args, &task).future
       end
 
-      # Short string representation.
-      # @return [String]
+      # @return [String] Short string representation.
       def to_s
-        "<##{self.class}:0x#{'%x' % (object_id << 1)} #{state.to_sym}>"
+        format '<#%s:0x%x %s>', self.class, object_id << 1, state
       end
 
-      # Longer string representation.
-      # @return [String]
-      def inspect
-        "#{to_s[0..-2]} blocks:[#{blocks.map(&:to_s).join(', ')}]>"
-      end
+      alias_method :inspect, :to_s
 
       # Resolves the resolvable when receiver is resolved.
       #
@@ -1288,12 +1283,10 @@ def touch
       end
 
       def to_s
-        format '<#%s:0x%x %s>', self.class, object_id << 1, state
+        format '<#%s:0x%x>', self.class, object_id << 1
       end
 
-      def inspect
-        to_s
-      end
+      alias_method :inspect, :to_s
 
       def delayed
         nil

spec/concurrent/edge/promises_spec.rb

@@ -332,7 +332,7 @@ def behaves_as_delay(delay, value)
       # meaningful to_s and inspect defined for Future and Promise
       expect(head.to_s).to match /<#Concurrent::Promises::Future:0x[\da-f]+ pending>/
       expect(head.inspect).to(
-          match(/<#Concurrent::Promises::Future:0x[\da-f]+ pending blocks:\[<#Concurrent::Promises::ThenPromise:0x[\da-f]+ pending>\]>/))
+          match(/<#Concurrent::Promises::Future:0x[\da-f]+ pending>/))
 
       # evaluates only up to three, four is left unevaluated
       expect(three.value!).to eq 3