add new etst and format

Author: Mat001

pkg/client/client_test.go

@@ -3197,7 +3197,6 @@ func (m *MockCmabService) GetDecision(projectConfig config.ProjectConfig, userCo
 	return decision, args.Error(1)
 }
 
-
 func TestHandleDecisionServiceError(t *testing.T) {
 	client := OptimizelyClient{
 		logger: logging.GetLogger("", ""),
@@ -3380,13 +3379,11 @@ func TestDecideWithCmabUUID(t *testing.T) {
 	// This should cover the CMAB UUID handling lines
 }
 
-
 func (m *MockProjectConfig) GetExperimentByID(experimentID string) (entities.Experiment, error) {
 	args := m.Called(experimentID)
 	return args.Get(0).(entities.Experiment), args.Error(1)
 }
 
-
 func TestClientTestSuiteAB(t *testing.T) {
 	suite.Run(t, new(ClientTestSuiteAB))
 }
@@ -3454,7 +3451,7 @@ func TestHandleDecisionServiceError_MoreCoverage(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			result := client.handleDecisionServiceError(tt.err, tt.key, userContext)
-			
+
 			assert.Equal(t, tt.expected.FlagKey, result.FlagKey)
 			assert.Equal(t, tt.expected.UserContext, result.UserContext)
 			assert.Equal(t, tt.expected.VariationKey, result.VariationKey)
@@ -3483,10 +3480,10 @@ func TestGetAllOptionsWithCMABOptions(t *testing.T) {
 	result := client.getAllOptions(userOptions)
 
 	// Verify all CMAB-related options are properly merged
-	assert.True(t, result.DisableDecisionEvent) // from default
-	assert.True(t, result.IgnoreCMABCache)      // from default
-	assert.True(t, result.ResetCMABCache)       // from user
+	assert.True(t, result.DisableDecisionEvent)    // from default
+	assert.True(t, result.IgnoreCMABCache)         // from default
+	assert.True(t, result.ResetCMABCache)          // from user
 	assert.True(t, result.InvalidateUserCMABCache) // from user
-	assert.False(t, result.EnabledFlagsOnly)    // neither
-	assert.False(t, result.ExcludeVariables)    // neither
+	assert.False(t, result.EnabledFlagsOnly)       // neither
+	assert.False(t, result.ExcludeVariables)       // neither
 }

pkg/client/factory_test.go

@@ -462,8 +462,8 @@ func TestStaticClientError(t *testing.T) {
 func TestFactoryWithCmabConfig(t *testing.T) {
 	factory := OptimizelyFactory{}
 	cmabConfig := cmab.Config{
-		CacheSize: 100,
-		CacheTTL:  time.Minute,
+		CacheSize:   100,
+		CacheTTL:    time.Minute,
 		HTTPTimeout: 30 * time.Second,
 		RetryConfig: &cmab.RetryConfig{
 			MaxRetries:        5,
@@ -537,3 +537,48 @@ func TestWithCmabConfigOption(t *testing.T) {
 	WithCmabConfig(testConfig)(factory)
 	assert.Equal(t, testConfig, factory.cmabConfig)
 }
+
+func TestClientWithCmabConfig(t *testing.T) {
+	// Test client creation with non-empty CMAB config (tests reflect.DeepEqual path)
+	cmabConfig := cmab.Config{
+		CacheSize:   200,
+		CacheTTL:    5 * time.Minute,
+		HTTPTimeout: 30 * time.Second,
+		RetryConfig: &cmab.RetryConfig{
+			MaxRetries:        5,
+			InitialBackoff:    200 * time.Millisecond,
+			MaxBackoff:        20 * time.Second,
+			BackoffMultiplier: 3.0,
+		},
+	}
+
+	factory := OptimizelyFactory{
+		SDKKey: "test_sdk_key",
+	}
+
+	client, err := factory.Client(WithCmabConfig(cmabConfig))
+	assert.NoError(t, err)
+	assert.NotNil(t, client)
+
+	// Verify the CMAB config was applied by checking if DecisionService exists
+	// This tests the reflect.DeepEqual check on lines 166-167
+	assert.NotNil(t, client.DecisionService)
+	client.Close()
+}
+
+func TestClientWithEmptyCmabConfig(t *testing.T) {
+	// Test client creation with empty CMAB config (tests reflect.DeepEqual returns true)
+	emptyCmabConfig := cmab.Config{}
+
+	factory := OptimizelyFactory{
+		SDKKey: "test_sdk_key",
+	}
+
+	client, err := factory.Client(WithCmabConfig(emptyCmabConfig))
+	assert.NoError(t, err)
+	assert.NotNil(t, client)
+
+	// Verify client still works with empty config
+	assert.NotNil(t, client.DecisionService)
+	client.Close()
+}

pkg/decision/composite_experiment_service_test.go

@@ -311,7 +311,7 @@ func (s *CompositeExperimentTestSuite) TestCmabServiceReturnsError() {
 	// Test that CMAB service error is properly propagated
 	mockCmabService := new(MockExperimentDecisionService)
 	testErr := errors.New("Failed to fetch CMAB data for experiment exp_123")
-	
+
 	mockCmabService.On("GetDecision", mock.Anything, mock.Anything, mock.Anything).Return(
 		ExperimentDecision{},
 		decide.NewDecisionReasons(s.options),
@@ -320,7 +320,7 @@ func (s *CompositeExperimentTestSuite) TestCmabServiceReturnsError() {
 
 	compositeService := &CompositeExperimentService{
 		experimentServices: []ExperimentService{mockCmabService},
-		logger:            logging.GetLogger("", "CompositeExperimentService"),
+		logger:             logging.GetLogger("", "CompositeExperimentService"),
 	}
 
 	userContext := entities.UserContext{ID: "test_user"}