@planning-inspectorate/dynamic-forms 3.2.0 → 3.3.1

package/README.md

@@ -146,12 +146,12 @@ to add conditions to questions in multiple ways.
 The simplest condition applies to just one question:
 
 ```javascript
-import { questionHasAnswer } from '@pins/dynamic-forms/src/components/utils/question-has-answer';
+import { whenQuestionHasAnswer } from '@pins/dynamic-forms/src/components/utils/question-has-answer';
 
 new Section('section-key', 'Section title')
         .addQuestion(questions.q1)
         .addQuestion(questions.q2)
-        .withCondition((response) => questionHasAnswer(response, questions.q1, 'yes'))
+        .withCondition(whenQuestionHasAnswer(questions.q1, 'yes'))
 ```
 
 In this scenario question two will only be shown if question one has an answer of 'yes'.
@@ -160,10 +160,10 @@ Conditions can also be added to an entire section:
 
 ```javascript
 new Section('section-key', 'Section title')
-        .withSectionCondition((response) => questionHasAnswer(response, questions.q1, 'yes'))
+        .withSectionCondition(whenQuestionHasAnswer(questions.q1, 'yes'))
         .addQuestion(questions.q3)
         .addQuestion(questions.q4)
-        .withCondition((response) => questionHasAnswer(response, questions.q3, 'yes'))
+        .withCondition(whenQuestionHasAnswer(questions.q3, 'yes'))
 ```
 
 In this case q3 and q4 will only be shown if q1 (from a previous section) is answered 'yes'. Q4 will only show if q1 is
@@ -173,16 +173,16 @@ Finally, conditions can be added to multiple questions - these can overlap:
 
 ```javascript
 new Section('section-key', 'Section title')
-        .withSectionCondition((response) => questionHasAnswer(response, questions.q1, 'yes'))
+        .withSectionCondition(whenQuestionHasAnswer(questions.q1, 'yes'))
         .addQuestion(questions.q2)
-        .startMultiQuestionCondition('group-1', (response) => questionHasAnswer(response, questions.q2, 'valid'))
+        .startMultiQuestionCondition('group-1', whenQuestionHasAnswer(questions.q2, 'valid'))
         .addQuestion(questions.q3)
         .addQuestion(questions.q4)
-        .withCondition((response) => questionHasAnswer(response, questions.q3, 'yes'))
+        .withCondition(whenQuestionHasAnswer(questions.q3, 'yes'))
         .endMultiQuestionCondition('group-1')
         .addQuestion(questions.q5)
-        .withCondition((response) => questionHasAnswer(response, questions.q4, 'yes'))
-        .startMultiQuestionCondition('group-2', (response) => questionHasAnswer(response, questions.q5, 90))
+        .withCondition(whenQuestionHasAnswer(questions.q4, 'yes'))
+        .startMultiQuestionCondition('group-2', whenQuestionHasAnswer(questions.q5, 90))
         .addQuestion(questions.q6)
         .addQuestion(questions.q7)
         .endMultiQuestionCondition('group-2')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@planning-inspectorate/dynamic-forms",
-  "version": "3.2.0",
+  "version": "3.3.1",
   "description": "This package is for building GDS forms using a configuration-based approach.",
   "main": "./src/index.js",
   "type": "module",

package/src/components/address/question.js

@@ -72,12 +72,11 @@ export default class AddressQuestion extends Question {
 	}
 
 	/**
-	 * returns the data to send to the DB
-	 * side effect: modifies journeyResponse with the new answers
+	 * Get the data to save from the request, returns an object of answers
 	 * @param {import('express').Request} req
 	 * @param {JourneyResponse} journeyResponse
 	 * @returns {Promise<{answers: Record<string, unknown>}>}
-	 */
+	 */ //eslint-disable-next-line no-unused-vars -- journeyResponse kept for other questions to use
 	async getDataToSave(req, journeyResponse) {
 		const data = {
 			addressLine1: req.body[this.fieldName + '_addressLine1'],
@@ -94,7 +93,6 @@ export default class AddressQuestion extends Question {
 		const answers = {
 			[this.fieldName]: address
 		};
-		journeyResponse.answers[this.fieldName] = address;
 
 		return {
 			answers

package/src/components/boolean/question.js

@@ -97,28 +97,23 @@ export default class BooleanQuestion extends RadioQuestion {
 	}
 
 	/**
-	 * returns the data to send to the DB
-	 * side effect: modifies journeyResponse with the new answers
+	 * Get the data to save from the request, returns an object of answers
 	 * @param {import('express').Request} req
 	 * @param {JourneyResponse} journeyResponse - current journey response, modified with the new answers
 	 * @returns {Promise.<Object>}
-	 */
+	 */ //eslint-disable-next-line no-unused-vars -- journeyResponse kept for other questions to use
 	async getDataToSave(req, journeyResponse) {
-		// set answer on response
-		let responseToSave = { answers: {} };
+		const answers = {};
 		const fieldValue = req.body[this.fieldName]?.trim();
 
-		responseToSave.answers[this.fieldName] = fieldValue === BOOLEAN_OPTIONS.YES;
+		answers[this.fieldName] = fieldValue === BOOLEAN_OPTIONS.YES;
 
 		for (const propName in req.body) {
 			if (propName.startsWith(this.fieldName + '_')) {
-				responseToSave.answers[propName] = req.body[propName]?.trim();
-				journeyResponse.answers[propName] = req.body[propName]?.trim();
+				answers[propName] = req.body[propName]?.trim();
 			}
 		}
 
-		journeyResponse.answers[this.fieldName] = fieldValue;
-
-		return responseToSave;
+		return { answers };
 	}
 }

package/src/components/date/question.js

@@ -27,27 +27,21 @@ export default class DateQuestion extends Question {
 	}
 
 	/**
-	 * returns the data to send to the DB
-	 * side effect: modifies journeyResponse with the new answers
+	 * Get the data to save from the request, returns an object of answers
 	 * @param {import('express').Request} req
 	 * @param {JourneyResponse} journeyResponse - current journey response, modified with the new answers
 	 * @returns {Promise.<Object>}
-	 */
+	 */ //eslint-disable-next-line no-unused-vars -- journeyResponse kept for other questions to use
 	async getDataToSave(req, journeyResponse) {
-		// set answer on response
-		let responseToSave = { answers: {} };
+		const answers = {};
 
 		const dayInput = req.body[`${this.fieldName}_day`];
 		const monthInput = req.body[`${this.fieldName}_month`];
 		const yearInput = req.body[`${this.fieldName}_year`];
 
-		const dateToSave = parseDateInput({ day: dayInput, month: monthInput, year: yearInput });
-
-		responseToSave.answers[this.fieldName] = dateToSave;
-
-		journeyResponse.answers[this.fieldName] = responseToSave.answers[this.fieldName];
+		answers[this.fieldName] = parseDateInput({ day: dayInput, month: monthInput, year: yearInput });
 
-		return responseToSave;
+		return { answers };
 	}
 
 	answerForViewModel(answers, isPayload) {

package/src/components/date-period/question.js

@@ -36,15 +36,13 @@ export default class DatePeriodQuestion extends Question {
 	}
 
 	/**
-	 * returns the data to send to the DB
-	 * side effect: modifies journeyResponse with the new answers
+	 * Get the data to save from the request, returns an object of answers
 	 * @param {import('express').Request} req
 	 * @param {JourneyResponse} journeyResponse - current journey response, modified with the new answers
 	 * @returns {Promise.<Object>}
-	 */
+	 */ //eslint-disable-next-line no-unused-vars -- journeyResponse kept for other questions to use
 	async getDataToSave(req, journeyResponse) {
-		// set answer on response
-		let responseToSave = { answers: {} };
+		const answers = {};
 
 		const startDayInput = req.body[`${this.fieldName}_start_day`];
 		const startMonthInput = req.body[`${this.fieldName}_start_month`];
@@ -70,11 +68,9 @@ export default class DatePeriodQuestion extends Question {
 			year: endYearInput
 		});
 
-		responseToSave.answers[this.fieldName] = { start: startDate, end: endDate };
-
-		journeyResponse.answers[this.fieldName] = responseToSave.answers[this.fieldName];
+		answers[this.fieldName] = { start: startDate, end: endDate };
 
-		return responseToSave;
+		return { answers };
 	}
 
 	answerForViewModel(answers, isPayload) {

package/src/components/date-time/question.js

@@ -23,14 +23,13 @@ export default class DateTimeQuestion extends Question {
 	}
 
 	/**
-	 * returns the data to send to the DB
-	 * side effect: modifies journeyResponse with the new answers
+	 * Get the data to save from the request, returns an object of answers
 	 * @param {import('express').Request} req
 	 * @param {JourneyResponse} journeyResponse - current journey response, modified with the new answers
 	 * @returns {Promise.<Object>}
-	 */
+	 */ //eslint-disable-next-line no-unused-vars -- journeyResponse kept for other questions to use
 	async getDataToSave(req, journeyResponse) {
-		let responseToSave = { answers: {} };
+		const answers = {};
 
 		const dayInput = req.body[`${this.fieldName}_day`];
 		const monthInput = req.body[`${this.fieldName}_month`];
@@ -39,7 +38,7 @@ export default class DateTimeQuestion extends Question {
 		const minutesInput = req.body[`${this.fieldName}_minutes`];
 		const periodInput = req.body[`${this.fieldName}_period`];
 
-		responseToSave.answers[this.fieldName] = parseDateInput({
+		answers[this.fieldName] = parseDateInput({
 			day: dayInput,
 			month: monthInput,
 			year: yearInput,
@@ -47,9 +46,7 @@ export default class DateTimeQuestion extends Question {
 			minute: minutesInput
 		});
 
-		journeyResponse.answers[this.fieldName] = responseToSave.answers[this.fieldName];
-
-		return responseToSave;
+		return { answers };
 	}
 
 	answerForViewModel(answers, isPayload) {

package/src/components/date-time/question.test.js

@@ -131,20 +131,13 @@ describe('./lib/forms/custom-components/date-time/question.js', () => {
 					siteVisitDate_period: 'am'
 				}
 			};
-			const journeyResponse = { answers: {} };
-
-			const responseToSave = await question.getDataToSave(req, journeyResponse);
+			const responseToSave = await question.getDataToSave(req);
 
 			assert.deepStrictEqual(responseToSave, {
 				answers: {
 					siteVisitDate: new Date('2025-02-21T00:12:00.000Z')
 				}
 			});
-			assert.deepStrictEqual(journeyResponse, {
-				answers: {
-					siteVisitDate: new Date('2025-02-21T00:12:00.000Z')
-				}
-			});
 		});
 		it('should get data to save when time period is pm', async () => {
 			const req = {
@@ -157,20 +150,13 @@ describe('./lib/forms/custom-components/date-time/question.js', () => {
 					siteVisitDate_period: 'pm'
 				}
 			};
-			const journeyResponse = { answers: {} };
-
-			const responseToSave = await question.getDataToSave(req, journeyResponse);
+			const responseToSave = await question.getDataToSave(req);
 
 			assert.deepStrictEqual(responseToSave, {
 				answers: {
 					siteVisitDate: new Date('2025-02-21T20:12:00.000Z')
 				}
 			});
-			assert.deepStrictEqual(journeyResponse, {
-				answers: {
-					siteVisitDate: new Date('2025-02-21T20:12:00.000Z')
-				}
-			});
 		});
 		it('should get data to save when bst date', async () => {
 			const req = {
@@ -183,20 +169,13 @@ describe('./lib/forms/custom-components/date-time/question.js', () => {
 					siteVisitDate_period: 'am'
 				}
 			};
-			const journeyResponse = { answers: {} };
-
-			const responseToSave = await question.getDataToSave(req, journeyResponse);
+			const responseToSave = await question.getDataToSave(req);
 
 			assert.deepStrictEqual(responseToSave, {
 				answers: {
 					siteVisitDate: new Date('2025-05-20T23:12:00.000Z')
 				}
 			});
-			assert.deepStrictEqual(journeyResponse, {
-				answers: {
-					siteVisitDate: new Date('2025-05-20T23:12:00.000Z')
-				}
-			});
 		});
 	});
 	describe('formatAnswerForSummary', () => {

package/src/components/manage-list/README.md

@@ -59,7 +59,7 @@ const questionProps = {
 
 ### 2. Set up the section within the journey
 
-You can use conditional logic as normal in the `ManageListSection`.
+You can use conditional logic in the `ManageListSection` (see [Conditions](#conditions))
 
 ```js
 new Section('New Holiday', 'holiday')
@@ -94,4 +94,23 @@ router.post(
         validationErrorHandler,
         buildSave(saveDataToSession)
 );
-```
+```
+
+### Conditions
+
+The usual conditions can be used in a `ManageListSection`. However, only answers in the `ManageListSection` will be available to check, not answers for main the journey. For example:
+
+```js
+new Section('New Holiday', 'holiday')
+        .addQuestion(questions.holidayDestination)
+        // add a manage list question like any other, but include the `ManageListSection` as well
+        .addQuestion(
+                questions.holidayActivitiesList,
+                new ManageListSection()
+                        .addQuestion(questions.holidayActivityType)
+                        .addQuestion(questions.holidayActivityDate)
+                        .withCondition(whenQuestionHasAnswer(questions.holidayActivityType, 'swimming'))
+        )
+        .addQuestion(questions.departureDate)
+        .addQuestion(questions.holidayPeriod)
+```

package/src/components/manage-list/index.njk

@@ -31,12 +31,12 @@
                             <dd class="govuk-summary-list__actions">
                                 <ul class="govuk-summary-list__actions-list">
                                     <li class="govuk-summary-list__actions-list-item">
-                                        <a class="govuk-link" href="{{ originalUrl }}/edit/{{ item.id }}/{{ question.firstQuestionUrl }}">
+                                        <a class="govuk-link" href="{{ util.trimTrailingSlash(originalUrl) }}/edit/{{ item.id }}/{{ question.firstQuestionUrl }}">
                                             Change <span class="govuk-visually-hidden">{{ titleSingular }} {{ loop.index }}</span>
                                         </a>
                                     </li>
                                     <li class="govuk-summary-list__actions-list-item">
-                                        <a class="govuk-link" href="{{ originalUrl }}/remove/{{ item.id }}/confirm">
+                                        <a class="govuk-link" href="{{ util.trimTrailingSlash(originalUrl) }}/remove/{{ item.id }}/confirm">
                                             Remove <span class="govuk-visually-hidden">{{ titleSingular }} {{ loop.index }}</span>
                                         </a>
                                     </li>
@@ -47,7 +47,7 @@
                 </dl>
 
                 <p class="govuk-body">
-                    <a href="{{ originalUrl }}/{{ question.addAnotherLink }}" class="govuk-link">Add another</a>
+                    <a href="{{ util.trimTrailingSlash(originalUrl) }}/{{ question.addAnotherLink }}" class="govuk-link">Add another</a>
                 </p>
 
                 {{ govukButton({

package/src/components/manage-list/question.js

@@ -71,26 +71,35 @@ export default class ManageListQuestion extends Question {
 			return [];
 		}
 		const mockJourney = {
-			getCurrentQuestionUrl() {}
+			getCurrentQuestionUrl() {},
+			response: {
+				answers: answer
+			}
 		};
-		return this.section.questions.map((q) => {
-			const formatted = q
-				.formatAnswerForSummary('', mockJourney, answer[q.fieldName])
-				.map((a) => a.value)
-				.join(', ');
+		return (
+			this.section.questions
+				// only show questions which should be displayed based on any conditional logic
+				.filter((q) => q.shouldDisplay({ answers: answer }))
+				.map((q) => {
+					const formatted = q
+						.formatAnswerForSummary('', mockJourney, answer[q.fieldName])
+						.map((a) => a.value)
+						.join(', ');
 
-			return {
-				question: q.title,
-				answer: formatted
-			};
-		});
+					return {
+						question: q.title,
+						answer: formatted
+					};
+				})
+		);
 	}
 
 	async getDataToSave(req, journeyResponse) {
-		let responseToSave = { answers: {} };
-		const data = journeyResponse.answers[this.fieldName];
-		responseToSave.answers[this.fieldName] = data || [];
-		return responseToSave;
+		return {
+			answers: {
+				[this.fieldName]: journeyResponse.answers[this.fieldName] || []
+			}
+		};
 	}
 
 	formatAnswerForSummary(sectionSegment, journey, answer) {

package/src/components/manage-list/question.test.js

@@ -3,7 +3,7 @@ import assert from 'node:assert';
 import ManageListQuestion from '#src/components/manage-list/question.js';
 import { ManageListSection } from '#src/components/manage-list/manage-list-section.js';
 import { mockJourney } from '#test/mock/journey.js';
-import { mockRandomUUID } from '#src/lib/uuid.test.js';
+import { mockRandomUUID } from '#test/mock/uuid.js';
 import { configureNunjucksTestEnv } from '#test/utils/nunjucks.js';
 import path from 'path';
 import { snapshotsDir } from '#test/utils/utils.js';
@@ -136,4 +136,41 @@ describe('components/manage-list/question', () => {
 			}
 		);
 	});
+
+	describe('getDataToSave', () => {
+		it('should return existing journey answers if present', async () => {
+			const q = newQuestion();
+			const existingData = [{ id: '1', name: 'Test' }];
+			const req = { body: {} };
+			const journeyResponse = {
+				answers: {
+					[FIELDNAME]: existingData
+				}
+			};
+
+			const result = await q.getDataToSave(req, journeyResponse);
+
+			assert.deepStrictEqual(result, {
+				answers: {
+					[FIELDNAME]: existingData
+				}
+			});
+		});
+
+		it('should return an empty array if no journey answers are present', async () => {
+			const q = newQuestion();
+			const req = { body: {} };
+			const journeyResponse = {
+				answers: {}
+			};
+
+			const result = await q.getDataToSave(req, journeyResponse);
+
+			assert.deepStrictEqual(result, {
+				answers: {
+					[FIELDNAME]: []
+				}
+			});
+		});
+	});
 });

package/src/components/manage-list/utils.js

@@ -0,0 +1,35 @@
+/**
+ *
+ * @param {import('#src/journey/journey-response.js').JourneyResponse} response
+ * @param {import('./question.js')} manageListQuestion
+ * @param {string} manageListItemId
+ * @returns {Record<string, unknown>}
+ */
+export function answerObjectForManageList(response, manageListQuestion, manageListItemId) {
+	const answers = response.answers[manageListQuestion.fieldName];
+	if (!Array.isArray(answers)) {
+		return {};
+	}
+	return answers.find((a) => a.id === manageListItemId) || {};
+}
+
+/**
+ * Similar to answerObjectForManageList but will edit response and add a new array entry if not found
+ *
+ * @param {import('#src/journey/journey-response.js').JourneyResponse} response
+ * @param {import('./question.js')} manageListQuestion
+ * @param {import('#src/journey/journey-types.d.ts').RouteParams} params
+ * @returns {import('#src/journey/journey-types.d.ts').ManageListAnswers}
+ */
+export function answerObjectForManageListSaving(response, manageListQuestion, params) {
+	const answersList =
+		response.answers[manageListQuestion?.fieldName] || (response.answers[manageListQuestion?.fieldName] = []);
+	let answers = answersList.find((item) => item.id === params.manageListItemId);
+	if (!answers) {
+		answers = {
+			id: params.manageListItemId
+		};
+		answersList.push(answers);
+	}
+	return answers;
+}

package/src/components/manage-list/utils.test.js

@@ -0,0 +1,80 @@
+import { describe, it } from 'node:test';
+import assert from 'node:assert';
+import { answerObjectForManageList, answerObjectForManageListSaving } from '#src/components/manage-list/utils.js';
+
+describe('manage-list-utils', () => {
+	describe('answerObjectForManageList', () => {
+		it('should return the manage list item object for manage list questions', () => {
+			const manageListQuestion = {
+				fieldName: 'manageListQuestion'
+			};
+			const answers = {
+				id: '1',
+				myField: 'my-value'
+			};
+			const journeyResponse = {
+				answers: {
+					manageListQuestion: [answers]
+				}
+			};
+			const manageListItemId = '1';
+			const got = answerObjectForManageList(journeyResponse, manageListQuestion, manageListItemId);
+			assert.strictEqual(got, answers);
+		});
+
+		it('should fallback to {} if manage list item id not found', () => {
+			const manageListQuestion = {
+				fieldName: 'manageListQuestion'
+			};
+			const answers = {
+				id: '2',
+				myField: 'my-value'
+			};
+			const journeyResponse = {
+				answers: {
+					manageListQuestion: [answers]
+				}
+			};
+			const manageListItemId = '1';
+			let got = answerObjectForManageList(journeyResponse, manageListQuestion, manageListItemId);
+			assert.deepStrictEqual(got, {});
+			// also fallback to {} if there is no manageListQuestion answer
+			delete journeyResponse.answers.manageListQuestion;
+			got = answerObjectForManageList(journeyResponse, manageListQuestion, manageListItemId);
+			assert.deepStrictEqual(got, {});
+		});
+	});
+
+	describe('answerObjectForManageListSaving', () => {
+		it('should add an array with a new object if not found', () => {
+			const manageListQuestion = {
+				fieldName: 'manageListQuestion'
+			};
+			const journeyResponse = {
+				answers: {}
+			};
+			const params = { manageListItemId: '1' };
+			const got = answerObjectForManageListSaving(journeyResponse, manageListQuestion, params);
+			assert.deepStrictEqual(got, { id: '1' });
+			assert.ok(Array.isArray(journeyResponse.answers.manageListQuestion));
+			assert.ok(journeyResponse.answers.manageListQuestion[0]?.id, '1');
+		});
+		it('should return the manage list item object for manage list questions', () => {
+			const manageListQuestion = {
+				fieldName: 'manageListQuestion'
+			};
+			const answers = {
+				id: '1',
+				myField: 'my-value'
+			};
+			const journeyResponse = {
+				answers: {
+					manageListQuestion: [answers]
+				}
+			};
+			const params = { manageListItemId: '1' };
+			const got = answerObjectForManageListSaving(journeyResponse, manageListQuestion, params);
+			assert.strictEqual(got, answers);
+		});
+	});
+});

package/src/components/multi-field-input/question.js

@@ -74,28 +74,23 @@ export default class MultiFieldInputQuestion extends Question {
 	}
 
 	/**
-	 * returns the data to send to the DB
-	 * side effect: modifies journeyResponse with the new answers
+	 * Get the data to save from the request, returns an object of answers
 	 * @param {import('express').Request} req
 	 * @param {JourneyResponse} journeyResponse - current journey response, modified with the new answers
 	 * @returns {Promise<{ answers: Record<string, unknown> }>}
-	 */
+	 */ //eslint-disable-next-line no-unused-vars -- journeyResponse kept for other questions to use
 	async getDataToSave(req, journeyResponse) {
-		/**
-		 * @type {{ answers: Record<string, unknown> }}
-		 */
-		let responseToSave = { answers: {} };
+		const answers = {};
 
 		for (const inputField of this.inputFields) {
 			let value = req.body[inputField.fieldName];
 			if (typeof value === 'string') {
 				value = value.trim();
 			}
-			responseToSave.answers[inputField.fieldName] = value;
-			journeyResponse.answers[inputField.fieldName] = responseToSave.answers[inputField.fieldName];
+			answers[inputField.fieldName] = value;
 		}
 
-		return responseToSave;
+		return { answers };
 	}
 
 	/**
@@ -123,7 +118,8 @@ export default class MultiFieldInputQuestion extends Question {
 		return [
 			{
 				key: `${this.title}`,
-				value: nl2br(escape(formattedAnswer)),
+				// Do not convert new lines to breaks in ManageListSection, as this causes a "doubled up" <br>
+				value: this.isInManageListSection ? escape(formattedAnswer) : nl2br(escape(formattedAnswer)),
 				action: this.getAction(sectionSegment, journey, summaryDetails)
 			}
 		];

package/src/components/multi-field-input/question.test.js

@@ -34,7 +34,7 @@ function createMultiFieldInputQuestion(
 	});
 }
 
-describe('./src/dynamic-forms/components/single-line-input/question.js', () => {
+describe('./src/dynamic-forms/components/multi-field-input/question.js', () => {
 	it('should create', () => {
 		const testQuestion = createMultiFieldInputQuestion();
 
@@ -320,5 +320,33 @@ describe('./src/dynamic-forms/components/single-line-input/question.js', () => {
 
 			assert.deepStrictEqual(result, expectedResult);
 		});
+		it('should return escaped text with newlines (no <br>) when isInManageListSection is true', () => {
+			const question = createMultiFieldInputQuestion();
+
+			question.isInManageListSection = true;
+
+			const journey = new Journey({
+				journeyId: 'TEST',
+				makeBaseUrl: () => 'base',
+				taskListUrl: 'cases/create-a-case/check-your-answers',
+				response: {
+					answers: {
+						testField1: 'Line 1',
+						testField2: 'Line & 2'
+					}
+				},
+				journeyTemplate: 'mock template',
+				taskListTemplate: 'mock path',
+				journeyTitle: 'mock title',
+				sections: []
+			});
+
+			const expectedValue = 'Line 1\nLine &amp; 2\n';
+
+			const result = question.formatAnswerForSummary('questions', journey);
+
+			assert.strictEqual(result[0].value, expectedValue);
+			assert.strictEqual(result[0].value.includes('<br>'), false);
+		});
 	});
 });