diff --git a/src/lib/features/feature-toggle/feature-toggle-service.ts b/src/lib/features/feature-toggle/feature-toggle-service.ts index d495502eb4..293ae8a6cc 100644 --- a/src/lib/features/feature-toggle/feature-toggle-service.ts +++ b/src/lib/features/feature-toggle/feature-toggle-service.ts @@ -698,8 +698,8 @@ export class FeatureToggleService { strategyConfig: Unsaved, existing?: IFeatureStrategy, ): Promise< - { strategyName: string } & Pick< - Partial, + { name: string } & Pick< + Partial, | 'title' | 'disabled' | 'variants' @@ -731,7 +731,7 @@ export class FeatureToggleService { } return { - strategyName: name, + name, title, disabled, sortOrder, @@ -770,6 +770,7 @@ export class FeatureToggleService { const newFeatureStrategy = await this.featureStrategiesStore.createStrategyFeatureEnv({ ...standardizedConfig, + strategyName: standardizedConfig.name, constraints: standardizedConfig.constraints || [], variants: standardizedConfig.variants || [], parameters: standardizedConfig.parameters || {}, diff --git a/src/lib/features/feature-toggle/tests/feature-toggle-service.e2e.test.ts b/src/lib/features/feature-toggle/tests/feature-toggle-service.e2e.test.ts index f63e609891..c89cc02d40 100644 --- a/src/lib/features/feature-toggle/tests/feature-toggle-service.e2e.test.ts +++ b/src/lib/features/feature-toggle/tests/feature-toggle-service.e2e.test.ts @@ -136,13 +136,18 @@ test('Should be able to update existing strategy configuration', async () => { expect(createdConfig.name).toEqual('default'); const updatedConfig = await service.updateStrategy( createdConfig.id, - { parameters: { b2b: 'true' } }, + { name: 'flexibleRollout', parameters: { b2b: 'true' } }, { projectId, featureName, environment: DEFAULT_ENV }, TEST_AUDIT_USER, ); expect(createdConfig.id).toEqual(updatedConfig.id); + expect(updatedConfig.name).toEqual('flexibleRollout'); expect(updatedConfig.parameters).toEqual({ b2b: 'true', + // flexible rollout default parameters + rollout: '100', + groupId: featureName, + stickiness: 'default', }); });