[OASIS-7757] - update copyright years and add more testcases for sdk_key and environment_key. Move decide tests to test_user_context from test_optimizely.

Author: The-inside-man

optimizely/optimizely_config.py

@@ -1,4 +1,4 @@
-# Copyright 2020, Optimizely
+# Copyright 2020-2021, Optimizely
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at

optimizely/optimizely_factory.py

@@ -113,7 +113,6 @@ def default_instance_with_config_manager(config_manager):
  def custom_instance(sdk_key, datafile=None, event_dispatcher=None, logger=None, error_handler=None,
  skip_json_validation=None, user_profile_service=None, config_manager=None,
  notification_center=None):
-
  """ Returns a new optimizely instance.
  if max_event_batch_size and max_event_flush_interval are None then default batch_size and flush_interval
  will be used to setup BatchEventProcessor.

optimizely/project_config.py

@@ -1,4 +1,4 @@
-# Copyright 2016-2019, Optimizely
+# Copyright 2016-2019, 2021, Optimizely
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at

tests/test_optimizely.py

@@ -26,7 +26,6 @@
 from optimizely import optimizely_config
 from optimizely import project_config
 from optimizely import version
-from optimizely.decision.optimizely_decide_option import OptimizelyDecideOption as DecideOption
 from optimizely.event.event_factory import EventFactory
 from optimizely.helpers import enums
 from . import base
@@ -266,19 +265,6 @@ def test_init__sdk_key_and_datafile_access_token(self):
 
  self.assertIs(type(opt_obj.config_manager), config_manager.AuthDatafilePollingConfigManager)
 
- def test_init__invalid_default_decide_options(self):
- """
- Test to confirm that default decide options passed not as a list will trigger setting
- self.deafulat_decide_options as an empty list.
- """
- invalid_decide_options = {"testKey": "testOption"}
-
- mock_client_logger = mock.MagicMock()
- with mock.patch('optimizely.logger.reset_logger', return_value=mock_client_logger):
- opt_obj = optimizely.Optimizely(default_decide_options=invalid_decide_options)
-
- self.assertEqual(opt_obj.default_decide_options, [])
-
  def test_invalid_json_raises_schema_validation_off(self):
  """ Test that invalid JSON logs error if schema validation is turned off. """
 
@@ -690,24 +676,6 @@ def on_activate(experiment, user_id, attributes, variation, event):
  self.assertEqual(1, mock_process.call_count)
  self.assertEqual(True, access_callback[0])
 
- def test_decide_experiment(self):
- """ Test that the feature is enabled for the user if bucketed into variation of a rollout.
- Also confirm that no impression event is processed. """
-
- opt_obj = optimizely.Optimizely(json.dumps(self.config_dict_with_features))
- project_config = opt_obj.config_manager.get_config()
-
- mock_experiment = project_config.get_experiment_from_key('test_experiment')
- mock_variation = project_config.get_variation_from_id('test_experiment', '111129')
- with mock.patch(
- 'optimizely.decision_service.DecisionService.get_variation_for_feature',
- return_value=(decision_service.Decision(mock_experiment,
- mock_variation, enums.DecisionSources.FEATURE_TEST), []),
- ):
- user_context = opt_obj.create_user_context('test_user')
- decision = user_context.decide('test_feature_in_experiment', [DecideOption.DISABLE_DECISION_EVENT])
- self.assertTrue(decision.enabled, "decision should be enabled")
-
  def test_activate__with_attributes__audience_match(self):
  """ Test that activate calls process with right params and returns expected
  variation when attributes are provided and audience conditions are met. """

tests/test_optimizely_config.py

@@ -1,4 +1,4 @@
-# Copyright 2020, Optimizely
+# Copyright 2020-2021, Optimizely
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
@@ -734,3 +734,59 @@ def test__get_datafile(self):
  actual_datafile = self.actual_config.get_datafile()
 
  self.assertEqual(expected_datafile, actual_datafile)
+
+ def test__get_sdk_key(self):
+ """ Test that get_sdk_key returns the expected value. """
+
+ config = optimizely_config.OptimizelyConfig(
+ revision='101',
+ experiments_map={},
+ features_map={},
+ sdk_key='testSdkKey',
+ )
+
+ expected_value = 'testSdkKey'
+
+ self.assertEqual(expected_value, config.get_sdk_key())
+
+ def test__get_sdk_key_invalid(self):
+ """ Negative Test that tests get_sdk_key does not return the expected value. """
+
+ config = optimizely_config.OptimizelyConfig(
+ revision='101',
+ experiments_map={},
+ features_map={},
+ sdk_key='testSdkKey',
+ )
+
+ invalid_value = 123
+
+ self.assertNotEqual(invalid_value, config.get_sdk_key())
+
+ def test__get_environment_key(self):
+ """ Test that get_environment_key returns the expected value. """
+
+ config = optimizely_config.OptimizelyConfig(
+ revision='101',
+ experiments_map={},
+ features_map={},
+ environment_key='TestEnvironmentKey'
+ )
+
+ expected_value = 'TestEnvironmentKey'
+
+ self.assertEqual(expected_value, config.get_environment_key())
+
+ def test__get_environment_key_invalid(self):
+ """ Negative Test that tests get_environment_key does not return the expected value. """
+
+ config = optimizely_config.OptimizelyConfig(
+ revision='101',
+ experiments_map={},
+ features_map={},
+ environment_key='testEnvironmentKey'
+ )
+
+ invalid_value = 321
+
+ self.assertNotEqual(invalid_value, config.get_environment_key())

tests/test_user_context.py

@@ -15,6 +15,7 @@
 import mock
 
 from optimizely.decision.optimizely_decision import OptimizelyDecision
+from optimizely.decision.optimizely_decide_option import OptimizelyDecideOption as DecideOption
 from optimizely.helpers import enums
 from . import base
 from optimizely import optimizely, decision_service
@@ -1264,3 +1265,34 @@ def test_decide_reasons__whitelisted_variation(self):
  expected_reasons = ['User "user_1" is forced in variation "control".']
 
  self.assertEqual(expected_reasons, actual.reasons)
+
+ def test_init__invalid_default_decide_options(self):
+ """
+ Test to confirm that default decide options passed not as a list will trigger setting
+ self.deafulat_decide_options as an empty list.
+ """
+ invalid_decide_options = {"testKey": "testOption"}
+
+ mock_client_logger = mock.MagicMock()
+ with mock.patch('optimizely.logger.reset_logger', return_value=mock_client_logger):
+ opt_obj = optimizely.Optimizely(default_decide_options=invalid_decide_options)
+
+ self.assertEqual(opt_obj.default_decide_options, [])
+
+ def test_decide_experiment(self):
+ """ Test that the feature is enabled for the user if bucketed into variation of a rollout.
+ Also confirm that no impression event is processed. """
+
+ opt_obj = optimizely.Optimizely(json.dumps(self.config_dict_with_features))
+ project_config = opt_obj.config_manager.get_config()
+
+ mock_experiment = project_config.get_experiment_from_key('test_experiment')
+ mock_variation = project_config.get_variation_from_id('test_experiment', '111129')
+ with mock.patch(
+ 'optimizely.decision_service.DecisionService.get_variation_for_feature',
+ return_value=(decision_service.Decision(mock_experiment,
+ mock_variation, enums.DecisionSources.FEATURE_TEST), []),
+ ):
+ user_context = opt_obj.create_user_context('test_user')
+ decision = user_context.decide('test_feature_in_experiment', [DecideOption.DISABLE_DECISION_EVENT])
+ self.assertTrue(decision.enabled, "decision should be enabled")