unleash-server 5.7.2 → 5.8.0

package/dist/test/e2e/services/project-service.e2e.test.js

@@ -9,12 +9,13 @@ const permissions_1 = require("../../../lib/types/permissions");
 const test_config_1 = require("../../config/test-config");
 const model_1 = require("../../../lib/types/model");
 const random_id_1 = require("../../../lib/util/random-id");
-const environment_service_1 = __importDefault(require("../../../lib/services/environment-service"));
+const environment_service_1 = __importDefault(require("../../../lib/features/project-environments/environment-service"));
 const incompatible_project_error_1 = __importDefault(require("../../../lib/error/incompatible-project-error"));
 const services_1 = require("../../../lib/services");
 const events_1 = require("../../../lib/types/events");
 const date_fns_1 = require("date-fns");
 const features_1 = require("../../../lib/features");
+const types_1 = require("../../../lib/types");
 let stores;
 let db;
 let projectService;
@@ -24,6 +25,7 @@ let environmentService;
 let featureToggleService;
 let user; // many methods in this test use User instead of IUser
 let group;
+const TEST_USER_ID = -9999;
 const isProjectUser = async (userId, projectName, condition) => {
     expect(await projectService.isProjectUser(userId, projectName)).toBe(condition);
 };
@@ -41,9 +43,6 @@ beforeAll(async () => {
     });
     const config = (0, test_config_1.createTestConfig)({
         getLogger: no_logger_1.default,
-        experimental: {
-            flags: { privateProjects: true },
-        },
     });
     eventService = new services_1.EventService(stores, config);
     accessService = (0, features_1.createAccessService)(db.rawDatabase, config);
@@ -143,6 +142,7 @@ test('should not be able to delete project with toggles', async () => {
     await projectService.createProject(project, user);
     await stores.featureToggleStore.create(project.id, {
         name: 'test-project-delete',
+        createdByUserId: 9999,
     });
     try {
         await projectService.deleteProject(project.id, user);
@@ -376,7 +376,7 @@ test('should remove user from the project', async () => {
     });
     const memberRole = await stores.roleStore.getRoleByName(model_1.RoleName.MEMBER);
     await projectService.addUser(project.id, memberRole.id, projectMember1.id, 'test');
-    await projectService.removeUser(project.id, memberRole.id, projectMember1.id, 'test');
+    await projectService.removeUser(project.id, memberRole.id, projectMember1.id, 'test', TEST_USER_ID);
     const { users } = await projectService.getAccessToProject(project.id);
     const memberUsers = users.filter((u) => u.roleId === memberRole.id);
     expect(memberUsers).toHaveLength(0);
@@ -391,7 +391,7 @@ test('should not change project if feature toggle project does not match current
     };
     const toggle = { name: 'test-toggle' };
     await projectService.createProject(project, user);
-    await featureToggleService.createFeatureToggle(project.id, toggle, user.email);
+    await featureToggleService.createFeatureToggle(project.id, toggle, user.email, TEST_USER_ID);
     try {
         await projectService.changeProject('newProject', toggle.name, user, 'wrong-project-id');
     }
@@ -410,7 +410,7 @@ test('should return 404 if no project is found with the project id', async () =>
     };
     const toggle = { name: 'test-toggle-2' };
     await projectService.createProject(project, user);
-    await featureToggleService.createFeatureToggle(project.id, toggle, user.email);
+    await featureToggleService.createFeatureToggle(project.id, toggle, user.email, TEST_USER_ID);
     try {
         await projectService.changeProject('newProject', toggle.name, user, project.id);
     }
@@ -440,7 +440,7 @@ test('should fail if user is not authorized', async () => {
     });
     await projectService.createProject(project, user);
     await projectService.createProject(projectDestination, projectAdmin1);
-    await featureToggleService.createFeatureToggle(project.id, toggle, user.email);
+    await featureToggleService.createFeatureToggle(project.id, toggle, user.email, TEST_USER_ID);
     try {
         await projectService.changeProject(projectDestination.id, toggle.name, user, project.id);
     }
@@ -465,7 +465,7 @@ test('should change project when checks pass', async () => {
     const toggle = { name: (0, random_id_1.randomId)() };
     await projectService.createProject(projectA, user);
     await projectService.createProject(projectB, user);
-    await featureToggleService.createFeatureToggle(projectA.id, toggle, user.email);
+    await featureToggleService.createFeatureToggle(projectA.id, toggle, user.email, TEST_USER_ID);
     await projectService.changeProject(projectB.id, toggle.name, user, projectA.id);
     const updatedFeature = await featureToggleService.getFeature({
         featureName: toggle.name,
@@ -488,7 +488,7 @@ test('changing project should emit event even if user does not have a username s
     const toggle = { name: (0, random_id_1.randomId)() };
     await projectService.createProject(projectA, user);
     await projectService.createProject(projectB, user);
-    await featureToggleService.createFeatureToggle(projectA.id, toggle, user.email);
+    await featureToggleService.createFeatureToggle(projectA.id, toggle, user.email, TEST_USER_ID);
     const eventsBeforeChange = await stores.eventStore.getEvents();
     await projectService.changeProject(projectB.id, toggle.name, user, projectA.id);
     const eventsAfterChange = await stores.eventStore.getEvents();
@@ -511,9 +511,9 @@ test('should require equal project environments to move features', async () => {
     const toggle = { name: (0, random_id_1.randomId)() };
     await projectService.createProject(projectA, user);
     await projectService.createProject(projectB, user);
-    await featureToggleService.createFeatureToggle(projectA.id, toggle, user.email);
+    await featureToggleService.createFeatureToggle(projectA.id, toggle, user.email, TEST_USER_ID);
     await stores.environmentStore.create(environment);
-    await environmentService.addEnvironmentToProject(environment.name, projectB.id);
+    await environmentService.addEnvironmentToProject(environment.name, projectB.id, 'test', TEST_USER_ID);
     await expect(() => projectService.changeProject(projectB.id, toggle.name, user, projectA.id)).rejects.toThrowError(incompatible_project_error_1.default);
 });
 test('A newly created project only gets connected to enabled environments', async () => {
@@ -607,6 +607,7 @@ test('should add a user to the project with a custom role', async () => {
                 id: 8, // DELETE_FEATURE
             },
         ],
+        createdByUserId: types_1.SYSTEM_USER_ID,
     });
     await projectService.addUser(project.id, customRole.id, projectMember1.id, 'test');
     const { users } = await projectService.getAccessToProject(project.id);
@@ -643,6 +644,7 @@ test('should delete role entries when deleting project', async () => {
                 id: 8, // DELETE_FEATURE
             },
         ],
+        createdByUserId: types_1.SYSTEM_USER_ID,
     });
     await projectService.addUser(project.id, customRole.id, user1.id, 'test');
     await projectService.addUser(project.id, customRole.id, user2.id, 'test');
@@ -676,6 +678,7 @@ test('should change a users role in the project', async () => {
                 id: 8, // DELETE_FEATURE
             },
         ],
+        createdByUserId: types_1.SYSTEM_USER_ID,
     });
     const member = await stores.roleStore.getRoleByName(model_1.RoleName.MEMBER);
     await projectService.addUser(project.id, member.id, projectUser.id, 'test');
@@ -684,7 +687,7 @@ test('should change a users role in the project', async () => {
     expect(memberUser).toHaveLength(1);
     expect(memberUser[0].id).toBe(projectUser.id);
     expect(memberUser[0].name).toBe(projectUser.name);
-    await projectService.removeUser(project.id, member.id, projectUser.id, 'test');
+    await projectService.removeUser(project.id, member.id, projectUser.id, 'test', TEST_USER_ID);
     await projectService.addUser(project.id, customRole.id, projectUser.id, 'test');
     const { users: updatedUsers } = await projectService.getAccessToProject(project.id);
     const customUser = updatedUsers.filter((u) => u.roleId === customRole.id);
@@ -708,7 +711,7 @@ test('should update role for user on project', async () => {
     const memberRole = await stores.roleStore.getRoleByName(model_1.RoleName.MEMBER);
     const ownerRole = await stores.roleStore.getRoleByName(model_1.RoleName.OWNER);
     await projectService.addUser(project.id, memberRole.id, projectMember1.id, 'test');
-    await projectService.changeRole(project.id, ownerRole.id, projectMember1.id, 'test');
+    await projectService.changeRole(project.id, ownerRole.id, projectMember1.id, 'test', TEST_USER_ID);
     const { users } = await projectService.getAccessToProject(project.id);
     const memberUsers = users.filter((u) => u.roleId === memberRole.id);
     const ownerUsers = users.filter((u) => u.roleId === ownerRole.id);
@@ -735,7 +738,7 @@ test('should able to assign role without existing members', async () => {
     });
     const memberRole = await stores.roleStore.getRoleByName(model_1.RoleName.MEMBER);
     await projectService.addUser(project.id, memberRole.id, projectMember1.id, 'test');
-    await projectService.changeRole(project.id, testRole.id, projectMember1.id, 'test');
+    await projectService.changeRole(project.id, testRole.id, projectMember1.id, 'test', TEST_USER_ID);
     const { users } = await projectService.getAccessToProject(project.id);
     const memberUsers = users.filter((u) => u.roleId === memberRole.id);
     const testUsers = users.filter((u) => u.roleId === testRole.id);
@@ -755,10 +758,10 @@ describe('ensure project has at least one owner', () => {
         const roles = await stores.roleStore.getRolesForProject(project.id);
         const ownerRole = roles.find((r) => r.name === model_1.RoleName.OWNER);
         await expect(async () => {
-            await projectService.removeUser(project.id, ownerRole.id, user.id, 'test');
+            await projectService.removeUser(project.id, ownerRole.id, user.id, 'test', TEST_USER_ID);
         }).rejects.toThrowError(new Error('A project must have at least one owner'));
         await expect(async () => {
-            await projectService.removeUserAccess(project.id, user.id, 'test');
+            await projectService.removeUserAccess(project.id, user.id, 'test', TEST_USER_ID);
         }).rejects.toThrowError(new Error('A project must have at least one owner'));
     });
     test('should be able to remove member user from the project when another is owner', async () => {
@@ -775,9 +778,9 @@ describe('ensure project has at least one owner', () => {
             name: 'Some Name',
             email: 'member@getunleash.io',
         });
-        await projectService.addAccess(project.id, [memberRole.id], [], [memberUser.id], 'test');
+        await projectService.addAccess(project.id, [memberRole.id], [], [memberUser.id], 'test', TEST_USER_ID);
         const usersBefore = await projectService.getProjectUsers(project.id);
-        await projectService.removeUserAccess(project.id, memberUser.id, 'test');
+        await projectService.removeUserAccess(project.id, memberUser.id, 'test', TEST_USER_ID);
         const usersAfter = await projectService.getProjectUsers(project.id);
         expect(usersBefore).toHaveLength(2);
         expect(usersAfter).toHaveLength(1);
@@ -798,10 +801,10 @@ describe('ensure project has at least one owner', () => {
         const memberRole = await stores.roleStore.getRoleByName(model_1.RoleName.MEMBER);
         await projectService.addUser(project.id, memberRole.id, projectMember1.id, 'test');
         await expect(async () => {
-            await projectService.changeRole(project.id, memberRole.id, user.id, 'test');
+            await projectService.changeRole(project.id, memberRole.id, user.id, 'test', TEST_USER_ID);
         }).rejects.toThrowError(new Error('A project must have at least one owner'));
         await expect(async () => {
-            await projectService.setRolesForUser(project.id, user.id, [memberRole.id], 'test');
+            await projectService.setRolesForUser(project.id, user.id, [memberRole.id], 'test', TEST_USER_ID);
         }).rejects.toThrowError(new Error('A project must have at least one owner'));
     });
     async function projectWithGroupOwner(projectId) {
@@ -815,10 +818,10 @@ describe('ensure project has at least one owner', () => {
         await projectService.createProject(project, user);
         const roles = await stores.roleStore.getRolesForProject(project.id);
         const ownerRole = roles.find((r) => r.name === model_1.RoleName.OWNER);
-        await projectService.addGroup(project.id, ownerRole.id, group.id, 'test');
+        await projectService.addGroup(project.id, ownerRole.id, group.id, 'test', TEST_USER_ID);
         // this should be fine, leaving the group as the only owner
         // note group has zero members, but it still acts as an owner
-        await projectService.removeUser(project.id, ownerRole.id, user.id, 'test');
+        await projectService.removeUser(project.id, ownerRole.id, user.id, 'test', TEST_USER_ID);
         return {
             project,
             group,
@@ -828,20 +831,20 @@ describe('ensure project has at least one owner', () => {
     test('should not remove group from the project', async () => {
         const { project, group, ownerRole } = await projectWithGroupOwner('remove-group-not-allowed');
         await expect(async () => {
-            await projectService.removeGroup(project.id, ownerRole.id, group.id, 'test');
+            await projectService.removeGroup(project.id, ownerRole.id, group.id, 'test', TEST_USER_ID);
         }).rejects.toThrowError(new Error('A project must have at least one owner'));
         await expect(async () => {
-            await projectService.removeGroupAccess(project.id, group.id, 'test');
+            await projectService.removeGroupAccess(project.id, group.id, 'test', TEST_USER_ID);
         }).rejects.toThrowError(new Error('A project must have at least one owner'));
     });
     test('should not update role for group on project when she is the owner', async () => {
         const { project, group } = await projectWithGroupOwner('update-group-not-allowed');
         const memberRole = await stores.roleStore.getRoleByName(model_1.RoleName.MEMBER);
         await expect(async () => {
-            await projectService.changeGroupRole(project.id, memberRole.id, group.id, 'test');
+            await projectService.changeGroupRole(project.id, memberRole.id, group.id, 'test', TEST_USER_ID);
         }).rejects.toThrowError(new Error('A project must have at least one owner'));
         await expect(async () => {
-            await projectService.setRolesForGroup(project.id, group.id, [memberRole.id], 'test');
+            await projectService.setRolesForGroup(project.id, group.id, [memberRole.id], 'test', TEST_USER_ID);
         }).rejects.toThrowError(new Error('A project must have at least one owner'));
     });
 });
@@ -870,8 +873,9 @@ test('Should allow bulk update of group permissions', async () => {
                 id: 2, // CREATE_FEATURE
             },
         ],
+        createdByUserId: types_1.SYSTEM_USER_ID,
     });
-    await projectService.addAccess(project.id, [createFeatureRole.id], [group1.id], [user1.id], 'some-admin-user');
+    await projectService.addAccess(project.id, [createFeatureRole.id], [group1.id], [user1.id], 'some-admin-user', TEST_USER_ID);
 });
 test('Should bulk update of only users', async () => {
     const project = 'bulk-update-project-users';
@@ -887,8 +891,9 @@ test('Should bulk update of only users', async () => {
                 id: 2, // CREATE_FEATURE
             },
         ],
+        createdByUserId: types_1.SYSTEM_USER_ID,
     });
-    await projectService.addAccess(project, [createFeatureRole.id], [], [user1.id], 'some-admin-user');
+    await projectService.addAccess(project, [createFeatureRole.id], [], [user1.id], 'some-admin-user', TEST_USER_ID);
 });
 test('Should allow bulk update of only groups', async () => {
     const project = {
@@ -911,8 +916,9 @@ test('Should allow bulk update of only groups', async () => {
                 id: 2, // CREATE_FEATURE
             },
         ],
+        createdByUserId: types_1.SYSTEM_USER_ID,
     });
-    await projectService.addAccess(project.id, [createFeatureRole.id], [group1.id], [], 'some-admin-user');
+    await projectService.addAccess(project.id, [createFeatureRole.id], [group1.id], [], 'some-admin-user', TEST_USER_ID);
 });
 test('Should allow permutations of roles, groups and users when adding a new access', async () => {
     const project = {
@@ -946,6 +952,7 @@ test('Should allow permutations of roles, groups and users when adding a new acc
                 id: 2, // CREATE_FEATURE
             },
         ],
+        createdByUserId: types_1.SYSTEM_USER_ID,
     });
     const role2 = await accessService.createRole({
         name: 'permutation-role-2',
@@ -955,8 +962,9 @@ test('Should allow permutations of roles, groups and users when adding a new acc
                 id: 7, // UPDATE_FEATURE
             },
         ],
+        createdByUserId: types_1.SYSTEM_USER_ID,
     });
-    await projectService.addAccess(project.id, [role1.id, role2.id], [group1.id, group2.id], [user1.id, user2.id], 'some-admin-user');
+    await projectService.addAccess(project.id, [role1.id, role2.id], [group1.id, group2.id], [user1.id, user2.id], 'some-admin-user', TEST_USER_ID);
     const { users, groups } = await projectService.getAccessToProject(project.id);
     const ownerRole = await stores.roleStore.getRoleByName(model_1.RoleName.OWNER);
     expect(users).toHaveLength(3); // the 2 added plus the one that created the project
@@ -976,9 +984,11 @@ test('should only count active feature toggles for project', async () => {
     await projectService.createProject(project, user);
     await stores.featureToggleStore.create(project.id, {
         name: 'only-active-t1',
+        createdByUserId: 9999,
     });
     await stores.featureToggleStore.create(project.id, {
         name: 'only-active-t2',
+        createdByUserId: 9999,
     });
     await featureToggleService.archiveToggle('only-active-t2', user);
     const projects = await projectService.getProjects();
@@ -996,6 +1006,7 @@ test('should list projects with all features archived', async () => {
     await projectService.createProject(project, user);
     await stores.featureToggleStore.create(project.id, {
         name: 'archived-toggle',
+        createdByUserId: 9999,
     });
     await featureToggleService.archiveToggle('archived-toggle', user);
     const projects = await projectService.getProjects();
@@ -1030,7 +1041,7 @@ test('should calculate average time to production', async () => {
         { name: 'average-prod-time-5' },
     ];
     const featureToggles = await Promise.all(toggles.map((toggle) => {
-        return featureToggleService.createFeatureToggle(project.id, toggle, user.email);
+        return featureToggleService.createFeatureToggle(project.id, toggle, user.email, TEST_USER_ID);
     }));
     await Promise.all(featureToggles.map((toggle) => {
         return eventService.storeEvent(new events_1.FeatureEnvironmentEvent({
@@ -1039,6 +1050,7 @@ test('should calculate average time to production', async () => {
             featureName: toggle.name,
             environment: 'default',
             createdBy: 'Fredrik',
+            createdByUserId: TEST_USER_ID,
         }));
     }));
     await updateEventCreatedAt((0, date_fns_1.subDays)(new Date(), 31), 'average-prod-time-5');
@@ -1062,6 +1074,7 @@ test('should calculate average time to production ignoring some items', async ()
         featureName,
         environment: 'default',
         createdBy: 'Fredrik',
+        createdByUserId: TEST_USER_ID,
         tags: [],
     });
     await projectService.createProject(project, user);
@@ -1069,10 +1082,10 @@ test('should calculate average time to production ignoring some items', async ()
         name: 'customEnv',
         type: 'development',
     });
-    await environmentService.addEnvironmentToProject('customEnv', project.id);
+    await environmentService.addEnvironmentToProject('customEnv', project.id, types_1.SYSTEM_USER.username, types_1.SYSTEM_USER.id);
     // actual toggle we take for calculations
     const toggle = { name: 'main-toggle' };
-    await featureToggleService.createFeatureToggle(project.id, toggle, user.email);
+    await featureToggleService.createFeatureToggle(project.id, toggle, user.email, TEST_USER_ID);
     await updateFeature(toggle.name, {
         created_at: (0, date_fns_1.subDays)(new Date(), 20),
     });
@@ -1082,22 +1095,22 @@ test('should calculate average time to production ignoring some items', async ()
     await eventService.storeEvent(new events_1.FeatureEnvironmentEvent(makeEvent(toggle.name)));
     // ignore toggles enabled in non-prod envs
     const devToggle = { name: 'dev-toggle' };
-    await featureToggleService.createFeatureToggle(project.id, devToggle, user.email);
+    await featureToggleService.createFeatureToggle(project.id, devToggle, user.email, TEST_USER_ID);
     await eventService.storeEvent(new events_1.FeatureEnvironmentEvent({
         ...makeEvent(devToggle.name),
         environment: 'customEnv',
     }));
     // ignore toggles from other projects
     const otherProjectToggle = { name: 'other-project' };
-    await featureToggleService.createFeatureToggle('default', otherProjectToggle, user.email);
+    await featureToggleService.createFeatureToggle('default', otherProjectToggle, user.email, TEST_USER_ID);
     await eventService.storeEvent(new events_1.FeatureEnvironmentEvent(makeEvent(otherProjectToggle.name)));
     // ignore non-release toggles
     const nonReleaseToggle = { name: 'permission-toggle', type: 'permission' };
-    await featureToggleService.createFeatureToggle(project.id, nonReleaseToggle, user.email);
+    await featureToggleService.createFeatureToggle(project.id, nonReleaseToggle, user.email, TEST_USER_ID);
     await eventService.storeEvent(new events_1.FeatureEnvironmentEvent(makeEvent(nonReleaseToggle.name)));
     // ignore toggles with events before toggle creation time
     const previouslyDeleteToggle = { name: 'previously-deleted' };
-    await featureToggleService.createFeatureToggle(project.id, previouslyDeleteToggle, user.email);
+    await featureToggleService.createFeatureToggle(project.id, previouslyDeleteToggle, user.email, TEST_USER_ID);
     await eventService.storeEvent(new events_1.FeatureEnvironmentEvent(makeEvent(previouslyDeleteToggle.name)));
     await updateEventCreatedAt((0, date_fns_1.subDays)(new Date(), 30), previouslyDeleteToggle.name);
     const result = await projectService.getStatusUpdates(project.id);
@@ -1118,7 +1131,7 @@ test('should get correct amount of features created in current and past window',
         { name: 'features-created-4' },
     ];
     await Promise.all(toggles.map((toggle) => {
-        return featureToggleService.createFeatureToggle(project.id, toggle, user.email);
+        return featureToggleService.createFeatureToggle(project.id, toggle, user.email, TEST_USER_ID);
     }));
     await Promise.all([
         updateFeature(toggles[2].name, { created_at: (0, date_fns_1.subDays)(new Date(), 31) }),
@@ -1143,7 +1156,7 @@ test('should get correct amount of features archived in current and past window'
         { name: 'features-archived-4' },
     ];
     await Promise.all(toggles.map((toggle) => {
-        return featureToggleService.createFeatureToggle(project.id, toggle, user.email);
+        return featureToggleService.createFeatureToggle(project.id, toggle, user.email, TEST_USER_ID);
     }));
     await Promise.all([
         updateFeature(toggles[0].name, {
@@ -1206,7 +1219,7 @@ test('should return average time to production per toggle', async () => {
         { name: 'average-prod-time-pt-5', subdays: 2 },
     ];
     const featureToggles = await Promise.all(toggles.map((toggle) => {
-        return featureToggleService.createFeatureToggle(project.id, toggle, user.email);
+        return featureToggleService.createFeatureToggle(project.id, toggle, user.email, TEST_USER_ID);
     }));
     await Promise.all(featureToggles.map((toggle) => {
         return eventService.storeEvent(new events_1.FeatureEnvironmentEvent({
@@ -1215,6 +1228,7 @@ test('should return average time to production per toggle', async () => {
             featureName: toggle.name,
             environment: 'default',
             createdBy: 'Fredrik',
+            createdByUserId: TEST_USER_ID,
         }));
     }));
     await Promise.all(toggles.map((toggle) => updateFeature(toggle.name, {
@@ -1250,10 +1264,10 @@ test('should return average time to production per toggle for a specific project
         { name: 'average-prod-time-pt-14', subdays: 2 },
     ];
     const featureTogglesProject1 = await Promise.all(togglesProject1.map((toggle) => {
-        return featureToggleService.createFeatureToggle(project1.id, toggle, user.email);
+        return featureToggleService.createFeatureToggle(project1.id, toggle, user.email, TEST_USER_ID);
     }));
     const featureTogglesProject2 = await Promise.all(togglesProject2.map((toggle) => {
-        return featureToggleService.createFeatureToggle(project2.id, toggle, user.email);
+        return featureToggleService.createFeatureToggle(project2.id, toggle, user.email, TEST_USER_ID);
     }));
     await Promise.all(featureTogglesProject1.map((toggle) => {
         return eventService.storeEvent(new events_1.FeatureEnvironmentEvent({
@@ -1262,6 +1276,7 @@ test('should return average time to production per toggle for a specific project
             featureName: toggle.name,
             environment: 'default',
             createdBy: 'Fredrik',
+            createdByUserId: TEST_USER_ID,
         }));
     }));
     await Promise.all(featureTogglesProject2.map((toggle) => {
@@ -1271,6 +1286,7 @@ test('should return average time to production per toggle for a specific project
             featureName: toggle.name,
             environment: 'default',
             createdBy: 'Fredrik',
+            createdByUserId: TEST_USER_ID,
         }));
     }));
     await Promise.all(togglesProject1.map((toggle) => updateFeature(toggle.name, {
@@ -1298,7 +1314,7 @@ test('should return average time to production per toggle and include archived t
         { name: 'average-prod-time-pta-12', subdays: 40 },
     ];
     const featureTogglesProject1 = await Promise.all(togglesProject1.map((toggle) => {
-        return featureToggleService.createFeatureToggle(project1.id, toggle, user.email);
+        return featureToggleService.createFeatureToggle(project1.id, toggle, user.email, TEST_USER_ID);
     }));
     await Promise.all(featureTogglesProject1.map((toggle) => {
         return eventService.storeEvent(new events_1.FeatureEnvironmentEvent({
@@ -1307,6 +1323,7 @@ test('should return average time to production per toggle and include archived t
             featureName: toggle.name,
             environment: 'default',
             createdBy: 'Fredrik',
+            createdByUserId: TEST_USER_ID,
         }));
     }));
     await Promise.all(togglesProject1.map((toggle) => updateFeature(toggle.name, {
@@ -1362,6 +1379,7 @@ test('deleting a project with archived toggles should result in any remaining ar
     await projectService.createProject(project, user);
     await stores.featureToggleStore.create(project.id, {
         name: toggleName,
+        createdByUserId: 9999,
     });
     await stores.featureToggleStore.archive(toggleName);
     await projectService.deleteProject(project.id, user);