feat: return project roles

src/lib/features/personal-dashboard/createPersonalDashboardService.ts

@@ -16,6 +16,8 @@ import { AccountStore } from '../../db/account-store';
 import { FakeAccountStore } from '../../../test/fixtures/fake-account-store';
 import { OnboardingReadModel } from '../onboarding/onboarding-read-model';
 import { FakeOnboardingReadModel } from '../onboarding/fake-onboarding-read-model';
+import { AccessStore } from '../../db/access-store';
+import FakeAccessStore from '../../../test/fixtures/fake-access-store';
 
 export const createPersonalDashboardService = (
     db: Db,
@@ -34,6 +36,7 @@ export const createPersonalDashboardService = (
         }),
         new PrivateProjectChecker(stores, config),
         new AccountStore(db, config.getLogger),
+        new AccessStore(db, config.eventBus, config.getLogger),
     );
 };
 
@@ -50,5 +53,6 @@ export const createFakePersonalDashboardService = (config: IUnleashConfig) => {
         }),
         new FakePrivateProjectChecker(),
         new FakeAccountStore(),
+        new FakeAccessStore(),
     );
 };

src/lib/features/personal-dashboard/personal-dashboard-controller.ts

@@ -106,6 +106,7 @@ export default class PersonalDashboardController extends Controller {
 
         const projectDetails =
             await this.personalDashboardService.getPersonalProjectDetails(
+                user.id,
                 req.params.projectId,
             );
 
@@ -115,8 +116,6 @@ export default class PersonalDashboardController extends Controller {
             personalDashboardProjectDetailsSchema.$id,
             {
                 ...projectDetails,
-                owners: [{ ownerType: 'user', name: 'placeholder' }],
-                roles: [{ name: 'placeholder', id: 0, type: 'project' }],
             },
         );
     }

src/lib/features/personal-dashboard/personal-dashboard-service.ts

@@ -10,6 +10,7 @@ import type {
 import type { IProjectReadModel } from '../project/project-read-model-type';
 import type { IPrivateProjectChecker } from '../private-project/privateProjectCheckerType';
 import type {
+    IAccessStore,
     IAccountStore,
     IEventStore,
     IOnboardingReadModel,
@@ -36,6 +37,8 @@ export class PersonalDashboardService {
 
     private onboardingReadModel: IOnboardingReadModel;
 
+    private accessStore: IAccessStore;
+
     constructor(
         personalDashboardReadModel: IPersonalDashboardReadModel,
         projectOwnersReadModel: IProjectOwnersReadModel,
@@ -45,6 +48,7 @@ export class PersonalDashboardService {
         featureEventFormatter: FeatureEventFormatter,
         privateProjectChecker: IPrivateProjectChecker,
         accountStore: IAccountStore,
+        accessStore: IAccessStore,
     ) {
         this.personalDashboardReadModel = personalDashboardReadModel;
         this.projectOwnersReadModel = projectOwnersReadModel;
@@ -54,6 +58,7 @@ export class PersonalDashboardService {
         this.featureEventFormatter = featureEventFormatter;
         this.privateProjectChecker = privateProjectChecker;
         this.accountStore = accountStore;
+        this.accessStore = accessStore;
     }
 
     getPersonalFeatures(userId: number): Promise<PersonalFeature[]> {
@@ -95,6 +100,7 @@ export class PersonalDashboardService {
     }
 
     async getPersonalProjectDetails(
+        userId: number,
         projectId: string,
     ): Promise<PersonalDashboardProjectDetailsSchema> {
         const recentEvents = await this.eventStore.searchEvents(
@@ -117,11 +123,26 @@ export class PersonalDashboardService {
         const owners =
             await this.projectOwnersReadModel.getProjectOwners(projectId);
 
+        const allRoles = await this.accessStore.getAllProjectRolesForUser(
+            userId,
+            projectId,
+        );
+
+        const projectRoles = allRoles
+            .filter((role) => ['project', 'custom'].includes(role.type))
+            .map((role) => ({
+                id: role.id,
+                name: role.name,
+                type: role.type as PersonalDashboardProjectDetailsSchema['roles'][number]['type'],
+            }));
+
+        console.log('project roles', projectRoles);
+
         return {
             latestEvents: formattedEvents,
             onboardingStatus,
             owners,
-            roles: [],
+            roles: projectRoles,
         };
     }
 

src/lib/services/access-service.test.ts

@@ -12,7 +12,7 @@ import FakeGroupStore from '../../test/fixtures/fake-group-store';
 import { FakeAccountStore } from '../../test/fixtures/fake-account-store';
 import FakeRoleStore from '../../test/fixtures/fake-role-store';
 import FakeEnvironmentStore from '../features/project-environments/fake-environment-store';
-import AccessStoreMock from '../../test/fixtures/fake-access-store';
+import FakeAccessStore from '../../test/fixtures/fake-access-store';
 import { GroupService } from '../services/group-service';
 import type { IRole } from '../../lib/types/stores/access-store';
 import {
@@ -271,7 +271,7 @@ test('throws error when trying to delete a project role in use by group', async
     const accountStore = new FakeAccountStore();
     const roleStore = new FakeRoleStore();
     const environmentStore = new FakeEnvironmentStore();
-    const accessStore = new AccessStoreMock();
+    const accessStore = new FakeAccessStore();
     accessStore.getGroupIdsForRole = groupIdResultOverride;
     accessStore.getUserIdsForRole = async (): Promise<number[]> => {
         return [];

src/test/fixtures/fake-access-store.ts

@@ -19,7 +19,7 @@ import {
 import FakeRoleStore from './fake-role-store';
 import type { PermissionRef } from '../../lib/services/access-service';
 
-class AccessStoreMock implements IAccessStore {
+export class FakeAccessStore implements IAccessStore {
     fakeRolesStore: IRoleStore;
 
     userToRoleMap: Map<number, number> = new Map();
@@ -327,6 +327,6 @@ class AccessStoreMock implements IAccessStore {
     }
 }
 
-module.exports = AccessStoreMock;
+module.exports = FakeAccessStore;
 
-export default AccessStoreMock;
+export default FakeAccessStore;