prefab-cloud-ruby 0.23.8 → 0.24.0

data/test/test_local_config_parser.rb

@@ -34,14 +34,13 @@ class TestLocalConfigParser < Minitest::Test
     assert_equal 1, config.rows[0].values.size
 
     value_row = config.rows[0].values[0]
-    assert_equal Prefab::WeightedValues, value_row.value.weighted_values.class
     assert_equal 'all-features', Prefab::ConfigValueUnwrapper.unwrap(value_row.value, key, {})
   end
 
-  def test_flag_in_lookup_key
-    key = :flag_in_lookup_key
-    value = stringify_keys({ "feature_flag": 'true', value: true,
-                             criterion: { operator: 'LOOKUP_KEY_IN', values: %w[abc123 xyz987] } })
+  def test_flag_in_user_key
+    key = :flag_in_user_key
+    value = stringify_keys({ 'feature_flag': 'true', value: true,
+                             criterion: { operator: 'PROP_IS_ONE_OF', property: 'user.key', values: %w[abc123 xyz987] } })
     parsed = Prefab::LocalConfigParser.parse(key, value, {}, FILE_NAME)[key]
     config = parsed[:config]
 
@@ -54,11 +53,10 @@ class TestLocalConfigParser < Minitest::Test
     assert_equal 1, config.rows[0].values[0].criteria.size
 
     value_row = config.rows[0].values[0]
-    assert_equal Prefab::WeightedValues, value_row.value.weighted_values.class
     assert_equal true, Prefab::ConfigValueUnwrapper.unwrap(value_row.value, key, {})
 
-    assert_equal Prefab::CriteriaEvaluator::LOOKUP_KEY, value_row.criteria[0].property_name
-    assert_equal :LOOKUP_KEY_IN, value_row.criteria[0].operator
+    assert_equal 'user.key', value_row.criteria[0].property_name
+    assert_equal :PROP_IS_ONE_OF, value_row.criteria[0].operator
     assert_equal %w[abc123 xyz987], value_row.criteria[0].value_to_match.string_list.values
   end
 

data/test/test_log_path_collector.rb

@@ -49,13 +49,10 @@ class TestLogPathCollector < Minitest::Test
   private
 
   def new_client(overrides = {})
-    options = Prefab::Options.new(**{
-      prefab_config_override_dir: 'none',
-      prefab_config_classpath_dir: 'test',
-      prefab_envs: ['unit_tests'],
-      api_key: '123-development-yourapikey-SDK'
+    super(**{
+      prefab_datasources: Prefab::Options::DATASOURCES::ALL,
+      api_key: '123-development-yourapikey-SDK',
+      collect_sync_interval: 1000 # we'll trigger sync manually in our test
     }.merge(overrides))
-
-    Prefab::Client.new(options)
   end
 end

data/test/test_logger.rb

@@ -192,7 +192,7 @@ class TestLogger < Minitest::Test
                 Prefab::Criterion.new(
                   operator: Prefab::Criterion::CriterionOperator::PROP_IS_ONE_OF,
                   value_to_match: string_list(['hotmail.com', 'gmail.com']),
-                  property_name: 'email_suffix'
+                  property_name: 'user.email_suffix'
                 )
               ],
               value: Prefab::ConfigValue.new(log_level: WRONG_ENV_VALUE)
@@ -209,7 +209,7 @@ class TestLogger < Minitest::Test
                 Prefab::Criterion.new(
                   operator: Prefab::Criterion::CriterionOperator::PROP_IS_ONE_OF,
                   value_to_match: string_list(['hotmail.com', 'gmail.com']),
-                  property_name: 'email_suffix'
+                  property_name: 'user.email_suffix'
                 )
               ],
               value: Prefab::ConfigValue.new(log_level: DESIRED_VALUE)
@@ -228,7 +228,7 @@ class TestLogger < Minitest::Test
     inject_project_env_id(prefab, PROJECT_ENV_ID)
 
     # without any context, the level should be the default for the env (info)
-    prefab.with_log_context(nil, {}) do
+    prefab.with_context({}) do
       prefab.log.debug 'Test debug'
       refute_logged io, 'Test debug'
 
@@ -242,7 +242,7 @@ class TestLogger < Minitest::Test
     reset_io(io)
 
     # with the wrong context, the level should be the default for the env (info)
-    prefab.with_log_context('user:1234', email_suffix: 'yahoo.com') do
+    prefab.with_context(user: { email_suffix: 'yahoo.com' }) do
       prefab.log.debug 'Test debug'
       refute_logged io, 'Test debug'
 
@@ -256,7 +256,7 @@ class TestLogger < Minitest::Test
     reset_io(io)
 
     # with the correct context, the level should be the desired value (debug)
-    prefab.with_log_context('user:1234', email_suffix: 'hotmail.com') do
+    prefab.with_context(user: { email_suffix: 'hotmail.com' }) do
       prefab.log.debug 'Test debug'
       assert_logged io, 'DEBUG', "#{prefix}.test.test_logger.test_logging_with_criteria_on_top_level_key", 'Test debug'
 
@@ -302,7 +302,7 @@ class TestLogger < Minitest::Test
                 Prefab::Criterion.new(
                   operator: Prefab::Criterion::CriterionOperator::PROP_IS_ONE_OF,
                   value_to_match: string_list(['hotmail.com', 'gmail.com']),
-                  property_name: 'email_suffix'
+                  property_name: 'user.email_suffix'
                 )
               ],
               value: Prefab::ConfigValue.new(log_level: DESIRED_VALUE)
@@ -311,9 +311,9 @@ class TestLogger < Minitest::Test
             Prefab::ConditionalValue.new(
               criteria: [
                 Prefab::Criterion.new(
-                  operator: Prefab::Criterion::CriterionOperator::LOOKUP_KEY_IN,
+                  operator: Prefab::Criterion::CriterionOperator::PROP_IS_ONE_OF,
                   value_to_match: string_list(%w[user:4567]),
-                  property_name: Prefab::CriteriaEvaluator::LOOKUP_KEY
+                  property_name: 'user.tracking_id'
                 )
               ],
               value: Prefab::ConfigValue.new(log_level: DESIRED_VALUE)
@@ -333,7 +333,7 @@ class TestLogger < Minitest::Test
     inject_project_env_id(prefab, PROJECT_ENV_ID)
 
     # without any context, the level should be the default for the env (info)
-    prefab.with_log_context(nil, {}) do
+    prefab.with_context({}) do
       prefab.log.debug 'Test debug'
       refute_logged io, 'Test debug'
 
@@ -347,7 +347,7 @@ class TestLogger < Minitest::Test
     reset_io(io)
 
     # with the wrong context, the level should be the default for the env (info)
-    prefab.with_log_context('user:1234', email_suffix: 'yahoo.com') do
+    prefab.with_context(user: { email_suffix: 'yahoo.com' }) do
       prefab.log.debug 'Test debug'
       refute_logged io, 'Test debug'
 
@@ -361,7 +361,7 @@ class TestLogger < Minitest::Test
     reset_io(io)
 
     # with the correct context, the level should be the desired value (debug)
-    prefab.with_log_context('user:1234', email_suffix: 'hotmail.com') do
+    prefab.with_context(user: { email_suffix: 'hotmail.com' }) do
       prefab.log.debug 'Test debug'
       assert_logged io, 'DEBUG', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test debug'
 
@@ -375,7 +375,7 @@ class TestLogger < Minitest::Test
     reset_io(io)
 
     # with the correct lookup key
-    prefab.with_log_context('user:4567', email_suffix: 'example.com') do
+    prefab.with_context(user: { tracking_id: 'user:4567' }) do
       prefab.log.debug 'Test debug'
       assert_logged io, 'DEBUG', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test debug'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prefab-cloud-ruby
 version: !ruby/object:Gem::Version
-  version: 0.23.8
+  version: 0.24.0
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-04-21 00:00:00.000000000 Z
+date: 2023-04-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -197,6 +197,7 @@ files:
 - lib/prefab/config_loader.rb
 - lib/prefab/config_resolver.rb
 - lib/prefab/config_value_unwrapper.rb
+- lib/prefab/context.rb
 - lib/prefab/criteria_evaluator.rb
 - lib/prefab/error.rb
 - lib/prefab/errors/initialization_timeout_error.rb
@@ -213,6 +214,7 @@ files:
 - lib/prefab/noop_cache.rb
 - lib/prefab/noop_stats.rb
 - lib/prefab/options.rb
+- lib/prefab/resolved_config_presenter.rb
 - lib/prefab/sse_logger.rb
 - lib/prefab/weighted_value_resolver.rb
 - lib/prefab/yaml_config_parser.rb
@@ -227,6 +229,7 @@ files:
 - test/test_config_loader.rb
 - test/test_config_resolver.rb
 - test/test_config_value_unwrapper.rb
+- test/test_context.rb
 - test/test_criteria_evaluator.rb
 - test/test_exponential_backoff.rb
 - test/test_feature_flag_client.rb