Addressing comments

Author: rashidsp

lib/optimizely/project_config.rb

@@ -54,7 +54,6 @@ class ProjectConfig
     attr_reader :feature_variable_key_map
     attr_reader :group_key_map
     attr_reader :rollout_id_map
-    attr_reader :rollout_experiment_id_map
     attr_reader :rollout_experiment_key_map
     attr_reader :variation_id_map
     attr_reader :variation_id_to_variable_usage_map
@@ -121,12 +120,10 @@ def initialize(datafile, logger, error_handler)
         end
       end
       @rollout_id_map = generate_key_map(@rollouts, 'id')
-      # split out the experiment id and key map for rollouts
-      @rollout_experiment_id_map = {}
+      # split out the experiment key map for rollouts
       @rollout_experiment_key_map = {}
       @rollout_id_map.each_value do |rollout|
         exps = rollout.fetch('experiments')
-        @rollout_experiment_id_map = @rollout_experiment_id_map.merge(generate_key_map(exps, 'id'))
         @rollout_experiment_key_map = @rollout_experiment_key_map.merge(generate_key_map(exps, 'key'))
       end
       @all_experiments = @experiment_key_map.merge(@rollout_experiment_key_map)

spec/project_config_spec.rb

@@ -666,15 +666,6 @@
         '166661' => config_body['rollouts'][1]
       }
 
-      expected_rollout_experiment_id_map = {
-        '177770' => config_body['rollouts'][0]['experiments'][0],
-        '177772' => config_body['rollouts'][0]['experiments'][1],
-        '177776' => config_body['rollouts'][0]['experiments'][2],
-        '177774' => config_body['rollouts'][1]['experiments'][0],
-        '177779' => config_body['rollouts'][1]['experiments'][1],
-        '177780' => config_body['rollouts'][1]['experiments'][2]
-      }
-
       expected_rollout_experiment_key_map = {
         '177770' => config_body['rollouts'][0]['experiments'][0],
         '177772' => config_body['rollouts'][0]['experiments'][1],
@@ -695,7 +686,6 @@
       expect(project_config.variation_key_map).to eq(expected_variation_key_map)
       expect(project_config.variation_id_to_variable_usage_map).to eq(expected_variation_id_to_variable_usage_map)
       expect(project_config.rollout_id_map).to eq(expected_rollout_id_map)
-      expect(project_config.rollout_experiment_id_map).to eq(expected_rollout_experiment_id_map)
       expect(project_config.rollout_experiment_key_map).to eq(expected_rollout_experiment_key_map)
     end