fix: update roadmap controller to use searchObject endpoint to retrieve file metadata from COMS

Signed-off-by: Sanjay Babu <sanjaytkbabu@gmail.com>

Author: sanjaytkbabu

app/src/controllers/roadmap.ts

@@ -3,7 +3,7 @@ import { v4 as uuidv4 } from 'uuid';
 import { PermitStage } from '../db/codes/enums.ts';
 import { transactionWrapper } from '../db/utils/transactionWrapper';
 import { generateCreateStamps, generateNullDeleteStamps, generateNullUpdateStamps } from '../db/utils/utils';
-import { getObject } from '../services/coms';
+import { getObject, searchObject } from '../services/coms';
 import { email } from '../services/email';
 import { createNote } from '../services/note';
 import { createNoteHistory } from '../services/noteHistory';
@@ -44,9 +44,10 @@ export const sendRoadmapController = async (
       // If succesful it is converted to base64 encoding and added to the attachment list
       const objectPromises = req.body.selectedFileIds.map(async (id) => {
         const { status, headers, data } = await getObject(req.currentContext.bearerToken!, id);
+        const searchResponse = await searchObject(req.currentContext.bearerToken!, id);
 
-        if (status === 200) {
-          const filename = headers['x-amz-meta-name'] as string;
+        if (status === 200 && searchResponse.data[0]) {
+          const filename = searchResponse.data[0].name;
           if (filename) {
             attachments.push({
               content: Buffer.from(data).toString('base64'),
@@ -55,7 +56,7 @@ export const sendRoadmapController = async (
               filename: filename
             });
           } else {
-            throw new Problem(status, { detail: `Unable to obtain filename for file ${id}` });
+            throw new Problem(500, { detail: `Unable to obtain filename for file ${id}` });
           }
         }
       });

app/src/services/coms.ts

@@ -84,6 +84,22 @@ export const getObject = async (bearerToken: string, objectId: string) => {
   return { status, headers, data };
 };
 
+/**
+ * Search for an object and obtain information about it
+ * @param bearerToken The bearer token of the authorized user
+ * @param objectId The id for the object to get metadata for
+ * @returns The obtained information about the object
+ */
+export const searchObject = async (bearerToken: string, objectId: string) => {
+  if (!uuidValidateV4(objectId)) {
+    throw new Problem(422, { detail: 'Invalid objectId parameter' });
+  }
+  const { status, headers, data } = await comsAxios({
+    headers: { Authorization: `Bearer ${bearerToken}` }
+  }).get('/object', { params: { objectId } });
+  return { status, headers, data };
+};
+
 /**
  * Obtain the current user information in COMS
  * @param currentContext The current context of the Express request

app/tests/unit/controllers/roadmap.spec.ts

@@ -8,6 +8,7 @@ import * as noteService from '../../../src/services/note.ts';
 import * as noteHistoryService from '../../../src/services/noteHistory.ts';
 import { uuidv4Pattern } from '../../../src/utils/regexp.ts';
 
+import type { InternalAxiosRequestConfig } from 'axios';
 import type { Request, Response } from 'express';
 import type { Mock } from 'vitest';
 import type { Email, Note, NoteHistory } from '../../../src/types';
@@ -34,6 +35,7 @@ afterEach(() => {
 describe('send', () => {
   const emailSpy = vi.spyOn(emailService, 'email');
   const getObjectSpy = vi.spyOn(comsService, 'getObject');
+  const searchObjectSpy = vi.spyOn(comsService, 'searchObject');
   const createNoteSpy = vi.spyOn(noteService, 'createNote');
   const createHistorySpy = vi.spyOn(noteHistoryService, 'createNoteHistory');
 
@@ -142,7 +144,7 @@ describe('send', () => {
     const req = {
       body: {
         activityId: 'ACTI1234',
-        selectedFileIds: ['123', '456'],
+        selectedFileIds: ['550e8400-e29b-41d4-a716-446655440001', '550e8400-e29b-41d4-a716-446655440002'],
         emailData: {
           body: 'Some message text',
           bodyType: 'text',
@@ -178,6 +180,14 @@ describe('send', () => {
       status: 200
     };
 
+    const searchObjectResponse = {
+      data: [{ name: 'foo' }],
+      status: 200,
+      statusText: 'OK',
+      headers: {},
+      config: {} as InternalAxiosRequestConfig
+    };
+
     const createdHistory: NoteHistory = {
       ...TEST_NOTE_HISTORY_1,
       type: 'Roadmap',
@@ -192,6 +202,7 @@ describe('send', () => {
     createHistorySpy.mockResolvedValue(createdHistory);
     createNoteSpy.mockResolvedValue(createdNote);
     getObjectSpy.mockResolvedValue(getObjectResponse);
+    searchObjectSpy.mockResolvedValue(searchObjectResponse);
     emailSpy.mockResolvedValue(TEST_EMAIL_RESPONSE);
 
     await sendRoadmapController(
@@ -212,7 +223,7 @@ describe('send', () => {
     const req = {
       body: {
         activityId: 'ACTI1234',
-        selectedFileIds: ['123', '456'],
+        selectedFileIds: ['550e8400-e29b-41d4-a716-446655440001', '550e8400-e29b-41d4-a716-446655440002'],
         emailData: {
           body: 'Some message text',
           bodyType: 'text',
@@ -270,7 +281,7 @@ describe('send', () => {
     const req = {
       body: {
         activityId: 'ACTI1234',
-        selectedFileIds: ['123', '456'],
+        selectedFileIds: ['550e8400-e29b-41d4-a716-446655440001', '550e8400-e29b-41d4-a716-446655440002'],
         emailData: {
           body: 'Some message text',
           bodyType: 'text',
@@ -315,8 +326,24 @@ describe('send', () => {
       status: 200
     };
 
-    const note1Name = `${getObjectResponse1.headers['x-amz-meta-name']}`;
-    const note2Name = `${getObjectResponse2.headers['x-amz-meta-name']}`;
+    const searchObjectResponse1 = {
+      data: [{ name: 'foo' }],
+      status: 200,
+      statusText: 'OK',
+      headers: {},
+      config: {} as InternalAxiosRequestConfig
+    };
+
+    const searchObjectResponse2 = {
+      data: [{ name: 'bar' }],
+      status: 200,
+      statusText: 'OK',
+      headers: {},
+      config: {} as InternalAxiosRequestConfig
+    };
+
+    const note1Name = `${searchObjectResponse1.data[0].name}`;
+    const note2Name = `${searchObjectResponse2.data[0].name}`;
 
     const createdHistory: NoteHistory = {
       ...TEST_NOTE_HISTORY_1,
@@ -334,6 +361,8 @@ describe('send', () => {
     emailSpy.mockResolvedValue(TEST_EMAIL_RESPONSE);
     getObjectSpy.mockResolvedValueOnce(getObjectResponse1);
     getObjectSpy.mockResolvedValueOnce(getObjectResponse2);
+    searchObjectSpy.mockResolvedValueOnce(searchObjectResponse1);
+    searchObjectSpy.mockResolvedValueOnce(searchObjectResponse2);
 
     await sendRoadmapController(
       req as unknown as Request<never, never, { activityId: string; selectedFileIds: string[]; emailData: Email }>,

frontend/tests/unit/components/projectCommon/ProjectRoadmapTab.spec.ts

@@ -0,0 +1,65 @@
+import ConfirmationService from 'primevue/confirmationservice';
+import ToastService from 'primevue/toastservice';
+import { createTestingPinia } from '@pinia/testing';
+import { flushPromises, shallowMount } from '@vue/test-utils';
+
+import ProjectRoadmapTab from '@/components/projectCommon/ProjectRoadmapTab.vue';
+import i18n from '@/i18n';
+import { roadmapService, userService } from '@/services';
+
+vi.mock('@/services', () => ({
+  roadmapService: { getRoadmapNote: vi.fn(), sendRoadmap: vi.fn() },
+  userService: { listUsers: vi.fn() }
+}));
+
+describe('ProjectRoadmapTab.vue', () => {
+  it('initializes BCC with multiple emails from config and assignee', async () => {
+    vi.mocked(roadmapService.getRoadmapNote).mockResolvedValue('Roadmap note');
+    vi.mocked(userService.listUsers).mockResolvedValue([
+      {
+        userId: 'user1',
+        email: 'navigator@bcgov.bc.ca',
+        firstName: 'Nav',
+        lastName: 'User',
+        fullName: 'Nav User',
+        sub: 'idir-123',
+        idp: 'idir',
+        active: true,
+        groups: [],
+        elevatedRights: false,
+        bceidBusinessName: ''
+      }
+    ]);
+
+    const wrapper = shallowMount(ProjectRoadmapTab, {
+      global: {
+        plugins: [
+          i18n,
+          ConfirmationService,
+          ToastService,
+          createTestingPinia({
+            initialState: {
+              app: { initiative: 'Housing' },
+              config: { config: { ches: { roadmap: { bcc: 'config1@bcgov.bc.ca; config2@bcgov.bc.ca' } } } },
+              project: {
+                project: { activityId: 'activity1', assignedUserId: 'user1' },
+                activityContacts: [{ contactId: 'c1', activityId: 'activity1', contact: { email: 'user@test.com' } }],
+                roadmapNote: 'Roadmap note'
+              }
+            }
+          })
+        ]
+      }
+    });
+
+    await flushPromises();
+
+    // Verify BCC field is initialized with config emails + assignee email
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    expect((wrapper.vm as any).initialFormValues.bcc).toEqual([
+      'config1@bcgov.bc.ca',
+      'config2@bcgov.bc.ca',
+      'navigator@bcgov.bc.ca'
+    ]);
+  });
+});