From 41c19929ad352f512cc50bb9de5b9265d771608c Mon Sep 17 00:00:00 2001 From: Dylan Thacker-Smith Date: Tue, 3 May 2022 12:47:43 -0400 Subject: [PATCH] Rename and alias Liquid::StaticRegisters to Liquid::Registers --- History.md | 2 +- lib/liquid.rb | 2 +- lib/liquid/context.rb | 4 +-- .../{static_registers.rb => registers.rb} | 7 +++-- test/integration/context_test.rb | 8 +++--- test/unit/partial_cache_unit_test.rb | 2 +- ...rs_unit_test.rb => registers_unit_test.rb} | 28 +++++++++---------- 7 files changed, 28 insertions(+), 25 deletions(-) rename lib/liquid/{static_registers.rb => registers.rb} (83%) rename test/unit/{static_registers_unit_test.rb => registers_unit_test.rb} (82%) diff --git a/History.md b/History.md index 7d619219..46013325 100644 --- a/History.md +++ b/History.md @@ -10,7 +10,7 @@ * Hash registers no longer leak into subcontexts as static registers (#1564) [Chris AtLee] ### Changed -* Liquid::Context#registers now always returns a Liquid::StaticRegisters object, though supports the most used Hash functions for compatibility (#1553) +* Liquid::Context#registers now always returns a Liquid::Registers object, though supports the most used Hash functions for compatibility (#1553) ## 5.3.0 2022-03-22 diff --git a/lib/liquid.rb b/lib/liquid.rb index a0a05881..e41b8c90 100644 --- a/lib/liquid.rb +++ b/lib/liquid.rb @@ -85,7 +85,7 @@ require 'liquid/parse_context' require 'liquid/partial_cache' require 'liquid/usage' require 'liquid/register' -require 'liquid/static_registers' +require 'liquid/registers' require 'liquid/template_factory' # Load all the tags of the standard library diff --git a/lib/liquid/context.rb b/lib/liquid/context.rb index 016bd309..eb6ddf9b 100644 --- a/lib/liquid/context.rb +++ b/lib/liquid/context.rb @@ -28,7 +28,7 @@ module Liquid @static_environments = [static_environments].flat_map(&:freeze).freeze @scopes = [(outer_scope || {})] - @registers = registers.is_a?(StaticRegisters) ? registers : StaticRegisters.new(registers) + @registers = registers.is_a?(Registers) ? registers : Registers.new(registers) @errors = [] @partial = false @strict_variables = false @@ -144,7 +144,7 @@ module Liquid self.class.build( resource_limits: resource_limits, static_environments: static_environments, - registers: StaticRegisters.new(registers) + registers: Registers.new(registers) ).tap do |subcontext| subcontext.base_scope_depth = base_scope_depth + 1 subcontext.exception_renderer = exception_renderer diff --git a/lib/liquid/static_registers.rb b/lib/liquid/registers.rb similarity index 83% rename from lib/liquid/static_registers.rb rename to lib/liquid/registers.rb index 4512c724..95cd8f9f 100644 --- a/lib/liquid/static_registers.rb +++ b/lib/liquid/registers.rb @@ -1,11 +1,11 @@ # frozen_string_literal: true module Liquid - class StaticRegisters + class Registers attr_reader :static def initialize(registers = {}) - @static = registers.is_a?(StaticRegisters) ? registers.static : registers + @static = registers.is_a?(Registers) ? registers.static : registers @registers = {} end @@ -45,4 +45,7 @@ module Liquid @registers.key?(key) || @static.key?(key) end end + + # Alias for backwards compatibility + StaticRegisters = Registers end diff --git a/test/integration/context_test.rb b/test/integration/context_test.rb index 7598c715..832e3e30 100644 --- a/test/integration/context_test.rb +++ b/test/integration/context_test.rb @@ -529,7 +529,7 @@ class ContextTest < Minitest::Test registers = { my_register: :my_value, } - super_context = Context.new({}, {}, StaticRegisters.new(registers)) + super_context = Context.new({}, {}, Registers.new(registers)) super_context.registers[:my_register] = :my_alt_value subcontext = super_context.new_isolated_subcontext assert_equal(:my_value, subcontext.registers[:my_register]) @@ -623,12 +623,12 @@ class ContextTest < Minitest::Test my_register: :my_value, } c = Context.new({}, {}, registers) - assert_instance_of(StaticRegisters, c.registers) + assert_instance_of(Registers, c.registers) assert_equal(:my_value, c.registers[:my_register]) - r = StaticRegisters.new(registers) + r = Registers.new(registers) c = Context.new({}, {}, r) - assert_instance_of(StaticRegisters, c.registers) + assert_instance_of(Registers, c.registers) assert_equal(:my_value, c.registers[:my_register]) end diff --git a/test/unit/partial_cache_unit_test.rb b/test/unit/partial_cache_unit_test.rb index 00052359..b0a2483b 100644 --- a/test/unit/partial_cache_unit_test.rb +++ b/test/unit/partial_cache_unit_test.rb @@ -132,7 +132,7 @@ class PartialCacheUnitTest < Minitest::Test 'my_partial' => 'my shared value' ) context = Liquid::Context.build( - registers: Liquid::StaticRegisters.new( + registers: Liquid::Registers.new( file_system: shared_file_system, ) ) diff --git a/test/unit/static_registers_unit_test.rb b/test/unit/registers_unit_test.rb similarity index 82% rename from test/unit/static_registers_unit_test.rb rename to test/unit/registers_unit_test.rb index f0c38a3a..caa718ea 100644 --- a/test/unit/static_registers_unit_test.rb +++ b/test/unit/registers_unit_test.rb @@ -2,11 +2,11 @@ require 'test_helper' -class StaticRegistersUnitTest < Minitest::Test +class RegistersUnitTest < Minitest::Test include Liquid def test_set - static_register = StaticRegisters.new(a: 1, b: 2) + static_register = Registers.new(a: 1, b: 2) static_register[:b] = 22 static_register[:c] = 33 @@ -16,13 +16,13 @@ class StaticRegistersUnitTest < Minitest::Test end def test_get_missing_key - static_register = StaticRegisters.new + static_register = Registers.new assert_nil(static_register[:missing]) end def test_delete - static_register = StaticRegisters.new(a: 1, b: 2) + static_register = Registers.new(a: 1, b: 2) static_register[:b] = 22 static_register[:c] = 33 @@ -37,7 +37,7 @@ class StaticRegistersUnitTest < Minitest::Test end def test_fetch - static_register = StaticRegisters.new(a: 1, b: 2) + static_register = Registers.new(a: 1, b: 2) static_register[:b] = 22 static_register[:c] = 33 @@ -61,7 +61,7 @@ class StaticRegistersUnitTest < Minitest::Test end def test_key - static_register = StaticRegisters.new(a: 1, b: 2) + static_register = Registers.new(a: 1, b: 2) static_register[:b] = 22 static_register[:c] = 33 @@ -72,7 +72,7 @@ class StaticRegistersUnitTest < Minitest::Test end def test_static_register_can_be_frozen - static_register = StaticRegisters.new(a: 1) + static_register = Registers.new(a: 1) static_register.static.freeze @@ -94,14 +94,14 @@ class StaticRegistersUnitTest < Minitest::Test end def test_new_static_retains_static - static_register = StaticRegisters.new(a: 1, b: 2) + static_register = Registers.new(a: 1, b: 2) static_register[:b] = 22 static_register[:c] = 33 - new_static_register = StaticRegisters.new(static_register) + new_static_register = Registers.new(static_register) new_static_register[:b] = 222 - newest_static_register = StaticRegisters.new(new_static_register) + newest_static_register = Registers.new(new_static_register) newest_static_register[:c] = 333 assert_equal(1, static_register[:a]) @@ -118,11 +118,11 @@ class StaticRegistersUnitTest < Minitest::Test end def test_multiple_instances_are_unique - static_register_1 = StaticRegisters.new(a: 1, b: 2) + static_register_1 = Registers.new(a: 1, b: 2) static_register_1[:b] = 22 static_register_1[:c] = 33 - static_register_2 = StaticRegisters.new(a: 10, b: 20) + static_register_2 = Registers.new(a: 10, b: 20) static_register_2[:b] = 220 static_register_2[:c] = 330 @@ -138,11 +138,11 @@ class StaticRegistersUnitTest < Minitest::Test end def test_initialization_reused_static_same_memory_object - static_register_1 = StaticRegisters.new(a: 1, b: 2) + static_register_1 = Registers.new(a: 1, b: 2) static_register_1[:b] = 22 static_register_1[:c] = 33 - static_register_2 = StaticRegisters.new(static_register_1) + static_register_2 = Registers.new(static_register_1) assert_equal(1, static_register_2[:a]) assert_equal(2, static_register_2[:b])