fix(validateProposalMessage): ensure submitted status is a boolean

src/queue/consumers/utils/validateProposalMessage.spec.ts

@@ -29,4 +29,49 @@ describe('Validate messages', () => {
       })
     ).not.toThrow();
   });
+
+  it('should accept a false submitted status', () => {
+    expect(() =>
+      validateProposalMessage({
+        title: 'Test proposal',
+        abstract: 'Test abstract',
+        members: [],
+        proposalPk: 1,
+        shortCode: '123123',
+        instruments: [{ id: 1, shortCode: 'TEST', allocatedTime: 1 }],
+        newStatus: 'REVIEW',
+        callId: 123,
+        submitted: false,
+        proposer: {
+          id: 1,
+          firstName: 'Test',
+          lastName: 'User',
+          email: 'test.user@email.com',
+          oidcSub: 'testOidcSub',
+        },
+      })
+    ).not.toThrow();
+  });
+
+  it('should throw when submitted status is missing', () => {
+    expect(() =>
+      validateProposalMessage({
+        title: 'Test proposal',
+        abstract: 'Test abstract',
+        members: [],
+        proposalPk: 1,
+        shortCode: '123123',
+        instruments: [{ id: 1, shortCode: 'TEST', allocatedTime: 1 }],
+        newStatus: 'REVIEW',
+        callId: 123,
+        proposer: {
+          id: 1,
+          firstName: 'Test',
+          lastName: 'User',
+          email: 'test.user@email.com',
+          oidcSub: 'testOidcSub',
+        },
+      })
+    ).toThrow('Proposal Submitted status is missing');
+  });
 });

src/queue/consumers/utils/validateProposalMessage.ts

@@ -33,7 +33,7 @@ export function validateProposalMessage(
     throw new Error('Proposal CallId is missing');
   }
 
-  if (!proposalMessage.submitted) {
+  if (typeof proposalMessage.submitted !== 'boolean') {
     throw new Error('Proposal Submitted status is missing');
   }