prefab-cloud-ruby 1.8.8.pre.1 → 1.8.9

data/test/test_evaluation_summary_aggregator.rb

@@ -59,86 +59,86 @@ class TestEvaluationSummaryAggregator < Minitest::Test
   end
 
   def test_sync
-    awhile_ago = Time.now - 60
-    now = Time.now
+    Timecop.freeze('2023-08-09 15:18:12 -0400') do
+      awhile_ago = Time.now - 60
+      now = Time.now
 
-    client = MockBaseClient.new
+      client = MockBaseClient.new
 
-    aggregator = nil
+      aggregator = nil
 
-    Timecop.freeze(awhile_ago) do
-      # start the aggregator in the past
-      aggregator = Prefab::EvaluationSummaryAggregator.new(client: client, max_keys: 10,
+      Timecop.freeze(awhile_ago) do
+        # start the aggregator in the past
+        aggregator = Prefab::EvaluationSummaryAggregator.new(client: client, max_keys: 10,
                                                            sync_interval: EFFECTIVELY_NEVER)
-    end
-
-    add_example_data(aggregator)
+      end
 
-    expected_post = PrefabProto::TelemetryEvents.new(
-      instance_hash: client.instance_hash,
-      events: [
-        PrefabProto::TelemetryEvent.new(
-          summaries:
-
-      PrefabProto::ConfigEvaluationSummaries.new(
-        start: awhile_ago.to_i * 1000,
-        end: now.to_i * 1000,
-        summaries: [
-          PrefabProto::ConfigEvaluationSummary.new(
-            key: 'config-1',
-            type: :CONFIG,
-            counters: [
-              PrefabProto::ConfigEvaluationCounter.new(
-                config_id: 1,
-                selected_index: 2,
-                config_row_index: 3,
-                conditional_value_index: 4,
-                weighted_value_index: 5,
-                selected_value: EXAMPLE_VALUE_1,
-                count: 3
-              ),
-              PrefabProto::ConfigEvaluationCounter.new(
-                config_id: 1,
-                selected_index: 3,
-                config_row_index: 7,
-                conditional_value_index: 8,
-                weighted_value_index: 10,
-                selected_value: EXAMPLE_VALUE_2,
-                count: 1
-              )
-            ]
-          ),
-          PrefabProto::ConfigEvaluationSummary.new(
-            key: 'config-2',
-            type: :FEATURE_FLAG,
-            counters: [
-              PrefabProto::ConfigEvaluationCounter.new(
-                config_id: 2,
-                selected_index: 3,
-                config_row_index: 5,
-                conditional_value_index: 7,
-                weighted_value_index: 6,
-                selected_value: EXAMPLE_VALUE_1,
-                count: 9
-              )
-            ]
+      add_example_data(aggregator)
+
+      expected_post = PrefabProto::TelemetryEvents.new(
+        instance_hash: client.instance_hash,
+        events: [
+          PrefabProto::TelemetryEvent.new(
+            summaries: PrefabProto::ConfigEvaluationSummaries.new(
+              start: awhile_ago.to_i * 1000,
+              end: now.to_i * 1000,
+              summaries: [
+                PrefabProto::ConfigEvaluationSummary.new(
+                  key: 'config-1',
+                  type: :CONFIG,
+                  counters: [
+                    PrefabProto::ConfigEvaluationCounter.new(
+                      config_id: 1,
+                      selected_index: 2,
+                      config_row_index: 3,
+                      conditional_value_index: 4,
+                      weighted_value_index: 5,
+                      selected_value: EXAMPLE_VALUE_1,
+                      count: 3
+                    ),
+                    PrefabProto::ConfigEvaluationCounter.new(
+                      config_id: 1,
+                      selected_index: 3,
+                      config_row_index: 7,
+                      conditional_value_index: 8,
+                      weighted_value_index: 10,
+                      selected_value: EXAMPLE_VALUE_2,
+                      count: 1
+                    )
+                  ]
+                ),
+                PrefabProto::ConfigEvaluationSummary.new(
+                  key: 'config-2',
+                  type: :FEATURE_FLAG,
+                  counters: [
+                    PrefabProto::ConfigEvaluationCounter.new(
+                      config_id: 2,
+                      selected_index: 3,
+                      config_row_index: 5,
+                      conditional_value_index: 7,
+                      weighted_value_index: 6,
+                      selected_value: EXAMPLE_VALUE_1,
+                      count: 9
+                    )
+                  ]
+                )
+              ]
+            )
           )
         ]
       )
-        )
-      ]
-    )
 
-    requests = wait_for_post_requests(client) do
-      Timecop.freeze(now) do
-        aggregator.sync
+      requests = wait_for_post_requests(client) do
+        Timecop.freeze(now) do
+          aggregator.sync
+        end
       end
-    end
 
-    assert_equal [[
-      '/api/v1/telemetry',
-      expected_post
-    ]], requests
+      assert_equal [[
+        '/api/v1/telemetry',
+        expected_post
+      ]], requests
+    end
   end
 
   private

data/test/test_fixed_size_hash.rb

@@ -0,0 +1,119 @@
+# frozen_string_literal: true
+
+require 'minitest/autorun'
+require 'test_helper'
+
+module Prefab
+  class FixedSizeHashTest < Minitest::Test
+    def setup
+      @max_size = 3
+      @hash = FixedSizeHash.new(@max_size)
+    end
+
+    def test_acts_like_a_regular_hash_when_under_max_size
+      @hash[:a] = 1
+      @hash[:b] = 2
+
+      assert_equal 1, @hash[:a]
+      assert_equal 2, @hash[:b]
+      assert_equal 2, @hash.size
+    end
+
+    def test_enforces_max_size_by_evicting_first_added_item
+      @hash[:a] = 1
+      @hash[:b] = 2
+      @hash[:c] = 3
+      assert_equal @max_size, @hash.size
+
+      @hash[:d] = 4
+      assert_equal @max_size, @hash.size
+      assert_nil @hash[:a]  # First item should be evicted
+      assert_equal 4, @hash[:d]
+    end
+
+    def test_updating_existing_key_does_not_trigger_eviction
+      @hash[:a] = 1
+      @hash[:b] = 2
+      @hash[:c] = 3
+
+      @hash[:b] = 'new value'  # Update existing key
+
+      assert_equal @max_size, @hash.size
+      assert_equal 1, @hash[:a]  # First item should still be present
+      assert_equal 'new value', @hash[:b]
+      assert_equal 3, @hash[:c]
+    end
+
+    def test_handles_nil_values
+      @hash[:a] = nil
+      @hash[:b] = 2
+      @hash[:c] = 3
+      @hash[:d] = 4
+
+      assert_nil @hash[:a]  # First item should be evicted
+      assert_equal 4, @hash[:d]
+    end
+
+    def test_preserves_hash_methods
+      @hash[:a] = 1
+      @hash[:b] = 2
+
+      assert_equal [:a, :b], @hash.keys
+      assert_equal [1, 2], @hash.values
+      assert @hash.key?(:a)
+      refute @hash.key?(:z)
+    end
+
+    def test_handles_string_keys
+      @hash['a'] = 1
+      @hash['b'] = 2
+      @hash['c'] = 3
+      @hash['d'] = 4
+
+      assert_nil @hash['a']  # First item should be evicted
+      assert_equal 4, @hash['d']
+    end
+
+    def test_handles_object_keys
+      key1 = Object.new
+      key2 = Object.new
+      key3 = Object.new
+      key4 = Object.new
+
+      @hash[key1] = 1
+      @hash[key2] = 2
+      @hash[key3] = 3
+      @hash[key4] = 4
+
+      assert_nil @hash[key1]  # First item should be evicted
+      assert_equal 4, @hash[key4]
+    end
+
+    def test_can_be_initialized_empty
+      assert_equal 0, @hash.size
+    end
+
+    def test_enumerable_methods
+      @hash[:a] = 1
+      @hash[:b] = 2
+
+      mapped = @hash.map { |k, v| [k, v * 2] }.to_h
+      assert_equal({ a: 2, b: 4 }, mapped)
+
+      filtered = @hash.select { |_, v| v > 1 }
+      assert_equal({ b: 2 }, filtered.to_h)
+    end
+
+    def test_clear_maintains_max_size_constraint
+      @hash[:a] = 1
+      @hash[:b] = 2
+      @hash.clear
+
+      assert_equal 0, @hash.size
+
+      # Should still enforce max size after clear
+      (@max_size + 1).times { |i| @hash[i] = i }
+      assert_equal @max_size, @hash.size
+    end
+  end
+end

data/test/test_semver.rb

@@ -0,0 +1,108 @@
+require 'test_helper'
+class TestSemanticVersion < Minitest::Test
+  def test_parse_valid_version
+    version = SemanticVersion.parse('1.2.3')
+    assert_equal 1, version.major
+    assert_equal 2, version.minor
+    assert_equal 3, version.patch
+    assert_nil version.prerelease
+    assert_nil version.build_metadata
+  end
+
+  def test_parse_version_with_prerelease
+    version = SemanticVersion.parse('1.2.3-alpha.1')
+    assert_equal 1, version.major
+    assert_equal 2, version.minor
+    assert_equal 3, version.patch
+    assert_equal 'alpha.1', version.prerelease
+    assert_nil version.build_metadata
+  end
+
+  def test_parse_version_with_build_metadata
+    version = SemanticVersion.parse('1.2.3+build.123')
+    assert_equal 1, version.major
+    assert_equal 2, version.minor
+    assert_equal 3, version.patch
+    assert_nil version.prerelease
+    assert_equal 'build.123', version.build_metadata
+  end
+
+  def test_parse_full_version
+    version = SemanticVersion.parse('1.2.3-alpha.1+build.123')
+    assert_equal 1, version.major
+    assert_equal 2, version.minor
+    assert_equal 3, version.patch
+    assert_equal 'alpha.1', version.prerelease
+    assert_equal 'build.123', version.build_metadata
+  end
+
+  def test_parse_invalid_version
+    assert_raises(ArgumentError) { SemanticVersion.parse('invalid') }
+    assert_raises(ArgumentError) { SemanticVersion.parse('1.2') }
+    assert_raises(ArgumentError) { SemanticVersion.parse('1.2.3.4') }
+    assert_raises(ArgumentError) { SemanticVersion.parse('') }
+  end
+
+  def test_parse_quietly
+    assert_nil SemanticVersion.parse_quietly('invalid')
+    refute_nil SemanticVersion.parse_quietly('1.2.3')
+  end
+
+  def test_to_string
+    assert_equal '1.2.3', SemanticVersion.parse('1.2.3').to_s
+    assert_equal '1.2.3-alpha.1', SemanticVersion.parse('1.2.3-alpha.1').to_s
+    assert_equal '1.2.3+build.123', SemanticVersion.parse('1.2.3+build.123').to_s
+    assert_equal '1.2.3-alpha.1+build.123', SemanticVersion.parse('1.2.3-alpha.1+build.123').to_s
+  end
+
+  def test_equality
+    v1 = SemanticVersion.parse('1.2.3')
+    v2 = SemanticVersion.parse('1.2.3')
+    v3 = SemanticVersion.parse('1.2.4')
+    v4 = SemanticVersion.parse('1.2.3-alpha')
+    v5 = SemanticVersion.parse('1.2.3+build.123')
+
+    assert_equal v1, v2
+    refute_equal v1, v3
+    refute_equal v1, v4
+    assert_equal v1, v5  # build metadata is ignored in equality
+  end
+
+  def test_comparison
+    versions = [
+      '1.0.0-alpha',
+      '1.0.0-alpha.1',
+      '1.0.0-beta.2',
+      '1.0.0-beta.11',
+      '1.0.0-rc.1',
+      '1.0.0',
+      '2.0.0',
+      '2.1.0',
+      '2.1.1'
+    ].map { |v| SemanticVersion.parse(v) }
+
+    # Test that each version is less than the next version
+    (versions.length - 1).times do |i|
+      assert versions[i] < versions[i + 1], "Expected #{versions[i]} < #{versions[i + 1]}"
+    end
+  end
+
+  def test_prerelease_comparison
+    # Test specific prerelease comparison cases
+    cases = [
+      ['1.0.0-alpha', '1.0.0-alpha.1', -1],
+      ['1.0.0-alpha.1', '1.0.0-alpha.beta', -1],
+      ['1.0.0-alpha.beta', '1.0.0-beta', -1],
+      ['1.0.0-beta', '1.0.0-beta.2', -1],
+      ['1.0.0-beta.2', '1.0.0-beta.11', -1],
+      ['1.0.0-beta.11', '1.0.0-rc.1', -1],
+      ['1.0.0-rc.1', '1.0.0', -1]
+    ]
+
+    cases.each do |v1_str, v2_str, expected|
+      v1 = SemanticVersion.parse(v1_str)
+      v2 = SemanticVersion.parse(v2_str)
+      assert_equal expected, (v1 <=> v2), "Expected #{v1} <=> #{v2} to be #{expected}"
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prefab-cloud-ruby
 version: !ruby/object:Gem::Version
-  version: 1.8.8.pre.1
+  version: 1.8.9
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-03 00:00:00.000000000 Z
+date: 2025-04-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -222,6 +222,7 @@ extra_rdoc_files:
 - README.md
 files:
 - ".envrc.sample"
+- ".github/pull_request_template.md"
 - ".github/workflows/ruby.yml"
 - ".gitmodules"
 - ".rubocop.yml"
@@ -240,6 +241,7 @@ files:
 - dev/console
 - dev/script_setup.rb
 - lib/prefab-cloud-ruby.rb
+- lib/prefab/caching_http_connection.rb
 - lib/prefab/client.rb
 - lib/prefab/config_client.rb
 - lib/prefab/config_client_presenter.rb
@@ -265,6 +267,7 @@ files:
 - lib/prefab/example_contexts_aggregator.rb
 - lib/prefab/exponential_backoff.rb
 - lib/prefab/feature_flag_client.rb
+- lib/prefab/fixed_size_hash.rb
 - lib/prefab/http_connection.rb
 - lib/prefab/internal_logger.rb
 - lib/prefab/javascript_stub.rb
@@ -277,6 +280,7 @@ files:
 - lib/prefab/prefab.rb
 - lib/prefab/rate_limit_cache.rb
 - lib/prefab/resolved_config_presenter.rb
+- lib/prefab/semver.rb
 - lib/prefab/sse_config_client.rb
 - lib/prefab/time_helpers.rb
 - lib/prefab/weighted_value_resolver.rb
@@ -292,6 +296,7 @@ files:
 - test/support/mock_base_client.rb
 - test/support/mock_config_client.rb
 - test/support/mock_config_loader.rb
+- test/test_caching_http_connection.rb
 - test/test_client.rb
 - test/test_config_client.rb
 - test/test_config_loader.rb
@@ -308,6 +313,7 @@ files:
 - test/test_example_contexts_aggregator.rb
 - test/test_exponential_backoff.rb
 - test/test_feature_flag_client.rb
+- test/test_fixed_size_hash.rb
 - test/test_helper.rb
 - test/test_integration.rb
 - test/test_internal_logger.rb
@@ -319,6 +325,7 @@ files:
 - test/test_options.rb
 - test/test_prefab.rb
 - test/test_rate_limit_cache.rb
+- test/test_semver.rb
 - test/test_sse_config_client.rb
 - test/test_weighted_value_resolver.rb
 homepage: http://github.com/prefab-cloud/prefab-cloud-ruby
@@ -336,11 +343,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
-rubygems_version: 3.2.32
+rubygems_version: 3.4.19
 signing_key:
 specification_version: 4
 summary: Prefab Ruby Infrastructure