psp-7082 ensure that primary contact logic for org teams works when t…

source/frontend/src/features/mapSideBar/acquisition/common/models.test.ts

@@ -0,0 +1,21 @@
+import { AcquisitionTeamFormModel } from './models';
+
+describe('acquisition model tests', () => {
+  it('omits the primary contact if unset', () => {
+    let model = new AcquisitionTeamFormModel('testType');
+    model.primaryContactId = '';
+    expect(model.toApi(1)?.primaryContactId).toBeUndefined();
+  });
+
+  it('omits the primary contact if null', () => {
+    let model = new AcquisitionTeamFormModel('testType');
+    model.primaryContactId = null as any;
+    expect(model.toApi(1)?.primaryContactId).toBeUndefined();
+  });
+
+  it('omits the primary contact if undefined', () => {
+    let model = new AcquisitionTeamFormModel('testType');
+    model.primaryContactId = undefined as any;
+    expect(model.toApi(1)?.primaryContactId).toBeUndefined();
+  });
+});

source/frontend/src/features/mapSideBar/acquisition/common/models.ts

@@ -42,7 +42,7 @@ export class AcquisitionTeamFormModel {
       person: undefined,
       organizationId: organizationId ?? undefined,
       organization: undefined,
-      primaryContactId: this.primaryContactId !== '' ? Number(this.primaryContactId) : undefined,
+      primaryContactId: !isNaN(+this.primaryContactId) ? Number(this.primaryContactId) : undefined,
       teamProfileType: toTypeCode(this.contactTypeCode),
       teamProfileTypeCode: this.contactTypeCode,
     };