Skip to content

Commit

Permalink
Update DS review page
Browse files Browse the repository at this point in the history
  • Loading branch information
JunTaoLuo committed Dec 27, 2024
1 parent a3eb46c commit 52a3eb5
Show file tree
Hide file tree
Showing 8 changed files with 27 additions and 15 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import { confirmAppointment } from '../redux/actions';
import AppointmentDate from '../../new-appointment/components/ReviewPage/AppointmentDate';
import { selectFeatureBreadcrumbUrlUpdate } from '../../redux/selectors';

const pageTitle = 'Review your appointment details';
const pageTitle = 'Review and confirm your appointment details';

function handleClick(history, contactInfo) {
return e => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,9 @@ describe('VAOS vaccine flow: ReviewPage', () => {
dateHeading,
facilityHeading,
] = screen.getAllByRole('heading');
expect(pageHeading).to.contain.text('Review your appointment details');
expect(pageHeading).to.contain.text(
'Review and confirm your appointment details',
);
expect(descHeading).to.contain.text('COVID-19 vaccine');
expect(descHeading).to.have.tagName('h2');

Expand Down
6 changes: 3 additions & 3 deletions src/applications/vaos/covid-19-vaccine/flow.js
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ export default function getPageFlow(state) {
url: featureBreadcrumbUrlUpdate
? 'review'
: '/new-covid-19-vaccine-appointment/review',
label: 'Review your appointment details',
label: 'Review and confirm your appointment details',
next: '',
},
root: {
Expand Down Expand Up @@ -147,11 +147,11 @@ export function routeToNextAppointmentPage(history, current, data) {
}

/* Function to get label from the flow
* The URL displayed in the browser address bar is compared to the
* The URL displayed in the browser address bar is compared to the
* flow URL
*
* @export
* @param {object} state
* @param {object} state
* @param {string} location - the pathname
* @returns {string} the label string
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,9 @@ describe('VAOS Page: ReviewPage direct scheduling with v2 api', () => {
reasonHeading,
contactHeading,
] = screen.getAllByRole('heading');
expect(pageHeading).to.contain.text('Review your appointment details');
expect(pageHeading).to.contain.text(
'Review and confirm your appointment details',
);
expect(descHeading).to.contain.text(
'scheduling a primary care appointment',
);
Expand Down
4 changes: 2 additions & 2 deletions src/applications/vaos/new-appointment/newAppointmentFlow.js
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,7 @@ const flow = {
},
review: {
url: '/new-appointment/review',
label: 'Review your appointment details',
label: 'Review and confirm your appointment details',
},
};

Expand Down Expand Up @@ -416,7 +416,7 @@ export default function getNewAppointmentFlow(state) {
...flow.review,
label:
FLOW_TYPES.DIRECT === flowType
? 'Review your appointment details'
? 'Review and confirm your appointment details'
: 'Review and submit your request',
url: featureBreadcrumbUrlUpdate ? 'review' : '/new-appointment/review',
},
Expand Down
8 changes: 8 additions & 0 deletions src/applications/vaos/services/mocks/v2/slots.json
Original file line number Diff line number Diff line change
Expand Up @@ -2487,6 +2487,14 @@
"start": "2024-08-22T21:15:00Z",
"end": "2024-08-22T00:00:00Z"
}
},
{
"id": "1913",
"type": "slots",
"attributes": {
"start": "2025-08-22T21:15:00Z",
"end": "2025-08-22T00:00:00Z"
}
}
]
}
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ describe('VAOS request schedule flow - Audiology', () => {

ReviewPageObject.assertUrl()
.assertHeading({
name: /Review your appointment details/i,
name: /Review and confirm your appointment details/i,
})
.assertHeading({
level: 2,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ describe('VAOS direct schedule flow - Primary care', () => {

ReviewPageObject.assertUrl()
.assertHeading({
name: /Review your appointment details/i,
name: /Review and confirm your appointment details/i,
})
.assertHeading({
level: 2,
Expand Down Expand Up @@ -209,7 +209,7 @@ describe('VAOS direct schedule flow - Primary care', () => {

ReviewPageObject.assertUrl()
.assertHeading({
name: /Review your appointment details/i,
name: /Review and confirm your appointment details/i,
})
.assertHeading({
level: 2,
Expand Down Expand Up @@ -291,7 +291,7 @@ describe('VAOS direct schedule flow - Primary care', () => {

ReviewPageObject.assertUrl()
.assertHeading({
name: /Review your appointment details/i,
name: /Review and confirm your appointment details/i,
})
.assertHeading({
level: 2,
Expand Down Expand Up @@ -481,7 +481,7 @@ describe('VAOS direct schedule flow - Primary care', () => {

ReviewPageObject.assertUrl()
.assertHeading({
name: /Review your appointment details/i,
name: /Review and confirm your appointment details/i,
})
.assertHeading({
level: 2,
Expand Down Expand Up @@ -548,7 +548,7 @@ describe('VAOS direct schedule flow - Primary care', () => {

ReviewPageObject.assertUrl()
.assertHeading({
name: /Review your appointment details/i,
name: /Review and confirm your appointment details/i,
})
.assertHeading({
level: 2,
Expand Down Expand Up @@ -711,7 +711,7 @@ describe('VAOS direct schedule flow - Primary care', () => {

ReviewPageObject.assertUrl()
.assertHeading({
name: /Review your appointment details/i,
name: /Review and confirm your appointment details/i,
})
.assertHeading({
level: 2,
Expand Down

0 comments on commit 52a3eb5

Please sign in to comment.