Map 625 uof new weapons observed question

assets/js/add-another-detail.js

@@ -0,0 +1,2 @@
+new AddAnother($('.add-another-body-worn-camera'), '.remove-button-container')
+new AddAnother($('.add-another-weapons-observed'), '.remove-button-container')

assets/js/add-another-evidence.js

@@ -1,2 +1 @@
 new AddAnother($('.add-another-evidence'), '.remove-button-container')
-new AddAnother($('.add-another-input'), '.remove-button-container')

integration-tests/integration/createIncident/form-submit.cy.js

@@ -84,6 +84,14 @@ context('Submit the incident report', () => {
     useOfForceDetailsPage.bodyWornCameraNumber(1).type('789')
     useOfForceDetailsPage.addAnotherBodyWornCamera()
     useOfForceDetailsPage.bodyWornCameraNumber(2).type('456')
+
+    useOfForceDetailsPage.weaponsObserved().check('YES')
+    useOfForceDetailsPage.weaponTypes(0).type('gun')
+    useOfForceDetailsPage.addAnotherWeapon()
+    useOfForceDetailsPage.weaponTypes(1).type('knife')
+    useOfForceDetailsPage.addAnotherWeapon()
+    useOfForceDetailsPage.weaponTypes(2).type('fork')
+
     const relocationAndInjuriesPage = useOfForceDetailsPage.save()
     relocationAndInjuriesPage.fillForm()
     const evidencePage = relocationAndInjuriesPage.save()

integration-tests/integration/reportPages/enter-use-of-force-details.cy.js

@@ -1,4 +1,3 @@
-const { use } = require('passport')
 const { offender } = require('../../mockApis/data')
 
 const ReportUseOfForcePage = require('../../pages/createReport/reportUseOfForcePage')
@@ -31,6 +30,7 @@ context('Enter use of force details page', () => {
     useOfForceDetailsPage.batonDrawn().check('true')
     useOfForceDetailsPage.batonUsed().check('true')
     useOfForceDetailsPage.pavaDrawn().check('true')
+    useOfForceDetailsPage.weaponsObserved().check('NO')
     useOfForceDetailsPage.pavaUsed().check('true')
     useOfForceDetailsPage.guidingHold().check('true')
     useOfForceDetailsPage.guidingHoldOfficersInvolved.check('2')
@@ -59,6 +59,7 @@ context('Enter use of force details page', () => {
         handcuffsApplied: true,
         pavaDrawn: true,
         pavaUsed: true,
+        weaponsObserved: 'NO',
         personalProtectionTechniques: true,
         positiveCommunication: true,
         restraintPositions: ['STANDING', 'ON_BACK', 'FACE_DOWN', 'KNEELING'],
@@ -83,6 +84,7 @@ context('Enter use of force details page', () => {
         escortingHold: true,
         handcuffsApplied: true,
         pavaDrawn: true,
+        weaponsObserved: 'NO',
         pavaUsed: true,
         personalProtectionTechniques: true,
         positiveCommunication: true,
@@ -127,12 +129,39 @@ context('Enter use of force details page', () => {
     selectUofReasonsPage.checkReason('FIGHT_BETWEEN_PRISONERS')
     selectUofReasonsPage.clickSaveAndContinue()
 
+    const useOfForceDetailsPage = UseOfForceDetailsPage.verifyOnPage()
+    useOfForceDetailsPage.positiveCommunication().check('true')
+    useOfForceDetailsPage.personalProtectionTechniques().check('true')
+    useOfForceDetailsPage.pavaDrawn().check('true')
+    useOfForceDetailsPage.pavaUsed().check('true')
+    useOfForceDetailsPage.guidingHold().check('true')
+    useOfForceDetailsPage.guidingHoldOfficersInvolved.check('2')
+    useOfForceDetailsPage.escortingHold().check('true')
+    useOfForceDetailsPage.handcuffsApplied().check('true')
+    useOfForceDetailsPage.clickSaveAndContinue()
+    useOfForceDetailsPage.errorSummary().contains('Select yes if a baton was drawn')
+    useOfForceDetailsPage
+      .errorSummary()
+      .contains('Select yes if any part of the incident was captured on a body-worn camera')
+    useOfForceDetailsPage.errorSummary().contains('Select if any pain inducing techniques were used')
+    useOfForceDetailsPage.errorSummary().contains('Select yes if any weapons were observed')
+  })
+
+  it('Displays secondary validation messages', () => {
+    cy.login()
+
+    const reportUseOfForcePage = ReportUseOfForcePage.visit(offender.bookingId)
+    const selectUofReasonsPage = reportUseOfForcePage.goToSelectUofReasonsPage()
+    selectUofReasonsPage.checkReason('FIGHT_BETWEEN_PRISONERS')
+    selectUofReasonsPage.clickSaveAndContinue()
+
     const useOfForceDetailsPage = UseOfForceDetailsPage.verifyOnPage()
     useOfForceDetailsPage.positiveCommunication().check('true')
     useOfForceDetailsPage.bodyWornCamera().check('YES')
     useOfForceDetailsPage.personalProtectionTechniques().check('true')
     useOfForceDetailsPage.pavaDrawn().check('true')
     useOfForceDetailsPage.pavaUsed().check('true')
+    useOfForceDetailsPage.weaponsObserved().check('YES')
     useOfForceDetailsPage.guidingHold().check('true')
     useOfForceDetailsPage.guidingHoldOfficersInvolved.check('2')
     useOfForceDetailsPage.escortingHold().check('true')
@@ -141,5 +170,31 @@ context('Enter use of force details page', () => {
     useOfForceDetailsPage.errorSummary().contains('Select yes if a baton was drawn')
     useOfForceDetailsPage.errorSummary().contains('Enter the body-worn camera number')
     useOfForceDetailsPage.errorSummary().contains('Select if any pain inducing techniques were used')
+    useOfForceDetailsPage.errorSummary().contains('Enter the type of weapon observed')
+  })
+
+  it('Displays validation messages when multiple inputs are not unique', () => {
+    cy.login()
+
+    const reportUseOfForcePage = ReportUseOfForcePage.visit(offender.bookingId)
+    const selectUofReasonsPage = reportUseOfForcePage.goToSelectUofReasonsPage()
+    selectUofReasonsPage.checkReason('FIGHT_BETWEEN_PRISONERS')
+    selectUofReasonsPage.clickSaveAndContinue()
+
+    const useOfForceDetailsPage = UseOfForceDetailsPage.verifyOnPage()
+    useOfForceDetailsPage.bodyWornCamera().check('YES')
+    useOfForceDetailsPage.bodyWornCameraNumber(0).type('1')
+    useOfForceDetailsPage.addAnotherBodyWornCamera()
+    useOfForceDetailsPage.bodyWornCameraNumber(1).type('1')
+
+    useOfForceDetailsPage.weaponsObserved().check('YES')
+    useOfForceDetailsPage.weaponTypes(0).type('gun')
+    useOfForceDetailsPage.addAnotherWeapon()
+    useOfForceDetailsPage.weaponTypes(1).type('gun')
+
+    useOfForceDetailsPage.clickSaveAndContinue()
+
+    useOfForceDetailsPage.errorSummary().contains("Camera '1' has already been added - remove this camera")
+    useOfForceDetailsPage.errorSummary().contains("Weapon 'gun' has already been added - remove this weapon")
   })
 })

integration-tests/integration/seedData.js

@@ -28,6 +28,8 @@ const expectedPayload = {
     pavaUsed: true,
     batonUsed: true,
     pavaDrawn: true,
+    weaponsObserved: 'YES',
+    weaponTypes: [{ weaponType: 'gun' }, { weaponType: 'knife' }, { weaponType: 'fork' }],
     batonDrawn: true,
     guidingHold: true,
     escortingHold: true,

integration-tests/pages/createReport/useOfForceDetailsPage.js

@@ -7,13 +7,17 @@ const useOfForceDetailsPage = () =>
 
     bodyWornCamera: () => cy.get('[name="bodyWornCamera"]'),
     bodyWornCameraNumber: index => cy.get(`[name="bodyWornCameraNumbers[${index}][cameraNum]"]`),
-    addAnotherBodyWornCamera: () => cy.get('[data-qa-add-another-input = true]').click(),
-    removeBodyWornCamera: index => cy.get('.add-another-input .add-another__remove-button').eq(index).click(),
+    addAnotherBodyWornCamera: () => cy.get('[dataqa=add-another-body-worn-camera]').click(),
+    removeBodyWornCamera: index =>
+      cy.get('.add-another-body-worn-camera .add-another__remove-button').eq(index).click(),
 
     personalProtectionTechniques: () => cy.get('[name="personalProtectionTechniques"]'),
     batonDrawn: () => cy.get('[name="batonDrawn"]'),
     batonUsed: () => cy.get('[name="batonUsed"]'),
     pavaDrawn: () => cy.get('[name="pavaDrawn"]'),
+    weaponsObserved: () => cy.get('[name="weaponsObserved"]'),
+    weaponTypes: index => cy.get(`[name="weaponTypes[${index}][weaponType]"]`),
+    addAnotherWeapon: () => cy.get('[dataqa=add-another-weapons-observed]').click(),
     pavaUsed: () => cy.get('[name="pavaUsed"]'),
     guidingHold: () => cy.get('[name="guidingHold"]'),
     guidingHoldOfficersInvolved: {
@@ -57,6 +61,7 @@ const useOfForceDetailsPage = () =>
       this.batonUsed().check('true')
       this.pavaDrawn().check('true')
       this.pavaUsed().check('true')
+      this.weaponsObserved().check('NO')
       this.guidingHold().check('true')
       this.guidingHoldOfficersInvolved.check('2')
       this.escortingHold().check('true')

server/config/forms/useOfForceDetailsForm.js

@@ -19,8 +19,7 @@ const completeSchema = joi.object({
 
   bodyWornCamera: requiredOneOfMsg(
     'YES',
-    'NO',
-    'NOT_KNOWN'
+    'NO'
   )('Select yes if any part of the incident was captured on a body-worn camera').alter(optionalForPartialValidation),
 
   bodyWornCameraNumbers: joi
@@ -53,6 +52,27 @@ const completeSchema = joi.object({
 
   pavaDrawn: requiredBooleanMsg('Select yes if PAVA was drawn').alter(optionalForPartialValidation),
 
+  weaponsObserved: requiredOneOfMsg(
+    'YES',
+    'NO'
+  )('Select yes if any weapons were observed').alter(optionalForPartialValidation),
+
+  weaponTypes: joi
+    .when('weaponsObserved', {
+      is: 'YES',
+      then: arrayOfObjects({
+        weaponType: requiredStringMsg('Enter the type of weapon observed').alter(optionalForPartialValidation),
+      })
+        .min(1)
+        .message('Enter the type of weapon observed')
+        .ruleset.unique('weaponType')
+        .message("Weapon '{#value.weaponType}' has already been added - remove this weapon")
+        .required()
+        .alter(minZeroForPartialValidation),
+      otherwise: joi.any().strip(),
+    })
+    .meta({ firstFieldName: 'weaponTypes[0]' }),
+
   pavaUsed: joi.when('pavaDrawn', {
     is: true,
     then: requiredBooleanMsg('Select yes if PAVA was used').alter(optionalForPartialValidation),