feat(EMS-4011-4022): policy - change your answers - broker address

[ttbarnes]

Introduction ✏️

This PR updates the user flow to allow for a "change your answers" journey - for changing the "broker address" answer.

Note: the "broker details" form has 4x fields. Depending on the original/origin URL and the current data shape, different redirects need to occur. For example:

• If the route is a "change" route and postcode or building number/ name has not changed, redirect to "Check your answers" URL.
• If the route is a "change" route and postcode or building number/ name has changed, redirect to the "broker addreses" URL.

Resolution ✔️

E2E

• UpdatecheckPolicySummaryList address fields to accept shouldRender and expectedValue params.
• Update constants.
• Add test coverage for:
  • When a broker is being used and is based in the UK, changing the "address" to not be based in the UK.
  • When a broker is being used and is not based in the UK, changing the "address" to be based in the UK - multiple addresses available to choose from.
  • When a broker is being used and is not based in the UK, changing the "address" to be based in the UK - single address available.
  • Changing "using broker" from no to yes, and then completing the forms with the broker being based in the UK.
  • Changing "using broker" from no to yes, and then completing the forms with the broker not being based in the UK.

UI

• Add BROKER_CONFIRM_ADDRESS "change" and "check and change" routes.
• Update the BROKER_CONFIRM_ADDRESS controller to:
  • Have pageVariables return a USE_DIFFERENT_ADDRESS_URL with or without a "change" suffix.
  • Redirect to CHECK_YOUR_ANSWERS if the original URL is a "change" URL.
• Update and Refactor the BROKER_DETAILS controller's POST redirection into it's own helper:
  • getBrokerDetailsPostRedirectUrl
  • basedInUkRedirectUrl
  • notBasedInUkRedirectUrl
• Update the policy summary list mapping so that:
  • The BROKER_MANUAL_ADDRESS field links to BROKER_CONFIRM_ADDRESS, instead of BROKER_MANUAL_ADDRESS.
  • The SELECT_THE_ADDRESS field's call to generateMultipleFieldHtml only consumes the existing addressLine1 value. Otherwise, it is incorrect.

Miscellaneous ➕

• Update BROKER_FULL_ADDRESS mocks, to be clearer.
• Align some E2E test descriptions.
• Remove some unnecessary usingBroker params being passed to cy.completeAndSubmitBrokerDetailsForm.
• Remove some unnecessary comments.
• Minor documentation improvements.
• Address some TODO comments

[abhi-markan]

LGTM 👍🏼