puppet 7.28.0-x64-mingw32 → 8.0.1-x64-mingw32

data/lib/puppet/pops/lookup/hiera_config.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'data_dig_function_provider'
 require_relative 'data_hash_function_provider'
 require_relative 'lookup_key_function_provider'
@@ -41,22 +42,22 @@ class HieraConfig
 
   CONFIG_FILE_NAME = 'hiera.yaml'
 
-  KEY_NAME = 'name'.freeze
-  KEY_VERSION = 'version'.freeze
-  KEY_DATADIR = 'datadir'.freeze
-  KEY_DEFAULT_HIERARCHY = 'default_hierarchy'.freeze
-  KEY_HIERARCHY = 'hierarchy'.freeze
-  KEY_PLAN_HIERARCHY = 'plan_hierarchy'.freeze
-  KEY_LOGGER = 'logger'.freeze
-  KEY_OPTIONS = 'options'.freeze
-  KEY_PATH = 'path'.freeze
-  KEY_PATHS = 'paths'.freeze
-  KEY_MAPPED_PATHS = 'mapped_paths'.freeze
-  KEY_GLOB = 'glob'.freeze
-  KEY_GLOBS = 'globs'.freeze
-  KEY_URI = 'uri'.freeze
-  KEY_URIS = 'uris'.freeze
-  KEY_DEFAULTS = 'defaults'.freeze
+  KEY_NAME = 'name'
+  KEY_VERSION = 'version'
+  KEY_DATADIR = 'datadir'
+  KEY_DEFAULT_HIERARCHY = 'default_hierarchy'
+  KEY_HIERARCHY = 'hierarchy'
+  KEY_PLAN_HIERARCHY = 'plan_hierarchy'
+  KEY_LOGGER = 'logger'
+  KEY_OPTIONS = 'options'
+  KEY_PATH = 'path'
+  KEY_PATHS = 'paths'
+  KEY_MAPPED_PATHS = 'mapped_paths'
+  KEY_GLOB = 'glob'
+  KEY_GLOBS = 'globs'
+  KEY_URI = 'uri'
+  KEY_URIS = 'uris'
+  KEY_DEFAULTS = 'defaults'
   KEY_DATA_HASH = DataHashFunctionProvider::TAG
   KEY_LOOKUP_KEY = LookupKeyFunctionProvider::TAG
   KEY_DATA_DIG = DataDigFunctionProvider::TAG
@@ -64,8 +65,8 @@ class HieraConfig
   KEY_V3_LOOKUP_KEY = V3LookupKeyFunctionProvider::TAG
   KEY_V3_BACKEND = V3BackendFunctionProvider::TAG
   KEY_V4_DATA_HASH = V4DataHashFunctionProvider::TAG
-  KEY_BACKEND = 'backend'.freeze
-  KEY_EXTENSION = 'extension'.freeze
+  KEY_BACKEND = 'backend'
+  KEY_EXTENSION = 'extension'
 
   FUNCTION_KEYS = [KEY_DATA_HASH, KEY_LOOKUP_KEY, KEY_DATA_DIG, KEY_V3_BACKEND]
   ALL_FUNCTION_KEYS = FUNCTION_KEYS + [KEY_V4_DATA_HASH]
@@ -227,7 +228,7 @@ class HieraConfig
       line_number += 1
       next if line_number < start_line
       quote = nil
-      stripped = ''
+      stripped = String.new
       line.each_codepoint do |cp|
         if cp == 0x22 || cp == 0x27 # double or single quote
           if quote == cp
@@ -252,7 +253,7 @@ class HieraConfig
       scope = lookup_invocation.scope
       lookup_invocation.without_explain do
         @scope_interpolations.all? do |key, root_key, segments, old_value|
-          value = scope[root_key]
+          value = Puppet.override(avoid_hiera_interpolation_errors: true) { scope[root_key] }
           unless value.nil? || segments.empty?
             found = nil;
             catch(:no_such_key) { found = sub_lookup(key, lookup_invocation, segments, value) }
@@ -282,8 +283,9 @@ class HieraConfig
   end
 
   def create_hiera3_backend_provider(name, backend, parent_data_provider, datadir, paths, hiera3_config)
-    # Custom backend. Hiera v3 must be installed, it's logger configured, and it must be made aware of the loaded config
-    require 'hiera'
+    # Custom backend. Hiera 3 must be installed, its logger configured, and it must be made aware of the loaded config
+    raise Puppet::DataBinding::LookupError, 'Hiera 3 is not installed' if !Puppet.features.hiera?
+
     if Hiera::Config.instance_variable_defined?(:@config) && (current_config = Hiera::Config.instance_variable_get(:@config)).is_a?(Hash)
       current_config.each_pair do |key, val|
         case key
@@ -345,9 +347,9 @@ end
 
 # @api private
 class HieraConfigV3 < HieraConfig
-  KEY_BACKENDS = 'backends'.freeze
-  KEY_MERGE_BEHAVIOR = 'merge_behavior'.freeze
-  KEY_DEEP_MERGE_OPTIONS = 'deep_merge_options'.freeze
+  KEY_BACKENDS = 'backends'
+  KEY_MERGE_BEHAVIOR = 'merge_behavior'
+  KEY_DEEP_MERGE_OPTIONS = 'deep_merge_options'
 
   def self.config_type
     return @@CONFIG_TYPE if class_variable_defined?(:@@CONFIG_TYPE)
@@ -632,23 +634,26 @@ class HieraConfigV5 < HieraConfig
       entry_datadir = @config_root + (he[KEY_DATADIR] || datadir)
       entry_datadir = Pathname(interpolate(entry_datadir.to_s, lookup_invocation, false))
       location_key = LOCATION_KEYS.find { |key| he.include?(key) }
-      locations = case location_key
-      when KEY_PATHS
-        resolve_paths(entry_datadir, he[location_key], lookup_invocation, @config_path.nil?)
-      when KEY_PATH
-        resolve_paths(entry_datadir, [he[location_key]], lookup_invocation, @config_path.nil?)
-      when KEY_GLOBS
-        expand_globs(entry_datadir, he[location_key], lookup_invocation)
-      when KEY_GLOB
-        expand_globs(entry_datadir, [he[location_key]], lookup_invocation)
-      when KEY_URIS
-        expand_uris(he[location_key], lookup_invocation)
-      when KEY_URI
-        expand_uris([he[location_key]], lookup_invocation)
-      when KEY_MAPPED_PATHS
-        expand_mapped_paths(entry_datadir, he[location_key], lookup_invocation)
-      else
-        nil
+      locations = []
+      Puppet.override(avoid_hiera_interpolation_errors: true) do
+        locations = case location_key
+                    when KEY_PATHS
+                      resolve_paths(entry_datadir, he[location_key], lookup_invocation, @config_path.nil?)
+                    when KEY_PATH
+                      resolve_paths(entry_datadir, [he[location_key]], lookup_invocation, @config_path.nil?)
+                    when KEY_GLOBS
+                      expand_globs(entry_datadir, he[location_key], lookup_invocation)
+                    when KEY_GLOB
+                      expand_globs(entry_datadir, [he[location_key]], lookup_invocation)
+                    when KEY_URIS
+                      expand_uris(he[location_key], lookup_invocation)
+                    when KEY_URI
+                      expand_uris([he[location_key]], lookup_invocation)
+                    when KEY_MAPPED_PATHS
+                      expand_mapped_paths(entry_datadir, he[location_key], lookup_invocation)
+                    else
+                      nil
+                    end
       end
       next if @config_path.nil? && !locations.nil? && locations.empty? # Default config and no existing paths found
       options = he[KEY_OPTIONS] || defaults[KEY_OPTIONS]

data/lib/puppet/pops/lookup/interpolation.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'hiera/scope'
 require_relative 'sub_lookup'
 module Puppet::Pops
@@ -90,11 +91,16 @@ module Interpolation
             lookup_invocation.report_found_in_overrides(root_key, ovr[root_key])
           else
             scope = lookup_invocation.scope
-            val = scope[root_key]
+            val = nil
+            if (default_key_exists = lookup_invocation.default_values.include?(root_key) )
+              catch(:undefined_variable) { val = scope[root_key] }
+            else
+              val = scope[root_key]
+            end
             if val.nil? && !nil_in_scope?(scope, root_key)
-              defaults = lookup_invocation.default_values
-              if defaults.include?(root_key)
-                lookup_invocation.report_found_in_defaults(root_key, defaults[root_key])
+              if default_key_exists
+                lookup_invocation.report_found_in_defaults(root_key,
+                                                           lookup_invocation.default_values[root_key])
               else
                 nil
               end

data/lib/puppet/pops/lookup/invocation.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/thread_local'
 
 require_relative 'explainer'

data/lib/puppet/pops/lookup/key_recorder.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This class defines the private API of the Lookup Key Recorder support.
 # @api private
 #

data/lib/puppet/pops/lookup/location_resolver.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'pathname'
 require_relative 'interpolation'
 

data/lib/puppet/pops/lookup/lookup_adapter.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'data_adapter'
 require_relative 'lookup_key'
 
@@ -12,12 +13,12 @@ class LookupAdapter < DataAdapter
 
   LOOKUP_OPTIONS_PREFIX = LOOKUP_OPTIONS + '.'
   LOOKUP_OPTIONS_PREFIX.freeze
-  LOOKUP_OPTIONS_PATTERN_START = '^'.freeze
+  LOOKUP_OPTIONS_PATTERN_START = '^'
 
-  HASH = 'hash'.freeze
-  MERGE = 'merge'.freeze
-  CONVERT_TO = 'convert_to'.freeze
-  NEW = 'new'.freeze
+  HASH = 'hash'
+  MERGE = 'merge'
+  CONVERT_TO = 'convert_to'
+  NEW = 'new'
 
   def self.create_adapter(compiler)
     new(compiler)
@@ -334,7 +335,7 @@ class LookupAdapter < DataAdapter
     end
   end
 
-  GLOBAL_ENV_MERGE = 'Global and Environment'.freeze
+  GLOBAL_ENV_MERGE = 'Global and Environment'
 
   # Retrieve lookup options that applies when using a specific module (i.e. a merge of the pre-cached
   # `env_lookup_options` and the module specific data)

data/lib/puppet/pops/lookup/lookup_key.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'sub_lookup'
 
 module Puppet::Pops

data/lib/puppet/pops/lookup/lookup_key_function_provider.rb

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
 require_relative 'function_provider'
 
 module Puppet::Pops
 module Lookup
 # @api private
 class LookupKeyFunctionProvider < FunctionProvider
-  TAG = 'lookup_key'.freeze
+  TAG = 'lookup_key'
 
   # Performs a lookup with the assumption that a recursive check has been made.
   #
@@ -64,7 +65,7 @@ end
 
 # @api private
 class V3LookupKeyFunctionProvider < LookupKeyFunctionProvider
-  TAG = 'v3_lookup_key'.freeze
+  TAG = 'v3_lookup_key'
 
   def initialize(name, parent_data_provider, function_name, options, locations)
     @datadir = options.delete(HieraConfig::KEY_DATADIR)

data/lib/puppet/pops/lookup/module_data_provider.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'configured_data_provider'
 
 module Puppet::Pops

data/lib/puppet/pops/lookup/sub_lookup.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Lookup
 module SubLookup

data/lib/puppet/pops/lookup.rb

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
 # This class is the backing implementation of the Puppet function 'lookup'.
 # See puppet/functions/lookup.rb for documentation.
 #
 module Puppet::Pops
 module Lookup
-  LOOKUP_OPTIONS = 'lookup_options'.freeze
-  GLOBAL = '__global__'.freeze
+  LOOKUP_OPTIONS = 'lookup_options'
+  GLOBAL = '__global__'
 
   # Performs a lookup in the configured scopes and optionally merges the default.
   #

data/lib/puppet/pops/merge_strategy.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'deep_merge/core'
 
 module Puppet::Pops

data/lib/puppet/pops/migration/migration_checker.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This class defines the private API of the MigrationChecker support.
 # @api private
 #

data/lib/puppet/pops/model/ast_transformer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/parser/ast'
 
 # The receiver of `import(file)` calls; once per imported file, or nil if imports are ignored

data/lib/puppet/pops/model/factory.rb

@@ -1112,7 +1112,7 @@ class Factory
   end
 
   def self.concat(*args)
-    result = ''.dup
+    result = String.new
     args.each do |e|
       if e.instance_of?(Factory) && e.model_class <= LiteralString
         result << e[KEY_VALUE]

data/lib/puppet/pops/model/model_label_provider.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Model
 # A provider of labels for model object, producing a human name for the model object.

data/lib/puppet/pops/model/model_tree_dumper.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Dumps a Pops::Model in reverse polish notation; i.e. LISP style
 # The intention is to use this for debugging output
 # TODO: BAD NAME - A DUMP is a Ruby Serialization

data/lib/puppet/pops/model/pn_transformer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/concurrent/thread_local_singleton'
 
 module Puppet::Pops

data/lib/puppet/pops/model/tree_dumper.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Base class for formatted textual dump of a "model"
 #
 class Puppet::Pops::Model::TreeDumper
@@ -20,7 +21,7 @@ class Puppet::Pops::Model::TreeDumper
   end
 
   def format(x)
-    result = ""
+    result = String.new
     parts = format_r(x)
     parts.each_index do |i|
       if i > 0

data/lib/puppet/pops/parser/code_merger.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 
 class Puppet::Pops::Parser::CodeMerger
 

data/lib/puppet/pops/parser/epp_parser.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The EppParser is a specialized Puppet Parser that starts parsing in Epp Text mode
 class Puppet::Pops::Parser::EppParser < Puppet::Pops::Parser::Parser
 

data/lib/puppet/pops/parser/epp_support.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Parser
 # This module is an integral part of the Lexer.
@@ -182,7 +183,7 @@ module EppSupport
       @skip_leading = skip_leading
 
       return nil if scanner.eos?
-      s = ""
+      s = String.new
       until scanner.eos?
         part = @scanner.scan_until(/(<%)|\z/)
         if @skip_leading

data/lib/puppet/pops/parser/evaluating_parser.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/concurrent/thread_local_singleton'
 
 module Puppet::Pops
@@ -124,7 +125,7 @@ class EvaluatingParser
   # @return [String] The quoted string
   #
   def self.quote(x)
-    escaped = '"'
+    escaped = String.new('"')
     p = nil
     x.each_char do |c|
       case p

data/lib/puppet/pops/parser/heredoc_support.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Parser
 module HeredocSupport

data/lib/puppet/pops/parser/interpolation_support.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This module is an integral part of the Lexer.
 # It defines interpolation support
 # PERFORMANCE NOTE: There are 4 very similar methods in this module that are designed to be as

data/lib/puppet/pops/parser/lexer_support.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 
 module Puppet::Pops
 module Parser

data/lib/puppet/pops/parser/locatable.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Interface for something that is "locatable" (holds offset and length).
 class Puppet::Pops::Parser::Locatable
 

data/lib/puppet/pops/parser/locator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Parser
 # Helper class that keeps track of where line breaks are located and can answer questions about positions.

data/lib/puppet/pops/parser/parser_support.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/parser/functions'
 require_relative '../../../puppet/parser/files'
 require_relative '../../../puppet/resource/type_collection'

data/lib/puppet/pops/parser/pn_parser.rb

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Parser
 
 class PNParser
-  LIT_TRUE = 'true'.freeze
-  LIT_FALSE = 'false'.freeze
-  LIT_NIL = 'nil'.freeze
+  LIT_TRUE = 'true'
+  LIT_FALSE = 'false'
+  LIT_NIL = 'nil'
 
   TOKEN_END = 0
   TOKEN_BOOL = 1
@@ -217,7 +218,7 @@ class PNParser
 
   def consume_string
     s = @pos
-    b = ''
+    b = String.new
     loop do
       c = next_cp
       case c

data/lib/puppet/pops/patterns.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The Patterns module contains common regular expression patters for the Puppet DSL language
 module Puppet::Pops::Patterns
 

data/lib/puppet/pops/pcore.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'uri'
 
 module Puppet::Pops
@@ -11,8 +12,8 @@ module Pcore
   TYPE_QUALIFIED_REFERENCE = Types::TypeFactory.pattern(/\A[A-Z][\w]*(?:::[A-Z][\w]*)*\z/)
   TYPE_MEMBER_NAME = Types::PPatternType.new([Types::PRegexpType.new(Patterns::PARAM_NAME)])
 
-  KEY_PCORE_URI = 'pcore_uri'.freeze
-  KEY_PCORE_VERSION = 'pcore_version'.freeze
+  KEY_PCORE_URI = 'pcore_uri'
+  KEY_PCORE_VERSION = 'pcore_version'
 
   PCORE_URI = 'http://puppet.com/2016.1/pcore'
   PCORE_VERSION = SemanticPuppet::Version.new(1,0,0)

data/lib/puppet/pops/pn.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module PN
   KEY_PATTERN = /^[A-Za-z_-][0-9A-Za-z_-]*$/
@@ -19,7 +20,7 @@ module PN
   end
 
   def to_s
-    s = ''
+    s = String.new
     format(nil, s)
     s
   end

data/lib/puppet/pops/puppet_stack.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/thread_local'
 
 module Puppet

data/lib/puppet/pops/resource/param.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # An implementation of the interface Puppet::Resource
 # that adapts the 3.x compiler and catalog expectations on
 # a resource instance. This instance is backed by a

data/lib/puppet/pops/resource/resource_type_impl.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'param'
 
 module Puppet::Pops

data/lib/puppet/pops/semantic_error.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Error that is used to raise an Issue. See {Puppet::Pops::Issues}.
 #
 class Puppet::Pops::SemanticError < RuntimeError

data/lib/puppet/pops/serialization/abstract_reader.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'extension'
 require_relative 'time_factory'
 

data/lib/puppet/pops/serialization/abstract_writer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'extension'
 
 module Puppet::Pops

data/lib/puppet/pops/serialization/deserializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'extension'
 
 module Puppet::Pops

data/lib/puppet/pops/serialization/extension.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Serialization
 module Extension

data/lib/puppet/pops/serialization/from_data_converter.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Serialization
   class Builder

data/lib/puppet/pops/serialization/instance_reader.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Serialization
   # An InstanceReader is responsible for reading an instance of a complex object using a deserializer. The read involves creating the

data/lib/puppet/pops/serialization/instance_writer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Serialization
   # An instance writer is responsible for writing complex objects using a {Serializer}

data/lib/puppet/pops/serialization/json.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/json'
 
 module Puppet::Pops

data/lib/puppet/pops/serialization/json_path.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/concurrent/thread_local_singleton'
 
 module Puppet::Pops
@@ -10,7 +11,7 @@ module JsonPath
   #
   # @api private
   def self.to_json_path(path)
-    p = '$'
+    p = String.new('$')
     path.each do |seg|
       if seg.nil?
         p << '[null]'

data/lib/puppet/pops/serialization/object.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'instance_reader'
 require_relative 'instance_writer'
 

data/lib/puppet/pops/serialization/serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'extension'
 
 module Puppet::Pops

data/lib/puppet/pops/serialization/time_factory.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Serialization
   # Implements all the constructors found in the Time class and ensures that

data/lib/puppet/pops/serialization/to_data_converter.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Serialization
   # Class that can process an arbitrary object into a value that is assignable to `Data`.
@@ -63,7 +64,7 @@ module Serialization
     private
 
     def path_to_s
-      s = @message_prefix || ''
+      s = String.new(@message_prefix || '')
       s << JsonPath.to_json_path(@path)[1..-1]
       s
     end

data/lib/puppet/pops/serialization/to_stringified_converter.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Serialization
 

data/lib/puppet/pops/serialization.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Serialization
   def self.not_implemented(impl, method_name)
@@ -7,28 +8,28 @@ module Serialization
   class SerializationError < Puppet::Error
   end
 
-  PCORE_TYPE_KEY = '__ptype'.freeze
+  PCORE_TYPE_KEY = '__ptype'
 
   # Key used when the value can be represented as, and recreated from, a single string that can
   # be passed to a `from_string` method or an array of values that can be passed to the default
   # initializer method.
-  PCORE_VALUE_KEY = '__pvalue'.freeze
+  PCORE_VALUE_KEY = '__pvalue'
 
   # Type key used for hashes that contain keys that are not of type String
-  PCORE_TYPE_HASH = 'Hash'.freeze
+  PCORE_TYPE_HASH = 'Hash'
 
   # Type key used for symbols
-  PCORE_TYPE_SENSITIVE = 'Sensitive'.freeze
-  PCORE_TYPE_BINARY = 'Binary'.freeze
+  PCORE_TYPE_SENSITIVE = 'Sensitive'
+  PCORE_TYPE_BINARY = 'Binary'
 
   # Type key used for symbols
-  PCORE_TYPE_SYMBOL = 'Symbol'.freeze
+  PCORE_TYPE_SYMBOL = 'Symbol'
 
   # Type key used for Default
-  PCORE_TYPE_DEFAULT = 'Default'.freeze
+  PCORE_TYPE_DEFAULT = 'Default'
 
   # Type key used for document local references
-  PCORE_LOCAL_REF_SYMBOL = 'LocalRef'.freeze
+  PCORE_LOCAL_REF_SYMBOL = 'LocalRef'
 end
 end