Move ObjectInspection.inspect

It isn't good to name a method `inspect`, even if it's on the class
level. This prevents the class from being inspected, which can easily
happen automatically in an IRB console.

Author: mcmire

lib/super_diff.rb

@@ -30,6 +30,14 @@ def self.configuration
     @_configuration ||= Configuration.new
   end
 
+  def self.inspect_object(object, as_single_line:, indent_level: 0)
+    ObjectInspection::Inspectors::Main.call(
+      object,
+      as_single_line: as_single_line,
+      indent_level: indent_level,
+    )
+  end
+
   def self.time_like?(value)
     # Check for ActiveSupport's #acts_like_time? for their time-like objects
     # (like ActiveSupport::TimeWithZone).

lib/super_diff/diff_formatters/collection.rb

@@ -94,7 +94,7 @@ def build_chunk_for(operation, prefix:, icon:)
       end
 
       def build_chunk_by_inspecting(value, prefix:, icon:)
-        inspection = ObjectInspection.inspect(
+        inspection = SuperDiff.inspect_object(
           value,
           as_single_line: false,
         )

lib/super_diff/equality_matchers/array.rb

@@ -13,14 +13,14 @@ def fail
             Helpers.style(
               :expected,
               "Expected: " +
-              ObjectInspection.inspect(expected, as_single_line: true),
+              SuperDiff.inspect_object(expected, as_single_line: true),
             )
           }
           #{
             Helpers.style(
               :actual,
               "  Actual: " +
-              ObjectInspection.inspect(actual, as_single_line: true),
+              SuperDiff.inspect_object(actual, as_single_line: true),
             )
           }
 

lib/super_diff/equality_matchers/default.rb

@@ -21,15 +21,15 @@ def expected_line
         Helpers.style(
           :expected,
           "Expected: " +
-          ObjectInspection.inspect(expected, as_single_line: true),
+          SuperDiff.inspect_object(expected, as_single_line: true),
         )
       end
 
       def actual_line
         Helpers.style(
           :actual,
           "  Actual: " +
-          ObjectInspection.inspect(actual, as_single_line: true),
+          SuperDiff.inspect_object(actual, as_single_line: true),
         )
       end
 

lib/super_diff/equality_matchers/hash.rb

@@ -13,14 +13,14 @@ def fail
             Helpers.style(
               :expected,
               "Expected: " +
-              ObjectInspection.inspect(expected, as_single_line: true),
+              SuperDiff.inspect_object(expected, as_single_line: true),
             )
           }
           #{
             Helpers.style(
               :actual,
               "  Actual: " +
-              ObjectInspection.inspect(actual, as_single_line: true),
+              SuperDiff.inspect_object(actual, as_single_line: true),
             )
           }
 

lib/super_diff/equality_matchers/multiline_string.rb

@@ -14,14 +14,14 @@ def fail
             Helpers.style(
               :expected,
               "Expected: " +
-              ObjectInspection.inspect(expected, as_single_line: true),
+              SuperDiff.inspect_object(expected, as_single_line: true),
             )
           }
           #{
             Helpers.style(
               :actual,
               "  Actual: " +
-              ObjectInspection.inspect(actual, as_single_line: true),
+              SuperDiff.inspect_object(actual, as_single_line: true),
             )
           }
 

lib/super_diff/equality_matchers/primitive.rb

@@ -17,14 +17,14 @@ def fail
             Helpers.style(
               :expected,
               "Expected: " +
-              ObjectInspection.inspect(expected, as_single_line: true),
+              SuperDiff.inspect_object(expected, as_single_line: true),
             )
           }
           #{
             Helpers.style(
               :actual,
               "  Actual: " +
-              ObjectInspection.inspect(actual, as_single_line: true),
+              SuperDiff.inspect_object(actual, as_single_line: true),
             )
           }
         OUTPUT

lib/super_diff/equality_matchers/singleline_string.rb

@@ -13,14 +13,14 @@ def fail
             Helpers.style(
               :expected,
               "Expected: " +
-              ObjectInspection.inspect(expected, as_single_line: true),
+              SuperDiff.inspect_object(expected, as_single_line: true),
             )
           }
           #{
             Helpers.style(
               :actual,
               "  Actual: " +
-              ObjectInspection.inspect(actual, as_single_line: true),
+              SuperDiff.inspect_object(actual, as_single_line: true),
             )
           }
         OUTPUT

lib/super_diff/object_inspection.rb

@@ -3,13 +3,5 @@ module ObjectInspection
     autoload :InspectionTree, "super_diff/object_inspection/inspection_tree"
     autoload :Inspectors, "super_diff/object_inspection/inspectors"
     autoload :Nodes, "super_diff/object_inspection/nodes"
-
-    def self.inspect(object, as_single_line:, indent_level: 0)
-      Inspectors::Main.call(
-        object,
-        as_single_line: as_single_line,
-        indent_level: indent_level,
-      )
-    end
   end
 end

lib/super_diff/object_inspection/nodes/inspection.rb

@@ -10,7 +10,7 @@ def evaluate(object, indent_level:, as_single_line:)
               immediate_value
             end
 
-          SuperDiff::ObjectInspection.inspect(
+          SuperDiff.inspect_object(
             value,
             indent_level: indent_level,
             as_single_line: as_single_line,