Merge pull request #1723 from OpenC3/inspect

Remove inpect override in array and hash

openc3/lib/openc3/core_ext.rb

@@ -27,7 +27,6 @@
 require 'openc3/core_ext/exception'
 require 'openc3/core_ext/faraday'
 require 'openc3/core_ext/file'
-require 'openc3/core_ext/hash'
 require 'openc3/core_ext/io'
 require 'openc3/core_ext/kernel'
 require 'openc3/core_ext/math'

openc3/lib/openc3/core_ext/array.rb

@@ -24,22 +24,6 @@
 
 # OpenC3 specific additions to the Ruby Array class
 class Array
-  # Redefine inspect to only print for small numbers of
-  # items. Prevents exceptions taking forever to be raise with
-  # large objects. See http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-talk/105145
-  alias old_inspect inspect
-
-  # @param max_elements [Integer] The maximum number of elements in the array to
-  #   print out before simply displaying the array class and object id
-  # @return [String] String representation of the array
-  def inspect(max_elements = 10)
-    if self.length <= max_elements
-      old_inspect()
-    else
-      '#<' + self.class.to_s + ':' + self.object_id.to_s + '>'
-    end
-  end
-
   # @return [Array] Cloned array after all elements called to_f
   def clone_to_f
     new_array = self.class.new(0)

openc3/spec/core_ext/array_spec.rb

@@ -14,23 +14,16 @@
 # GNU Affero General Public License for more details.
 
 # Modified by OpenC3, Inc.
-# All changes Copyright 2022, OpenC3, Inc.
+# All changes Copyright 2024, OpenC3, Inc.
 # All Rights Reserved
 #
-# This file may also be used under the terms of a commercial license 
+# This file may also be used under the terms of a commercial license
 # if purchased from OpenC3, Inc.
 
 require 'spec_helper'
 require 'openc3/core_ext/array'
 
 describe Array do
-  describe "inspect" do
-    it "limits the number of items to 10" do
-      expect(Array.new([1, 2, 3, 4, 5, 6, 7, 8, 9, 10]).inspect).to eql "[1, 2, 3, 4, 5, 6, 7, 8, 9, 10]"
-      expect(Array.new([1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11]).inspect).to match(/#<Array:\d+>/)
-    end
-  end
-
   describe "clone_to_f" do
     it "clones the array and convert all the values to floats" do
       expect(Array.new([1, 2, 3, 4, 5]).clone_to_f).to eql [1.0, 2.0, 3.0, 4.0, 5.0]