unleash-server 7.2.1 → 7.2.3

package/dist/lib/features/feature-toggle/tests/feature-toggles.e2e.test.js

@@ -7,7 +7,6 @@ import ApiUser from '../../../types/api-user.js';
 import { ApiTokenType } from '../../../types/model.js';
 import IncompatibleProjectError from '../../../error/incompatible-project-error.js';
 import { RoleName } from '../../../types/model.js';
-import { v4 as uuidv4 } from 'uuid';
 import { randomId } from '../../../util/random-id.js';
 import { DEFAULT_PROJECT, TEST_AUDIT_USER } from '../../../types/index.js';
 import { ForbiddenError } from '../../../error/index.js';
@@ -135,8 +134,8 @@ test('Trying to add a strategy configuration to environment not connected to fla
     });
 });
 test('should list dependencies and children', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(parent, 'default');
     await app.createFeature(child, 'default');
     await app.addDependency(child, parent);
@@ -152,8 +151,8 @@ test('should list dependencies and children', async () => {
     });
 });
 test('should not allow to change project with dependencies', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(parent, 'default');
     await app.createFeature(child, 'default');
     await app.addDependency(child, parent);
@@ -170,8 +169,8 @@ test('should not allow to change project with dependencies', async () => {
     user, 'default', TEST_AUDIT_USER)).rejects.errorWithMessage(new ForbiddenError('Changing project not allowed. Feature has dependencies.'));
 });
 test('Should not allow to archive/delete feature with children', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(parent, 'default');
     await app.createFeature(child, 'default');
     await app.addDependency(child, parent);
@@ -187,8 +186,8 @@ test('Should not allow to archive/delete feature with children', async () => {
     expect(deleteBody.message).toBe('You can not archive/delete this feature since other features depend on it.');
 });
 test('Should allow to archive/delete feature with children if no orphans are left', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(parent, 'default');
     await app.createFeature(child, 'default');
     await app.addDependency(child, parent);
@@ -199,9 +198,9 @@ test('Should allow to archive/delete feature with children if no orphans are lef
         .expect(200);
 });
 test('Should not allow to archive/delete feature when orphans are left', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
-    const orphan = uuidv4();
+    const parent = randomId();
+    const child = randomId();
+    const orphan = randomId();
     await app.createFeature(parent, 'default');
     await app.createFeature(child, 'default');
     await app.createFeature(orphan, 'default');
@@ -215,9 +214,9 @@ test('Should not allow to archive/delete feature when orphans are left', async (
     expect(deleteBody.message).toBe('You can not archive/delete those features since other features depend on them.');
 });
 test('should clone feature with parent dependencies', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
-    const childClone = uuidv4();
+    const parent = randomId();
+    const child = randomId();
+    const childClone = randomId();
     await app.createFeature(parent, 'default');
     await app.createFeature(child, 'default');
     await app.addDependency(child, parent);
@@ -1955,7 +1954,7 @@ test('Can create flag with impression data on different project', async () => {
     });
 });
 test('should handle strategy variants', async () => {
-    const feature = { name: uuidv4(), impressionData: false };
+    const feature = { name: randomId(), impressionData: false };
     await app.createFeature(feature.name);
     const strategyWithInvalidVariant = {
         name: 'flexibleRollout',
@@ -2017,13 +2016,13 @@ test('should handle strategy variants', async () => {
 });
 test('should reject invalid constraint values for multi-valued constraints', async () => {
     const project = await db.stores.projectStore.create({
-        id: uuidv4(),
-        name: uuidv4(),
+        id: randomId(),
+        name: randomId(),
         description: '',
         mode: 'open',
     });
     const flag = await db.stores.featureToggleStore.create(project.id, {
-        name: uuidv4(),
+        name: randomId(),
         impressionData: true,
         createdByUserId: 9999,
     });
@@ -2059,13 +2058,13 @@ test('should reject invalid constraint values for multi-valued constraints', asy
 });
 test('should add default constraint values for single-valued constraints', async () => {
     const project = await db.stores.projectStore.create({
-        id: uuidv4(),
-        name: uuidv4(),
+        id: randomId(),
+        name: randomId(),
         description: '',
         mode: 'open',
     });
     const flag = await db.stores.featureToggleStore.create(project.id, {
-        name: uuidv4(),
+        name: randomId(),
         impressionData: true,
         createdByUserId: 9999,
     });
@@ -2111,13 +2110,13 @@ test('should add default constraint values for single-valued constraints', async
 });
 test('should allow long parameter values', async () => {
     const project = await db.stores.projectStore.create({
-        id: uuidv4(),
-        name: uuidv4(),
-        description: uuidv4(),
+        id: randomId(),
+        name: randomId(),
+        description: randomId(),
         mode: 'open',
     });
     const flag = await db.stores.featureToggleStore.create(project.id, {
-        name: uuidv4(),
+        name: randomId(),
         createdByUserId: 9999,
     });
     const strategy = {
@@ -2130,7 +2129,7 @@ test('should allow long parameter values', async () => {
         .expect(200);
 });
 test('should change strategy sort order when payload is valid', async () => {
-    const flag = { name: uuidv4(), impressionData: false };
+    const flag = { name: randomId(), impressionData: false };
     await app.request
         .post('/api/admin/projects/default/features')
         .send({
@@ -2180,7 +2179,7 @@ test('should change strategy sort order when payload is valid', async () => {
     expect(strategiesOrdered[1].id).toBe(strategyOne.id);
 });
 test('should reject set sort order request when payload is invalid', async () => {
-    const flag = { name: uuidv4(), impressionData: false };
+    const flag = { name: randomId(), impressionData: false };
     await app.request
         .post(`/api/admin/projects/default/features/${flag.name}/environments/${DEFAULT_ENV}/strategies/set-sort-order`)
         .send([
@@ -2194,7 +2193,7 @@ test('should reject set sort order request when payload is invalid', async () =>
         .expect(400);
 });
 test('should return strategies in correct order when new strategies are added', async () => {
-    const flag = { name: uuidv4(), impressionData: false };
+    const flag = { name: randomId(), impressionData: false };
     await app.request
         .post('/api/admin/projects/default/features')
         .send({
@@ -2284,7 +2283,7 @@ test('should return strategies in correct order when new strategies are added',
     expect(strategiesReOrdered[3].id).toBe(strategyThree.id);
 });
 test('should create a strategy with segments', async () => {
-    const feature = { name: uuidv4(), impressionData: false };
+    const feature = { name: randomId(), impressionData: false };
     await app.createFeature(feature.name);
     const segment = await createSegment('segmentOne');
     const { body: strategyOne } = await createStrategy(feature.name, {
@@ -2318,7 +2317,7 @@ test('should create a strategy with segments', async () => {
     });
 });
 test('should add multiple segments to a strategy', async () => {
-    const feature = { name: uuidv4(), impressionData: false };
+    const feature = { name: randomId(), impressionData: false };
     await app.createFeature(feature.name);
     const segment = await createSegment('seg1');
     const segmentTwo = await createSegment('seg2');
@@ -2444,7 +2443,7 @@ test('Should batch stale features', async () => {
     expect(body.stale).toBeTruthy();
 });
 test('should return disabled strategies', async () => {
-    const flag = { name: uuidv4(), impressionData: false };
+    const flag = { name: randomId(), impressionData: false };
     await app.request
         .post('/api/admin/projects/default/features')
         .send({
@@ -2477,7 +2476,7 @@ test('should return disabled strategies', async () => {
     expect(strategies[1].disabled).toBe(strategyTwo.disabled);
 });
 test('should disable strategies in place', async () => {
-    const flag = { name: uuidv4(), impressionData: false };
+    const flag = { name: randomId(), impressionData: false };
     await app.request
         .post('/api/admin/projects/default/features')
         .send({
@@ -2749,7 +2748,7 @@ test('Updating feature strategy sort-order should trigger a an event', async ()
     });
 });
 test('should not be allowed to create with invalid strategy type name', async () => {
-    const feature = { name: uuidv4(), impressionData: false };
+    const feature = { name: randomId(), impressionData: false };
     await app.createFeature(feature.name);
     await createStrategy(feature.name, {
         name: 'random-type',
@@ -2759,7 +2758,7 @@ test('should not be allowed to create with invalid strategy type name', async ()
     }, 400);
 });
 test('should not be allowed to update with invalid strategy type name', async () => {
-    const feature = { name: uuidv4(), impressionData: false };
+    const feature = { name: randomId(), impressionData: false };
     await app.createFeature(feature.name);
     const { body: strategyOne } = await createStrategy(feature.name, {
         name: 'default',