Update inspect method to handle nil serializable_hash (#384)

* Update inspect method to handle nil serializable_hash

* Refactor inspect method to use object instead of hash

* Add changelog and rspec

* Update CHANGELOG.md

* Fix rubocop offenses

* Makes nil entity inspect consistent with the Ruby-style #<ClassName:object_id …> format

---------

Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]>
Co-authored-by: Andrei Subbota <[email protected]>

Author: 3 people

CHANGELOG.md

@@ -6,8 +6,9 @@
 
 #### Fixes
 
+* [#388](https://github.com/ruby-grape/grape-entity/pull/388): Drop ruby-head from test matrix - [@numbata](https://github.com/numbata).
+* [#384](https://github.com/ruby-grape/grape-entity/pull/384): Fix `inspect` to correctly handle `nil` values - [@fcce](https://github.com/fcce).
 * Your contribution here.
-* [#388](https://github.com/ruby-grape/grape-entity/pull/388): Drop ruby-head from test matrix to keep builds stable - [@numbata](https://github.com/numbata).
 
 ### ### 1.0.1 (2024-04-10)
 

lib/grape_entity/entity.rb

@@ -472,8 +472,13 @@ def presented
 
     # Prevent default serialization of :options or :delegator.
     def inspect
-      fields = serializable_hash.map { |k, v| "#{k}=#{v}" }
-      "#<#{self.class.name}:#{object_id} #{fields.join(' ')}>"
+      object = serializable_hash
+      if object.nil?
+        "#<#{self.class.name}:#{object_id} nil>"
+      else
+        fields = object.map { |k, v| "#{k}=#{v}" }
+        "#<#{self.class.name}:#{object_id} #{fields.join(' ')}>"
+      end
     end
 
     def initialize(object, options = {})

spec/grape_entity/entity_spec.rb

@@ -1778,6 +1778,11 @@ class NoPathCharacterEntity < Grape::Entity
         expect(data).to_not include '@options'
         expect(data).to_not include '@delegator'
       end
+
+      it 'returns a nil string when subject is nil' do
+        data = subject.class.new(nil).inspect
+        expect(data).to include 'nil'
+      end
     end
 
     describe '#value_for' do