Fix unit test issue

Author: esrakartalOpt

spec/config/datafile_project_config_spec.rb

@@ -1974,9 +1974,9 @@ def build_datafile(experiments: [], rollouts: [], feature_flags: [])
       end
     end
 
-    describe '#get_global_holdouts' do
+    describe '#global_holdouts' do
       it 'returns only holdouts without includedRules (global holdouts)' do
-        global_holdouts = config_with_local_holdouts.get_global_holdouts
+        global_holdouts = config_with_local_holdouts.global_holdouts
 
         expect(global_holdouts).not_to be_nil
         expect(global_holdouts).to be_an(Array)
@@ -1988,14 +1988,14 @@ def build_datafile(experiments: [], rollouts: [], feature_flags: [])
       end
 
       it 'does not include local holdouts (those with includedRules array)' do
-        global_holdouts = config_with_local_holdouts.get_global_holdouts
+        global_holdouts = config_with_local_holdouts.global_holdouts
         local_holdout = config_with_local_holdouts.get_holdout('holdout_local_1')
 
         expect(global_holdouts).not_to include(local_holdout)
       end
 
       it 'does not include holdouts with empty includedRules array' do
-        global_holdouts = config_with_local_holdouts.get_global_holdouts
+        global_holdouts = config_with_local_holdouts.global_holdouts
         empty_local_holdout = config_with_local_holdouts.get_holdout('holdout_local_empty_rules')
 
         # Empty [] is local, not global — must not appear in global_holdouts
@@ -2022,7 +2022,7 @@ def build_datafile(experiments: [], rollouts: [], feature_flags: [])
           logger,
           error_handler
         )
-        expect(config_no_global.get_global_holdouts).to eq([])
+        expect(config_no_global.global_holdouts).to eq([])
       end
     end
 
@@ -2079,7 +2079,7 @@ def build_datafile(experiments: [], rollouts: [], feature_flags: [])
 
         # Must be classified as global
         expect(config_global_only.holdout_global?(holdout)).to be true
-        expect(config_global_only.get_global_holdouts).to include(holdout)
+        expect(config_global_only.global_holdouts).to include(holdout)
         expect(config_global_only.get_holdouts_for_rule('any_rule')).to eq([])
       end
 

spec/decision_service_holdout_spec.rb

@@ -783,7 +783,7 @@
         expect(config_with_local_holdouts.holdout_global?(global_holdout)).to be true
 
         # Verify it appears in global_holdouts list
-        expect(config_with_local_holdouts.get_global_holdouts).to include(global_holdout)
+        expect(config_with_local_holdouts.global_holdouts).to include(global_holdout)
 
         # Mock the holdout to return a decision to simulate user being in global holdout
         allow(decision_service_local).to receive(:get_variation_for_holdout)
@@ -836,10 +836,10 @@
         expect(rule_holdouts).to include(local_holdout)
 
         # Verify it is NOT in global holdouts
-        expect(config_with_local_holdouts.get_global_holdouts).not_to include(local_holdout)
+        expect(config_with_local_holdouts.global_holdouts).not_to include(local_holdout)
 
         # Set up global holdouts to miss (no global holdout decision)
-        config_with_local_holdouts.get_global_holdouts.each do |gh|
+        config_with_local_holdouts.global_holdouts.each do |gh|
           allow(decision_service_local).to receive(:get_variation_for_holdout)
             .with(gh, anything, anything)
             .and_return(Optimizely::DecisionService::DecisionResult.new(nil, false, []))
@@ -889,7 +889,7 @@
         expect(local_holdout).not_to be_nil
 
         # All global holdouts miss
-        config_with_local_holdouts.get_global_holdouts.each do |gh|
+        config_with_local_holdouts.global_holdouts.each do |gh|
           allow(decision_service_local).to receive(:get_variation_for_holdout)
             .with(gh, anything, anything)
             .and_return(Optimizely::DecisionService::DecisionResult.new(nil, false, []))