Fix tests

Author: esrakartalOpt

lib/optimizely/config/datafile_project_config.rb

@@ -120,10 +120,11 @@ def initialize(datafile, logger, error_handler)
       @excluded_holdouts = {}
       @flag_holdouts_map = {}
 
-      @holdout_id_map = generate_key_map(@holdouts, 'id')
       @holdouts.each do |holdout|
         next unless holdout['status'] == 'Running'
 
+        @holdout_id_map[holdout['id']] = holdout
+
         if holdout['includedFlags'].nil? || holdout['includedFlags'].empty?
           @global_holdouts[holdout['id']] = holdout
 

spec/config/datafile_project_config_spec.rb

@@ -1298,16 +1298,16 @@
       holdout = config_with_holdouts.get_holdout('holdout_1')
       expect(holdout).not_to be_nil
       expect(holdout['id']).to eq('holdout_1')
-      expect(holdout['key']).to eq('test_holdout')
+      expect(holdout['key']).to eq('global_holdout')
       expect(holdout['status']).to eq('Running')
     end
 
     it 'should return holdout regardless of status when valid ID is provided' do
       holdout = config_with_holdouts.get_holdout('holdout_2')
       expect(holdout).not_to be_nil
       expect(holdout['id']).to eq('holdout_2')
-      expect(holdout['key']).to eq('another_holdout')
-      expect(holdout['status']).to eq('Inactive')
+      expect(holdout['key']).to eq('specific_holdout')
+      expect(holdout['status']).to eq('Running')
     end
 
     it 'should return nil for non-existent holdout ID' do
@@ -1363,20 +1363,20 @@
           'key' => 'global',
           'status' => 'Running',
           'includedFlags' => [],
-          'excludedFlags' => %w[boolean_feature string_single_variable_feature]
+          'excludedFlags' => ['553339214', '594060']
         },
         {
           'id' => 'specific_holdout',
           'key' => 'specific',
           'status' => 'Running',
-          'includedFlags' => %w[multi_variate_feature empty_feature],
+          'includedFlags' => ['594089', '594032'],
           'excludedFlags' => []
         },
         {
           'id' => 'inactive_holdout',
           'key' => 'inactive',
           'status' => 'Inactive',
-          'includedFlags' => ['boolean_feature'],
+          'includedFlags' => ['553339214'],
           'excludedFlags' => []
         }
       ]