add missing tests for coveralls increase

Author: Mat001

pkg/client/factory_test.go

@@ -385,3 +385,52 @@ func TestOptimizelyClientWithNoTracer(t *testing.T) {
 	tracer := optimizelyClient.tracer.(*tracing.NoopTracer)
 	assert.NotNil(t, tracer)
 }
+
+func TestConvertDecideOptionsWithCMABOptions(t *testing.T) {
+	// Test with IgnoreCMABCache option
+	options := []decide.OptimizelyDecideOptions{decide.IgnoreCMABCache}
+	convertedOptions := convertDecideOptions(options)
+	assert.True(t, convertedOptions.IgnoreCMABCache)
+	assert.False(t, convertedOptions.ResetCMABCache)
+	assert.False(t, convertedOptions.InvalidateUserCMABCache)
+
+	// Test with ResetCMABCache option
+	options = []decide.OptimizelyDecideOptions{decide.ResetCMABCache}
+	convertedOptions = convertDecideOptions(options)
+	assert.False(t, convertedOptions.IgnoreCMABCache)
+	assert.True(t, convertedOptions.ResetCMABCache)
+	assert.False(t, convertedOptions.InvalidateUserCMABCache)
+
+	// Test with InvalidateUserCMABCache option
+	options = []decide.OptimizelyDecideOptions{decide.InvalidateUserCMABCache}
+	convertedOptions = convertDecideOptions(options)
+	assert.False(t, convertedOptions.IgnoreCMABCache)
+	assert.False(t, convertedOptions.ResetCMABCache)
+	assert.True(t, convertedOptions.InvalidateUserCMABCache)
+
+	// Test with all CMAB options
+	options = []decide.OptimizelyDecideOptions{
+		decide.IgnoreCMABCache,
+		decide.ResetCMABCache,
+		decide.InvalidateUserCMABCache,
+	}
+	convertedOptions = convertDecideOptions(options)
+	assert.True(t, convertedOptions.IgnoreCMABCache)
+	assert.True(t, convertedOptions.ResetCMABCache)
+	assert.True(t, convertedOptions.InvalidateUserCMABCache)
+
+	// Test with CMAB options mixed with other options
+	options = []decide.OptimizelyDecideOptions{
+		decide.DisableDecisionEvent,
+		decide.IgnoreCMABCache,
+		decide.EnabledFlagsOnly,
+		decide.ResetCMABCache,
+		decide.InvalidateUserCMABCache,
+	}
+	convertedOptions = convertDecideOptions(options)
+	assert.True(t, convertedOptions.DisableDecisionEvent)
+	assert.True(t, convertedOptions.EnabledFlagsOnly)
+	assert.True(t, convertedOptions.IgnoreCMABCache)
+	assert.True(t, convertedOptions.ResetCMABCache)
+	assert.True(t, convertedOptions.InvalidateUserCMABCache)
+}

pkg/config/datafileprojectconfig/config_test.go

@@ -655,6 +655,34 @@ func TestCmabExperimentsNil(t *testing.T) {
 	}
 }
 
+func TestGetExperimentByID(t *testing.T) {
+	// Create a test config with some experiments
+	testConfig := DatafileProjectConfig{
+		experimentMap: map[string]entities.Experiment{
+			"exp1": {ID: "exp1", Key: "experiment_1"},
+			"exp2": {ID: "exp2", Key: "experiment_2"},
+		},
+	}
+
+	// Test getting an experiment that exists
+	experiment, err := testConfig.GetExperimentByID("exp1")
+	assert.NoError(t, err)
+	assert.Equal(t, "exp1", experiment.ID)
+	assert.Equal(t, "experiment_1", experiment.Key)
+
+	// Test getting another experiment that exists
+	experiment, err = testConfig.GetExperimentByID("exp2")
+	assert.NoError(t, err)
+	assert.Equal(t, "exp2", experiment.ID)
+	assert.Equal(t, "experiment_2", experiment.Key)
+
+	// Test getting an experiment that doesn't exist
+	experiment, err = testConfig.GetExperimentByID("non_existent")
+	assert.Error(t, err)
+	assert.Equal(t, `experiment with ID "non_existent" not found`, err.Error())
+	assert.Equal(t, entities.Experiment{}, experiment)
+}
+
 func TestGetAttributeKeyByID(t *testing.T) {
 	// Setup
 	id := "id"