fix: enable disabled strategies keeps settings (#7952)

src/lib/features/feature-toggle/feature-toggle-service.ts

@@ -1886,7 +1886,7 @@ class FeatureToggleService {
                     strategies.map((strategy) =>
                         this.updateStrategy(
                             strategy.id,
-                            { disabled: false },
+                            { ...strategy, disabled: false },
                             {
                                 environment,
                                 projectId: project,

src/lib/features/feature-toggle/tests/feature-toggle-service.e2e.test.ts

@@ -744,3 +744,51 @@ test('Should return "default" for stickiness when creating a flexibleRollout str
         strategies: [{ parameters: { stickiness: 'default' } }],
     });
 });
+
+test('Should enable disabled strategies on feature environment enabled', async () => {
+    const flagName = 'enableThisFlag';
+    const project = 'default';
+    const environment = 'default';
+    const shouldActivateDisabledStrategies = true;
+    await service.createFeatureToggle(
+        project,
+        {
+            name: flagName,
+        },
+        TEST_AUDIT_USER,
+    );
+    const config: Omit<FeatureStrategySchema, 'id'> = {
+        name: 'default',
+        constraints: [
+            { contextName: 'userId', operator: 'IN', values: ['1', '1'] },
+        ],
+        parameters: { param: 'a' },
+        variants: [
+            {
+                name: 'a',
+                weight: 100,
+                weightType: 'variable',
+                stickiness: 'random',
+            },
+        ],
+        disabled: true,
+    };
+    const createdConfig = await service.createStrategy(
+        config,
+        { projectId: project, featureName: flagName, environment },
+        TEST_AUDIT_USER,
+    );
+
+    await service.updateEnabled(
+        project,
+        flagName,
+        environment,
+        true,
+        TEST_AUDIT_USER,
+        { email: '[email protected]' } as User,
+        shouldActivateDisabledStrategies,
+    );
+
+    const strategy = await service.getStrategy(createdConfig.id);
+    expect(strategy).toMatchObject({ ...config, disabled: false });
+});