Implemented fix for VIH-11057 - external ref id not being mapped corr…

…ectly for endpoints models, on preexisting endpoints.
hmcts · Oct 21, 2024 · 259892a · 259892a
1 parent 21cc208
commit 259892a
Show file tree

Hide file tree

Showing 17 changed files with 141 additions and 88 deletions.
diff --git a/AdminWebsite/AdminWebsite/ClientApp/src/app/booking/endpoints/endpoints.component.spec.ts b/AdminWebsite/AdminWebsite/ClientApp/src/app/booking/endpoints/endpoints.component.spec.ts
@@ -57,7 +57,8 @@ function initHearingRequest(): HearingModel {
             pin: 'pin',
             username: '[email protected]',
             contactEmail: '[email protected]',
-            interpretationLanguage: undefined
+            interpretationLanguage: undefined,
+            externalReferenceId: 'ex1'
         }
     ];
     return newHearing;
@@ -212,7 +213,14 @@ describe('EndpointsComponent', () => {
             featureServiceSpy.getFlag.withArgs(FeatureFlags.multiDayBookingEnhancements).and.returnValue(of(true));
             component.ngOnInit();
             component.videoEndpoints = [
-                { id: '1', displayName: 'Test', defenceAdvocate: null, interpretationLanguage: undefined, screening: undefined }
+                {
+                    id: '1',
+                    displayName: 'Test',
+                    defenceAdvocate: null,
+                    interpretationLanguage: undefined,
+                    screening: undefined,
+                    externalReferenceId: '1'
+                }
             ];
             component.saveEndpoints();
             expect(routerSpy.navigate).toHaveBeenCalledWith(['/other-information']);
@@ -232,7 +240,8 @@ describe('EndpointsComponent', () => {
                 displayName: 'Test',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             component.onEndpointAdded(endpoint);
             expect(component.videoEndpoints).toContain(endpoint);
@@ -244,7 +253,8 @@ describe('EndpointsComponent', () => {
                 displayName: 'Test',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             component.videoEndpoints = [endpoint];
             component.onEndpointAdded(endpoint);
@@ -255,8 +265,22 @@ describe('EndpointsComponent', () => {
     describe('onEndpointUpdated', () => {
         beforeEach(() => {
             component.videoEndpoints = [
-                { id: '1', displayName: 'Test', defenceAdvocate: null, interpretationLanguage: undefined, screening: undefined },
-                { id: '2', displayName: 'Test2', defenceAdvocate: null, interpretationLanguage: undefined, screening: undefined }
+                {
+                    id: '1',
+                    displayName: 'Test',
+                    defenceAdvocate: null,
+                    interpretationLanguage: undefined,
+                    screening: undefined,
+                    externalReferenceId: '1'
+                },
+                {
+                    id: '2',
+                    displayName: 'Test2',
+                    defenceAdvocate: null,
+                    interpretationLanguage: undefined,
+                    screening: undefined,
+                    externalReferenceId: '1'
+                }
             ];
         });
 
@@ -266,15 +290,17 @@ describe('EndpointsComponent', () => {
                 displayName: 'Test',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             component.videoEndpoints = [endpoint];
             const updatedEndpoint = {
                 id: '1',
                 displayName: 'Updated',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             component.onEndpointUpdated({ original: endpoint, updated: updatedEndpoint });
             expect(component.videoEndpoints).toContain(updatedEndpoint);
@@ -286,14 +312,16 @@ describe('EndpointsComponent', () => {
                 displayName: 'DoesNotExist',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             const updatedEndpoint = {
                 id: '1',
                 displayName: 'Updated',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             component.onEndpointUpdated({ original: endpoint, updated: updatedEndpoint });
             expect(component.videoEndpoints).not.toContain(endpoint);
@@ -303,8 +331,22 @@ describe('EndpointsComponent', () => {
     describe('onEndpointSelectedForDeletion', () => {
         beforeEach(() => {
             component.videoEndpoints = [
-                { id: '1', displayName: 'Test', defenceAdvocate: null, interpretationLanguage: undefined, screening: undefined },
-                { id: '2', displayName: 'Test2', defenceAdvocate: null, interpretationLanguage: undefined, screening: undefined }
+                {
+                    id: '1',
+                    displayName: 'Test',
+                    defenceAdvocate: null,
+                    interpretationLanguage: undefined,
+                    screening: undefined,
+                    externalReferenceId: '1'
+                },
+                {
+                    id: '2',
+                    displayName: 'Test2',
+                    defenceAdvocate: null,
+                    interpretationLanguage: undefined,
+                    screening: undefined,
+                    externalReferenceId: '1'
+                }
             ];
         });
 
@@ -314,7 +356,8 @@ describe('EndpointsComponent', () => {
                 displayName: 'Test',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             component.onEndpointSelectedForDeletion(endpoint);
             expect(component.videoEndpoints).not.toContain(endpoint);
@@ -326,7 +369,8 @@ describe('EndpointsComponent', () => {
                 displayName: 'Test3',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             component.onEndpointSelectedForDeletion(endpoint);
             expect(component.videoEndpoints.length).toBe(2);
@@ -340,7 +384,8 @@ describe('EndpointsComponent', () => {
                 displayName: 'Test',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: '1'
             };
             component.onEndpointSelectedForEdit(endpoint);
             expect(component.videoEndpointToEdit).toBe(endpoint);

diff --git a/AdminWebsite/AdminWebsite/ClientApp/src/app/booking/endpoints/endpoints.component.ts b/AdminWebsite/AdminWebsite/ClientApp/src/app/booking/endpoints/endpoints.component.ts
@@ -95,7 +95,7 @@ export class EndpointsComponent extends BookingBaseComponent implements OnInit,
 
         const newEndpointsArray: EndpointModel[] = [];
         for (const vapDto of this.videoEndpoints) {
-            const endpointModel = new EndpointModel();
+            const endpointModel = new EndpointModel(vapDto.externalReferenceId);
             endpointModel.id = vapDto.id;
             endpointModel.displayName = vapDto.displayName;
             endpointModel.defenceAdvocate = vapDto.defenceAdvocate?.email;
@@ -197,7 +197,8 @@ export class EndpointsComponent extends BookingBaseComponent implements OnInit,
                       }
                     : null,
                 interpretationLanguage: e.interpretationLanguage,
-                screening: e.screening
+                screening: e.screening,
+                externalReferenceId: e.externalReferenceId
             };
         });
     }

diff --git a/...bsite/AdminWebsite/ClientApp/src/app/booking/endpoints/models/video-access-point.model.ts b/...bsite/AdminWebsite/ClientApp/src/app/booking/endpoints/models/video-access-point.model.ts
@@ -7,6 +7,7 @@ export interface VideoAccessPointDto {
     defenceAdvocate?: EndpointLink;
     interpretationLanguage: InterpreterSelectedDto; // This should not be optional once the backend is implemented
     screening: ScreeningDto;
+    externalReferenceId: string;
 }
 
 export interface EndpointLink {

diff --git a/...ntApp/src/app/booking/endpoints/video-endpoint-form/video-endpoint-form.component.spec.ts b/...ntApp/src/app/booking/endpoints/video-endpoint-form/video-endpoint-form.component.spec.ts
@@ -1,4 +1,4 @@
-import { ComponentFixture, TestBed, fakeAsync, flush } from '@angular/core/testing';
+import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { FormBuilder, ReactiveFormsModule } from '@angular/forms';
 import { VideoEndpointFormComponent } from './video-endpoint-form.component';
 import { VideoAccessPointDto } from '../models/video-access-point.model';
@@ -83,7 +83,8 @@ describe('VideoEndpointFormComponent', () => {
                 displayName: 'Test',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: undefined
             };
             component.form.setValue({
                 displayName: dto.displayName,
@@ -110,14 +111,16 @@ describe('VideoEndpointFormComponent', () => {
                 displayName: 'Original',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: undefined
             };
             const updatedDto: VideoAccessPointDto = {
                 id: '1',
                 displayName: 'Updated',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: undefined
             };
             component.existingVideoEndpoint = originalDto;
             component.form.setValue({
@@ -134,11 +137,11 @@ describe('VideoEndpointFormComponent', () => {
 
             const rep = component.availableRepresentatives[0];
             // update the input field with the email of the first participant via the debug fixture element
-            const displayNameInput = fixture.nativeElement.querySelector('[id="displayName"') as HTMLInputElement;
+            const displayNameInput = fixture.nativeElement.querySelector('[id="displayName"]') as HTMLInputElement;
             displayNameInput.value = 'Test Endpoint';
             displayNameInput.dispatchEvent(new Event('input'));
 
-            const linkedRepresentativeInput = fixture.nativeElement.querySelector('[id="representative"') as HTMLSelectElement;
+            const linkedRepresentativeInput = fixture.nativeElement.querySelector('[id="representative"]') as HTMLSelectElement;
             linkedRepresentativeInput.value = rep.email;
             linkedRepresentativeInput.dispatchEvent(new Event('change'));
 
@@ -150,7 +153,8 @@ describe('VideoEndpointFormComponent', () => {
                     displayName: rep.display_name
                 },
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: undefined
             });
         });
 
@@ -171,15 +175,17 @@ describe('VideoEndpointFormComponent', () => {
                     displayName: 'Test',
                     defenceAdvocate: null,
                     interpretationLanguage: undefined,
-                    screening: undefined
+                    screening: undefined,
+                    externalReferenceId: undefined
                 }
             ];
             component.videoEndpoint = {
                 id: '2',
                 displayName: 'Test',
                 defenceAdvocate: null,
                 interpretationLanguage: undefined,
-                screening: undefined
+                screening: undefined,
+                externalReferenceId: undefined
             };
             component.form.setValue({
                 displayName: 'Test',

diff --git a/.../ClientApp/src/app/booking/endpoints/video-endpoint-form/video-endpoint-form.component.ts b/.../ClientApp/src/app/booking/endpoints/video-endpoint-form/video-endpoint-form.component.ts
@@ -94,7 +94,8 @@ export class VideoEndpointFormComponent {
             displayName: this.form.value.displayName,
             defenceAdvocate,
             interpretationLanguage: this.interpreterSelection,
-            screening: this.videoEndpoint?.screening
+            screening: this.videoEndpoint?.screening,
+            externalReferenceId: this.videoEndpoint?.externalReferenceId
         };
         if (this.editMode) {
             this.endpointUpdated.emit({ original: this.videoEndpoint, updated: dto });

diff --git a/...ntApp/src/app/booking/endpoints/video-endpoint-item/video-endpoint-item.component.spec.ts b/...ntApp/src/app/booking/endpoints/video-endpoint-item/video-endpoint-item.component.spec.ts
@@ -26,7 +26,8 @@ describe('VideoEndpointItemComponent', () => {
             displayName: 'Test',
             defenceAdvocate: null,
             interpretationLanguage: undefined,
-            screening: undefined
+            screening: undefined,
+            externalReferenceId: undefined
         };
         component.videoEndpoint = dto;
         component.edit();
@@ -39,7 +40,8 @@ describe('VideoEndpointItemComponent', () => {
             displayName: 'Test',
             defenceAdvocate: null,
             interpretationLanguage: undefined,
-            screening: undefined
+            screening: undefined,
+            externalReferenceId: undefined
         };
         component.videoEndpoint = dto;
         component.delete();

diff --git a/...ntApp/src/app/booking/endpoints/video-endpoint-list/video-endpoint-list.component.spec.ts b/...ntApp/src/app/booking/endpoints/video-endpoint-list/video-endpoint-list.component.spec.ts
@@ -26,7 +26,8 @@ describe('VideoEndpointListComponent', () => {
             displayName: 'Test',
             defenceAdvocate: null,
             interpretationLanguage: undefined,
-            screening: undefined
+            screening: undefined,
+            externalReferenceId: undefined
         };
         component.editEndpoint.emit(dto);
         expect(component.editEndpoint.emit).toHaveBeenCalledWith(dto);
@@ -38,7 +39,8 @@ describe('VideoEndpointListComponent', () => {
             displayName: 'Test',
             defenceAdvocate: null,
             interpretationLanguage: undefined,
-            screening: undefined
+            screening: undefined,
+            externalReferenceId: undefined
         };
         component.deleteEndpoint.emit(dto);
         expect(component.deleteEndpoint.emit).toHaveBeenCalledWith(dto);

diff --git a/...Website/AdminWebsite/ClientApp/src/app/booking/screening/screening-form.component.spec.ts b/...Website/AdminWebsite/ClientApp/src/app/booking/screening/screening-form.component.spec.ts
@@ -25,12 +25,12 @@ describe('ScreeningFormComponent', () => {
         participant2.email = 'email2';
         participant2.display_name = 'Participant2';
 
-        const endpoint1 = new EndpointModel();
+        const endpoint1 = new EndpointModel(null);
         endpoint1.id = '3';
         endpoint1.displayName = 'Endpoint 1';
         endpoint1.sip = 'sip1';
 
-        const endpoint2 = new EndpointModel();
+        const endpoint2 = new EndpointModel(null);
         endpoint2.id = '4';
         endpoint2.displayName = 'Endpoint 2';
         endpoint2.sip = 'sip2';

diff --git a/...te/AdminWebsite/ClientApp/src/app/booking/screening/screening-list-item.component.spec.ts b/...te/AdminWebsite/ClientApp/src/app/booking/screening/screening-list-item.component.spec.ts
@@ -4,7 +4,7 @@ import { ScreeningListItemComponent } from './screening-list-item.component';
 import { EndpointModel } from 'src/app/common/model/endpoint.model';
 import { HearingModel } from 'src/app/common/model/hearing.model';
 import { ParticipantModel } from 'src/app/common/model/participant.model';
-import { SimpleChanges, SimpleChange } from '@angular/core';
+import { SimpleChange, SimpleChanges } from '@angular/core';
 
 describe('ScreeningListItemComponent', () => {
     let component: ScreeningListItemComponent;
@@ -28,11 +28,11 @@ describe('ScreeningListItemComponent', () => {
         };
         hearing.participants = [participantWithoutScreening, participantWithScreening];
 
-        const endpointWithoutScreening = new EndpointModel();
+        const endpointWithoutScreening = new EndpointModel(null);
         endpointWithoutScreening.id = '3';
         endpointWithoutScreening.displayName = 'Endpoint No Screening';
 
-        const endpointWithScreening = new EndpointModel();
+        const endpointWithScreening = new EndpointModel(null);
         endpointWithScreening.id = '4';
         endpointWithScreening.displayName = 'Endpoint With Screening';
         endpointWithScreening.screening = {

diff --git a/...Website/AdminWebsite/ClientApp/src/app/booking/screening/screening-list.component.spec.ts b/...Website/AdminWebsite/ClientApp/src/app/booking/screening/screening-list.component.spec.ts
@@ -1,4 +1,4 @@
-import { ComponentFixture, TestBed, tick } from '@angular/core/testing';
+import { ComponentFixture, TestBed } from '@angular/core/testing';
 
 import { ScreeningListComponent } from './screening-list.component';
 import { HearingModel } from 'src/app/common/model/hearing.model';
@@ -26,11 +26,11 @@ describe('ScreeningListComponent', () => {
         };
         hearing.participants = [participantWithoutScreening, participantWithScreening];
 
-        const endpointWithoutScreening = new EndpointModel();
+        const endpointWithoutScreening = new EndpointModel(null);
         endpointWithoutScreening.id = '3';
         endpointWithoutScreening.displayName = 'Endpoint No Screening';
 
-        const endpointWithScreening = new EndpointModel();
+        const endpointWithScreening = new EndpointModel(null);
         endpointWithScreening.id = '4';
         endpointWithScreening.displayName = 'Endpoint With Screening';
         endpointWithScreening.screening = {
@@ -67,7 +67,7 @@ describe('ScreeningListComponent', () => {
     describe('onEndpointScreeningDeleted', () => {
         it('should emit deleteEndpointScreening event', () => {
             // Arrange
-            const endpoint = new EndpointModel();
+            const endpoint = new EndpointModel(null);
             spyOn(component.deleteEndpointScreening, 'emit');
 
             // Act