Fix uses of .keys.any?, .keys.include?, .keys == %w(), .keys.length

Author: jperville

lib/json/ld/api.rb

@@ -174,7 +174,7 @@ def self.expand(input, options = {}, &block)
       end
 
       # If, after the algorithm outlined above is run, the resulting element is an JSON object with just a @graph property, element is set to the value of @graph's value.
-      result = result['@graph'] if result.is_a?(Hash) && result.keys == %w(@graph)
+      result = result['@graph'] if result.is_a?(Hash) && result.length == 1 && result.key?('@graph')
 
       # Finally, if element is a JSON object, it is wrapped into an array.
       result = [result].compact unless result.is_a?(Array)

lib/json/ld/compact.rb

@@ -43,7 +43,7 @@ def compact(element, property: nil)
         # @null objects are used in framing
         return nil if element.has_key?('@null')
 
-        if element.keys.any? {|k| %w(@id @value).include?(k)}
+        if element.key?('@id') || element.key?('@value')
           result = context.compact_value(property, element, log_depth: @options[:log_depth])
           unless result.is_a?(Hash)
             #log_debug("") {"=> scalar result: #{result.inspect}"}

lib/json/ld/context.rb

@@ -615,7 +615,7 @@ def create_term_definition(local_context, term, defined)
 
         if value.has_key?('@reverse')
           raise JsonLdError::InvalidReverseProperty, "unexpected key in #{value.inspect} on term #{term.inspect}" if
-            value.keys.any? {|k| %w(@id @nest).include?(k)}
+            value.key?('@id') || value.key?('@nest')
           raise JsonLdError::InvalidIRIMapping, "expected value of @reverse to be a string: #{value['@reverse'].inspect} on term #{term.inspect}" unless
             value['@reverse'].is_a?(String)
 
@@ -1297,7 +1297,7 @@ def expand_value(property, value, useNativeTypes: false, **options)
     def compact_value(property, value, options = {})
       #log_debug("compact_value") {"property: #{property.inspect}, value: #{value.inspect}"}
 
-      num_members = value.keys.length
+      num_members = value.length
 
       num_members -= 1 if index?(value) && container(property) == '@index'
       if num_members > 2

lib/json/ld/expand.rb

@@ -87,23 +87,23 @@ def expand(input, active_property, context, ordered: true)
         elsif !output_object.fetch('@type', []).is_a?(Array)
           # Otherwise, if result contains the key @type and its associated value is not an array, set it to an array containing only the associated value.
           output_object['@type'] = [output_object['@type']]
-        elsif output_object.keys.any? {|k| %w(@set @list).include?(k)}
+        elsif output_object.key?('@set') || output_object.key?('@list')
           # Otherwise, if result contains the key @set or @list:
           # The result must contain at most one other key and that key must be @index. Otherwise, an invalid set or list object error has been detected and processing is aborted.
           raise JsonLdError::InvalidSetOrListObject,
                 "@set or @list may only contain @index: #{output_object.keys.inspect}" unless
                 (output_object.keys - %w(@set @list @index)).empty?
 
           # If result contains the key @set, then set result to the key's associated value.
-          return output_object['@set'] if output_object.keys.include?('@set')
+          return output_object['@set'] if output_object.key?('@set')
         end
 
         # If result contains only the key @language, set result to null.
-        return nil if output_object.keys == %w(@language)
+        return nil if output_object.length == 1 && output_object.key?('@language')
 
         # If active property is null or @graph, drop free-floating values as follows:
         if (active_property || '@graph') == '@graph' &&
-          (output_object.keys.any? {|k| %w(@value @list).include?(k)} ||
+          (output_object.key?('@value') || output_object.key?('@list') ||
            (output_object.keys - %w(@id)).empty? && !framing)
           #log_debug(" =>") { "empty top-level: " + output_object.inspect}
           return nil