Skip to content

Commit

Permalink
Merge pull request #1057 from ministryofjustice/CAS-489/alternative-pdu
Browse files Browse the repository at this point in the history
CAS-489: alternative PDU - content fixes
  • Loading branch information
froddd authored Aug 14, 2024
2 parents ea3106b + e7d637b commit fcb4648
Show file tree
Hide file tree
Showing 9 changed files with 213 additions and 76 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ export default class ProbationPractitioner extends ApplyPage {
const pdu = this.application.data['contact-details']['practitioner-pdu'].name
this.clickLink(/(Change PDU|Enter a PDU)/)
// Type the first 3 letters of the PDU then click the option shown
this.completeTextInputByLabel('What’s your PDU (probation delivery unit)?', pdu.slice(0, 3))
this.completeTextInputByLabel('What’s your PDU (Probation Delivery Unit)?', pdu.slice(0, 3))
cy.get('li[role="option"]').contains(pdu).click()
this.clickSubmit()
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import ApplyPage from '../../applyPage'
export default class AlternativePduPage extends ApplyPage {
constructor(application: TemporaryAccommodationApplication) {
super(
'Is placement required in an alternative PDU (probation delivery unit)?',
'Is placement required in an alternative PDU (Probation Delivery Unit)?',
application,
'placement-location',
'alternative-pdu',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import ApplyPage from '../../applyPage'
export default class AlternativePduReasonPage extends ApplyPage {
constructor(application: TemporaryAccommodationApplication) {
super(
'Provide a reason for choosing a different PDU (probation delivery unit)',
'Provide a reason for choosing a different PDU (Probation Delivery Unit)',
application,
'placement-location',
'alternative-pdu-reason',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export type PractitionerPduBody = ProbationDeliveryUnit

@Page({ name: 'practitioner-pdu', bodyProperties: ['id'] })
export default class PractitionerPdu implements TasklistPage {
title: string = 'What’s your PDU (probation delivery unit)?'
title: string = 'What’s your PDU (Probation Delivery Unit)?'

htmlDocumentTitle = this.title

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ describe('ProbationPractitioner', () => {
},
},
{
key: { text: 'PDU (Probation delivery unit)' },
key: { text: 'PDU (Probation Delivery Unit)' },
value: {
html: `<a href="${paths.applications.pages.show({
id: application.id,
Expand Down Expand Up @@ -218,7 +218,7 @@ describe('ProbationPractitioner', () => {
},
},
{
key: { text: 'PDU (Probation delivery unit)' },
key: { text: 'PDU (Probation Delivery Unit)' },
value: { text: 'PDU Name' },
actions: {
items: [
Expand All @@ -229,7 +229,7 @@ describe('ProbationPractitioner', () => {
page: 'practitioner-pdu',
}),
text: 'Change',
visuallyHiddenText: 'PDU (Probation delivery unit)',
visuallyHiddenText: 'PDU (Probation Delivery Unit)',
},
],
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ export default class ProbationPractitioner implements TasklistPage {
this.summaryListItem('name', 'Name', 'Enter a name'),
this.summaryListItem('email', 'Email address', 'Enter an email address'),
this.summaryListItem('phone', 'Phone number', 'Enter a phone number'),
this.summaryListItem('pdu', 'PDU (Probation delivery unit)', 'Enter a PDU'),
this.summaryListItem('pdu', 'PDU (Probation Delivery Unit)', 'Enter a PDU'),
]
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export type AlternativePduBody = {

@Page({ name: 'alternative-pdu', bodyProperties: ['alternativePdu', 'pduId', 'pduName'] })
export default class AlternativePdu implements TasklistPage {
title = 'Is placement required in an alternative PDU (probation delivery unit)?'
title = 'Is placement required in an alternative PDU (Probation Delivery Unit)?'

htmlDocumentTitle = this.title

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export type AlternativePduReasonBody = {

@Page({ name: 'alternative-pdu-reason', bodyProperties: ['reason'] })
export default class AlternativePduReason implements TasklistPage {
title = 'Provide a reason for choosing a different PDU (probation delivery unit)'
title = 'Provide a reason for choosing a different PDU (Probation Delivery Unit)'

htmlDocumentTitle = this.title

Expand Down
Loading

0 comments on commit fcb4648

Please sign in to comment.